Skip to content

Airflow environment variables convention #1649

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
Closed
Tutuchan opened this issue Sep 6, 2023 · 2 comments
Closed

Airflow environment variables convention #1649

Tutuchan opened this issue Sep 6, 2023 · 2 comments

Comments

@Tutuchan
Copy link
Contributor

Tutuchan commented Sep 6, 2023

Hello,

The Airflow doc indicates that environment variables should be declared as AIRFLOW_VAR_<MY_VAR> to be used as Airflow Variables.

Currently, in the Data Platform blueprints, they are declared without the AIRFLOW_VAR prefix, which prevents us from using them in templated fields with Airflow's usual process.

Would you be open to a change on this ?
@wiktorn
Copy link
Collaborator

wiktorn commented Sep 6, 2023

Hi @Tutuchan,

You mean the blueprint provided variables in data-platform-minimal and data-platform-foundations ?

It looks like a great improvement. Are you willing to create a PR for that?

@Tutuchan
Copy link
Contributor Author

Tutuchan commented Sep 6, 2023

Yes, sorry, that was unclear, I am talking about these blueprints.

I'll get started on a PR ASAP.

@Tutuchan Tutuchan mentioned this issue Sep 6, 2023
Merged
4 tasks
@Tutuchan Tutuchan closed this as completed Sep 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@wiktorn @Tutuchan