Skip to content

Comments

Added the support for cluster ID in App config query#1917

Merged
rishkarajgi merged 3 commits intomainfrom
gh_7571
Aug 21, 2025
Merged

Added the support for cluster ID in App config query#1917
rishkarajgi merged 3 commits intomainfrom
gh_7571

Conversation

@rujhan-arora-astronomer
Copy link
Contributor

@rujhan-arora-astronomer rujhan-arora-astronomer commented Aug 21, 2025

Description

Added support for cluster ID in App config query. This is required because different clusters can have different clusters' deployment' configs.

🎟 Issue(s)

Related https://github.com/astronomer/issues/issues/7571

🧪 Functional Testing

Locally
Unit tests

📸 Screenshots

The commands are same as before, in terms of UX.

📋 Checklist

  • Rebased from the main (or release if patching) branch (before testing)
  • Ran make test before taking out of draft
  • Ran make lint before 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

Copy link

@karankhanchandani 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.

@rishkarajgi rishkarajgi merged commit 0d94581 into main Aug 21, 2025
4 of 5 checks passed
@rishkarajgi rishkarajgi deleted the gh_7571 branch August 21, 2025 14:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants