Skip to content
This repository has been archived by the owner on Feb 22, 2022. It is now read-only.

quote boolean S3 environment variables #5066

Merged
merged 2 commits into from
Apr 17, 2018
Merged

quote boolean S3 environment variables #5066

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
1ed9aa1
add if statements for optional S3 env variables, and quote them becau…
emsixteeen Apr 16, 2018
ead2669
bump version, per CI request
emsixteeen Apr 16, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion stable/docker-registry/Chart.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
apiVersion: v1
description: A Helm chart for Docker Registry
name: docker-registry
version: 1.1.1
version: 1.1.2
appVersion: 2.6.2
home: https://hub.docker.com/_/registry/
icon: https://hub.docker.com/public/images/logos/mini-logo.svg
Expand Down
8 changes: 6 additions & 2 deletions stable/docker-registry/templates/deployment.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -84,10 +84,14 @@ spec:
value: {{ required ".Values.s3.region is required" .Values.s3.region }}
- name: REGISTRY_STORAGE_S3_BUCKET
value: {{ required ".Values.s3.bucket is required" .Values.s3.bucket }}
{{- if .Values.s3.encrypt }}
- name: REGISTRY_STORAGE_S3_ENCRYPT
value: {{ .Values.s3.encrypt }}
value: {{ .Values.s3.encrypt | quote }}
{{- end }}
{{- if .Values.s3.secure }}
- name: REGISTRY_STORAGE_S3_SECURE
value: {{ .Values.s3.secure }}
value: {{ .Values.s3.secure | quote }}
{{- end }}
{{- end }}
volumeMounts:
{{- if .Values.secrets.htpasswd }}
Expand Down