Skip to content

Commit afe286b

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

File tree

2 files changed

+16
-6
lines changed

2 files changed

+16
-6
lines changed

.github/workflows/test-pr.yml

+16-2
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,14 @@ jobs:
3535
chart_validation:
3636
needs: [publish_images]
3737
runs-on: ubuntu-latest
38+
strategy:
39+
fail-fast: false
40+
matrix:
41+
chart:
42+
- azimuth-llm-chat
43+
- azimuth-llm-image-analysis
44+
- azimuth-llm
45+
- flux-image-gen
3846
env:
3947
CLUSTER_NAME: chart-testing
4048
RELEASE_NAME: ci-test
@@ -59,13 +67,16 @@ jobs:
5967
uses: helm/chart-testing-action@v2
6068

6169
- name: Run chart linting
62-
run: ct lint --config ct.yaml
70+
run: ct lint --config ct.yaml --charts charts/${{ matrix.chart }}
6371

6472
- name: Create Kind Cluster
6573
uses: helm/kind-action@v1
6674
with:
6775
cluster_name: ${{ env.CLUSTER_NAME }}
6876

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

88+
- name: Debug
89+
run: (df -h && docker ps -a && docker image ls && kubectl get nodes && kubectl get pods --all-namespaces) || true
90+
7791
# https://github.com/helm/charts/blob/master/test/README.md#providing-custom-test-values
7892
# Each chart/ci/*-values.yaml file will be treated as a separate test case with it's
7993
# own helm install/test process.
8094
- name: Run chart install and test
81-
run: ct install --config ct.yaml
95+
run: ct install --config ct.yaml --charts charts/${{ matrix.chart }}
8296

8397
publish_charts:
8498
needs: [chart_validation]

ct.yaml

-4
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)