Skip to content

Add helm chart linting to CI#227

Merged
clux merged 4 commits into
kube-rs:mainfrom
doxxx93:feat/lint-helm-#41
Feb 16, 2026
Merged

Add helm chart linting to CI#227
clux merged 4 commits into
kube-rs:mainfrom
doxxx93:feat/lint-helm-#41

Conversation

@doxxx93

@doxxx93 doxxx93 commented Feb 13, 2026

Copy link
Copy Markdown
Member

Motivation

Closes #41

The project uses Helm charts but the CI lint job only covers Rust (rustfmt, clippy). There is no validation for the Helm chart structure or generated Kubernetes manifests.

Solution

  • Add lint-helm recipe to justfile for local and CI use
  • Integrate into the existing lint job in CI:
    • helm lint to validate chart structure and values
    • helm template | kubeconform --strict to validate generated manifests against Kubernetes schemas
  • Tools installed in CI: azure/setup-helm@v4, kubeconform, extractions/setup-just@v3

@doxxx93 doxxx93 requested a review from clux February 13, 2026 09:02
@doxxx93

doxxx93 commented Feb 13, 2026

Copy link
Copy Markdown
Member Author

@clux This addresses the same issue as #64 (draft). I took a slightly different approach using helm lint + helm template | kubeconform --strict with CLI installation instead of the Docker container method. Happy to adjust if you have any preferences!

Signed-off-by: doxxx93 <doxxx93@gmail.com>
…chemas

Signed-off-by: doxxx93 <doxxx93@gmail.com>
…ation

Signed-off-by: doxxx93 <doxxx93@gmail.com>
@doxxx93

doxxx93 commented Feb 13, 2026

Copy link
Copy Markdown
Member Author

Switched to the official kubeconform Docker image (docker://ghcr.io/yannh/kubeconform:latest) as described in the kubeconform README.

@clux clux left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks great. thank you.

@clux clux merged commit 350457c into kube-rs:main Feb 16, 2026
8 checks passed
@doxxx93 doxxx93 deleted the feat/lint-helm-#41 branch February 17, 2026 03:21
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.

add linting of yaml

2 participants