Add workers.celery.extraVolumes & workers.kubernetes.extraVolumes#64746
Add workers.celery.extraVolumes & workers.kubernetes.extraVolumes#64746jscheffl merged 6 commits intoapache:mainfrom
Conversation
Miretpl
force-pushed
the
add-workers-celery-k8s-extra-volumes
branch
from
cf8a0d1 to
2a732e2
Compare
Miretpl
force-pushed
the
add-workers-celery-k8s-extra-volumes
branch
from
2a732e2 to
33f83ef
Compare
Miretpl
force-pushed
the
add-workers-celery-k8s-extra-volumes
branch
from
33f83ef to
bc09b57
Compare
Miretpl
requested review from
bugraoz93,
dstandish,
hussein-awala,
jedcunningham and
jscheffl
as code owners
|
Missclicked on "Ready for review" |
jscheffl
added
the
backport-to-chart/v1-2x-test
jscheffl
left a comment
jscheffl
left a comment
There was a problem hiding this comment.
Extra-Volumes also smells very much like a good Kustomize Feature actually :-D
…es.extraVolumes (#64746) * Seperate extraVolumes & extraVolumeMounts in values.yaml * Refactor workers extraVolumes-related tests * Add workers.celery.extraVolumes & workers.kubernetes.extraVolumes * Add newsfragment (cherry picked from commit 6cbf4c8) Co-authored-by: Przemysław Mirowski <17602603+Miretpl@users.noreply.github.com>
Backport successfully created: chart/v1-2x-testNote: As of Merging PRs targeted for Airflow 3.X In matter of doubt please ask in #release-management Slack channel.
|
|
I'm starting to think that basically everything with |
Maybe 80% of these - pareto principly prbably applies here as well :-D (ExtraEnv is a pretty common thing for example... but I see potentially low coverage for extra containers on top of kerberos... |
…es.extraVolumes (#64746) (#65032) * Seperate extraVolumes & extraVolumeMounts in values.yaml * Refactor workers extraVolumes-related tests * Add workers.celery.extraVolumes & workers.kubernetes.extraVolumes * Add newsfragment (cherry picked from commit 6cbf4c8) Co-authored-by: Przemysław Mirowski <17602603+Miretpl@users.noreply.github.com>
Agreed |
related: #28880
This PR introduces two new fields:
workers.celery.extraVolumesandworkers.kubernetes.extraVolumes. Theworkers.extraVolumesfield is now deprecated.Was generative AI tooling used to co-author this PR?
{pr_number}.significant.rst, in airflow-core/newsfragments. You can add this file in a follow-up commit after the PR is created so you know the PR number.