Skip to content

Commit

Permalink
add helm linter github action (Azure#612)
Browse files Browse the repository at this point in the history
* add helm linter github action
* use sha for dependabot version management
* use sha for dependabot version management
  • Loading branch information
tony-schndr authored Sep 18, 2024
1 parent 915e6fa commit bf5117c
Show file tree
Hide file tree
Showing 3 changed files with 58 additions and 3 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/aro-hcp-dev-env-cd.yml
Original file line number Diff line number Diff line change
Expand Up @@ -379,7 +379,7 @@
chmod +x /usr/local/bin/oc
# Used to deploy Maestro Server, Frontend
- uses: azure/[email protected]
- uses: azure/setup-helm@fe7b79cd5ee1e45176fcad797de68ecaf3ca4814 # v4.2.0
with:
version: 'v3.13.3'

Expand Down Expand Up @@ -473,7 +473,7 @@
chmod +x /usr/local/bin/oc
# Used to deploy Maestro Agent
- uses: azure/[email protected]
- uses: azure/setup-helm@fe7b79cd5ee1e45176fcad797de68ecaf3ca4814 # v4.2.0
with:
version: 'v3.13.3'

Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/cs-integration-env-cd.yml
Original file line number Diff line number Diff line change
Expand Up @@ -254,7 +254,7 @@
chmod +x /usr/local/bin/oc
# Used to deploy Maestro Server
- uses: azure/[email protected]
- uses: azure/setup-helm@fe7b79cd5ee1e45176fcad797de68ecaf3ca4814 # v4.2.0
with:
version: 'v3.13.3'

Expand Down
55 changes: 55 additions & 0 deletions .github/workflows/helm-lint.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,55 @@
---
# Helm chart lint
# More info at https://github.com/helm/chart-testing-action
name: Lint Helm Charts

on: pull_request

jobs:
lint-test:
runs-on: ubuntu-latest
steps:
- name: Checkout
uses: actions/checkout@v3
with:
fetch-depth: 0

- name: Set up Helm
uses: azure/setup-helm@fe7b79cd5ee1e45176fcad797de68ecaf3ca4814 # v4.2.0
with:
version: v3.13.3

- uses: actions/setup-python@f677139bbe7f9c59b41e40162b753c062f5d49a3 # v5.2.0
with:
python-version: '3.x'
check-latest: true

- name: Set up chart-testing
uses: helm/chart-testing-action@e6669bcd63d7cb57cb4380c33043eebe5d111992 # v2.6.1

- name: Collect chart directories
id: collect-chart-dirs
run: |
# Find all parent directories of directories containing 'Chart.yaml', output via csv without the leading './'
CHART_DIRS=$(find . -type f -name 'Chart.yaml' -printf '%h\n' |sed 's|^\./||' | xargs -I {} dirname {} | sort -u | tr '\n' ',' | sed 's/,$//' )
echo "found charts in the following directories ${CHART_DIRS}"
echo "CHART_DIRS=$CHART_DIRS" >> "$GITHUB_ENV"
- name: Run chart-testing (list-changed)
id: list-changed
run: |
changed=$(ct list-changed --chart-dirs=${CHART_DIRS} --target-branch ${{ github.event.repository.default_branch }})
if [[ -n "$changed" ]]; then
echo "changed=true" >> "$GITHUB_ENV"
fi
- name: Run chart-testing (lint)
id: lint-helm
if: env.changed == 'true'
run: |
echo "Running helm linter on charts in the following directories: $CHART_DIRS"
ct lint \
--chart-dirs=$CHART_DIRS \
--all \
--validate-maintainers=false \
--target-branch ${{ github.event.repository.default_branch }}

0 comments on commit bf5117c

Please sign in to comment.