Repository navigation
charts/redpanda, multicluster: share init container rendering - #1888
Merged
Merged
Conversation
chrisseto
added this pull request to stack #1882
September 21, 2026 17:16
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
chrisseto
force-pushed
the
chris/dedup-initcontainer
branch
from
September 22, 2026 20:43
ecf59e2 to
5f8416b
Compare
david-yu
force-pushed
the
chris/dedup-initcontainer
branch
from
September 23, 2026 20:34
5f8416b to
d37919a
Compare
chrisseto
force-pushed
the
chris/dedup-initcontainer
branch
from
September 25, 2026 19:30
d37919a to
26ca979
Compare
chrisseto
marked this pull request as ready for review
September 25, 2026 19:35
chrisseto
requested review from
RafalKorepta,
andrewstucki,
gene-redpanda and
hidalgopl
as code owners
September 25, 2026 19:35
chrisseto
force-pushed
the
chris/dedup-initcontainer
branch
from
September 25, 2026 19:59
26ca979 to
9dccfbb
Compare
chrisseto
force-pushed
the
chris/dedup-initcontainer
branch
from
September 30, 2026 19:15
9dccfbb to
f08937e
Compare
chrisseto
force-pushed
the
chris/dedup-initcontainer
branch
from
September 30, 2026 20:30
f08937e to
220cb32
Compare
RafalKorepta
approved these changes
Oct 1, 2026
RafalKorepta
left a comment
RafalKorepta
left a comment
Contributor
There was a problem hiding this comment.
LGTM only non blocking questions
chrisseto
force-pushed
the
chris/dedup-initcontainer
branch
from
October 1, 2026 14:32
220cb32 to
6a046b2
Compare
Extract, deduplicate, and refactor redpanda init container rendering
into charts/redpanda from charts/redpanda/chart and
multicluster/operator.
The change is largely inert but a few changes had to be made:
- The configurator and fs-validator script volumes are now named
`configurator` and `fs-validator` rather than after the release or
pool. This was done to clean up internal plumbing as there's no need
to "namespace" volumes.
- StretchCluster's redpanda-configurator init container gains
securityContext.runAsNonRoot and allowPrivilegeEscalation: false,
to match the chart.
chrisseto
force-pushed
the
chris/dedup-initcontainer
branch
from
October 1, 2026 17:35
6a046b2 to
3888019
Compare
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation and see the Github Action logs for details |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Extract, deduplicate, and refactor redpanda init container rendering into charts/redpanda from charts/redpanda/chart and multicluster/operator.
The change is largely inert but a few changes had to be made:
The configurator and fs-validator script volumes are now named
configuratorandfs-validatorrather than after the release or pool. This was done to clean up internal plumbing as there's no need to "namespace" volumes.StretchCluster's redpanda-configurator init container gains securityContext.runAsNonRoot and allowPrivilegeEscalation: false, to match the chart.