azure-cli-extensions icon indicating copy to clipboard operation
azure-cli-extensions copied to clipboard

[cosmosdb-preview] Add support for new api version 2024-02-15-preview

Open carjackson-msft opened this issue 1 year ago • 15 comments


This checklist is used to make sure that common guidelines for a pull request are followed.

Related command

General Guidelines

  • [x] Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
  • [x] Have you run python scripts/ci/test_index.py -q locally? (pip install wheel==0.30.0 required)
  • [x] My extension version conforms to the Extension version schema

For new extensions:

About Extension Publish

There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

carjackson-msft avatar Feb 08 '24 00:02 carjackson-msft

❌Azure CLI Extensions Breaking Change Test
❌cosmosdb-preview
rule cmd_name rule_message suggest_message
1007 - ParaRemove cosmosdb create cmd cosmosdb create removed parameter enable_public_network please add back parameter enable_public_network for cmd cosmosdb create
1007 - ParaRemove cosmosdb restore cmd cosmosdb restore removed parameter enable_public_network please add back parameter enable_public_network for cmd cosmosdb restore
1007 - ParaRemove cosmosdb update cmd cosmosdb update removed parameter enable_public_network please add back parameter enable_public_network for cmd cosmosdb update
⚠️ 1001 - CmdAdd cosmosdb copy complete cmd cosmosdb copy complete added
⚠️ 1006 - ParaAdd cosmosdb copy create cmd cosmosdb copy create added parameter mode
⚠️ 1006 - ParaAdd cosmosdb create cmd cosmosdb create added parameter enable_partition_merge
⚠️ 1006 - ParaAdd cosmosdb create cmd cosmosdb create added parameter enable_prpp_autoscale
⚠️ 1006 - ParaAdd cosmosdb create cmd cosmosdb create added parameter public_network_access
⚠️ 1006 - ParaAdd cosmosdb gremlin database restore cmd cosmosdb gremlin database restore added parameter disable_ttl
⚠️ 1009 - ParaPropRemove cosmosdb gremlin database restore cmd cosmosdb gremlin database restore update parameter restore_timestamp: removed property required=True
⚠️ 1006 - ParaAdd cosmosdb gremlin graph restore cmd cosmosdb gremlin graph restore added parameter disable_ttl
⚠️ 1009 - ParaPropRemove cosmosdb gremlin graph restore cmd cosmosdb gremlin graph restore update parameter restore_timestamp: removed property required=True
⚠️ 1006 - ParaAdd cosmosdb mongodb collection restore cmd cosmosdb mongodb collection restore added parameter disable_ttl
⚠️ 1009 - ParaPropRemove cosmosdb mongodb collection restore cmd cosmosdb mongodb collection restore update parameter restore_timestamp: removed property required=True
⚠️ 1006 - ParaAdd cosmosdb mongodb database restore cmd cosmosdb mongodb database restore added parameter disable_ttl
⚠️ 1009 - ParaPropRemove cosmosdb mongodb database restore cmd cosmosdb mongodb database restore update parameter restore_timestamp: removed property required=True
⚠️ 1006 - ParaAdd cosmosdb restore cmd cosmosdb restore added parameter disable_ttl
⚠️ 1006 - ParaAdd cosmosdb restore cmd cosmosdb restore added parameter public_network_access
⚠️ 1006 - ParaAdd cosmosdb sql container restore cmd cosmosdb sql container restore added parameter disable_ttl
⚠️ 1009 - ParaPropRemove cosmosdb sql container restore cmd cosmosdb sql container restore update parameter restore_timestamp: removed property required=True
⚠️ 1006 - ParaAdd cosmosdb sql database restore cmd cosmosdb sql database restore added parameter disable_ttl
⚠️ 1009 - ParaPropRemove cosmosdb sql database restore cmd cosmosdb sql database restore update parameter restore_timestamp: removed property required=True
⚠️ 1006 - ParaAdd cosmosdb table restore cmd cosmosdb table restore added parameter disable_ttl
⚠️ 1009 - ParaPropRemove cosmosdb table restore cmd cosmosdb table restore update parameter restore_timestamp: removed property required=True
⚠️ 1006 - ParaAdd cosmosdb update cmd cosmosdb update added parameter enable_partition_merge
⚠️ 1006 - ParaAdd cosmosdb update cmd cosmosdb update added parameter enable_prpp_autoscale
⚠️ 1006 - ParaAdd cosmosdb update cmd cosmosdb update added parameter public_network_access
⚠️ 1008 - ParaPropAdd managed-cassandra cluster deallocate cmd managed-cassandra cluster deallocate update parameter force: added property default=false

Hi @carjackson-msft, Please write the description of changes which can be perceived by customers into HISTORY.rst. If you want to release a new extension version, please update the version in setup.py as well.

cosmosdb-preview

yonzhan avatar Feb 08 '24 00:02 yonzhan

Please fix CI issues.

yonzhan avatar Mar 01 '24 00:03 yonzhan

Please fix CI issues

yonzhan avatar Mar 07 '24 23:03 yonzhan

/azp run

ghost avatar Mar 08 '24 00:03 ghost

Commenter does not have sufficient privileges for PR 7271 in repo Azure/azure-cli-extensions

azure-pipelines[bot] avatar Mar 08 '24 00:03 azure-pipelines[bot]

Hi @yonzhan, can you please help in triggering CLI checks to run? Seems I do not have sufficient privileges.

ghost avatar Mar 08 '24 00:03 ghost

/azp run

yonzhan avatar Mar 08 '24 00:03 yonzhan

Azure Pipelines successfully started running 2 pipeline(s).

azure-pipelines[bot] avatar Mar 08 '24 00:03 azure-pipelines[bot]

Please fix CI failure on style check

evelyn-ys avatar Mar 08 '24 01:03 evelyn-ys

Please fix CI failure on style check

Hi @evelyn-ys, is the style check required? According to the pipeline "This task does not block the PR merge." I also ran the azdev style check locally and don't see any errors:

image

I took a look at the errors in the pipeline, they seem to be for existing tests/changes. Given this is not a required check, can we disregard?

ghost avatar Mar 08 '24 18:03 ghost

Pls upgrade your local azdev to latest version and run azdev style again. Or you can check the failure details in CI logs

evelyn-ys avatar Mar 12 '24 05:03 evelyn-ys

@evelyn-ys style has been fixed.

ghost avatar Mar 12 '24 18:03 ghost

Hi @evelyn-ys, this has been fixed. Can we please merge this?

ghost avatar Mar 14 '24 16:03 ghost

[Release] Update index.json for extension [ cosmosdb-preview ] : https://dev.azure.com/azclitools/internal/_build/results?buildId=140834&view=results

azclibot avatar Mar 20 '24 02:03 azclibot