Skip to content

Commit

Permalink
Add digest option to btp-operator chart (#169)
Browse files Browse the repository at this point in the history
* add sha option to deployment images
  • Loading branch information
mordaby authored May 26, 2022
1 parent e778f90 commit 76cd05d
Show file tree
Hide file tree
Showing 2 changed files with 18 additions and 2 deletions.
14 changes: 12 additions & 2 deletions sapbtp-operator-charts/templates/deployment.yml
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,13 @@ spec:
- --upstream=http://127.0.0.1:8080/
- --logtostderr=true
- --v=10
image: quay.io/brancz/kube-rbac-proxy:v0.11.0

{{- if .Values.manager.rbacProxy.image.sha }}
image: "{{.Values.manager.rbacProxy.image.repository}}@sha256:{{.Values.manager.rbacProxy.image.sha}}"
{{- else }}
image: "{{.Values.manager.rbacProxy.image.repository}}:{{.Values.manager.rbacProxy.image.tag}}"
{{- end }}

{{- if .Values.manager.securityContext }}
securityContext:
{{ toYaml .Values.manager.securityContext | indent 2 }}
Expand All @@ -50,7 +56,11 @@ spec:
envFrom:
- configMapRef:
name: sap-btp-operator-config
image: {{.Values.manager.image.repository}}:{{.Values.manager.image.tag}}
{{- if .Values.manager.image.sha}}
image: "{{.Values.manager.image.repository}}@sha256:{{.Values.manager.image.sha}}"
{{- else }}
image: "{{.Values.manager.image.repository}}:{{.Values.manager.image.tag}}"
{{- end }}
imagePullPolicy: IfNotPresent
{{- if .Values.manager.securityContext }}
securityContext:
Expand Down
6 changes: 6 additions & 0 deletions sapbtp-operator-charts/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@ manager:
image:
repository: ghcr.io/sap/sap-btp-service-operator/controller
tag: master
sha: ""
secret:
b64encoded: false
enabled: true
Expand All @@ -26,6 +27,11 @@ manager:
sm_url: ""
tokenurl: ""
tokenurlsuffix: "/oauth/token"
rbacProxy:
image:
repository: quay.io/brancz/kube-rbac-proxy
sha: ""
tag: v0.11.0
certificates:
# Configure if https://github.com/jetstack/cert-manager is used
certManager: true
Expand Down

0 comments on commit 76cd05d

Please sign in to comment.