Skip to content
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

Cherry pick improvements from helm charts repo #365

Merged

Conversation

The Deployment Spec token auto mount takes precendese over Service Account token
auto mount. Now both token auto mount properties should be set to true in order
to not mount volume.

Reference
redpanda-data/helm-charts@5d1314e
Prior to this commit objects produced from the console helm chart would
inconsistently set `namespace`. This isn't particularly noticeable as `kubectl`
and `helm` often set namespace transparently but it can cause issues when using
other tooling.

This commit sets namespace on all objects it was set on, adds a test assertion
that namespace is present across all objects, and removes usages of
`extraInitContainers` from tests as they generated invalid manifests.

Reference
redpanda-data/helm-charts@592a079
Set default value for tiered storage cache to `none` which will defer tiered
storage cache path to Redpanda process. Previously the `emptyDir` volume could
fill Kubernetes VM quickly and tip over such K8S Node. Then user would need to
restart Pod to clean the content of the `emptyDir` volume.

Reference
redpanda-data/helm-charts@78a4c0b
Copy link
Contributor

@andrewstucki andrewstucki left a comment

Choose a reason for hiding this comment

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

looks like the comment for the helm compat check needs to be updated still, but aside from the LGTM

@RafalKorepta RafalKorepta force-pushed the rk/cherry-pick-some-improvements-from-helm-charts-repo branch from dfd90ab to dedb1e4 Compare December 19, 2024 13:40
@andrewstucki
Copy link
Contributor

Heads up, but I manually ran TestRedpandaController/TestIntegrationStableUIDAndGeneration and TestHelmValuesCompat with the latest changes and just to make sure we were generating the same things between our now locally sourced manifests and the latest helm-release, so we should be good now.

@andrewstucki andrewstucki force-pushed the rk/cherry-pick-some-improvements-from-helm-charts-repo branch from d96888f to 4620e61 Compare December 20, 2024 17:48
@andrewstucki andrewstucki merged commit d453875 into main Dec 20, 2024
8 checks passed
@andrewstucki andrewstucki deleted the rk/cherry-pick-some-improvements-from-helm-charts-repo branch December 20, 2024 19:28
RafalKorepta added a commit to redpanda-data/helm-charts that referenced this pull request Jan 8, 2025
The source of truth is now redpanda-operator repostiory where gotohelm,
genpartial and genschema cli are located. Those tools will generate inside
redpanda-operator repository the following charts:
* connectors
* console
* operator
* redpanda

Reference
redpanda-data/redpanda-operator#339
redpanda-data/redpanda-operator#354
redpanda-data/redpanda-operator#365
RafalKorepta added a commit to redpanda-data/helm-charts that referenced this pull request Jan 8, 2025
The source of truth is now redpanda-operator repostiory where gotohelm,
genpartial and genschema cli are located. Those tools will generate inside
redpanda-operator repository the following charts:
* connectors
* console
* operator
* redpanda

Reference
redpanda-data/redpanda-operator#339
redpanda-data/redpanda-operator#354
redpanda-data/redpanda-operator#365
RafalKorepta added a commit to redpanda-data/helm-charts that referenced this pull request Jan 8, 2025
The source of truth is now redpanda-operator repostiory where gotohelm,
genpartial and genschema cli are located. Those tools will generate inside
redpanda-operator repository the following charts:
* connectors
* console
* operator
* redpanda

Reference
redpanda-data/redpanda-operator#339
redpanda-data/redpanda-operator#354
redpanda-data/redpanda-operator#365
RafalKorepta added a commit to redpanda-data/helm-charts that referenced this pull request Jan 8, 2025
The source of truth is now redpanda-operator repostiory where gotohelm,
genpartial and genschema cli are located. Those tools will generate inside
redpanda-operator repository the following charts:
* connectors
* console
* operator
* redpanda

Reference
redpanda-data/redpanda-operator#339
redpanda-data/redpanda-operator#354
redpanda-data/redpanda-operator#365
RafalKorepta added a commit to redpanda-data/helm-charts that referenced this pull request Jan 8, 2025
The source of truth is now redpanda-operator repostiory where gotohelm,
genpartial and genschema cli are located. Those tools will generate inside
redpanda-operator repository the following charts:
* connectors
* console
* operator
* redpanda

Reference
redpanda-data/redpanda-operator#339
redpanda-data/redpanda-operator#354
redpanda-data/redpanda-operator#365
RafalKorepta added a commit to redpanda-data/helm-charts that referenced this pull request Jan 8, 2025
The source of truth is now redpanda-operator repostiory where gotohelm,
genpartial and genschema cli are located. Those tools will generate inside
redpanda-operator repository the following charts:
* connectors
* console
* operator
* redpanda

Reference
redpanda-data/redpanda-operator#339
redpanda-data/redpanda-operator#354
redpanda-data/redpanda-operator#365
RafalKorepta added a commit to redpanda-data/helm-charts that referenced this pull request Jan 8, 2025
The source of truth is now redpanda-operator repostiory where gotohelm,
genpartial and genschema cli are located. Those tools will generate inside
redpanda-operator repository the following charts:
* connectors
* console
* operator
* redpanda

Reference
redpanda-data/redpanda-operator#339
redpanda-data/redpanda-operator#354
redpanda-data/redpanda-operator#365
RafalKorepta added a commit to redpanda-data/helm-charts that referenced this pull request Jan 8, 2025
The source of truth is now redpanda-operator repostiory where gotohelm,
genpartial and genschema cli are located. Those tools will generate inside
redpanda-operator repository the following charts:
* connectors
* console
* operator
* redpanda

Reference
redpanda-data/redpanda-operator#339
redpanda-data/redpanda-operator#354
redpanda-data/redpanda-operator#365
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants