Skip to content

Commit 82d1298

Browse files
committed
Add debug logs && matrix strategy for charts
1 parent 4ab144f commit 82d1298

File tree

2 files changed

+12
-6
lines changed

2 files changed

+12
-6
lines changed

.github/workflows/test-pr.yml

Lines changed: 12 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,10 @@ jobs:
3535
chart_validation:
3636
needs: [publish_images]
3737
runs-on: ubuntu-latest
38+
strategy:
39+
fail-fast: false
40+
matrix:
41+
chart: ["azimuth-llm-chat", "azimuth-llm-image-analysis", "azimuth-llm", "flux-image-gen"]
3842
env:
3943
CLUSTER_NAME: chart-testing
4044
RELEASE_NAME: ci-test
@@ -59,13 +63,16 @@ jobs:
5963
uses: helm/chart-testing-action@v2
6064

6165
- name: Run chart linting
62-
run: ct lint --config ct.yaml
66+
run: ct lint --config ct.yaml --charts charts/${{ matrix.chart }}
6367

6468
- name: Create Kind Cluster
6569
uses: helm/kind-action@v1
6670
with:
6771
cluster_name: ${{ env.CLUSTER_NAME }}
6872

73+
- name: Debug
74+
run: (df -h && docker ps -a && docker image ls && kubectl get nodes && kubectl get pods --all-namespaces) || true
75+
6976
# NOTE(scott): Since the local Chart.yaml uses "appVersion: latest" and this
7077
# only gets overwritten to the correct commit SHA during Helm chart build,
7178
# we need to pull these published images and load them into the kind cluster
@@ -74,11 +81,14 @@ jobs:
7481
run: ./kind-images.sh $(git rev-parse --short ${{ github.event.pull_request.head.sha }}) ${{ env.CLUSTER_NAME }}
7582
working-directory: web-apps
7683

84+
- name: Debug
85+
run: (df -h && docker ps -a && docker image ls && kubectl get nodes && kubectl get pods --all-namespaces) || true
86+
7787
# https://github.com/helm/charts/blob/master/test/README.md#providing-custom-test-values
7888
# Each chart/ci/*-values.yaml file will be treated as a separate test case with it's
7989
# own helm install/test process.
8090
- name: Run chart install and test
81-
run: ct install --config ct.yaml
91+
run: ct install --config ct.yaml --charts charts/${{ matrix.chart }}
8292

8393
publish_charts:
8494
needs: [chart_validation]

ct.yaml

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,5 @@
11
# Complains about invalid maintainer URLs
22
validate-maintainers: false
3-
# Skip version bump detection and lint all charts
4-
# since we're using the azimuth-cloud Helm chart publish
5-
# workflow which doesn't use Chart.yaml's version key
6-
all: true
73
# Split output to make it look nice in GitHub Actions tab
84
github-groups: true
95
# Allow for long running install and test processes

0 commit comments

Comments
 (0)