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

{AKS} Add support for option --aks-custom-headers to some aks commands #6430

Merged
merged 8 commits into from
Jun 27, 2023

Conversation

FumingZhang
Copy link
Member

@FumingZhang FumingZhang commented Jun 21, 2023


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

Related command

{AKS} Add support for option --aks-custom-headers to the following aks commands, feature request from aurora team.

  • aks get-credentials
  • aks nodepool scale
  • aks nodepool update
  • aks enable-addons
  • aks show
  • aks scale

Fix recordings as the inherited command az aks get-version bumped API version yesterday (by PR #26718). Overwrite the implementation in aks-preview, so the issue won't happen again.

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.

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

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

@ghost ghost requested review from zhoxing-ms and yanzhudd June 21, 2023 07:52
@ghost ghost assigned zhoxing-ms Jun 21, 2023
@ghost ghost added the Auto-Assign Auto assign by bot label Jun 21, 2023
@ghost ghost requested a review from yonzhan June 21, 2023 07:52
@ghost ghost added the AKS label Jun 21, 2023
@yonzhan
Copy link
Collaborator

yonzhan commented Jun 21, 2023

AKS

@FumingZhang FumingZhang force-pushed the fuming/aks-custom-headers-0623 branch from 00c47df to a6bed07 Compare June 26, 2023 05:35
@FumingZhang FumingZhang marked this pull request as ready for review June 26, 2023 07:57
@FumingZhang FumingZhang changed the title {AKS} Add support for option --aks-custom-headers to some aks commands {AKS} Add support for option --aks-custom-headers to some aks commands Jun 26, 2023
@FumingZhang FumingZhang marked this pull request as draft June 26, 2023 10:02

@AllowLargeResponse()
@AKSCustomResourceGroupPreparer(random_name_length=17, name_prefix='clitest', location='eastus', preserve_default_location=True)
def test_aks_create_with_custom_headers(self, resource_group, resource_group_location):
Copy link
Member Author

@FumingZhang FumingZhang Jun 26, 2023

Choose a reason for hiding this comment

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

Queued live test to validate the change.

@FumingZhang FumingZhang marked this pull request as ready for review June 26, 2023 11:09
@zhoxing-ms zhoxing-ms merged commit 5b3055f into Azure:main Jun 27, 2023
@azclibot
Copy link
Collaborator

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

@FumingZhang FumingZhang deleted the fuming/aks-custom-headers-0623 branch June 28, 2023 09:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AKS Auto-Assign Auto assign by bot
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants