Skip to content

Commit

Permalink
remove ct install check
Browse files Browse the repository at this point in the history
Signed-off-by: Ryan Lo <[email protected]>
  • Loading branch information
lowc1012 committed Mar 16, 2024
1 parent 1bc5e9d commit 4173437
Showing 1 changed file with 8 additions and 7 deletions.
15 changes: 8 additions & 7 deletions .github/workflows/validate-helm-charts.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -51,13 +51,14 @@ jobs:
--chart-repos twuni=https://helm.twun.io/ \
--chart-repos kubernetes-dashboard=https://kubernetes.github.io/dashboard/
- name: Create KinD cluster
if: steps.charts-changed.outputs.changed == 'true'
uses: helm/kind-action@v1

- name: Run chart-testing (install)
if: steps.charts-changed.outputs.changed == 'true'
run: ct install --target-branch ${{ github.event.repository.default_branch }}
# TODO: Prepare dependencies (e.g. db) for testing. Currently `ct install` fails because no database & object store, etc
# - name: Create KinD cluster
# if: steps.charts-changed.outputs.changed == 'true'
# uses: helm/kind-action@v1
#
# - name: Run chart-testing (install)
# if: steps.charts-changed.outputs.changed == 'true'
# run: ct install --target-branch ${{ github.event.repository.default_branch }}

validate-manifests:
needs:
Expand Down

0 comments on commit 4173437

Please sign in to comment.