Skip to content

Commit

Permalink
Merge pull request goharbor#49 from holyhope/dev/46-deploy-certmanage…
Browse files Browse the repository at this point in the history
…r-new-way

fix(ci) New way to deploy CertManager CRDs
  • Loading branch information
steven-zou committed Apr 16, 2020
2 parents c8e922b + 5e9cc5d commit 1e2a59e
Showing 1 changed file with 8 additions and 3 deletions.
11 changes: 8 additions & 3 deletions .github/workflows/tests.yml
Expand Up @@ -76,8 +76,8 @@ jobs:
strategy:
fail-fast: false
matrix:
# https://github.com/jetstack/cert-manager/branches/active
certManager: ["0.12", "0.13", "0.14", "0.15"]
# https://github.com/jetstack/cert-manager/tags
certManager: ["0.12.0", "0.13.1", "0.14.2", "0.15.0-alpha.0"]
# https://snapcraft.io/microk8s
k8sVersion: ["1.15", "1.16", "1.17"]

Expand All @@ -102,7 +102,12 @@ jobs:
sleep 1
done
- name: Install CertManager v${{ matrix.certManager }}
run: sudo microk8s.kubectl apply -f "https://raw.githubusercontent.com/jetstack/cert-manager/release-${{ matrix.certManager }}/deploy/manifests/00-crds.yaml"
run: |
# Try the recet way to install crd or fallback to the old one
version='${{ matrix.certManager }}'
shortVersion=""${version%.*}""
sudo microk8s.kubectl apply -f "https://github.com/jetstack/cert-manager/releases/download/v${version}/cert-manager.crds.yaml" ||
sudo microk8s.kubectl apply -f "https://raw.githubusercontent.com/jetstack/cert-manager/release-${shortVersion}/deploy/manifests/00-crds.yaml"
- uses: actions/checkout@v2
- name: go tests
run: |
Expand Down

0 comments on commit 1e2a59e

Please sign in to comment.