Skip to content

Commit

Permalink
Update Helm chart repo actions to latest versions. Note we may want t…
Browse files Browse the repository at this point in the history
…o add GPG signing

Signed-off-by: Scott Rigby <[email protected]>
  • Loading branch information
scottrigby committed Dec 14, 2020
1 parent c9937aa commit 905fecc
Show file tree
Hide file tree
Showing 2 changed files with 33 additions and 14 deletions.
4 changes: 3 additions & 1 deletion .github/workflows/helm-cd.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,8 @@ jobs:
git config user.email "[email protected]"
- name: Run chart-releaser
uses: helm/[email protected]
uses: helm/[email protected]
with:
charts_dir: charts
env:
CR_TOKEN: "${{ secrets.GITHUB_TOKEN }}"
43 changes: 30 additions & 13 deletions .github/workflows/helm-ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -14,20 +14,37 @@ jobs:
with:
fetch-depth: 0

- name: Lint chart
id: lint
uses: helm/[email protected]
- name: Set up Helm
uses: azure/setup-helm@v1
with:
command: lint
config: tests/ct.yaml
version: v3.4.1

- name: Create kind cluster
uses: helm/[email protected]
if: steps.lint.outputs.changed == 'true'
# Python is required because `ct lint` runs Yamale (https://github.com/23andMe/Yamale) and
# yamllint (https://github.com/adrienverge/yamllint) which require Python
- name: Set up Python
uses: actions/setup-python@v2
with:
python-version: 3.7

- name: Install chart
uses: helm/[email protected]
if: steps.lint.outputs.changed == 'true'
- name: Set up chart-testing
uses: helm/[email protected]
with:
command: install
config: tests/ct.yaml
version: v3.3.1

- name: Run chart-testing (list-changed)
id: list-changed
run: |
changed=$(ct list-changed --config ct.yaml)
if [[ -n "$changed" ]]; then
echo "::set-output name=changed::true"
fi
- name: Run chart-testing (lint)
run: ct lint --config tests/ct.yaml

- name: Create kind cluster
uses: helm/[email protected]
if: steps.list-changed.outputs.changed == 'true'

- name: Run chart-testing (install)
run: ct install --config tests/ct.yaml

0 comments on commit 905fecc

Please sign in to comment.