Skip to content
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

Merged
merged 12 commits into from
Mar 19, 2018
Merged

Conversation

lmazuel
Copy link
Member

@lmazuel lmazuel commented Feb 8, 2018

No description provided.

@codecov-io
Copy link

codecov-io commented Feb 8, 2018

Codecov Report

Merging #1931 into master will decrease coverage by 2.21%.
The diff coverage is 51.19%.

Impacted file tree graph

@@            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
Impacted Files Coverage Δ
...azure/mgmt/monitor/models/action_group_resource.py 35.71% <10%> (-9.75%) ⬇️
...onitor/azure/mgmt/monitor/models/metric_trigger.py 35.71% <10%> (-2.75%) ⬇️
...ure/mgmt/monitor/models/rule_metric_data_source.py 100% <100%> (ø) ⬆️
...r/azure/mgmt/monitor/models/alert_rule_resource.py 100% <100%> (ø) ⬆️
...-monitor/azure/mgmt/monitor/operations/__init__.py 100% <100%> (ø) ⬆️
...r/azure/mgmt/monitor/models/time_series_element.py 100% <100%> (ø) ⬆️
...itor/azure/mgmt/monitor/models/rule_data_source.py 100% <100%> (ø) ⬆️
...re/mgmt/monitor/models/threshold_rule_condition.py 100% <100%> (ø) ⬆️
...onitor/azure/mgmt/monitor/models/rule_condition.py 100% <100%> (ø) ⬆️
...tor/azure/mgmt/monitor/models/rule_email_action.py 100% <100%> (ø) ⬆️
... and 1251 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f746c62...b779c2d. Read the comment docs.

@lmazuel
Copy link
Member Author

lmazuel commented Feb 16, 2018

@AutorestCI rebuild https://github.com/Azure/azure-rest-api-specs/blob/master/specification/monitor/resource-manager/readme.md

AutorestCI added a commit that referenced this pull request Feb 16, 2018
@AutorestCI
Copy link
Contributor

Build SDK finished and cleaned

@lmazuel
Copy link
Member Author

lmazuel commented Feb 16, 2018

@AutorestCI install

@AutorestCI
Copy link
Contributor

Installation instruction

Package azure-mgmt-monitor

You can install the package azure-mgmt-monitor of this PR using the following command:
pip install "git+https://github.com/Azure/azure-sdk-for-python@restapi_auto_monitor/resource-manager#egg=azure-mgmt-monitor&subdirectory=azure-mgmt-monitor"

You can build a wheel to distribute for test using the following command:
pip wheel --no-deps "git+https://github.com/Azure/azure-sdk-for-python@restapi_auto_monitor/resource-manager#egg=azure-mgmt-monitor&subdirectory=azure-mgmt-monitor"

If you have a local clone of this repository, you can also do:

  • git checkout restapi_auto_monitor/resource-manager
  • pip install -e ./azure-mgmt-monitor

Or build a wheel file to distribute for testing:

  • git checkout restapi_auto_monitor/resource-manager
  • pip wheel --no-deps ./azure-mgmt-monitor

@lmazuel
Copy link
Member Author

lmazuel commented Feb 28, 2018

@AutorestCI rebase

AutorestCI and others added 6 commits February 28, 2018 18:22
[Monitor] Make all attributes of EventData class readonly (#2254)
… 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.
@AutorestCI
Copy link
Contributor

Rebase done and pushed to the branch

[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)
@AutorestCI
Copy link
Contributor

AutorestCI commented Mar 14, 2018

(message created by the CI based on PR content)

Installation instruction

Package azure-mgmt-monitor

You can install the package azure-mgmt-monitor of this PR using the following command:
pip install "git+https://github.com/Azure/azure-sdk-for-python@restapi_auto_monitor/resource-manager#egg=azure-mgmt-monitor&subdirectory=azure-mgmt-monitor"

You can build a wheel to distribute for test using the following command:
pip wheel --no-deps "git+https://github.com/Azure/azure-sdk-for-python@restapi_auto_monitor/resource-manager#egg=azure-mgmt-monitor&subdirectory=azure-mgmt-monitor"

If you have a local clone of this repository, you can also do:

  • git checkout restapi_auto_monitor/resource-manager
  • pip install -e ./azure-mgmt-monitor

Or build a wheel file to distribute for testing:

  • git checkout restapi_auto_monitor/resource-manager
  • pip wheel --no-deps ./azure-mgmt-monitor

Direct download

Your 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
Copy link
Member Author

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]'},
Copy link
Member Author

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(
Copy link
Member Author

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
Copy link
Member Author

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"
Copy link
Member Author

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
Copy link
Member Author

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
Copy link
Member Author

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
Copy link
Member Author

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
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

new parameter

@lmazuel lmazuel merged commit 5e33876 into master Mar 19, 2018
lmazuel pushed a commit that referenced this pull request Mar 19, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants