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} Fix regression of option --uptime-sla and --no-uptime-sla caused by change in #7478 #7510

Merged
merged 3 commits into from
Apr 17, 2024

Conversation

FumingZhang
Copy link
Member

@FumingZhang FumingZhang commented Apr 16, 2024


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

Related command

  • Fix the issue that option --uptime-sla is ignored in command az aks create.
  • Fix the issue that option --uptime-sla and --no-uptime-sla is ignored in command az aks update.
  • Regression caused by PR {AKS} add --sku to az aks create/update command #7478 in aks-preview 3.0.0b2 and 3.0.0b3.

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? (pip install wheel==0.30.0 required)
  • 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.

Copy link

azure-client-tools-bot-prd bot commented Apr 16, 2024

️✔️Azure CLI Extensions Breaking Change Test
️✔️Non Breaking Changes

@yonzhan
Copy link
Collaborator

yonzhan commented Apr 16, 2024

AKS

Copy link

⚠️ Release Suggestions

Module: aks-preview

  • Update version to 3.0.0b4 in setup.py

Notes

  • Stable/preview tag is inherited from last release. If needed, please add stable/preview label to modify it.
  • Major/minor/patch/pre increment of version number is calculated by pull request code changes automatically. If needed, please add major/minor/patch/pre label to adjust it.
  • For more info about extension versioning, please refer to Extension version schema

Copy link
Contributor

@xmzhao0822 xmzhao0822 left a comment

Choose a reason for hiding this comment

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

Approved

@FumingZhang
Copy link
Member Author

FumingZhang commented Apr 17, 2024

Queued live test for the change, test passed!

  • test_aks_automatic_sku
  • test_aks_create_with_standard_sku
  • test_aks_create_with_premium_sku
  • test_aks_update_with_premium_sku

@yanzhudd yanzhudd merged commit fbae3ee into Azure:main Apr 17, 2024
15 checks passed
@azclibot
Copy link
Collaborator

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

blackchoey pushed a commit to blackchoey/azure-cli-extensions that referenced this pull request Apr 29, 2024
@FumingZhang FumingZhang deleted the fuming/aks-fix-sku-0416 branch May 11, 2024 07:58
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.

6 participants