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] Minor Updates - description text #2513

Merged
merged 2 commits into from
May 3, 2018

Conversation

AutorestCI
Copy link
Contributor

Merge pull request #1 from abversqr/MinorUpdates

Minor updates
@AutorestCI
Copy link
Contributor Author

AutorestCI commented May 2, 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_2998#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_2998#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_2998
  • pip install -e ./azure-mgmt-monitor

Or build a wheel file to distribute for testing:

  • git checkout restapi_auto_2998
  • pip wheel --no-deps ./azure-mgmt-monitor

Direct download

Your files can be directly downloaded here:

@codecov-io
Copy link

codecov-io commented May 2, 2018

Codecov Report

Merging #2513 into restapi_auto_monitor/resource-manager will increase coverage by 0.16%.
The diff coverage is 33.33%.

Impacted file tree graph

@@                            Coverage Diff                            @@
##           restapi_auto_monitor/resource-manager    #2513      +/-   ##
=========================================================================
+ Coverage                                   54.3%   54.47%   +0.16%     
=========================================================================
  Files                                       6022     5472     -550     
  Lines                                     136291   128396    -7895     
=========================================================================
- Hits                                       74011    69941    -4070     
+ Misses                                     62280    58455    -3825
Impacted Files Coverage Δ
...nitor/azure/mgmt/monitor/models/alerting_action.py 45.45% <ø> (ø) ⬆️
...r/azure/mgmt/monitor/models/alerting_action_py3.py 45.45% <ø> (ø) ⬆️
...or/azure/mgmt/monitor/models/az_ns_action_group.py 50% <ø> (ø) ⬆️
...azure/mgmt/monitor/models/action_group_resource.py 29.41% <ø> (ø) ⬆️
...zure/mgmt/monitor/models/az_ns_action_group_py3.py 50% <ø> (ø) ⬆️
...e/mgmt/monitor/models/action_group_resource_py3.py 29.41% <ø> (ø) ⬆️
...e-mgmt-monitor/azure/mgmt/monitor/models/source.py 50% <0%> (ø) ⬆️
...mt-monitor/azure/mgmt/monitor/models/source_py3.py 50% <50%> (ø) ⬆️
..._12_01/models/virtual_machine_instance_view_py3.py 23.52% <0%> (-76.48%) ⬇️
.../compute/v2017_12_01/models/virtual_machine_py3.py 25% <0%> (-75%) ⬇️
... and 756 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 d19dcb8...e3ce710. Read the comment docs.

@AutorestCI AutorestCI merged commit 336124b into restapi_auto_monitor/resource-manager May 3, 2018
@AutorestCI
Copy link
Contributor Author

This PR has been merged into #2460

AutorestCI added a commit that referenced this pull request Jun 6, 2018
…2513)

* Generated from 0c826790271a1be2d4af5905ee0eb089b4260feb

Merge pull request #1 from abversqr/MinorUpdates

Minor updates

* Generated from 1610a74a5c49c73a546972daca1e7a43798a8cea

Merge branch 'master' into master
lmazuel pushed a commit that referenced this pull request Jun 6, 2018
* [AutoPR monitor/resource-manager] Added GA API version for Scheduled Query Rule (#2416)

* Generated from d183bad923d3a0a4092d21203f850db6cde9f4e9

Resolving Comments

Resolving Comments

* Generated from df40ac3bbf17ea0180dd4adff2af3094065feca0

Made SKU as top level proprty and including generic error response format

Couple of changes -
1. Made SKU as top level proprty as mentioned here - https://github.com/Azure/azure-resource-manager-rpc/blob/master/v1.0/resource-api-reference.md#put-resource
 2. including generic error response format

* Generated from 7578160670750a68d05f77ec8163b25bb72f391e

Minor Changes to align implementation

1. Removed SKU - Billing Model is still under discussion, hence need not to be exposed to customers.
2.  Enabled field ichanged to align with camel case.
3.  'throttleTillDate' in properties.action changed to 'throttlingInMin' to denote correct meaning  and it’s data type changed to number instead of date.
4. 'status' to be removed in properties.action from examples.
5.  'severity' made a required field in properties.action.
6. Removed Examples from original spec json

* Generated from fb180fb311d73934d0f5150f72638cc3de155ea0

Added Default Response payload in all APIs

Changes -
1. Added Default Response payload in all APIs
2. Made azNs description more clear

* Generated from fb180fb311d73934d0f5150f72638cc3de155ea0

Added Default Response payload in all APIs

Changes -
1. Added Default Response payload in all APIs
2. Made azNs description more clear

* [AutoPR monitor/resource-manager] Minor Updates - description text (#2513)

* Generated from 0c826790271a1be2d4af5905ee0eb089b4260feb

Merge pull request #1 from abversqr/MinorUpdates

Minor updates

* Generated from 1610a74a5c49c73a546972daca1e7a43798a8cea

Merge branch 'master' into master

* [AutoPR monitor/resource-manager] Add "autoMitigate" to Metric Alert API spec for 2018-03-01 (#2592)

* Generated from d5b9b4545a8d15c70d3c8318d39e2f7b43229387

AutoMitigate changes

* Generated from d5b9b4545a8d15c70d3c8318d39e2f7b43229387

AutoMitigate changes

* [AutoPR monitor/resource-manager] [Monitor] Deconflict "Action" name in MetricAlert (#2617)

* Generated from 35adc5e493c20a57e6e2cb35d87c578133d2b7a6

[Monitor] Deconflict "Action" name in MetricAlert

Generates bad SDK.

* Generated from ebfaf0bdd7f2bcf736598d8ec501f490a83d1268

Update reference to MetricAlertAction

* [AutoPR monitor/resource-manager] Adding patch support for SQR API (#2596)

* Generated from f53526896e5c001f79bd46ca892f825172b53abf

Merge branch 'master' into patchsupport

* Generated from f53526896e5c001f79bd46ca892f825172b53abf

Merge branch 'master' into patchsupport

* Generated from b37d3ace14508e3bc3f87ee29b7f2ad0b7025f67

Resolving multiple model issue

Changes - 
1. Renamed "MetricTrigger" to "LogMetricTrigger"
2. Aligned "Resources" to other specs

* Rebuild by #2460

* Auto packaging
@lmazuel lmazuel deleted the restapi_auto_2998 branch June 14, 2018 18:15
azure-sdk pushed a commit to openapi-env-test/azure-sdk-for-python that referenced this pull request Jul 13, 2022
Merge pull request Azure#2513 from test-repo-billy/xiaoxu/swaggertest071322

update swagger file
azure-sdk pushed a commit to openapi-env-test/azure-sdk-for-python that referenced this pull request Jul 13, 2022
Merge pull request Azure#2513 from test-repo-billy/xiaoxu/swaggertest071322

update swagger file
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.

2 participants