use trim for multiple watch namespaces

stakater · May 3, 2021 · 3c189ee · 3c189ee
1 parent 8d430df
commit 3c189ee
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 5 deletions.
diff --git a/.github/workflows/pull_request.yml b/.github/workflows/pull_request.yml
@@ -20,8 +20,7 @@ jobs:
     - name: Check out code
       uses: actions/checkout@v2
       with:
-        fetch-depth: 0
-        ref: ${{ github.head_ref }}
+        ref: ${{github.event.pull_request.head.sha}}
 
     - name: Set up Go
       id: go

diff --git a/deploy/chart/ingressmonitorcontroller/templates/role.yaml b/deploy/chart/ingressmonitorcontroller/templates/role.yaml
@@ -4,7 +4,7 @@ apiVersion: rbac.authorization.k8s.io/v1
 kind: Role
 metadata:
   name: {{ template "name" $ }}-role
-  namespace: {{ . | quote }}
+  namespace: {{ . | trim }}
   labels:
 {{ include "labels" $ | indent 4 }}
 rules:

diff --git a/deploy/chart/ingressmonitorcontroller/templates/rolebinding.yaml b/deploy/chart/ingressmonitorcontroller/templates/rolebinding.yaml
@@ -4,14 +4,14 @@ kind: RoleBinding
 apiVersion: rbac.authorization.k8s.io/v1
 metadata:
   name: {{ template "name" $ }}-role-binding
-  namespace: {{ . | quote }}
+  namespace: {{ . | trim }}
   labels:
 {{ include "labels" $ | indent 4 }}
 roleRef:
   kind: Role
   name: {{ template "name" $ }}-role
   apiGroup: rbac.authorization.k8s.io
-  namespace: {{ . | quote }}
+  namespace: {{ . | trim }}
 subjects:
 - kind: ServiceAccount
   name: {{ template "serviceAccountName" $ }}