Fix PAAS cluster role

OSC · Apr 3, 2024 · 57faaef · 57faaef
1 parent a6ea5bc
commit 57faaef
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 9 deletions.
diff --git a/.github/workflows/test.yaml b/.github/workflows/test.yaml
@@ -95,12 +95,11 @@ jobs:
         run: |
           helm repo add kyverno https://kyverno.github.io/kyverno/
           helm install kyverno kyverno/kyverno -n kyverno --create-namespace --version 3.1.4
-      # TODO: Implement if/when need to validate with cert-manager resources
-      #- name: Install cert-manager
-      #  if: matrix.chart == 'paas'
-      #  run: |
-      #    helm repo add jetstack https://charts.jetstack.io
-      #    helm install cert-manager jetstack/cert-manager -n cert-manager --create-namespace --version v1.14.4 --set installCRDs=true
+      - name: Install cert-manager
+        if: matrix.chart == 'paas'
+        run: |
+          helm repo add jetstack https://charts.jetstack.io
+          helm install cert-manager jetstack/cert-manager -n cert-manager --create-namespace --version v1.14.4 --set installCRDs=true
       - name: Add namespaces
         run: kubectl create namespace ${{ matrix.chart }}
       - name: Run chart-testing (install)

diff --git a/charts/paas/Chart.yaml b/charts/paas/Chart.yaml
@@ -2,7 +2,7 @@ apiVersion: v2
 name: paas
 description: OSC PAAS bootstrap Helm Chart
 type: application
-version: 0.2.0
+version: 0.2.1
 appVersion: "0.1.0"
 maintainers:
   - name: treydock

diff --git a/charts/paas/templates/clusterrole.yaml b/charts/paas/templates/clusterrole.yaml
@@ -68,7 +68,7 @@ rules:
     - update
     - delete
 {{ if .Capabilities.APIVersions.Has "cert-manager.io/v1" -}}
-- apiGroup: ["cert-manager.io"]
+- apiGroups: ["cert-manager.io"]
   resources:
     - certificates
     - certificaterequests
@@ -78,7 +78,7 @@ rules:
     - watch
 {{ end -}}
 {{ if .Capabilities.APIVersions.Has "acme.cert-manager.io/v1" -}}
-- apiGroup: ["acme.cert-manager.io"]
+- apiGroups: ["acme.cert-manager.io"]
   resources:
     - orders
     - challenges