Skip to content

{aimanager} replace an aimanager role name - #10398

Merged
Ethan Yang (necusjz) merged 1 commit into
Azure:mainfrom
chengliangli0918:main
Sep 28, 2026
Merged

Ethan Yang (necusjz) merged 1 commit into
Azure:mainfrom
chengliangli0918:main

Conversation

@chengliangli0918

Copy link
Copy Markdown
Contributor

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

Related command

{aimanager} replace an aimanager role name

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 azdev 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.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟢 Approval recommended

No unresolved issues were identified.

Review effort: Lite
Findings: None

What changed in this PR

Updates the AIManager RBAC Reader role name and bumps the extension version to 2.0.0b3.

Changes:

  • Renames the role in constants and command help.
  • Records the change in release history.
  • Updates package and extension metadata versions.
File Description
src/​aimanager/​setup.py Updates package version.
src/​aimanager/​HISTORY.rst Documents the role-name change.
src/​aimanager/​azext_aimanager/​constants.py Updates the role display name.
src/​aimanager/​azext_aimanager/​azext_metadata.json Updates extension metadata version.
src/​aimanager/​azext_aimanager/​_help.py Updates role names in command help.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@tnsimon tnsimon left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@tnsimon tnsimon left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Signed-off-by: Simon Tien tiensimon@microsoft.com

@x-engineering-agent

Copy link
Copy Markdown

Automated sensitive-information remediation ran on this pull request.

  • Detected categories: email address
  • Replaced with typed [REDACTED:category] placeholders in: no PR metadata fields
  • Comment/review owners notified because X Engineering Agent cannot edit another user's text: tnsimon

X Engineering Agent does not modify source files. The PR creator must remove or replace each suspected value at the linked line:

  • No changed-file findings

If a credential was exposed, rotate or revoke it immediately. Detected values are never copied into this comment.

✅ Confirm the finding · ❌ Dispute the finding

GitHub only supports a fixed reaction set, so 👍 represents ✅ and 👎 represents ❌. The bot-created reactions are only poll choices.

@PugDeveloper

Copy link
Copy Markdown
Contributor

Julie Zhu (@yanzhudd) can we get approval please. Thanks!

@yonzhan

Copy link
Copy Markdown
Collaborator

aimanager

1 similar comment
@yonzhan

Copy link
Copy Markdown
Collaborator

aimanager

@necusjz
Ethan Yang (necusjz) merged commit 4a0a0d7 into Azure:main Sep 28, 2026
38 checks passed
@azclibot

Copy link
Copy Markdown
Collaborator

[Release] Update index.json for extension [ aimanager-2.0.0b3 ] : https://dev.azure.com/msazure/One/_build/results?buildId=183101607&view=results

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.

10 participants