Skip to content

Commit

Permalink
feat: imagePullSecrets for servers and controllers (argoproj#361)
Browse files Browse the repository at this point in the history
  • Loading branch information
hobti01 authored Aug 30, 2020
1 parent e1e2dd9 commit ffc9220
Show file tree
Hide file tree
Showing 18 changed files with 50 additions and 4 deletions.
1 change: 1 addition & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -3,3 +3,4 @@ output
.DS_Store
.idea
**/*.tgz
**/charts/*/charts
2 changes: 1 addition & 1 deletion charts/argo-ci/Chart.yaml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
apiVersion: v1
description: A Helm chart for Argo-CI
name: argo-ci
version: 0.1.6
version: 0.1.7
icon: https://raw.githubusercontent.com/argoproj/argo/master/docs/assets/argo.png
appVersion: v1.0.0-alpha2
home: https://github.com/argoproj/argo-helm
Expand Down
4 changes: 4 additions & 0 deletions charts/argo-ci/templates/ci-deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -33,3 +33,7 @@ spec:
ports:
- containerPort: 8001
- containerPort: 8002
{{- with .Values.imagePullSecrets }}
imagePullSecrets:
{{- toYaml . | nindent 8 }}
{{- end }}
3 changes: 3 additions & 0 deletions charts/argo-ci/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,9 @@ imageNamespace: argoproj
ciImage: argoci
imageTag: v1.0.0-alpha2
imagePullPolicy: Always
# Secrets with credentials to pull images from a private registry
imagePullSecrets: []
# - name: argo-pull-secret
workflowNamespace: default

argo:
Expand Down
2 changes: 1 addition & 1 deletion charts/argo-events/Chart.yaml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
apiVersion: v1
description: A Helm chart to install Argo-Events in k8s Cluster
name: argo-events
version: 0.17.0
version: 0.17.1
keywords:
- argo-events
- sensor-controller
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -37,3 +37,7 @@ spec:
value: {{ .Values.eventbusController.natsStreamingImage }}
- name: NATS_METRICS_EXPORTER_IMAGE
value: {{ .Values.eventbusController.natsMetricsExporterImage }}
{{- with .Values.imagePullSecrets }}
imagePullSecrets:
{{- toYaml . | nindent 8 }}
{{- end }}
3 changes: 3 additions & 0 deletions charts/argo-events/templates/eventbus-crd.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,9 @@ apiVersion: apiextensions.k8s.io/v1beta1
kind: CustomResourceDefinition
metadata:
name: eventbus.argoproj.io
annotations:
helm.sh/hook: crd-install
helm.sh/hook-delete-policy: before-hook-creation
spec:
group: argoproj.io
names:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -35,3 +35,7 @@ spec:
fieldPath: metadata.namespace
- name: EVENTSOURCE_IMAGE
value: "{{ .Values.registry }}/{{ .Values.eventsourceController.eventsourceImage }}:{{ .Values.eventsourceController.tag }}"
{{- with .Values.imagePullSecrets }}
imagePullSecrets:
{{- toYaml . | nindent 8 }}
{{- end }}
3 changes: 3 additions & 0 deletions charts/argo-events/templates/eventsource-crd.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,9 @@ apiVersion: apiextensions.k8s.io/v1beta1
kind: CustomResourceDefinition
metadata:
name: eventsources.argoproj.io
annotations:
helm.sh/hook: crd-install
helm.sh/hook-delete-policy: before-hook-creation
spec:
group: argoproj.io
scope: Namespaced
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -35,3 +35,7 @@ spec:
fieldPath: metadata.namespace
- name: SENSOR_IMAGE
value: "{{ .Values.registry }}/{{ .Values.sensorController.sensorImage }}:{{ .Values.sensorController.tag }}"
{{- with .Values.imagePullSecrets }}
imagePullSecrets:
{{- toYaml . | nindent 8 }}
{{- end }}
3 changes: 3 additions & 0 deletions charts/argo-events/templates/sensor-crd.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,9 @@ apiVersion: apiextensions.k8s.io/v1beta1
kind: CustomResourceDefinition
metadata:
name: sensors.argoproj.io
annotations:
helm.sh/hook: crd-install
helm.sh/hook-delete-policy: before-hook-creation
spec:
group: argoproj.io
names:
Expand Down
4 changes: 4 additions & 0 deletions charts/argo-events/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,10 @@ registry: argoproj
# The image pull policy
imagePullPolicy: Always

# Secrets with credentials to pull images from a private registry
imagePullSecrets: []
# - name: argo-pull-secret

# If set to false, skip installing the CRDs. Requires user to have them installed prior to helm chart installation.
installCRD: true

Expand Down
2 changes: 1 addition & 1 deletion charts/argo-rollouts/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ apiVersion: v1
appVersion: "0.8.3"
description: A Helm chart for Argo Rollouts
name: argo-rollouts
version: 0.3.5
version: 0.3.6
icon: https://raw.githubusercontent.com/argoproj/argo/master/argo.png
home: https://github.com/argoproj/argo-helm
maintainers:
Expand Down
2 changes: 2 additions & 0 deletions charts/argo-rollouts/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -40,4 +40,6 @@ serviceAnnotations: {}
##
podLabels: {}

# Secrets with credentials to pull images from a private registry
imagePullSecrets: []
# - name: argo-pull-secret
2 changes: 1 addition & 1 deletion charts/argo/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ apiVersion: v1
appVersion: v2.8.0
description: A Helm chart for Argo Workflows
name: argo
version: 0.9.9
version: 0.9.10
icon: https://raw.githubusercontent.com/argoproj/argo/master/docs/assets/argo.png
home: https://github.com/argoproj/argo-helm
maintainers:
Expand Down
4 changes: 4 additions & 0 deletions charts/argo/templates/server-deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -67,6 +67,10 @@ spec:
volumeMounts:
{{- toYaml . | nindent 12}}
{{- end }}
{{- with .Values.images.pullSecrets }}
imagePullSecrets:
{{- toYaml . | nindent 8 }}
{{- end }}
{{- with .Values.server.volumes }}
volumes:
{{- toYaml . | nindent 8}}
Expand Down
4 changes: 4 additions & 0 deletions charts/argo/templates/workflow-controller-deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -63,6 +63,10 @@ spec:
ports:
- containerPort: 8080
{{- end }}
{{- with .Values.images.pullSecrets }}
imagePullSecrets:
{{- toYaml . | nindent 8 }}
{{- end }}
{{- with .Values.controller.nodeSelector }}
nodeSelector:
{{- toYaml . | nindent 8 }}
Expand Down
3 changes: 3 additions & 0 deletions charts/argo/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,9 @@ images:
server: argocli
executor: argoexec
pullPolicy: Always
# Secrets with credentials to pull images from a private registry
pullSecrets: []
# - name: argo-pull-secret
tag: v2.7.6

crdVersion: v1alpha1
Expand Down

0 comments on commit ffc9220

Please sign in to comment.