Skip to content

Commit

Permalink
Merge pull request jupyterhub#2215 from consideRatio/pr/helm-template…
Browse files Browse the repository at this point in the history
…-cert-manager-also

ci: helm template --validate the cert-manager chart as well
  • Loading branch information
consideRatio authored May 28, 2022
2 parents 0cf568b + 6ef0445 commit 330dd64
Showing 1 changed file with 13 additions and 4 deletions.
17 changes: 13 additions & 4 deletions .github/workflows/test-helm-template.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,9 @@ on:
workflow_dispatch:

jobs:
# Validate rendered helm templates against a k8s api-server with a
# matching version of where we look to deploy.
#
helm-template:
runs-on: ubuntu-20.04
strategy:
Expand Down Expand Up @@ -73,10 +76,16 @@ jobs:
- name: Install cert-manager CRDs
run: kubectl apply -f https://github.com/jetstack/cert-manager/releases/download/${CERT_MANAGER_VERSION}/cert-manager.crds.yaml

# Validate rendered helm templates against a k8s api-server with a
# matching version of where we look to deploy.
#
- name: "helm template --validate"
- name: "helm template --validate (cert-manager chart)"
run: |
echo "cert-manager version: ${CERT_MANAGER_VERSION}"
helm template --validate cert-manager cert-manager \
--repo=https://charts.jetstack.io \
--version=${CERT_MANAGER_VERSION} \
--values=config/cert-manager.yaml
- name: "helm template --validate (mybinder chart)"
run: |
helm template --validate mybinder ./mybinder \
--values=config/${{ matrix.release }}.yaml \
Expand Down

0 comments on commit 330dd64

Please sign in to comment.