feat: Allow arbitrary podSpec in migration-job and crd-install jobs for operator chart#1288
feat: Allow arbitrary podSpec in migration-job and crd-install jobs for operator chart#1288localleon wants to merge 3 commits intoredpanda-data:mainfrom
Conversation
…or operator chart
|
This PR is stale because it has been open 5 days with no activity. Remove stale label or comment or this will be closed in 5 days. |
|
Dear all, would you be so kind and review the PR? Thanks in advance! |
|
This PR is stale because it has been open 5 days with no activity. Remove stale label or comment or this will be closed in 5 days. |
|
Still there :-) |
|
This PR is stale because it has been open 5 days with no activity. Remove stale label or comment or this will be closed in 5 days. |
|
@andrewstucki @ivotron @RafalKorepta @hidalgopl @david-yu would anyone of you please be so kind and take a look at the PR? Thanks in advance! |
|
This PR is stale because it has been open 5 days with no activity. Remove stale label or comment or this will be closed in 5 days. |
|
Still here ;-) |
|
Hi @localleon -- been quite busy with a number of other things so haven't had too much time to look at this. At minimum one of us will have to close and re-open this as a local-to-this-repo branch as that's the only way CI will run. That said, I'm generally fine with this idea since we have the ability to merge in arbitrary pod spec to actual brokers today, so why not the migration/CRD jobs? But just need to discuss internally whether this is something we want to support. Thanks for the contribution and we'll get back about it soon! |
|
Thanks for the response @andrewstucki! If theres anything to do, let me know! |
|
This PR is stale because it has been open 5 days with no activity. Remove stale label or comment or this will be closed in 5 days. |
Closes #1287
This extends the Redpanda Operator Helm chart to allow configurable security contexts and other pod spec settings for pre-install CRD job and post-upgrade migration job hooks via new top-level values fields.
Values Configuration
Implementation Details
I've regenerated the charts using the taskfile and performed manual validation (helm template), if the rednering is successfull with both jobs!