Skip to content

ci: bump trivy-action workflow#314

Closed
krishnaGajabi wants to merge 1 commit intoopenebs:developfrom
krishnaGajabi:develop
Closed

ci: bump trivy-action workflow#314
krishnaGajabi wants to merge 1 commit intoopenebs:developfrom
krishnaGajabi:develop

Conversation

@krishnaGajabi
Copy link
Copy Markdown

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 container image registry.
  imageRegistry: ""
  # - secret
  imagePullSecrets: []
  # Global overide for image pull policy
  imagePullPolicy: ""
  analytics:
    # Global overide for analytics
    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.

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: Prateek Chandra <Prateek.Chandra@datacore.com>
@krishnaGajabi krishnaGajabi requested a review from a team as a code owner March 18, 2026 08:17
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 18, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 54.00%. Comparing base (0c375ea) to head (2e17afb).
⚠️ Report is 2 commits behind head on develop.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #314   +/-   ##
========================================
  Coverage    54.00%   54.00%           
========================================
  Files            1        1           
  Lines          474      474           
========================================
  Hits           256      256           
  Misses         209      209           
  Partials         9        9           
Flag Coverage Δ
integrationtests 54.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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