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

update command to get astronomer provider version #1331

Merged
merged 1 commit into from
Sep 21, 2023

Conversation

vatsrahul1001
Copy link
Contributor

update command to get astronomer provider version

@@ -209,7 +209,7 @@ def prepare_dag_dependency(task_info, execution_time):

get_astronomer_providers_version = BashOperator(
task_id="get_astronomer_providers_version",
bash_command="airflow providers get astronomer-providers -o json | jq '.[0].Version'",
bash_command="pip show astronomer-providers | grep -i version | awk '{print $2}'",
Copy link
Contributor

@pankajastro pankajastro Sep 21, 2023

Choose a reason for hiding this comment

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

We should add a comment or TODO here for this workaround

Copy link
Contributor

Choose a reason for hiding this comment

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

or even fine in current state since you are now getting it using pip command

@vatsrahul1001 vatsrahul1001 merged commit 52c4db4 into main Sep 21, 2023
@vatsrahul1001 vatsrahul1001 deleted the update-proviver-version-check branch September 21, 2023 11:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants