Skip to content

Resources: fix api-version from method property to client property#7753

Merged
ChenTanyi merged 1 commit intomasterfrom
fix-resources
Nov 21, 2019
Merged

Resources: fix api-version from method property to client property#7753
ChenTanyi merged 1 commit intomasterfrom
fix-resources

Conversation

@xseeseesee
Copy link
Contributor

Latest improvements:

MSFT employees can try out our new experience at OpenAPI Hub - one location for using our validation tools and finding your workflow.

Contribution checklist:

  • I have reviewed the documentation for the workflow.
  • Validation tools were run on swagger spec(s) and have all been fixed in this PR.
  • The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

  • Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
  • adding/removing APIs.
  • adding/removing properties.
  • adding/removing API-version.
  • adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

  • If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
    Please follow the link to find more details on API review process.

@AutorestCI
Copy link

AutorestCI commented Nov 11, 2019

Automation for azure-sdk-for-python

A PR has been created for you:
Azure/azure-sdk-for-python#8776

@AutorestCI
Copy link

AutorestCI commented Nov 11, 2019

Automation for azure-sdk-for-go

Nothing to generate for azure-sdk-for-go

@ChenTanyi ChenTanyi requested a review from tfitzmac November 11, 2019 03:40
@ChenTanyi ChenTanyi added the WaitForARMFeedback <valid label in PR review process> add this label when ARM review is required label Nov 11, 2019
@azuresdkci
Copy link
Contributor

Can one of the admins verify this patch?

@qiaozha
Copy link
Member

qiaozha commented Nov 11, 2019

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@ravbhatnagar ravbhatnagar added ARMSignedOff <valid label in PR review process>add this label when ARM approve updates after review and removed WaitForARMFeedback <valid label in PR review process> add this label when ARM review is required labels Nov 13, 2019
@xseeseesee
Copy link
Contributor Author

@anthony-c-martin I noticed you are reviewer of ARM 2019-10-01. Can you please help verify if my changes make sense? If so, please also advise whether these changes should apply to package 2019-10-01. Thanks.

@ChenTanyi ChenTanyi merged commit 1551635 into master Nov 21, 2019
TalluriAnusha pushed a commit to AsrOneSdk/azure-rest-api-specs that referenced this pull request Dec 11, 2019
Grayer123 added a commit to Grayer123/azure-rest-api-specs that referenced this pull request Jan 17, 2020
ArcturusZhang pushed a commit that referenced this pull request Jan 22, 2020
…y" for Resources within 2019-08-01 and 2019-10-01 (#8184)

* Added previous version spec as initial commit.

* Modified all the api-version for the existing example files.

* Added tag related api spec to resource file as well add example files.

* Modified readme related files.

* Execute prettier to reorganize the file.

* Pull request comments related fixes.

* Use scope to support nested resources for tags.

* Update examples based on scope change.

* Remove first slash to resolve model validation error.

* Combine tracked resource and subscription apis for tags.

* Modifed the examples to match the changes in resource.json.

* Utilize the shared parameters, and update the examples.

* Removing "required" from TagPatchRequest as getting R2016 error.

* Collapse TagRequest and TagResponse into TagsResource.

* Removed 404 response as covered in default scenario, as well updated corresponding examples.

* Apply fix api-version from method property to client property for 2019-10-01.

* Revert "Resources: fix api-version from method property to client property (#7753)"

This reverts commit 1551635.

* Revert changes "fix api-version from method property to client property" for resources.
@xseeseesee xseeseesee deleted the fix-resources branch August 24, 2020 05:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ARMSignedOff <valid label in PR review process>add this label when ARM approve updates after review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants