Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

BUG: upgrade messing output in deployment files #1622

Closed
1 of 3 tasks
chomipi88 opened this issue Apr 17, 2024 · 2 comments
Closed
1 of 3 tasks

BUG: upgrade messing output in deployment files #1622

chomipi88 opened this issue Apr 17, 2024 · 2 comments
Assignees

Comments

@chomipi88
Copy link

Describe the bug

When inspecting a deployment and saving the output to a file, the CLI is including the upgrade message. With the included upgrade message, deployment updates using the deployment file error out.

A newer version of Astro CLI is available: 1.25.0
Please update to the latest version using 'brew upgrade astro'

If you don't want to see this message again run 'astro config set -g upgrade_message false'

When trying to update a deployment, the below error is returned:

Error: error converting YAML to JSON: yaml: line 5: could not find expected ':'

What CLI Version did you experience this bug? 1.24.1

This CLI bug is related to which Astronomer Platform?

  • Astro
  • Software
  • None/Unknown

What Operating System is the above CLI installed on? MAC

🪜 Steps To Reproduce

astro deployment inspect > depl.yaml

astro deployment update --deployment-file depl.yaml

📸 Screenshots

@sunkickr
Copy link
Contributor

we need to make sure this message is not shown when a user is logged in with an API key or token. It would also be nice if this message was not shown when the --clean-output flag is used

@kushalmalani
Copy link
Contributor

@chomipi88 I merged the PR with this fix. Fix will be released next week as part of 1.27.0

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

No branches or pull requests

3 participants