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] [Monitor] Deconflict "Action" name in MetricAlert #2617

Merged
merged 2 commits into from
May 30, 2018

Conversation

AutorestCI
Copy link
Contributor

[Monitor] Deconflict "Action" name in MetricAlert

Generates bad SDK.
@AutorestCI
Copy link
Contributor Author

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

Or build a wheel file to distribute for testing:

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

Direct download

Your files can be directly downloaded here:

@codecov-io
Copy link

codecov-io commented May 23, 2018

Codecov Report

Merging #2617 into restapi_auto_monitor/resource-manager will decrease coverage by 1.56%.
The diff coverage is 62.5%.

Impacted file tree graph

@@                            Coverage Diff                            @@
##           restapi_auto_monitor/resource-manager    #2617      +/-   ##
=========================================================================
- Coverage                                  56.04%   54.48%   -1.57%     
=========================================================================
  Files                                       6865     5474    -1391     
  Lines                                     156009   128412   -27597     
=========================================================================
- Hits                                       87442    69965   -17477     
+ Misses                                     68567    58447   -10120
Impacted Files Coverage Δ
...zure/mgmt/monitor/models/az_ns_action_group_py3.py 50% <ø> (ø) ⬆️
...mgmt/monitor/models/metric_alert_resource_patch.py 29.41% <ø> (ø) ⬆️
...nitor/azure/mgmt/monitor/models/alerting_action.py 45.45% <ø> (ø) ⬆️
.../monitor/models/metric_alert_resource_patch_py3.py 29.41% <ø> (ø) ⬆️
...azure/mgmt/monitor/models/metric_alert_resource.py 31.25% <ø> (ø) ⬆️
...e/mgmt/monitor/models/metric_alert_resource_py3.py 31.25% <ø> (ø) ⬆️
...e-mgmt-monitor/azure/mgmt/monitor/models/action.py 75% <ø> (+15%) ⬆️
...or/azure/mgmt/monitor/models/az_ns_action_group.py 50% <ø> (ø) ⬆️
...e-mgmt-monitor/azure/mgmt/monitor/models/source.py 50% <0%> (ø) ⬆️
...mt-monitor/azure/mgmt/monitor/models/action_py3.py 75% <100%> (+15%) ⬆️
... and 1819 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 e2bd0f4...b9a906b. Read the comment docs.

Update reference to MetricAlertAction
@lmazuel lmazuel merged commit 94bcfb2 into restapi_auto_monitor/resource-manager May 30, 2018
@lmazuel lmazuel deleted the restapi_auto_3128 branch May 30, 2018 18:24
AutorestCI added a commit that referenced this pull request Jun 6, 2018
…in MetricAlert (#2617)

* Generated from 35adc5e493c20a57e6e2cb35d87c578133d2b7a6

[Monitor] Deconflict "Action" name in MetricAlert

Generates bad SDK.

* Generated from ebfaf0bdd7f2bcf736598d8ec501f490a83d1268

Update reference to MetricAlertAction
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
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