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

helm: Remove deprecated tetragonOperator.skipCRDCreation value #2907

Conversation

BonySmoke
Copy link
Contributor

@BonySmoke BonySmoke commented Sep 11, 2024

Description

In this PR #2498, the tetragon.skipCRDCreation key was removed from the values.yaml file; however, there is still a reference to it in the operator config map.
From what I understand, the operator expects a boolean value, and in case the installation method is operator, it should be set to false.

The patch makes the following changes:

  1. Remove deprecated tetragon.skipCRDCreation from the operator config map and set false as the static value in skip-crd-creation.
  2. Rename tetragon.skipCRDCreation to tetragonOperator.skipCRDCreation in the upgrade notes for Tetragon 1.2.0.

Changelog

Remove deprecated `tetragonOperator.skipCRDCreation` from the operator config map and set "false" when `.Values.crds.installMethod` is `operator`

The patch makes the following changes:
1. Remove deprecated "tetragon.skipCRDCreation" from the operator config map and set "false" as the static value in "skip-crd-creation"
2. Rename "tetragon.skipCRDCreation" to "tetragonOperator.skipCRDCreation" in the upgrade notes for Tetragon 1.2.0

Signed-off-by: Oleh Neichev <[email protected]>
@BonySmoke BonySmoke requested a review from a team as a code owner September 11, 2024 11:19
@BonySmoke BonySmoke requested a review from tpapagian September 11, 2024 11:19
@BonySmoke BonySmoke changed the title helm: Remove deprecated tetragon.skipCRDCreation value helm: Remove deprecated tetragonOperator.skipCRDCreation value Sep 12, 2024
@mtardy mtardy added the release-note/misc This PR makes changes that have no direct user impact. label Sep 12, 2024
@lambdanis lambdanis added area/helm Related to the Helm chart kind/bug Something isn't working labels Sep 13, 2024
Copy link
Contributor

@lambdanis lambdanis left a comment

Choose a reason for hiding this comment

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

Thanks!

@lambdanis lambdanis added the needs-backport/1.2 This PR needs backporting to 1.2 label Sep 13, 2024
@lambdanis lambdanis merged commit 4697850 into cilium:main Sep 13, 2024
42 of 45 checks passed
@lambdanis lambdanis added backport-pending/1.2 The backport for this PR is in progress. backport-done/1.2 PR backport done. and removed needs-backport/1.2 This PR needs backporting to 1.2 backport-pending/1.2 The backport for this PR is in progress. labels Sep 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/helm Related to the Helm chart backport-done/1.2 PR backport done. kind/bug Something isn't working release-note/misc This PR makes changes that have no direct user impact.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants