Api Management 2025-03-01-preview to new structure #38751
Closed
Conversation
This file contains hidden or 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
Co-authored-by: catalinaperalta <caperal@microsoft.com>
Co-authored-by: catalinaperalta <caperal@microsoft.com>
* Change logging functions to use Write-Host Change logging helpers to always write to host and either set color or use devops/gh formatting. We do not want to use Write-Error or Write-Warning directly because they can stop the script or not depending on preferences which makes it difficult to ensure local runs of scripts work the same as in the pipelines. So, we should never depend on these logging commands to cause a script to stop execution. * Clean up error handling in CommandInvocation-Helpers Removed legacy error handling for command failures. * Fix error logging for command execution * Add option to skip exiting --------- Co-authored-by: Wes Haggard <weshaggard@users.noreply.github.com> Co-authored-by: Wes Haggard <Wes.Haggard@microsoft.com>
* Enable dataPlane for azure-sdk-for-java and azure-sdk-for-js * Change isSpecGenSdkCheckRequired to true * Fix expected result for azure-sdk-for-js test Update expected result for JS SDK setting check.
* added 2025-11-15preview, removed 2025-05-15preview and 2024-11-15preview * added new types to pii categories * added docorator for 20251115preview and compile * updated the examples for 2025-11-15preview * added 2025-11-15-preview in readme * added IBAN to scpell.yaml * removed all older preview * fix typespec validation * update enum names * fix spellcheck * fix typespec validation * updated readme * fix typo --------- Co-authored-by: Bidisha Chakraborty <29803402+bidisha-c@users.noreply.github.com>
Co-authored-by: Summer Warren <summerwarren@microsoft.com>
Co-authored-by: Summer Warren <summerwarren@microsoft.com>
…ure-sdk-tools repo are unnecessary - Unnecessary checkout hurts perf and increases security footprint (#38362) Co-authored-by: Mike Harder <mharder@microsoft.com>
Co-authored-by: catalinaperalta <caperal@microsoft.com>
* mitigate breakings * Update client.tsp * Update client.tsp
* refactor the folder structure * gix js sdk validation ci error --------- Co-authored-by: Qiaoqiao Zhang <55688292+qiaozha@users.noreply.github.com>
…#37112) * refactor the folder structure * fix semantic error * fix validation error --------- Co-authored-by: Qiaoqiao Zhang <55688292+qiaozha@users.noreply.github.com>
* Refactor migrateprojects folder structure to comply with Azure Guidelines * Removed files readme.go.md, and go swagger-to-sdk section in readme.md
…elines (#37652) Co-authored-by: Qiaoqiao Zhang <55688292+qiaozha@users.noreply.github.com>
#37653) Co-authored-by: Qiaoqiao Zhang <55688292+qiaozha@users.noreply.github.com>
* Refactor maps folder structure to comply with Azure Guidelines * Resolve conflicts
Co-authored-by: Yuchao Yan <yuchaoyan@microsoft.com> Co-authored-by: Qiaoqiao Zhang <55688292+qiaozha@users.noreply.github.com>
* refactor the folder structure * fix go sdk ci error * remove invalid info --------- Co-authored-by: Qiaoqiao Zhang <55688292+qiaozha@users.noreply.github.com>
* Refactor redhatopenshift folder structure to comply with Azure Guidelines * Rename folder openshiftclusters to OpenShiftClusters (case change) * Update suppressions.yaml
…37505) * refactor the folder structure * refactor the folder structure * remove data plane --------- Co-authored-by: Qiaoqiao Zhang <55688292+qiaozha@users.noreply.github.com>
* refactor the folder structure * resolve conflict --------- Co-authored-by: Qiaoqiao Zhang <55688292+qiaozha@users.noreply.github.com>
* Allow forced creation when release plans already exist * Update creation instructions * fix small markdown issues * Update creation steps --------- Co-authored-by: Summer Warren <summerwarren@microsoft.com>
Copied the files in a separate commit. This allows reviewers to easily diff subsequent changes against the previous spec.
Updated the API version from preview/2024-10-01-preview to preview/2025-03-01-preview.
…SSL certificate (#37047) * hostname refresh * fix prettier error * fix uuid * revert breaking change
…s for load balancer (#36818) * feat(apim): Make url/protocol optional in Backend and provide examples for load balancer * Extend example Signed-off-by: Tom Kerkhove <kerkhove.tom@gmail.com> * Remove empty required section Signed-off-by: Tom Kerkhove <kerkhove.tom@gmail.com> * Fix samples Signed-off-by: Tom Kerkhove <kerkhove.tom@gmail.com> * Update description Signed-off-by: Tom Kerkhove <kerkhove.tom@gmail.com> * Add simple LB example --------- Signed-off-by: Tom Kerkhove <kerkhove.tom@gmail.com>
* Add hostname bindings endpoints * fixes * Rename keyVaultUrl to keyVault * update * Remove optiona * update * Add pattern * Update pattern
…d resource (#36737) * feat(apim): Introduce azureRegion in Backend resource Signed-off-by: Tom Kerkhove <kerkhove.tom@gmail.com> * feat(apim): Introduce preferredCarbonEmission in Backend resource Signed-off-by: Tom Kerkhove <kerkhove.tom@gmail.com> * feat(apim): Introduce preferredCarbonEmission in Backend resource Signed-off-by: Tom Kerkhove <kerkhove.tom@gmail.com> * Fix misspelling Signed-off-by: Tom Kerkhove <kerkhove.tom@gmail.com> * Fix sample Signed-off-by: Tom Kerkhove <kerkhove.tom@gmail.com> * Fix sample Signed-off-by: Tom Kerkhove <kerkhove.tom@gmail.com> * Prettier Signed-off-by: Tom Kerkhove <kerkhove.tom@gmail.com> * Fix sample Signed-off-by: Tom Kerkhove <kerkhove.tom@gmail.com> * Fix invalid sample Signed-off-by: Tom Kerkhove <kerkhove.tom@gmail.com> * Fix invalid sample Signed-off-by: Tom Kerkhove <kerkhove.tom@gmail.com> * Url and protocol are no longer required Signed-off-by: Tom Kerkhove <kerkhove.tom@gmail.com> * Revert "Url and protocol are no longer required" This reverts commit 8b3b7d4. * Fix LB sample Signed-off-by: Tom Kerkhove <kerkhove.tom@gmail.com> * preferredCarbonEmission is not read-only Signed-off-by: Tom Kerkhove <kerkhove.tom@gmail.com> --------- Signed-off-by: Tom Kerkhove <kerkhove.tom@gmail.com>
Signed-off-by: Tom Kerkhove <kerkhove.tom@gmail.com>
* fix the modifications * add examples * run prettier * md 1 * corerct exampels
* Added APIM spec changes for mcp tools api * Fix for ToolContract definition not found * Fix prettier
* New ZoneRedundant Property * Revert other change in apimdeployment * Change to Enum instead of bool * Change description * Remove the default value
* Change ZoneRedundant property to nullable bool * add nullable true * prettier * Remove nullable
Signed-off-by: Tom Kerkhove <kerkhove.tom@gmail.com>
…refresh secret support for CM and added examples (#37231) * added definitions for key vault and refresh secret support for CM and added examples * fixed apimauthorizationproviders.json * fixed prettier issues * fixed model validation issues * fix kv contract ref * fix remaining model validation issue * fixing lint diff errors * addressed PR comments * fixed remaining v6 reference * reverted changes to authorization code grant type with key vaults and added keyvault to grant type * fixed to move key vault out of grant types into oauth2settings --------- Co-authored-by: Ji Hoon Kim <kimjihoon@microsoft.com>
solankisamir
requested review from
a team,
benbp,
live1206,
maririos,
mikeharder,
praveenkuttappan,
pshao25,
raych1 and
weshaggard
as code owners
Next Steps to Merge⌛ Please wait. Next steps to merge this PR are being evaluated by automation. ⌛Comment generated by summarize-checks workflow run. |
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.
ARM (Control Plane) API Specification Update Pull Request
Tip
Overwhelmed by all this guidance? See the
Getting helpsection at the bottom of this PR description.PR review workflow diagram
Please understand this diagram before proceeding. It explains how to get your PR approved & merged.
Purpose of this PR
What's the purpose of this PR? Check the specific option that applies. This is mandatory!
Due diligence checklist
To merge this PR, you must go through the following checklist and confirm you understood
and followed the instructions by checking all the boxes:
ARM resource provider contract and
REST guidelines (estimated time: 4 hours).
I understand this is required before I can proceed to the diagram Step 2, "ARM API changes review", for this PR.
Additional information
Viewing API changes
For convenient view of the API changes made by this PR, refer to the URLs provided in the table
in the
Generated ApiViewcomment added to this PR. You can use ApiView to show API versions diff.Suppressing failures
If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the
suppressions guide to get approval.
Getting help
Purpose of this PRandDue diligence checklist.write accessper aka.ms/azsdk/access#request-access-to-rest-api-or-sdk-repositoriesNext Steps to Mergecomment. It will appear within few minutes of submitting this PR and will continue to be up-to-date with current PR state.and https://aka.ms/ci-fix.
queuedstate, please add a comment with contents/azp run.This should result in a new comment denoting a
PR validation pipelinehas started and the checks should be updated after few minutes.