Skip to content

Commit

Permalink
ci: merge chart linting steps into ci.yml
Browse files Browse the repository at this point in the history
  • Loading branch information
Ben Blackmore committed Feb 1, 2023
1 parent 6d3bb5c commit 13e6d3a
Show file tree
Hide file tree
Showing 3 changed files with 28 additions and 48 deletions.
47 changes: 0 additions & 47 deletions .github/workflows/chart-testing.yaml

This file was deleted.

28 changes: 27 additions & 1 deletion .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -78,7 +78,7 @@ jobs:
platforms: linux/amd64,linux/arm64

test-helm-charts:
name: "Unit Test Helm Charts"
name: "Test Helm Charts"
runs-on: ubuntu-latest
needs:
- build-images
Expand Down Expand Up @@ -108,6 +108,32 @@ jobs:
- name: Run unit tests
run: make charttesting

- name: Set up chart-testing
uses: helm/chart-testing-action@v2.3.1
with:
version: v3.5.1

- name: Run chart-testing (list-changed)
id: list-changed
if: github.event_name == 'pull_request'
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 chartTesting.yaml
if: github.event_name == 'pull_request'

- name: Create kind cluster
uses: helm/kind-action@v1.2.0
if: steps.list-changed.outputs.changed == 'true'

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


release-helm-chart:
name: "Release Helm Chart"
runs-on: ubuntu-latest
Expand Down
1 change: 1 addition & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@
- add PERSONAL_ACCESS_TOKEN_USED_BY_CLA_FROM_ANSGAR as secret if you want to use the CLA support
- Docker image published where/how/how to make the image publicly accessible
- Helm chart
- when are image tags set

## Getting started

Expand Down

0 comments on commit 13e6d3a

Please sign in to comment.