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.

Sign up for GitHub

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

Jump to bottom

Allow variables to be printed to STDOUT #31851

Closed
2 tasks done
vedantlodha opened this issue Jun 12, 2023 · 4 comments · Fixed by #33279
Closed
2 tasks done

Allow variables to be printed to STDOUT #31851

vedantlodha opened this issue Jun 12, 2023 · 4 comments · Fixed by #33279
Assignees
@vedantlodha
Labels
area:core good first issue kind:feature Feature Requests

Comments

@vedantlodha
Copy link
Contributor

Description

Currently, the airflow variables export command requires an explicit file path and does not support output to stdout. However connections can be printed to stdout using airflow connections export -. This inconsistency between the two export commands can lead to confusion and limits the flexibility of the variables export command.

Use case/motivation

To bring some consistency, similar to connections, variables should also be printed to STDOUT, using - instead of filename.

Related issues

No response

Are you willing to submit a PR?

  • Yes I am willing to submit a PR!

Code of Conduct
@vedantlodha vedantlodha added kind:feature Feature Requests needs-triage label for new issues that we didn't triage yet labels Jun 12, 2023
@boring-cyborg
Copy link

boring-cyborg bot commented Jun 12, 2023

Thanks for opening your first issue here! Be sure to follow the issue template! If you are willing to raise PR to address this issue please do so, no need to wait for approval.

@vedantlodha vedantlodha changed the title Allow airflow variables export - to Output to stdout Allow variables to be printed to STDOUT Jun 12, 2023
@vedantlodha
Copy link
Contributor Author

vedantlodha commented Jun 12, 2023 •
edited
Loading

If the folks think this is a valid request, i would like to work on this. I kinda have a draft patch ready for this.
Thanks!

@potiuk
Copy link
Member

potiuk commented Jun 12, 2023

Feel free. Sounds like a good first contribution

@phanikumv phanikumv added good first issue and removed needs-triage label for new issues that we didn't triage yet labels Jun 12, 2023
@phanikumv
Copy link
Contributor

Assigned it to you @vedantlodha

@vedantlodha vedantlodha mentioned this issue Jun 12, 2023
Closed
vedantlodha added a commit to vedantlodha/airflow that referenced this issue Jun 20, 2023
@vedantlodha
Allow variables to be printed to STDOUT with airflow variables export.
c3294af 
 Similar to connections, this change allows the cli to export the variables to the stdout using 'airflow variables export -'

 Fixes apache#31851
@uranusjr uranusjr mentioned this issue Aug 10, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@vedantlodha vedantlodha

Labels
area:core good first issue kind:feature Feature Requests
Projects
None yet
Milestone
No milestone
3 participants
@potiuk @vedantlodha @phanikumv