Skip to content

CSPL-4631: add Helm chart support for feature gates#1860

Open
kubabuczak wants to merge 1 commit intodevelopfrom
CSPL-4631-helm-feature-gates
Open

CSPL-4631: add Helm chart support for feature gates#1860
kubabuczak wants to merge 1 commit intodevelopfrom
CSPL-4631-helm-feature-gates

Conversation

@kubabuczak
Copy link
Copy Markdown
Collaborator

Description

Add splunkOperator.featureGates map to the Helm chart so operators can enable feature gates via helm install/upgrade without per-gate chart modifications.

Key Changes

  • Add featureGates map to values and a helper template that formats it into --feature-gates=Key=bool,...
  • Deployment template conditionally renders the arg when the map is non-empty
  • Add helm-unittest framework with 25 tests and make helm-check target
  • Update FeatureGates.md and ValidationWebhook.md with Helm usage

Testing and Verification

  • helm lint passes
  • helm template verified for 0, 1, and 2 gates
  • 25 helm-unittest tests pass (make helm-check)

Related Issues

PR Checklist

  • Code changes adhere to the project's coding standards
  • Relevant unit and integration tests are included
  • Documentation has been updated accordingly
  • If test framework files were changed (test/testenv/, test/run-tests.sh, test/env.sh), docs/IntegrationTesting.md has been updated
  • All tests pass locally
  • The PR description follows the project's guidelines

Add splunkOperator.featureGates map to values so operators can
enable feature gates via helm install/upgrade without per-gate
chart modifications. A helper template formats the map into a
single --feature-gates=Key1=true,Key2=false arg; the deployment
template conditionally includes it when the map is non-empty.

Also introduces helm-unittest with Makefile targets (helm-lint,
helm-test, helm-check) and fixes a pre-existing nindent bug in
the extraEnvs rendering.
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.

4 participants