[Cosmos DB] az cosmosdb create/update: Add support for minimum allowed TLS version and burst capacity configuration
#27322
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
AdrianSibajaRetana
requested review from
calvinhzy and
evelyn-ys
as code owners
🔄AzureCLI-FullTest
|
|
| rule | cmd_name | rule_message | suggest_message |
|---|---|---|---|
| cosmosdb create | cmd cosmosdb create added parameter enable_burst_capacity |
||
| cosmosdb create | cmd cosmosdb create added parameter minimal_tls_version |
||
| cosmosdb update | cmd cosmosdb update added parameter enable_burst_capacity |
||
| cosmosdb update | cmd cosmosdb update added parameter minimal_tls_version |
Collaborator
|
Thank you for your contribution! We will review the pull request and get back to you soon. |
microsoft-github-policy-service
bot
added
CosmosDB
AdrianSibajaRetana
changed the title
When using the key-uri parameter on an update, the non-CMK to CMK migration is triggered. This workflow is independent from the CMK provisioning workflow and is currently in preview.
rsharm7
reviewed
Sep 14, 2023
AdrianSibajaRetana
changed the title
AdrianSibajaRetana
changed the title
yonzhan
modified the milestones:
September 2023 (2023-09-26),
October 2023 (2023-11-15)❗ Breaking change release ❗
Contributor
Author
|
/azp run |
|
Commenter does not have sufficient privileges for PR 27322 in repo Azure/azure-cli |
…m/AdrianSibajaRetana/azure-cli into users/adriansib/CosmosDB-09152023
dani99vargas
requested review from
jsntcy,
kairu-ms,
necusjz and
wangzelin007
as code owners
Collaborator
|
Please fix CI issues. |
Contributor
Author
|
@yonzhan CI changes have been fixed. Can you please approve? |
Contributor
Author
|
@calvinhzy, @evelyn-ys, @jsntcy, @kairu-ms, @necusjz, and/or @wangzelin007. Can any of you folks review the changes as well please? |
Member
|
@evelyn-ys Can you review and merge this PR? Thanks. |
evelyn-ys
approved these changes
Oct 10, 2023
necusjz
approved these changes
Oct 10, 2023
evelyn-ys
changed the title
az cosmosdb create/update: Add support for minimum allowed TLS version and burst capacity configuration
evelyn-ys
pushed a commit
to Azure/azure-cli-extensions
that referenced
this pull request
Mar 20, 2024
…#7271) * add new sdk * update version * update sdk * cli changes for PRPP * variable name fix, added test case * Add mode in copy job (cherry picked from commit a28144517ad4022098561e83d8607c93933dd36c) * Add complete job API (cherry picked from commit 3d721cb) (cherry picked from commit f4e6f01f99eb1299e8c536423f074e9832b71e16) * Add test * history.rst changes * Add history * Introduce disable_ttl parameter * Syntax error fixes * Update tests * Fix syntax error * Change test location to WestCentralUS where 0201 batch1 is completed, add remaining asserts, rerecordings * Update history file * Changed cluster deallocate force input to string * add missing changes from stable Azure/azure-cli#26226 * add stable changes https://github.com/Azure/azure-cli/pull/27175/files * https://github.com/Azure/azure-cli/pull/27322/files * add missing stable changes from https://github.com/Azure/azure-cli/pull/26392/files * Revert "https://github.com/Azure/azure-cli/pull/27322/files" This reverts commit f779e16. * add missing stable changes for minimal tls version from Azure/azure-cli#27322 * nit * Revert "nit" This reverts commit d4f4abe. * Revert "add missing stable changes for minimal tls version from Azure/azure-cli#27322" This reverts commit be4709e. * fix style checl * Address multi character short flag issue * Add missing parameters * Fix syntax issue * Remove short flag * fix test * poc for test changes * inital changes in custom.py * removing recordings * changes in params as well as test and recording for sql ntbr flow * fixing imports * fixing test * fixing recordings * config name change from --enable-per-region-per-partition-autoscale to --enable-prpp-autoscale * re-record tests * nit style * Revert "fixing recordings" This reverts commit ea5c847. * Revert "removing recordings" This reverts commit 583371c. * fixing incorrect merge * Added test * 6 out of 8 test failures * fixing remaining 2 tests * fixing style issues * Update HISTORY.rst to reflect changes ported from stable CLI * re-record cassandra tests * Update version to 1.0.0 due to breaking changes * azdev style fixes * azdev style fixes * azdev style fixes * azdev style fixes * azdev style fixes * azdev style fixes * Update HISTORY.rst version to 1.0.0 --------- Co-authored-by: Kushal Warke <[email protected]> Co-authored-by: Nitesh Vijay <[email protected]> Co-authored-by: Nitesh Vijay <[email protected]> Co-authored-by: Min Ho Kang <[email protected]> Co-authored-by: drewkaufmann <[email protected]> Co-authored-by: Vivek Agarwal (from Dev Box) <[email protected]> Co-authored-by: AgarwalVivek <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
7 participants
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.
Related command
az cosmosdb create
Description
Removed preview parameters and added test for new property.
Testing Guide
Added test for CMK status.