Skip to content

Conversation

@csp33
Copy link
Contributor

@csp33 csp33 commented Nov 18, 2022

closes: #27687
Adds a global way to add volumes & volumeMounts for all containers (same as env for env vars)
image
image

@boring-cyborg boring-cyborg bot added the area:helm-chart Airflow Helm Chart label Nov 18, 2022
@csp33 csp33 force-pushed the feature/27687-add-global-volume-volumemounts-to-chart branch from 87e11ff to af0378a Compare November 21, 2022 14:45
@potiuk
Copy link
Member

potiuk commented Nov 25, 2022

Small fix in static checks. But otherwise LGTM

Copy link
Member

@potiuk potiuk left a comment

Choose a reason for hiding this comment

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

Others?

@csp33 csp33 force-pushed the feature/27687-add-global-volume-volumemounts-to-chart branch from af0378a to afd540c Compare November 29, 2022 11:24
@csp33
Copy link
Contributor Author

csp33 commented Nov 29, 2022

Small fix in static checks. But otherwise LGTM

should be fixed now :)

@csp33 csp33 force-pushed the feature/27687-add-global-volume-volumemounts-to-chart branch 2 times, most recently from afd540c to c9c0801 Compare November 29, 2022 11:32
@eladkal eladkal added this to the Airflow Helm Chart 1.8.0 milestone Nov 29, 2022
@eladkal
Copy link
Contributor

eladkal commented Nov 29, 2022

Static checks are failing

@csp33 csp33 force-pushed the feature/27687-add-global-volume-volumemounts-to-chart branch 3 times, most recently from 02db772 to 2f7fc17 Compare November 30, 2022 08:08
@csp33 csp33 force-pushed the feature/27687-add-global-volume-volumemounts-to-chart branch from 2f7fc17 to 5df056d Compare November 30, 2022 08:09
@csp33
Copy link
Contributor Author

csp33 commented Nov 30, 2022

Static checks are failing

Fixed

@csp33
Copy link
Contributor Author

csp33 commented Dec 9, 2022

Can this PR be merged now? @potiuk @jedcunningham
Thanks!

@jedcunningham jedcunningham merged commit 20a1c4d into apache:main Dec 9, 2022
@jedcunningham
Copy link
Member

Thanks @csp33!

@csp33 csp33 deleted the feature/27687-add-global-volume-volumemounts-to-chart branch December 12, 2022 07:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:helm-chart Airflow Helm Chart

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Allow global volume & volumeMounts in Helm chart

4 participants