Skip to content

Commit

Permalink
Fix PAAS cluster role (#233)
Browse files Browse the repository at this point in the history
  • Loading branch information
treydock authored Apr 3, 2024
1 parent a6ea5bc commit ba4603a
Show file tree
Hide file tree
Showing 4 changed files with 9 additions and 10 deletions.
11 changes: 5 additions & 6 deletions .github/workflows/test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -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)
Expand Down
2 changes: 1 addition & 1 deletion charts/paas/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down
2 changes: 1 addition & 1 deletion charts/paas/README.md
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
# paas

![Version: 0.2.0](https://img.shields.io/badge/Version-0.2.0-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: 0.1.0](https://img.shields.io/badge/AppVersion-0.1.0-informational?style=flat-square)
![Version: 0.2.1](https://img.shields.io/badge/Version-0.2.1-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: 0.1.0](https://img.shields.io/badge/AppVersion-0.1.0-informational?style=flat-square)

OSC PAAS bootstrap Helm Chart

Expand Down
4 changes: 2 additions & 2 deletions charts/paas/templates/clusterrole.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand All @@ -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
Expand Down

0 comments on commit ba4603a

Please sign in to comment.