Skip to content

Commit

Permalink
fix: Update oss charts (#15964)
Browse files Browse the repository at this point in the history
* fix: revert extraEnv. Move updated extraEnv to env_vars

* fix: Add node pool selector feature for gke deployments

* fix: hotfix. Fix webapp values and imagepullsecrets

* fix: remove nodeSelector duplicate
  • Loading branch information
xpuska513 authored Aug 25, 2022
1 parent 121bbcc commit edc6891
Show file tree
Hide file tree
Showing 7 changed files with 7 additions and 12 deletions.
3 changes: 0 additions & 3 deletions charts/airbyte-bootloader/templates/pod.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -20,9 +20,7 @@ spec:
serviceAccountName: {{ .Values.global.serviceAccountName }}
{{- if .Values.global.imagePullSecrets }}
imagePullSecrets:
{{- range .Values.global.imagePullSecrets }}
{{- printf "- name: %s" . | nindent 2 }}
{{- end }}
{{- end }}
restartPolicy: Never
{{- if .Values.nodeSelector }}
Expand Down Expand Up @@ -95,5 +93,4 @@ spec:
{{- toYaml .Values.extraEnv | nindent 8 }}
{{- end }}


resources: {{- toYaml .Values.resources | nindent 8 }}
1 change: 0 additions & 1 deletion charts/airbyte-bootloader/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -67,4 +67,3 @@ extraEnv: []
secrets: {}
env_vars: {}


3 changes: 1 addition & 2 deletions charts/airbyte-server/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -29,9 +29,7 @@ spec:
serviceAccountName: {{ .Values.global.serviceAccountName }}
{{- if .Values.global.imagePullSecrets }}
imagePullSecrets:
{{- range .Values.global.imagePullSecrets }}
{{- printf "- name: %s" . | nindent 2 }}
{{- end }}
{{- end }}
{{- if .Values.nodeSelector }}
nodeSelector: {{- include "common.tplvalues.render" (dict "value" .Values.nodeSelector "context" $) | nindent 8 }}
Expand Down Expand Up @@ -257,6 +255,7 @@ spec:
{{- if .Values.global.extraContainers }}
{{ toYaml .Values.global.extraContainers | indent 8 }}
{{- end }}

volumes:
{{- if eq .Values.deploymentMode "oss" }}
- name: gcs-log-creds-volume
Expand Down
4 changes: 3 additions & 1 deletion charts/airbyte-server/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -188,4 +188,6 @@ extraContainers: []
extraInitContainers: []

secrets: {}
env_vars: {}

env_vars: {}

2 changes: 0 additions & 2 deletions charts/airbyte-webapp/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -35,9 +35,7 @@ spec:
{{- end }}
{{- if .Values.global.imagePullSecrets }}
imagePullSecrets:
{{- range .Values.global.imagePullSecrets }}
{{- printf "- name: %s" . | nindent 2 }}
{{- end }}
{{- end }}
containers:
- name: airbyte-webapp-container
Expand Down
4 changes: 3 additions & 1 deletion charts/airbyte-webapp/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -176,4 +176,6 @@ extraContainers: []

secrets: {}

env_vars: []

env_vars: {}

2 changes: 0 additions & 2 deletions charts/airbyte-worker/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -28,9 +28,7 @@ spec:
automountServiceAccountToken: true
{{- if .Values.global.imagePullSecrets }}
imagePullSecrets:
{{- range .Values.global.imagePullSecrets }}
{{- printf "- name: %s" . | nindent 2 }}
{{- end }}
{{- end }}
{{- if .Values.nodeSelector }}
nodeSelector: {{- include "common.tplvalues.render" (dict "value" .Values.nodeSelector "context" $) | nindent 8 }}
Expand Down

0 comments on commit edc6891

Please sign in to comment.