Skip to content
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

[k8s-extension] Update extension CLI to v1.4.3 #6630

Open
wants to merge 104 commits into
base: main
Choose a base branch
from

Conversation

bavneetsingh16
Copy link
Contributor


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

Related command

General Guidelines

  • Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
  • Have you run python scripts/ci/test_index.py -q locally?

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.

Miraj50 and others added 30 commits September 29, 2022 13:17
* First draft for Data Protection K8s backup extension (Pending internal review)

* Removing tracing

* Minor changes to improve azdev style

* Internal PR review feedback

Co-authored-by: Rishabh Raj <[email protected]>
…ad of workspace (#175)

* fix: Update DCR creation to Clusters resource group instead of workspace

* .

* .

* casing check
adding the api version to the operation definition in the client factory
…tests

adding tests for all 4 extension types calls
…tensiontypetests

adding upstream test for extension types
…eate (#188)

* Add more validations and user prompt for existing installation scenario

Signed-off-by: Shubham Sharma <[email protected]>

* Add Dapr test'

Signed-off-by: Shubham Sharma <[email protected]>

* Handle stateful set

Signed-off-by: Shubham Sharma <[email protected]>

* Update default handling

Signed-off-by: Shubham Sharma <[email protected]>

* Fix HA handling

Signed-off-by: Shubham Sharma <[email protected]>

* Add placement service todo

Signed-off-by: Shubham Sharma <[email protected]>

* Add non-interactive mode

Signed-off-by: Shubham Sharma <[email protected]>

* Fix lint

Signed-off-by: Shubham Sharma <[email protected]>

* Update tests

Signed-off-by: Shubham Sharma <[email protected]>

* Reset configuration for StatefulSet during k8s upgrade

Signed-off-by: Shubham Sharma <[email protected]>

* Fix lint

Signed-off-by: Shubham Sharma <[email protected]>

* Retrigger tests

Signed-off-by: Shubham Sharma <[email protected]>

* Add changes to manage ha and placement params

Signed-off-by: Shubham Sharma <[email protected]>

* Update message

Signed-off-by: Shubham Sharma <[email protected]>

* nits

Signed-off-by: Shubham Sharma <[email protected]>

Signed-off-by: Shubham Sharma <[email protected]>
@azure-client-tools-bot-prd
Copy link

azure-client-tools-bot-prd bot commented Aug 11, 2023

❌Azure CLI Extensions Breaking Change Test
❌k8s-extension
rule cmd_name rule_message suggest_message
1002 - CmdRemove k8s-extension extension-types list cmd k8s-extension extension-types list removed please confirm cmd k8s-extension extension-types list removed
1002 - CmdRemove k8s-extension extension-types list-versions cmd k8s-extension extension-types list-versions removed please confirm cmd k8s-extension extension-types list-versions removed
1002 - CmdRemove k8s-extension extension-types show cmd k8s-extension extension-types show removed please confirm cmd k8s-extension extension-types show removed
⚠️ 1001 - CmdAdd k8s-extension extension-types list-by-cluster cmd k8s-extension extension-types list-by-cluster added
⚠️ 1006 - ParaAdd k8s-extension extension-types list-by-location cmd k8s-extension extension-types list-by-location added parameter cluster_type
⚠️ 1006 - ParaAdd k8s-extension extension-types list-by-location cmd k8s-extension extension-types list-by-location added parameter plan_name
⚠️ 1006 - ParaAdd k8s-extension extension-types list-by-location cmd k8s-extension extension-types list-by-location added parameter plan_product
⚠️ 1006 - ParaAdd k8s-extension extension-types list-by-location cmd k8s-extension extension-types list-by-location added parameter plan_publisher
⚠️ 1006 - ParaAdd k8s-extension extension-types list-by-location cmd k8s-extension extension-types list-by-location added parameter release_train
⚠️ 1001 - CmdAdd k8s-extension extension-types list-versions-by-cluster cmd k8s-extension extension-types list-versions-by-cluster added
⚠️ 1001 - CmdAdd k8s-extension extension-types list-versions-by-location cmd k8s-extension extension-types list-versions-by-location added
⚠️ 1001 - CmdAdd k8s-extension extension-types show-by-cluster cmd k8s-extension extension-types show-by-cluster added
⚠️ 1001 - CmdAdd k8s-extension extension-types show-by-location cmd k8s-extension extension-types show-by-location added
⚠️ 1001 - CmdAdd k8s-extension extension-types show-version-by-cluster cmd k8s-extension extension-types show-version-by-cluster added
⚠️ 1001 - CmdAdd k8s-extension extension-types show-version-by-location cmd k8s-extension extension-types show-version-by-location added

@azure-client-tools-bot-prd
Copy link

Hi @bavneetsingh16,
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.

@yonzhan
Copy link
Collaborator

yonzhan commented Aug 11, 2023

k8s-extension

@zhoxing-ms
Copy link
Contributor

❌ 1002 - CmdRemove | k8s-extension extension-types list | cmd k8s-extension extension-types list removed | please confirm cmd k8s-extension extension-types list removed
-- | -- | -- | --
❌ 1002 - CmdRemove | k8s-extension extension-types list-versions | cmd k8s-extension extension-types list-versions removed | please confirm cmd k8s-extension extension-types list-versions removed
❌ 1002 - CmdRemove | k8s-extension extension-types show | cmd k8s-extension extension-types show removed | please confirm cmd k8s-extension extension-types show removed

Please be do not release breaking changes in the non breaking change Sprint. Please deprecate and hide them first. For more details about the breaking process, please refer to this doc breaking change process

@zhoxing-ms
Copy link
Contributor

May I ask why you closed the original PR #6575 and opened this new PR? What are the differences between them?

@bavneetsingh16
Copy link
Contributor Author

May I ask why you closed the original PR #6575 and opened this new PR? What are the differences between them?

Since the original pr was not yet merged, we had some additional changes that needed to be released. The pr links for the new changes are:

Add old commands back with deprecated status
Comment on lines +117 to +128
c.argument('plan_name',
arg_group="Marketplace",
options_list=['--plan-name'],
help='The plan name is referring to the Marketplace Plan ID of the extension.')
c.argument('plan_product',
arg_group="Marketplace",
options_list=['--plan-product'],
help='The plan product is referring to the Marketplace Product ID of the extension.')
c.argument('plan_publisher',
arg_group="Marketplace",
options_list=['--plan-publisher'],
help='The plan publisher is referring to the Marketplace Publisher ID of the extension')
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please do not explicitly declare the default option name

Suggested change
c.argument('plan_name',
arg_group="Marketplace",
options_list=['--plan-name'],
help='The plan name is referring to the Marketplace Plan ID of the extension.')
c.argument('plan_product',
arg_group="Marketplace",
options_list=['--plan-product'],
help='The plan product is referring to the Marketplace Product ID of the extension.')
c.argument('plan_publisher',
arg_group="Marketplace",
options_list=['--plan-publisher'],
help='The plan publisher is referring to the Marketplace Publisher ID of the extension')
c.argument('plan_name',
arg_group="Marketplace",
help='The plan name is referring to the Marketplace Plan ID of the extension.')
c.argument('plan_product',
arg_group="Marketplace",
help='The plan product is referring to the Marketplace Product ID of the extension.')
c.argument('plan_publisher',
arg_group="Marketplace",
help='The plan publisher is referring to the Marketplace Publisher ID of the extension')

@@ -33,7 +33,7 @@
# TODO: Add any additional SDK dependencies here
DEPENDENCIES = []

VERSION = "1.4.2"
VERSION = "1.4.3"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
VERSION = "1.4.3"
VERSION = "2.0.0"

Since this PR includes breaking changes, we usually recommend upgrading the major version

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Auto-Assign Auto assign by bot
Projects
None yet
Development

Successfully merging this pull request may close these issues.