Skip to content

Feat/delete unessecary #3

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

Closed
michi8620 wants to merge 2 commits into from
Closed

Feat/delete unessecary #3

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
0167039
deleted kubeconform action
mg365348 Nov 8, 2025
dee281d
return helm script
mg365348 Nov 8, 2025
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
6 changes: 0 additions & 6 deletions .github/workflows/ci-kind-deploy.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -32,12 +32,6 @@ jobs:
- name: Render chart manifests
run: helm template ./helm > rendered-manifests.yaml

- name: Validate rendered manifests with kubeconform
uses: instrumenta/kubeconform-action@v1
with:
files: rendered-manifests.yaml
args: -strict

unit-tests:
name: Unit tests
runs-on: ubuntu-latest
Expand Down
18 changes: 18 additions & 0 deletions scripts/helm_lint.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,18 @@
#!/usr/bin/env bash
# Lint a Helm chart directory. Exits non-zero on failure.

set -euo pipefail

CHART_DIR="${1:-./helm}"

echo "Running helm lint for chart directory: ${CHART_DIR}"

if ! command -v helm >/dev/null 2>&1; then
echo "helm is required but not found in PATH" >&2
exit 2
fi

helm lint "${CHART_DIR}" \
|| { echo "helm lint failed" >&2; exit 1; }

echo "helm lint succeeded for ${CHART_DIR}"
Comment on lines +1 to +18
Copy link

@coderabbitai coderabbitai bot Nov 8, 2025

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🔴 Critical

Script is not executable—add execute permissions.

The pipeline failure (exit code 126) indicates the script lacks execute permissions. This is a blocker preventing the script from running.

Add execute permissions by running:

chmod +x scripts/helm_lint.sh

Or include this change in the PR by adding it to .gitattributes or committing with execute permissions set in git:

git update-index --chmod=+x scripts/helm_lint.sh
🧰 Tools
🪛 GitHub Actions: CI - Kind Helm PR workflow

[error] 1-1: Permission denied while attempting to execute script './scripts/helm_lint.sh ./helm'. Ensure the script is executable (chmod +x scripts/helm_lint.sh). Process exited with code 126.

🤖 Prompt for AI Agents 
In scripts/helm_lint.sh around lines 1 to 18, the script file lacks execute
permissions causing the pipeline to fail with exit code 126; fix by making the
file executable in the repo — run chmod +x scripts/helm_lint.sh locally and
commit the permission change, or use git update-index --chmod=+x
scripts/helm_lint.sh before committing so the executable bit is recorded
(alternatively ensure your CI patches set the executable bit or add appropriate
.gitattributes config).

Loading