Skip to content

[Build] update helm lint command to match with helm v3/v4 #395

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
sajmera-pensando merged 1 commit into from
Dec 17, 2025
Merged

[Build] update helm lint command to match with helm v3/v4 #395

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions Makefile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -332,7 +332,7 @@ helm-k8s: helmify manifests kustomize clean-helm-k8s gen-kmm-charts-k8s ## Build
mkdir -p $(shell pwd)/helm-charts-k8s/charts/remediation/templates
cp $(shell pwd)/hack/k8s-patch/k8s-remediation-patch/metadata-patch/*.yaml $(shell pwd)/helm-charts-k8s/charts/remediation/
cp $(shell pwd)/hack/k8s-patch/k8s-remediation-patch/template-patch/*.yaml $(shell pwd)/helm-charts-k8s/charts/remediation/templates/
cd $(shell pwd)/helm-charts-k8s; helm dependency update; helm lint; cd ..;
cd $(shell pwd)/helm-charts-k8s; helm dependency update; helm lint .; cd ..;
mkdir $(shell pwd)/helm-charts-k8s/crds
echo "moving crd yaml files to crds folder"
@for file in $(CRD_YAML_FILES); do \
Expand All @@ -341,7 +341,7 @@ helm-k8s: helmify manifests kustomize clean-helm-k8s gen-kmm-charts-k8s ## Build
rm $(shell pwd)/helm-charts-k8s/templates/*crd.yaml
$(MAKE) helm-docs
echo "dependency update, lint and pack charts"
cd $(shell pwd)/helm-charts-k8s; helm dependency update; helm lint; cd ..; helm package helm-charts-k8s/ --destination ./helm-charts-k8s
cd $(shell pwd)/helm-charts-k8s; helm dependency update; helm lint .; cd ..; helm package helm-charts-k8s/ --destination ./helm-charts-k8s
mv $(shell pwd)/helm-charts-k8s/gpu-operator-charts-$(PROJECT_VERSION).tgz $(shell pwd)/helm-charts-k8s/gpu-operator-helm-k8s-$(PROJECT_VERSION).tgz

.PHONY: helm-openshift
Expand All @@ -361,15 +361,15 @@ helm-openshift: helmify manifests kustomize clean-helm-openshift gen-nfd-charts-
rm $(shell pwd)/helm-charts-openshift/charts/kmm/templates/device-plugin-rbac.yaml
# opeartor already has module-loader rbac yaml, removing the redundant rbac yaml from subchart
rm $(shell pwd)/helm-charts-openshift/charts/kmm/templates/module-loader-rbac.yaml
cd $(shell pwd)/helm-charts-openshift; helm dependency update; helm lint; cd ..;
cd $(shell pwd)/helm-charts-openshift; helm dependency update; helm lint .; cd ..;
mkdir $(shell pwd)/helm-charts-openshift/crds
echo "moving crd yaml files to crds folder"
@for file in $(CRD_YAML_FILES); do \
helm template amd-gpu helm-charts-openshift -s templates/$$file > $(shell pwd)/helm-charts-openshift/crds/$$file; \
done
rm $(shell pwd)/helm-charts-openshift/templates/*crd.yaml
echo "dependency update, lint and pack charts"
cd $(shell pwd)/helm-charts-openshift; helm dependency update; helm lint; cd ..; helm package helm-charts-openshift/ --destination ./helm-charts-openshift
cd $(shell pwd)/helm-charts-openshift; helm dependency update; helm lint .; cd ..; helm package helm-charts-openshift/ --destination ./helm-charts-openshift
mv $(shell pwd)/helm-charts-openshift/gpu-operator-charts-$(PROJECT_VERSION).tgz $(shell pwd)/helm-charts-openshift/gpu-operator-helm-openshift-$(PROJECT_VERSION).tgz

.PHONY: bundle-build
Expand Down
Loading