Skip to content

[MonitorMgmt] Status code fixes#1353

Merged
lmazuel merged 2 commits intoAzure:masterfrom
AutorestCI:RestAPI-PR1481
Aug 9, 2017
Merged

[MonitorMgmt] Status code fixes#1353
lmazuel merged 2 commits intoAzure:masterfrom
AutorestCI:RestAPI-PR1481

Conversation

@AutorestCI
Copy link
Copy Markdown
Contributor

Generated from RestAPI PR: Azure/azure-rest-api-specs#1481

@codecov-io
Copy link
Copy Markdown

codecov-io commented Aug 8, 2017

Codecov Report

Merging #1353 into master will decrease coverage by <.01%.
The diff coverage is 91.66%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1353      +/-   ##
==========================================
- Coverage   56.13%   56.13%   -0.01%     
==========================================
  Files        2720     2720              
  Lines       71615    71617       +2     
==========================================
  Hits        40201    40201              
- Misses      31414    31416       +2
Impacted Files Coverage Δ
...onitor/models/activity_log_alert_resource_paged.py 80% <ø> (ø) ⬆️
...onitor/azure/mgmt/monitor/models/incident_paged.py 80% <ø> (ø) ⬆️
...e/mgmt/monitor/models/alert_rule_resource_paged.py 100% <ø> (ø) ⬆️
...onitor/azure/mgmt/monitor/models/email_receiver.py 62.5% <ø> (ø) ⬆️
...-monitor/azure/mgmt/monitor/models/sms_receiver.py 55.55% <ø> (ø) ⬆️
...mgmt/monitor/models/action_group_resource_paged.py 80% <ø> (ø) ⬆️
.../mgmt/monitor/models/log_profile_resource_paged.py 80% <ø> (ø) ⬆️
...monitor/models/autoscale_setting_resource_paged.py 80% <ø> (ø) ⬆️
...itor/operations/alert_rule_incidents_operations.py 26% <100%> (ø) ⬆️
...erations/service_diagnostic_settings_operations.py 20.89% <100%> (ø) ⬆️
... and 8 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 065d27f...1d9c763. Read the comment docs.

@lmazuel lmazuel changed the title Automatic PR from RestAPI-PR1481 [MonitorMgmt] Status code fixes Aug 9, 2017
@lmazuel
Copy link
Copy Markdown
Member

lmazuel commented Aug 9, 2017

@vishrutshah LGTM to me, looking at the Python code there is just a few HTTP status code check different that seems reasonable.

@lmazuel lmazuel merged commit bde32c3 into Azure:master Aug 9, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants