Skip to content

[AKS] BREAKING CHANGE: az aks trustedaccess rolebinding create: Remove deprecated -r and -s options#8243

Merged
yanzhudd merged 5 commits into
Azure:mainfrom
YitongFeng-git:yife/ext
Nov 7, 2024
Merged

[AKS] BREAKING CHANGE: az aks trustedaccess rolebinding create: Remove deprecated -r and -s options#8243
yanzhudd merged 5 commits into
Azure:mainfrom
YitongFeng-git:yife/ext

Conversation

@YitongFeng-git
Copy link
Copy Markdown
Contributor


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

Related command

az aks trustedaccess rolebinding create

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.

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

azure-client-tools-bot-prd Bot commented Nov 6, 2024

⚠️Azure CLI Extensions Breaking Change Test
⚠️aks-preview
rule cmd_name rule_message suggest_message
⚠️ 1009 - ParaPropRemove aks trustedaccess rolebinding create cmd aks trustedaccess rolebinding create update parameter source_resource_id: removed property options_deprecate_info=[{'target': '-s', 'redirect': '--source-resource-id', 'hide': True}, {'target': '-r', 'redirect': '--source-resource-id', 'hide': True}]

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

Hi @YitongFeng-git,
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
Copy Markdown
Collaborator

yonzhan commented Nov 6, 2024

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

@YitongFeng-git YitongFeng-git changed the title [AKS] BREAKING CHANGE: az aks trustedaccess rolebinding create: Remove deprecated -r options [AKS] BREAKING CHANGE: az aks trustedaccess rolebinding create: Remove deprecated -r and -s options Nov 6, 2024
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Nov 6, 2024

Copy link
Copy Markdown
Contributor

@karataliu karataliu left a comment

Choose a reason for hiding this comment

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

LGTM

Comment thread src/aks-preview/HISTORY.rst Outdated
@github-actions github-actions Bot added release-version-block Updates do not qualify release version rules. NOTE: please do not edit it manually. and removed release-version-block Updates do not qualify release version rules. NOTE: please do not edit it manually. labels Nov 6, 2024
@github-actions github-actions Bot added the release-version-block Updates do not qualify release version rules. NOTE: please do not edit it manually. label Nov 7, 2024
Copy link
Copy Markdown
Member

@FumingZhang FumingZhang left a comment

Choose a reason for hiding this comment

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

LGTM

@yanzhudd yanzhudd added the major release extension module with version major upgraded label Nov 7, 2024
@github-actions github-actions Bot removed the release-version-block Updates do not qualify release version rules. NOTE: please do not edit it manually. label Nov 7, 2024
@yanzhudd yanzhudd merged commit 2e3be93 into Azure:main Nov 7, 2024
@azclibot
Copy link
Copy Markdown
Collaborator

azclibot commented Nov 7, 2024

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

@YitongFeng-git YitongFeng-git deleted the yife/ext branch November 7, 2024 07:09
chrisribe pushed a commit to chrisribe/azure-cli-extensions that referenced this pull request Nov 13, 2024
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 major release extension module with version major upgraded

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants