-
Notifications
You must be signed in to change notification settings - Fork 3.2k
[AutoPr] monitor/resource-manager #1931
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
Merged
Merged
Changes from 1 commit
Commits
Show all changes
12 commits
Select commit
Hold shift + click to select a range
2fc289f
[Monitor] Add Public Preview APIs of Metric Baseline (#1710)
AutorestCI a36f114
Generated from df75ee5f47b030dfa0d66e0cac2cf68fcc29b071 (#1891)
lmazuel 98c65ea
Generated from 00a5689e14e49f0c822c7e5a9807982ec2ecda45 (#1927)
AutorestCI d017994
Rebuild by https://github.com/Azure/azure-sdk-for-python/pull/1931
AutorestCI deb50ba
Generated from eabf2683eee832655c5e028aa1f3501a6113780b (#2009)
AutorestCI d27e72f
[AutoPR monitor/resource-manager] [DO NOT MERGE, until feb 20] Adding…
AutorestCI 6de6322
Generated from 067e5a9fee3acee4afed33296cadde18b73a3f1c (#2080)
AutorestCI 0875231
Generated from b0b757a970cf10c190a722bef8a33de12b42ee93 (#2087)
AutorestCI 18d253a
Generated from 189d112e0d578ff72b4e4531174f8d3c80c0b2f3 (#2176)
AutorestCI 9afc4f7
Generated from a878bd64cf43e5cb9155fdecff440bfee98a49ce (#2174)
AutorestCI 98dac47
azure-mgmt-monitor 0.5.0
lmazuel b779c2d
Re-record Monitor tests
lmazuel File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Generated from 067e5a9fee3acee4afed33296cadde18b73a3f1c (#2080)
[Monitor] Fixing issue with metricnamespace specification as client level argument #2565
- Loading branch information
commit 6de632206753e28864ba7d59905015e84510dc2f
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -37,7 +37,7 @@ def __init__(self, client, config, serializer, deserializer): | |
| self.config = config | ||
|
|
||
| def list( | ||
| self, resource_uri, timespan=None, interval=None, metric=None, aggregation=None, top=None, orderby=None, filter=None, result_type=None, custom_headers=None, raw=False, **operation_config): | ||
| self, resource_uri, timespan=None, interval=None, metric=None, aggregation=None, top=None, orderby=None, filter=None, result_type=None, metricnamespace=None, custom_headers=None, raw=False, **operation_config): | ||
| """**Lists the metric values for a resource**. | ||
|
|
||
| :param resource_uri: The identifier of the resource. | ||
|
|
@@ -76,6 +76,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 | ||
|
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. new parameter |
||
| for. | ||
| :type metricnamespace: str | ||
| :param dict custom_headers: headers that will be added to the request | ||
| :param bool raw: returns the direct response alongside the | ||
| deserialized response | ||
|
|
@@ -113,8 +116,8 @@ def list( | |
| if result_type is not None: | ||
| query_parameters['resultType'] = self._serialize.query("result_type", result_type, 'ResultType') | ||
| query_parameters['api-version'] = self._serialize.query("self.api_version", self.api_version, 'str') | ||
| if self.config.metricnamespace is not None: | ||
| query_parameters['metricnamespace'] = self._serialize.query("self.config.metricnamespace", self.config.metricnamespace, 'str') | ||
| if metricnamespace is not None: | ||
| query_parameters['metricnamespace'] = self._serialize.query("metricnamespace", metricnamespace, 'str') | ||
|
|
||
| # Construct headers | ||
| header_parameters = {} | ||
|
|
||
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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