Skip to content

Commit

Permalink
update post submit helm ci and clean up (#11220)
Browse files Browse the repository at this point in the history
Signed-off-by: cpanato <[email protected]>
  • Loading branch information
cpanato authored Apr 5, 2024
1 parent 531b007 commit 8ede0f7
Showing 1 changed file with 12 additions and 10 deletions.
22 changes: 12 additions & 10 deletions .github/workflows/helm.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -11,12 +11,13 @@ permissions:
jobs:

changes:
runs-on: ubuntu-latest

permissions:
contents: read # for dorny/paths-filter to fetch a list of changed files
pull-requests: read # for dorny/paths-filter to read pull requests
runs-on: ubuntu-latest
if: |
(github.repository == 'kubernetes/ingress-nginx')

if: github.repository == 'kubernetes/ingress-nginx'

outputs:
docs: ${{ steps.filter.outputs.docs }}
charts: ${{ steps.filter.outputs.charts }}
Expand All @@ -33,9 +34,11 @@ jobs:
./ah lint -p charts/ingress-nginx || exit 1
rm -f ./ah ./ah_1.5.0_linux_amd64.tar.gz
- name: Lint
run: |
./build/run-in-docker.sh ./hack/verify-chart-lint.sh
- name: Set up chart-testing
uses: helm/chart-testing-action@e6669bcd63d7cb57cb4380c33043eebe5d111992 # v2.6.1

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

- uses: dorny/paths-filter@de90cc6fb38fc0963ad72b210f1f284cd68cea36 # v3.0.2
id: filter
Expand All @@ -55,9 +58,8 @@ jobs:

needs:
- changes
if: |
(github.repository == 'kubernetes/ingress-nginx') &&
(needs.changes.outputs.charts == 'true')

if: ${{ needs.changes.outputs.charts == 'true' }}

steps:
- name: Checkout master
Expand Down

0 comments on commit 8ede0f7

Please sign in to comment.