Skip to content

Conversation

nilgaar
Copy link

@nilgaar nilgaar commented Oct 7, 2025

What kind of change does this PR introduce?

This PR splits the Helm chart CI workflow into two separate workflows — one for linting and one for installation tests.
This change improves clarity, reduces runtime for simple validations, and enables maintainers to approve or trigger heavy install tests only when needed.

What is the current behavior?

Currently, the Lint and Test Charts workflow performs both linting and installation tests in a single run, requiring manual approval before execution.
This setup slows down the feedback loop and consumes resources even for minor changes.

What is the new behavior?

  • Introduces two distinct GitHub Actions:

    • lint-charts.yml: runs automatically on PRs for fast lint feedback.
    • test-charts.yml: runs only after lint passes and is gated by environment approval (github-cicd).
  • Maintains the same chart-testing logic and cluster setup for installation tests.

  • Keeps existing validation parity while improving CI efficiency and control.

Additional context

a github-cicd environment has to be created under Repository Settings → Environments, with the appropriate required reviewers configured to control when installation tests can run.

@nilgaar
Copy link
Author

nilgaar commented Oct 7, 2025

Fixes #126

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