Skip to content

feat: add global helm global values#831

Open
krishnaGajabi wants to merge 5 commits intoopenebs:developfrom
krishnaGajabi:develop
Open

feat: add global helm global values#831
krishnaGajabi wants to merge 5 commits intoopenebs:developfrom
krishnaGajabi:develop

Conversation

@krishnaGajabi
Copy link
Copy Markdown
Contributor

Why is this PR required? What issue does it fix?

Currently, the Helm charts do not provide global variables for configuring common parameters such as the image registry, image pull secrets, image pull policy, and analytics settings.

When deploying the OpenEBS Helm charts, users often need to override these values across multiple subcharts and components. This makes it difficult to locate and configure all the required fields consistently. Providing global variables simplifies configuration and improves the user experience.

What does this PR do?

This PR introduces the following global configuration variables:

global:
  # -- Global override for image registry
  imageRegistry: ""
  # -- Global overide for image pull policy
  imagePullPolicy: ""
  # -- Global override for image pull secrets
  # - secret
  imagePullSecrets: []
  analytics:
    # -- Global overide for call home
    enabled:

These variables allow users to configure common settings in a single place, which will then be applied across the chart components.

Does this PR require any upgrade changes?

No.
This change is backward compatible and does not require any upgrade steps.

Verification

The changes were verified using the following scenarios:

Rendered the templates using helm template to confirm correct value substitution.

Deployed the chart on a Kubernetes cluster with 3 worker nodes (kubeadm-based setup).

Verified that the global values are rendered and applied correctly across the components.

Signed-off-by: krishnaGajabi <gajbikrishna23@gmail.com>
Signed-off-by: krishnaGajabi <gajbikrishna23@gmail.com>
Signed-off-by: krishnaGajabi <gajbikrishna23@gmail.com>
Signed-off-by: krishnaGajabi <gajbikrishna23@gmail.com>
@krishnaGajabi krishnaGajabi requested a review from a team as a code owner April 1, 2026 12:28
Signed-off-by: krishnaGajabi <gajbikrishna23@gmail.com>
@krishnaGajabi
Copy link
Copy Markdown
Contributor Author

CI tests(images.sh script) are falling because we have set global.imageRegistry to an empty string. earlier the value used to be from localpv helm chart where it was explicitly set to docker.io. need the changes to be done in localpv before this changes are merged

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.

1 participant