-
Notifications
You must be signed in to change notification settings - Fork 474
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
feat: allow labels on sync
and db-migrations
Deployments/Jobs
#467
Conversation
Signed-off-by: Oleksandr Kovalenko <oleksandr.kovalenko@gmail.com>
fce47e8
to
fc59cee
Compare
sync
and db-migrations
Deployments/Jobs
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@oleksandrkovalenko thanks for the PR!
However, you have missed a adding the labels to the Job resources:
- https://github.com/airflow-helm/charts/blob/main/charts/airflow/templates/db-migrations/db-migrations-job.yaml
- https://github.com/airflow-helm/charts/blob/main/charts/airflow/templates/sync/sync-connections-job.yaml
- https://github.com/airflow-helm/charts/blob/main/charts/airflow/templates/sync/sync-pools-job.yaml
- https://github.com/airflow-helm/charts/blob/main/charts/airflow/templates/sync/sync-users-job.yaml
- https://github.com/airflow-helm/charts/blob/main/charts/airflow/templates/sync/sync-variables-job.yaml
Can you please add the labels to these resources also?
Signed-off-by: Oleksandr Kovalenko <oleksandr.kovalenko@gmail.com>
@thesuperzapper I just make the changes. Could you please check them |
Could somebody please take a look at this one. The issue was already marked as staled. Sorry, I just afraid it will be forgotten. Many thanks |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@oleksandrkovalenko thanks for the PR, it looks good-to-go for the 8.6.0
release, I will put it in "ready to merge", and merge it just before we cut the release.
Signed-off-by: Mathew Wicks <thesuperzapper@users.noreply.github.com>
Signed-off-by: Mathew Wicks <thesuperzapper@users.noreply.github.com>
What issues does your PR fix?
sync
anddb-migrations
Deployments/Jobs #466What does your PR do?
sync
anddb-migrations
Deployments/Jobs:airflow.dbMigrations.labels
airflow.sync.labels
pgbouncer.labels
value so that it is respectedChecklist
For all Pull Requests
For releasing ONLY