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/release 1.5.0 #6779

Merged
merged 132 commits into from
Oct 11, 2023

Conversation

Arif-lakhani
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 <rishraj@microsoft.com>
…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 <shubhash@microsoft.com>

* Add Dapr test'

Signed-off-by: Shubham Sharma <shubhash@microsoft.com>

* Handle stateful set

Signed-off-by: Shubham Sharma <shubhash@microsoft.com>

* Update default handling

Signed-off-by: Shubham Sharma <shubhash@microsoft.com>

* Fix HA handling

Signed-off-by: Shubham Sharma <shubhash@microsoft.com>

* Add placement service todo

Signed-off-by: Shubham Sharma <shubhash@microsoft.com>

* Add non-interactive mode

Signed-off-by: Shubham Sharma <shubhash@microsoft.com>

* Fix lint

Signed-off-by: Shubham Sharma <shubhash@microsoft.com>

* Update tests

Signed-off-by: Shubham Sharma <shubhash@microsoft.com>

* Reset configuration for StatefulSet during k8s upgrade

Signed-off-by: Shubham Sharma <shubhash@microsoft.com>

* Fix lint

Signed-off-by: Shubham Sharma <shubhash@microsoft.com>

* Retrigger tests

Signed-off-by: Shubham Sharma <shubhash@microsoft.com>

* Add changes to manage ha and placement params

Signed-off-by: Shubham Sharma <shubhash@microsoft.com>

* Update message

Signed-off-by: Shubham Sharma <shubhash@microsoft.com>

* nits

Signed-off-by: Shubham Sharma <shubhash@microsoft.com>

Signed-off-by: Shubham Sharma <shubhash@microsoft.com>
@azure-client-tools-bot-prd
Copy link

azure-client-tools-bot-prd bot commented Sep 20, 2023

❌Azure CLI Extensions Breaking Change Test
❌k8s-extension
rule cmd_name rule_message suggest_message
1006 - ParaAdd k8s-extension extension-types list-versions cmd k8s-extension extension-types list-versions added parameter cluster_name please remove parameter cluster_name for cmd k8s-extension extension-types list-versions
1006 - ParaAdd k8s-extension extension-types list-versions cmd k8s-extension extension-types list-versions added parameter cluster_type please remove parameter cluster_type for cmd k8s-extension extension-types list-versions
1006 - ParaAdd k8s-extension extension-types list-versions cmd k8s-extension extension-types list-versions added parameter resource_group_name please remove parameter resource_group_name for cmd k8s-extension extension-types list-versions
1007 - ParaRemove k8s-extension extension-types list-versions cmd k8s-extension extension-types list-versions removed parameter location please add back parameter location for cmd k8s-extension extension-types list-versions
1006 - ParaAdd k8s-extension extension-types show cmd k8s-extension extension-types show added parameter location please remove parameter location for cmd k8s-extension extension-types show
1007 - ParaRemove k8s-extension extension-types show cmd k8s-extension extension-types show removed parameter cluster_name please add back parameter cluster_name for cmd k8s-extension extension-types show
1007 - ParaRemove k8s-extension extension-types show cmd k8s-extension extension-types show removed parameter cluster_type please add back parameter cluster_type for cmd k8s-extension extension-types show
1007 - ParaRemove k8s-extension extension-types show cmd k8s-extension extension-types show removed parameter resource_group_name please add back parameter resource_group_name for cmd k8s-extension extension-types show
⚠️ 1006 - ParaAdd k8s-extension extension-types list cmd k8s-extension extension-types list added parameter plan_name
⚠️ 1006 - ParaAdd k8s-extension extension-types list cmd k8s-extension extension-types list added parameter plan_product
⚠️ 1006 - ParaAdd k8s-extension extension-types list cmd k8s-extension extension-types list added parameter plan_publisher
⚠️ 1006 - ParaAdd k8s-extension extension-types list cmd k8s-extension extension-types list added parameter release_train
⚠️ 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
⚠️ 1006 - ParaAdd k8s-extension extension-types list-versions cmd k8s-extension extension-types list-versions added parameter major_version
⚠️ 1006 - ParaAdd k8s-extension extension-types list-versions cmd k8s-extension extension-types list-versions added parameter release_train
⚠️ 1006 - ParaAdd k8s-extension extension-types list-versions cmd k8s-extension extension-types list-versions added parameter show_latest
⚠️ 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 @Arif-lakhani,
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.

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

Hi @Arif-lakhani,
Since the current milestone time is less than 7 days, this pr will be reviewed in the next milestone.

@yonzhan
Copy link
Collaborator

yonzhan commented Sep 20, 2023

Thank you for your contribution! We will review the pull request and get back to you soon.

@Arif-lakhani Arif-lakhani force-pushed the k8s-extension/release-1.5.0 branch from 073751f to ac3a9bb Compare September 20, 2023 18:47
@yanzhudd
Copy link
Contributor

Could you please add some change details in the PR description #6779 (comment)?

@Arif-lakhani Arif-lakhani force-pushed the k8s-extension/release-1.5.0 branch from ac3a9bb to 10e1709 Compare September 25, 2023 18:20
# --------------------------------------------------------------------------

VERSION = "2.0.0"
VERSION = "0.1.0"
Copy link
Contributor

Choose a reason for hiding this comment

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

May I ask if it is better to use 2.1.0 instead of 0.1.0 as the version?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

this file was replaced as a part of the generated files from the swagger sdk.

@Arif-lakhani Arif-lakhani force-pushed the k8s-extension/release-1.5.0 branch from 10e1709 to ed41084 Compare September 27, 2023 22:33
@yanzhudd yanzhudd merged commit 32b82b9 into Azure:main Oct 11, 2023
@azclibot
Copy link
Collaborator

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

bgashirabake pushed a commit to bgashirabake/azure-cli-extensions that referenced this pull request Oct 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.