Skip to content

Comments

Adding a CI action to validate helm chart renderings#199

Merged
Shivs11 merged 4 commits intomainfrom
improvements-to-ci
Feb 19, 2026
Merged

Adding a CI action to validate helm chart renderings#199
Shivs11 merged 4 commits intomainfrom
improvements-to-ci

Conversation

@Shivs11
Copy link
Member

@Shivs11 Shivs11 commented Feb 17, 2026 •
edited
Loading

What was changed

  • WISOTT
  • I have added some very basic "helm lint and helm template" validation checks in this PR. One could very well build on this by adding something like "is the image that is being pulled a valid docker image". I decided to keep it small and just get this in, for now, so that we have a pipeline to test things against.

Why?

  • to detect failures quite early

Checklist

  1. Closes
    Automate validation of PR's changing the Helm chart. #191

  2. How was this tested:

  1. Any docs updates needed?

@Shivs11 Shivs11 marked this pull request as ready for review February 17, 2026 20:14
@Shivs11 Shivs11 requested review from a team and jlegrone as code owners February 17, 2026 20:14
Shivs11
Shivs11 commented Feb 17, 2026
View reviewed changes
@carlydf
Copy link
Collaborator

carlydf commented Feb 19, 2026

closes #200

@Shivs11 Shivs11 enabled auto-merge (squash) February 19, 2026 19:46
@Shivs11 Shivs11 merged commit 0d0da97 into main Feb 19, 2026
14 checks passed
@Shivs11 Shivs11 deleted the improvements-to-ci branch February 19, 2026 19:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@carlydf carlydf carlydf approved these changes

@jlegrone jlegrone Awaiting requested review from jlegrone jlegrone is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Shivs11 @carlydf