Skip to content

Comments

Add "REMOTE EXECUTION" column to deployment tables#1952

Merged
ronidelacruz-astro merged 1 commit intomainfrom
feat/add_deployment_re_col
Oct 7, 2025
Merged

Add "REMOTE EXECUTION" column to deployment tables#1952
ronidelacruz-astro merged 1 commit intomainfrom
feat/add_deployment_re_col

Conversation

@ronidelacruz-astro
Copy link
Contributor

@ronidelacruz-astro ronidelacruz-astro commented Oct 7, 2025 •
edited
Loading

Description

Add remote execution status col to deployment display table

🎟 Issue(s)

Related #1897

🧪 Functional Testing

./astro deployment create -n "test_cli_remote_execution_create_3"  --remote-execution-enabled --executor Astro --type dedicated 

Current Workspace: roni


Please select a Cluster for your Deployment:
 #     CLUSTER NAME             CLOUD PROVIDER     CLUSTER ID                    
 1     Astro Agent CI_DND       AZURE              cmbjc2euc03ak01nlfpy5xtwr     
 2     airflow3-testing_DND     AZURE              cm4sauxuo03ig01rfux2aun5d     

> 2
 NAME                                   NAMESPACE                 CLUSTER                  CLOUD PROVIDER     REGION      DEPLOYMENT ID                 RUNTIME VERSION                    DAG DEPLOY ENABLED     CI-CD ENFORCEMENT     DEPLOYMENT TYPE     REMOTE EXECUTION     
 test_cli_remote_execution_create_3     uniform-buoyancy-7912     airflow3-testing_DND     AZURE              westus2     cmgguhtpl011501owe9woezt9     3.1-1 (based on Airflow 3.1.0)     false                  false                 DEDICATED           true                 

 Successfully created Deployment: test_cli_remote_execution_create_3
 Deployment can be accessed at the following URLs 

 Deployment Dashboard: cloud.astronomer-dev.io/cmfvjgjhv00hg01qj5115yydj/deployments/cmgguhtpl011501owe9woezt9
 Airflow Dashboard: clztuyh0a01z701mwxjahsza2.astronomer-dev.run/d9woezt9

📸 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

@ronidelacruz-astro ronidelacruz-astro self-assigned this Oct 7, 2025
@ronidelacruz-astro ronidelacruz-astro linked an issue Oct 7, 2025 that may be closed by this pull request
Add a clear indicator for whether a deployment is Remote Execution or not #1897
Closed
feluelle
feluelle approved these changes Oct 7, 2025
View reviewed changes
Copy link
Member

@feluelle feluelle left a comment

Choose a reason for hiding this comment

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

Are there any existing UTs for this we should update?

@ronidelacruz-astro ronidelacruz-astro merged commit ee0682a into main Oct 7, 2025
5 of 6 checks passed
@ronidelacruz-astro ronidelacruz-astro deleted the feat/add_deployment_re_col branch October 7, 2025 18:09
@ronidelacruz-astro
Copy link
Contributor Author

ronidelacruz-astro commented Oct 7, 2025

No, we check for success, nothing to check for outputs

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@feluelle feluelle feluelle approved these changes

@neel-astro neel-astro Awaiting requested review from neel-astro

@jeremybeard jeremybeard Awaiting requested review from jeremybeard

@kushalmalani kushalmalani Awaiting requested review from kushalmalani

Assignees

@ronidelacruz-astro ronidelacruz-astro

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Add a clear indicator for whether a deployment is Remote Execution or not

2 participants

@ronidelacruz-astro @feluelle