-
Notifications
You must be signed in to change notification settings - Fork 5k
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
Copy the legacy API contracts to 2022 02 01 preview #19799
Merged
raych1
merged 257 commits into
Azure:dev-sql-Microsoft.Sql-2022-02-01-preview
from
ericshape:dev-sql-Microsoft.Sql-2022-02-01-preview
Jul 19, 2022
Merged
Copy the legacy API contracts to 2022 02 01 preview #19799
raych1
merged 257 commits into
Azure:dev-sql-Microsoft.Sql-2022-02-01-preview
from
ericshape:dev-sql-Microsoft.Sql-2022-02-01-preview
Jul 19, 2022
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* update access mode values * update examples * Add 202 in delete spec * Add provisioning issues changes * update schema * review changs * update * typo in model fix * make provisioning issues readonly * Add one example in list association, provisioning state enum update * Add logggin categories in profile * examples update * update * update examples * reformat code * Add x-msidentifier for logger * update * provisioning state fix * Fix provisioning state * Add location header in delete nsp association * update location header * update location header * location header update Co-authored-by: Kaushal Kumar <kumarkaushal@microsoft.com>
…sion stable/2022-03-10 (Azure#18458) * Adds base for updating Microsoft.HybridCompute from version preview/2021-12-10-preview to version 2022-03-10 * Updates readme * Updates API version in new specs and examples * updated the swagger * fixed typo * fixed examples * fixed swagger issues * removed columns * fixed the x-ms-identifier format * fixed the identifier again * fixed the readme tag * ran prettier * removed extra tag in the readme.md * Update HybridCompute.json updated serviceStatuses and agentConfiguration descriptions * removed resources * removed resources in the examples * addressed issues * ran prettier * A small fix to the newly added field's name * updated the az cli related files Co-authored-by: Justin Chen <justin99chen@gmail.com>
… preview/2021-12-01-preview (Azure#18297) * New Swagger Spec File * New Swagger Example Spec File * New Readme Config File * New Azure AZ Readme Config File * New Azure CLI Readme Config File * New Go Language Readme Config File * New Python Language Readme Config File * New Typescript Language Readme Config File * New C# Language Readme Config File * merging educationrp spec to public branch * Delete OperationGroupGet.json fix avocado error
…Azure#17382) * adds new version for credentials api * adding new version to ReadMe * tagging /credentials * tagging /credential * rearranging ReadMe so that default tag could be detected * rearranging ReadMe so that default tag could be detected * testing avacado * adding 2022-02-01 back to ReadMe * testing avacado readme checker * testing avacado readme checker * testing a new readme file * Revert "testing a new readme file" This reverts commit e5536c9. * testing AzureStorage discriminator * pr feedback * pr feedback * adds description for credential models * fixes typo
…es PUT (v1.0 only). (Azure#18104) Co-authored-by: Chang Liu <changliu3@microsoft.com>
…on.md files (Azure#18731) * Corrected the Tag: name * Update readme.python.md Co-authored-by: Yuchao Yan <yuchaoyan@microsoft.com>
* Promote 2022-05-01-preview api from RPSaaSMaster to main * Fixing lint errors
* Add action start/end time. * Update examples with action start/end times. * fix typo in description. Co-authored-by: kipartha <kipartha@microsoft.com>
* Add new apis for test button feature * Fix bug * Revert to old package version * Fix errors * Fix errors * Fix error * Remove " * Only include the actiongroups changes in this new tag as per Dapeng. * Revert "Only include the actiongroups changes in this new tag as per Dapeng." This reverts commit 529a149. * Add "x-ms-identifiers": [], * Update the default tag * Add location headers * No need to return response * build * build * format json * Prettier check Co-authored-by: Thomas Pham <thompham@microsoft.com>
* Adds base for updating Microsoft.Network from version stable/2021-05-01 to version 2021-08-01 * Updates readme * Updates API version in new specs and examples * Add AppGw swagger changes for L4 proxy (Azure#17561) * Add AppGw swagger changes for L4 proxy * Fix Lint Errors * fix prettier checks * HubRoutingPreference in VirtualHub (Azure#17609) * commit1 * commit2 Co-authored-by: Khushboo Baheti <khbaheti@microsoft.com> * MultipleApipa feature VpnSiteLinkConnection and VirtualNetworkGatewayConnection (Azure#17672) * VngConnection * VpnSiteLinkConnection * fixes * fixes * fix2 * fixes Co-authored-by: Khushboo Baheti <khbaheti@microsoft.com> * Virtual Wan P2S MultiPool feature swagger changes (Azure#17620) * Virtual Wan P2S MultiPool feature swagger changes * Fix Swagger LintDiff errors * Fix LintDiff errors * Fix errors * Fix spec * Fix spec * Fix spec * Fix LintDiff errors * Fix LintDiff errors * Fix SDK azure-sdk-for-net generation error * Remove suppression * Fix errors * Fix Lintdiff error * Fix PrettierCheck * changes (Azure#18002) * Revert "changes (Azure#18002)" (Azure#18014) This reverts commit 320ed6a. * Add nic auxiliary mode (Azure#17577) * Add nic auxiliary mode * fix spacing * Fixing prettier check * Restoring package-lock file * Restoring package json Co-authored-by: Prachi Bhavsar <prbhavsar@microsoft.com> * Connection Draining add new properties (Azure#18052) * merge * fix * fix * Adding express route port authorization apis (Azure#17582) * adding apis and updating resource to support ports auth * moving change to 2021-08-01 * minor: removing change from 2020-07-01 * lintdiff : adding type object * minor: fixing prettier * adding authorizations to ports property * fixing circuitResourceUri property name * fixing model validation * changing circuit resource uri type to string * removing authorizations child reosurce from parent property * Fix Azure Firewall Policy regressions. Back fix validation issues (Azure#18233) * Fix regressions in Firewall Policy Swagger / give firewallPolicy.json some love * Additional lint violations * remove breaking changes for next time. TO DO * Revert "remove breaking changes for next time. TO DO" This reverts commit 8f44a17. * Reintroduce api-version for idps signature based routes. Create better names for enums to be generated in SDKs * standardize enum names with FirewallPolicy prefix * Azure Firewall Support of Private IP Ranges in IDPS (Azure#18320) * Azure Firewall Support of Private IP Ranges in IDPS * make sure all arrays have x-ms-identifiers * FirewallPolicy not Firewall policy fix spellcheck validation * Ability to update tags on firewall policies (Azure#18322) * Support updating of Azure Firewall Policy Tags. Includes HTTP Patch example * Use common-types ErrorDetail * Ability to update tags for Firewall Policies * spell check fix for firewallpolicy * Use future release api-version for example * Added flush conn to nsg (Azure#18393) * Added flush conn to nsg * Updated flushConn to correct location * Updated description * Modified T/F to Enabled/Disabled * Refactoring so that null value appears first * Reverted FlushConnection to boolean value instead of string * Revert "Added flush conn to nsg (Azure#18393)" (Azure#18576) This reverts commit 6541d30. * Fixing typo in response of idps private ip range feature (Azure#18574) * Use common-type api version (Azure#18729) Co-authored-by: Ben Eshed <bewaters@microsoft.com> * fix (Azure#18417) Co-authored-by: Tianen <347142915@qq.com> Co-authored-by: gk-ms <97893166+gk-ms@users.noreply.github.com> Co-authored-by: Khushboo Baheti <37917868+Khushboo-Baheti@users.noreply.github.com> Co-authored-by: Khushboo Baheti <khbaheti@microsoft.com> Co-authored-by: Nilambari <nilamd@microsoft.com> Co-authored-by: nimaller <71352534+nimaller@users.noreply.github.com> Co-authored-by: pracsb <78512712+pr-work@users.noreply.github.com> Co-authored-by: Prachi Bhavsar <prbhavsar@microsoft.com> Co-authored-by: Matthew Yang <79727592+matyang22@users.noreply.github.com> Co-authored-by: utbarn-ms <66377251+utbarn-ms@users.noreply.github.com> Co-authored-by: Ben Eshed <thebenwaters@users.noreply.github.com> Co-authored-by: Satya-anshu <70507845+Satya-anshu@users.noreply.github.com> Co-authored-by: bewaterspassover <103988461+bewaterspassover@users.noreply.github.com> Co-authored-by: Ben Eshed <bewaters@microsoft.com>
* add openapi spec for scvmm * lint fixes, spell fixes * typo fix * prettier * force delete query param * removing some deprecated properties * remove additional props
* update doc * Update configure-python-sdk.md * Update configure-python-sdk.md * Update configure-python-sdk.md
…zure#18713) * add groupids in response * fix prettier issues * fix consistency problem with two swagger file Co-authored-by: Chenjie Shi <tadelesh.shi@live.cn>
Co-authored-by: Alex Wilcox <alex.wilcox@cloudsecure.ltd>
* Create 2022-05-01 * Delete 2022-05-01 * Create changes.json * Create GetChange.json * Create ListChanges.json * Update readme.md * Update changes.json * Update readme.md * Update readme.md * Update readme.md * Update readme.md * Update readme.md * Update readme.md
* Update readme.python.md * update * fix
* Copy existing FabricLocation * AzureStack Admin API - Fabric RP - FabricLocation - 2022-03-01 * Revert to original Fabric.json for common type
* swagger review initial commit * Swagger Changes for new API * Swagger Changes for new API * Swagger Changes for new API * Swagger Changes for new API * Swagger Changes for new API * fix validation issue * Swagger Changes for new API * Update readme.md changes to readme * manifest changes for new API
… preview/2019-10-11-preview (Azure#18482) * add x-ms-enum * Fix validation issues * Delete Notebooks spec Co-authored-by: Geoffrey Troyer <geoffreymtroyer@gmail.com> Co-authored-by: Kyle Cutler <kycutler@microsoft.com>
* Enable azure-powershell for all manage RPs * Change integrationRepository to Azure/azure-powershell-pr * Change azure-powershell in api-spec repo * Change azure-powershell in api-spec repo * Remove azure-powershell from some deprecated RPs Co-authored-by: v-yuzhichen@microsoft.com <v-yuzhichen@microsoft.com>
…ion stable/2021-01-01 (Azure#18603) * Update Microsoft.MixedReality swagger to fix quality issues in s360 * Fixing linter issue * Revert "Fixing linter issue" This reverts commit 9d3b03a. Co-authored-by: Ramón Argüelles-García <rgarcia@ntdev.microsoft.com>
Add Chenjie shi as assignee
* Add new Sql VA API * Update server level API * Update API attributes * Update controlers * Add missing examples * Fix examples * Format JSON file * Update examples * Updae scan record API * Add ref to README file * Update version * Update enums * Add SystemData * add systemData * sort readme files * Update examples * Update remove baseline API * Add 204 respond to delete API * Add 204 error * Fixed error * Update description * Update swagger * fix PR comments, seperated baseline operations to /baselines/default/rules/ruleid * fix validations errors Co-authored-by: Ahmad Abas <ahabas@microsoft.com>
…er requirements. (Azure#19240) Co-authored-by: Ahmad Abas <ahabas@microsoft.com>
…v-sql-Microsoft.Sql-2022-02-01-preview
Hi, @ericshape Thanks for your PR. I am workflow bot for review process. Here are some small tips. Any feedback about review process or workflow bot, pls contact swagger and tools team. vscswagger@microsoft.com |
Swagger Validation Report
|
Rule | Message |
---|---|
"details":"Attention: There are some existing APIs currently documented in a new spec file. The validation may not be able to report breaking changes with these APIs. It is recommended not to rename swagger file or move public APIs to a new file when creating a new API version.The existing APIs being moved are:ServerUsages_ListByServer;", "location":"https://github.com/Azure/azure-rest-api-specs/blob/dev-sql-Microsoft.Sql-2022-02-01-preview/specification/sql/resource-manager/Microsoft.Sql/preview/2022-02-01-preview/ServerUsages.json" |
️️✔️
LintDiff succeeded [Detail] [Expand]
Validation passes for LintDiff.
compared tags (via openapi-validator v1.13.0) | new version | base version |
---|---|---|
default | default(234d7a3) | default(dev-sql-Microsoft.Sql-2022-02-01-preview) |
️❌
Avocado: 5 Errors, 1 Warnings failed [Detail]
️️✔️
ModelValidation succeeded [Detail] [Expand]
Validation passes for ModelValidation.
️️✔️
SemanticValidation succeeded [Detail] [Expand]
Validation passes for SemanticValidation.
️️✔️
SDK Track2 Validation succeeded [Detail] [Expand]
Validation passes for SDKTrack2Validation
️️✔️
PrettierCheck succeeded [Detail] [Expand]
Validation passes for PrettierCheck.
️️✔️
SpellCheck succeeded [Detail] [Expand]
Validation passes for SpellCheck.
️️✔️
CadlValidation succeeded [Detail] [Expand]
Validation passes for CadlValidation.
Swagger Generation Artifacts
|
Hi @ericshape, Your PR has some issues. Please fix the CI sequentially by following the order of
|
This was referenced Jul 18, 2022
raych1
approved these changes
Jul 19, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
MSFT employees can try out our new experience at OpenAPI Hub - one location for using our validation tools and finding your workflow.
Copy the following legacy API contracts and example files
from
2014-04-01-stable
to
latest 2022-02-01-preview:
We did not modify any API contract. We just bumped up the version. So there is no breaking change.
The breaking Change(Cross-Version) pipeline has passed.
Request to ignore the Avacado validation because we will update V5 SDK tag after 2022-02-01-preview release.
Changelog
Add a changelog entry for this PR by answering the following questions:
Contribution checklist:
If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist
Otherwise your PR may be subject to ARM review requirements. Complete the following:
Check this box if any of the following appy to the PR so that the label "ARMReview" and "WaitForARMFeedback" will be added by bot to kick off ARM API Review. Missing to check this box in the following scenario may result in delays to the ARM manifest review and deployment.
-[ ] To review changes efficiently, ensure you are using OpenAPIHub to initialize the PR for adding a new version. More details, refer to the wiki.
Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.
If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Breaking Change Review Checklist
If any of the following scenarios apply to the PR, request approval from the Breaking Change Review Board as defined in the Breaking Change Policy.
Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.