Merge pull request #37 from deyaeddin/dev-01

fix: service account in the namespace as cert-manager
deyaeddin · May 20, 2021 · d859abd · d859abd
2 parents 8b2ed5d + fb3aacd
commit d859abd
Show file tree

Hide file tree

Showing 5 changed files with 6 additions and 2 deletions.
diff --git a/Dockerfile b/Dockerfile
@@ -21,6 +21,7 @@ FROM alpine:3.13.5
 RUN apk add --no-cache ca-certificates
 
 COPY --from=build /workspace/webhook /usr/local/bin/webhook
+
 RUN apk add libcap && setcap 'cap_net_bind_service=+ep' /usr/local/bin/webhook
 
 USER 1001

diff --git a/charts/cert-manager-webhook-hetzner/templates/deployment.yaml b/charts/cert-manager-webhook-hetzner/templates/deployment.yaml
@@ -2,6 +2,7 @@ apiVersion: apps/v1
 kind: Deployment
 metadata:
   name: {{ include "cert-manager-webhook-hetzner.fullname" . }}
+  namespace: {{ .Release.Namespace | quote }}
   labels:
     app: {{ include "cert-manager-webhook-hetzner.name" . }}
     chart: {{ include "cert-manager-webhook-hetzner.chart" . }}

diff --git a/charts/cert-manager-webhook-hetzner/templates/rbac.yaml b/charts/cert-manager-webhook-hetzner/templates/rbac.yaml
@@ -2,6 +2,7 @@ apiVersion: v1
 kind: ServiceAccount
 metadata:
   name: {{ include "cert-manager-webhook-hetzner.fullname" . }}
+  namespace: {{ .Release.Namespace | quote }}
   labels:
     app: {{ include "cert-manager-webhook-hetzner.name" . }}
     chart: {{ include "cert-manager-webhook-hetzner.chart" . }}

diff --git a/charts/cert-manager-webhook-hetzner/templates/service.yaml b/charts/cert-manager-webhook-hetzner/templates/service.yaml
@@ -2,6 +2,7 @@ apiVersion: v1
 kind: Service
 metadata:
   name: {{ include "cert-manager-webhook-hetzner.fullname" . }}
+  namespace: {{ .Release.Namespace | quote }}
   labels:
     app: {{ include "cert-manager-webhook-hetzner.name" . }}
     chart: {{ include "cert-manager-webhook-hetzner.chart" . }}

diff --git a/charts/cert-manager-webhook-hetzner/values.yaml b/charts/cert-manager-webhook-hetzner/values.yaml
@@ -69,7 +69,7 @@ affinity: {}
 
 
 
-## NGINX pods' Security Context.
+## Webhook pods' Security Context.
 ## ref: https://kubernetes.io/docs/tasks/configure-pod-container/security-context/#set-the-security-context-for-a-pod
 ##
 podSecurityContext:
@@ -83,7 +83,7 @@ podSecurityContext:
   ##
 #  sysctls: []
 
-## NGINX Core containers' Security Context (only main container).
+## Webhook Core containers' Security Context (only main container).
 ## ref: https://kubernetes.io/docs/tasks/configure-pod-container/security-context/#set-the-security-context-for-a-container
 ##
 containerSecurityContext: