Added support for cluster id in the astro deployment list command by rujhan-arora-astronomer · Pull Request #1910 · astronomer/astro-cli
Navigation Menu
{{ message }}
astronomer / astro-cli Public
- Notifications You must be signed in to change notification settings
- Fork 106
Merged
Added support for cluster id in the astro deployment list command#1910
pgvishnuram merged 2 commits intomainfrom
Added support for cluster id in the astro deployment list command#1910
pgvishnuram merged 2 commits intomainfrom
Conversation
Copy link
Contributor
Description
Added support for cluster ID in the astro deployment list command. Customers will have an option to pass the cluster ID, too.
🎟 Issue(s)
🧪 Functional Testing
Tested locally against Houston local
📸 Screenshots
📋 Checklist
- Rebased from the main (or release if patching) branch (before testing)
- Ran
make testbefore taking out of draft - Ran
make lintbefore taking out of draft - Added/updated applicable tests
- Tested against Astro-API (if necessary).
- Tested against Houston-API and Astronomer (if necessary).
- Communicated to/tagged owners of respective clients potentially impacted by these changes.
- Updated any related documentation
rujhan-arora-astronomer
requested review from
jeremybeard,
karankhanchandani,
kushalmalani,
neel-astro and
pgvishnuram
as code owners
for more information, see https://pre-commit.ci
karankhanchandani approved these changes Aug 12, 2025
Copy link
karankhanchandani
left a comment
karankhanchandani
left a comment
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR. LGTM.
pgvishnuram approved these changes Aug 12, 2025
pgvishnuram
deleted the
f_7590
branch
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
