Improve remote deploy messaging by neel-astro · Pull Request #1970 · astronomer/astro-cli

Conversation

@neel-astro

Description

Changes:

  • Improve remote deploy messaging to be more descriptive and helpful
  • Add tiny logic to skip login to Astronomer if the base image in the Dockerfile.client is not from the Astronomer registry
  • Add logic to pick the best available client image when the runtime version is passed
  • Bump the default timeout for deployment create wait time to 10 mins, since there are a decent number of times when deployment create takes slightly more than 5 mins
  • Set the remote base image registry as the image host in Dockerfile.client
  • Fix issue around deployment health poll failing for deployment that hibernates right after creation

🎟 Issue(s)

https://astronomer.slack.com/archives/C07FWJXBQ3X/p1761225561411039

🧪 Functional Testing

image image image image image

📸 Screenshots

Add screenshots to illustrate the validity of these changes.

📋 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
… based on runtime version

@coveralls-official

feluelle

yanmastin-astro

danalogalbo-astronomer

danalogalbo-astronomer

neel-astro added a commit that referenced this pull request

Oct 24, 2025

Reviewers

@danalogalbo-astronomer danalogalbo-astronomer danalogalbo-astronomer left review comments

@feluelle feluelle feluelle approved these changes

@jeremybeard jeremybeard Awaiting requested review from jeremybeard

@kushalmalani kushalmalani Awaiting requested review from kushalmalani

@lzdanski lzdanski Awaiting requested review from lzdanski

+1 more reviewer
Reviewers whose approvals may not affect merge requirements