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 web/resource-manager] [Hub Generated] Review request for Microsoft.Web to add version 2018-02-01 #4436

Merged
merged 4 commits into from
Mar 4, 2019

Conversation

AutorestCI
Copy link
Contributor

@AutorestCI
Copy link
Contributor Author

AutorestCI commented Feb 27, 2019

(message created by the CI based on PR content)

Installation instruction

Package azure-mgmt-web

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

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_5137#egg=azure-mgmt-web&subdirectory=azure-mgmt-web"

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

  • git checkout restapi_auto_5137
  • pip install -e ./azure-mgmt-web

Or build a wheel file to distribute for testing:

  • git checkout restapi_auto_5137
  • pip wheel --no-deps ./azure-mgmt-web

Direct download

Your files can be directly downloaded here:

@codecov-io
Copy link

codecov-io commented Feb 27, 2019

Codecov Report

Merging #4436 into restapi_auto_web/resource-manager will increase coverage by 3.83%.
The diff coverage is 40.74%.

Impacted file tree graph

@@                          Coverage Diff                          @@
##           restapi_auto_web/resource-manager    #4436      +/-   ##
=====================================================================
+ Coverage                              49.53%   53.37%   +3.83%     
=====================================================================
  Files                                   6227    10466    +4239     
  Lines                                 174600   219577   +44977     
=====================================================================
+ Hits                                   86491   117198   +30707     
- Misses                                 88109   102379   +14270
Impacted Files Coverage Δ
.../operations/app_service_environments_operations.py 10.54% <10.52%> (-0.01%) ⬇️
azure-mgmt-web/azure/mgmt/web/models/__init__.py 100% <100%> (+44.13%) ⬆️
...t-web/azure/mgmt/web/models/endpoint_detail_py3.py 44.44% <44.44%> (ø)
...-mgmt-web/azure/mgmt/web/models/endpoint_detail.py 44.44% <44.44%> (ø)
...gmt/web/models/inbound_environment_endpoint_py3.py 50% <50%> (ø)
...re/mgmt/web/models/inbound_environment_endpoint.py 50% <50%> (ø)
...t-web/azure/mgmt/web/models/endpoint_dependency.py 57.14% <57.14%> (ø)
...e/mgmt/web/models/outbound_environment_endpoint.py 57.14% <57.14%> (ø)
...b/azure/mgmt/web/models/endpoint_dependency_py3.py 57.14% <57.14%> (ø)
...mt/web/models/outbound_environment_endpoint_py3.py 57.14% <57.14%> (ø)
... and 4355 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 60727bd...eb084e3. Read the comment docs.

@AutorestCI AutorestCI merged commit 11726bc into restapi_auto_web/resource-manager Mar 4, 2019
@AutorestCI AutorestCI deleted the restapi_auto_5137 branch March 4, 2019 22:27
@AutorestCI
Copy link
Contributor Author

This PR has been merged into #4429

lmazuel pushed a commit that referenced this pull request May 24, 2019
* Generated from b358cc2d9448ec292d6d67de69f587080731c85e (#4428)

Add examples of 201 response to AppServicePlans_CreateOrUpdate.

* Generated from c2d7ae74564edf027d2baa5d39864ba26a4f593a (#4448)

Update UserAssignedIdentities spec in Web

Microsoft.Web recently added support for UserAssigned managed service identities (in preview), but the swagger spec is incorrect for the identity payload as it follows the old construct of sending an array of identityIds as opposed to the updated spec which instead sends a dictionary object called userAssignedIdentities. 

Spec + history outlined here:
http://sharepoint/sites/AzureUX/Sparta/_layouts/15/WopiFrame2.aspx?sourcedoc=%7b633632B4-02E4-4DE2-92A9-EDCAF9BE73D4%7d&file=ARM%2BMSI.docx&action=default 

Our swagger contains identityids[] because of previous work to future support UserAssigned identities, but never released the feature in that state, rather we opted to wait for the new spec to be supported and then integrated with that on our end. 

Our MSI implementation reflects Microsoft.Compute's spec for user assigned.

https://github.com/Azure/azure-rest-api-specs/blob/78e1feab142c605839aa3cec23442ca26b8ef04d/specification/compute/resource-manager/Microsoft.Compute/stable/2018-06-01/compute.json

* Packaging update of azure-mgmt-web

* [AutoPR web/resource-manager] [Hub Generated] Review request for Microsoft.Web to add version 2018-02-01 (#4436)

* Generated from b772c09b4e8132be83594f0a834cf7b363445a3c

Address PR description feedback

* Generated from 61d39f048374e3c8dd07112e7d9e8554e8664ea5

Fix casing

* Packaging update of azure-mgmt-web

* Generated from 66114e58530dd7027b63182defcf1c8f73d1d502

Remove incorrect pagination from examples.

* Generated from 13b235cccbe934e60dba491a2a49edcae9c7f275 (#4481)

Fix Hosting Environment Network Dependencies pagination

* [AutoPR web/resource-manager] Update managedserviceidentity type enum (#4761)

* Generated from d605882bdcb7e402badbf4aeca8d66f20a9d197e

Update managedserviceidentity type enum

Enum is currently missing the ua + sa combination + "none" type. We do support those values in our API and as such need the swagger to reflect that.

* Packaging update of azure-mgmt-web

* Generated from 215b006997fc053d43337cd0455d8bdfdbe18898 (#4768)

Add Managed Identity envelope to SitePatchResource

Add Managed Identity envelope to SitePatchResource object to allow PATCH identity operations to a site through Swagger

* [AutoPR web/resource-manager] VnetInfo CertBlob format changed to string (#4860)

* Generated from d0b3b0dc2238ba14b6a41ced8e7ce66e3755f279

VnetInfo CertBlob format changed to string

* Packaging update of azure-mgmt-web

* Generated from 310b5ada2616fb0b8f5808d6d1422df350d459e6 (#5169)

Add examples

* Generated from ca964030410907fee5bc6c2e08d6e5dd934e49a3 (#5190)

Add Source Web App Location

* Generated from 96295fc95567dcd52215bafad85d53dc931adc32 (#5212)

Add Api-version 2018-11-01 for WebApps Collection

Add package Tag

Fix path to common definitions

* Web in new folder

* Generated from ac4cdaf9da165cd1951ce8e10cdd63c78ff04612 (#5472)

Web new path

* ChangeLog
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