-
Notifications
You must be signed in to change notification settings - Fork 2.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[AutoPr] monitor/resource-manager #1931
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1931 +/- ##
==========================================
- Coverage 53.06% 50.85% -2.22%
==========================================
Files 4774 5626 +852
Lines 119483 144305 +24822
==========================================
+ Hits 63406 73386 +9980
- Misses 56077 70919 +14842
Continue to review full report at Codecov.
|
Build SDK finished and cleaned |
@AutorestCI install |
Installation instructionPackage azure-mgmt-monitorYou can install the package You can build a wheel to distribute for test using the following command: If you have a local clone of this repository, you can also do:
Or build a wheel file to distribute for testing:
|
@AutorestCI rebase |
* Generated from e6af07c * Update Monitor version
fix python sdk failure
[Monitor] Make all attributes of EventData class readonly (#2254)
Update monitor Py conf
… 2018-01-01 version for metrics API (#2007) * Generated from 25392731dd0415e1e5700412428dd5a96a2d7813 Fix PR comments 1 * Generated from f6739fd0e07e3d040fc0b1f47f3297a7a4a69cba Fixing top/orderby names, Please be reminded, there is no code change, it's just a typo of the names. * Generated from 8ddf699be2bc7bd7a429b97db29d4abb3f238cad Fix meta data sample as well.
Rebase done and pushed to the branch |
2dd7bbb
to
d27e72f
Compare
[Monitor] Fixing issue with metricnamespace specification as client level argument #2565
[Monitor] Fixing issue with metrics api queryparameter name (#2575)
[Monitor] Fixing incorrect specification of top parameter for Metrics API (#2655)
(message created by the CI based on PR content) Installation instructionPackage azure-mgmt-monitorYou can install the package You can build a wheel to distribute for test using the following command: If you have a local clone of this repository, you can also do:
Or build a wheel file to distribute for testing:
Direct downloadYour files can be directly downloaded here: |
Support azureFunctionReceivers in actionGroup APIs
@@ -96,6 +97,8 @@ class MonitorManagementClient(object): | |||
:vartype metric_definitions: azure.mgmt.monitor.operations.MetricDefinitionsOperations | |||
:ivar metrics: Metrics operations | |||
:vartype metrics: azure.mgmt.monitor.operations.MetricsOperations | |||
:ivar metric_baseline: MetricBaseline operations |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
New operation group
@@ -66,12 +79,18 @@ class ActionGroupResource(Resource): | |||
'email_receivers': {'key': 'properties.emailReceivers', 'type': '[EmailReceiver]'}, | |||
'sms_receivers': {'key': 'properties.smsReceivers', 'type': '[SmsReceiver]'}, | |||
'webhook_receivers': {'key': 'properties.webhookReceivers', 'type': '[WebhookReceiver]'}, | |||
'itsm_receivers': {'key': 'properties.itsmReceivers', 'type': '[ItsmReceiver]'}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
new attribute
|
||
if response.status_code not in [200, 204]: | ||
raise models.ErrorResponseException(self._deserialize, response) | ||
|
||
if raw: | ||
client_raw_response = ClientRawResponse(None, response) | ||
return client_raw_response | ||
delete.metadata = {'url': '/subscriptions/{subscriptionId}/resourceGroups/{resourceGroupName}/providers/microsoft.insights/actionGroups/{actionGroupName}'} | ||
|
||
def update( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
mew method
:ivar api_version: Client Api Version. Constant value: "2015-04-01". | ||
""" | ||
|
||
models = models |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
models
def __init__(self, client, config, serializer, deserializer): | ||
|
||
self._client = client | ||
self._serialize = serializer | ||
self._deserialize = deserializer | ||
self.api_version = "2017-05-01-preview" | ||
self.api_version = "2018-01-01" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
new api version
@@ -45,15 +47,16 @@ def list( | |||
:type timespan: str | |||
:param interval: The interval (i.e. timegrain) of the query. | |||
:type interval: timedelta | |||
:param metric: The name of the metric to retrieve. | |||
:type metric: str | |||
:param metricnames: The names of the metrics (comma separated) to |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
new name
:param aggregation: The list of aggregation types (comma separated) to | ||
retrieve. | ||
:type aggregation: str | ||
:param top: The maximum number of records to retrieve. | ||
Valid only if $filter is specified. | ||
Defaults to 10. | ||
:type top: float | ||
:type top: int |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
new type
@@ -74,6 +77,9 @@ def list( | |||
allowed depends on the operation. See the operation's description for | |||
details. Possible values include: 'Data', 'Metadata' | |||
:type result_type: str or ~azure.mgmt.monitor.models.ResultType | |||
:param metricnamespace: Metric namespace to query metric definitions |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
new parameter
"""Lists the metric definitions for the resource. | ||
|
||
:param resource_uri: The identifier of the resource. | ||
:type resource_uri: str | ||
:param metricnamespace: Metric namespace to query metric definitions |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
new parameter
No description provided.