Skip to content

ci: add helm lint workflow triggered on helm chart changes#1160

Merged
TaylorMutch merged 3 commits intotmutch/kube-e2efrom
tmutch/helm-lint-ci
May 5, 2026
Merged

ci: add helm lint workflow triggered on helm chart changes#1160
TaylorMutch merged 3 commits intotmutch/kube-e2efrom
tmutch/helm-lint-ci

Conversation

@TaylorMutch
Copy link
Copy Markdown
Collaborator

Summary

  • Adds a dedicated helm-lint.yml workflow that runs mise run helm:lint on PRs that touch deploy/helm/**
  • Path filter on the trigger means the workflow is skipped entirely for unrelated PRs
  • Uses the same pr_metadata gate and CI image as the other branch-checks workflows

Related Issue

Stacked on #1159 (which introduces the helm:lint matrix task and the ci/ values directory).

Changes

  • .github/workflows/helm-lint.yml — new workflow: triggers on deploy/helm/** path changes, runs mise run helm:lint in the CI container

Testing

  • Verify workflow triggers on a PR with helm chart changes
  • Verify workflow is skipped on PRs with no helm chart changes

Checklist

  • Follows conventional commits format
  • No secrets or credentials committed

@TaylorMutch TaylorMutch requested a review from a team as a code owner May 5, 2026 00:03
@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot Bot commented May 5, 2026

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@TaylorMutch
Copy link
Copy Markdown
Collaborator Author

/ok to test d605eab

@TaylorMutch
Copy link
Copy Markdown
Collaborator Author

/ok to test 6b6b0a5

@TaylorMutch TaylorMutch merged commit d15cafe into tmutch/kube-e2e May 5, 2026
4 checks passed
@TaylorMutch TaylorMutch deleted the tmutch/helm-lint-ci branch May 5, 2026 00:22
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