Skip to content

Commit

Permalink
feat(deployment): allow existing service account usage (#102)
Browse files Browse the repository at this point in the history
* feat(deployment): allow existing service account usage

* revert Chart version update

* fix README.md file synthax
  • Loading branch information
pchanvallon authored Aug 22, 2024
1 parent a8ba0f7 commit bd42523
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion charts/gateway/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -106,6 +106,7 @@ Shared kubernetes configuration of the chart
| Name | Description | Value |
| ----------------------- | -------------------------------------------------------------- | ------- |
| `serviceAccount.create` | Create Kubernetes service account. Default kube value if false | `false` |
| `serviceAccount.name` | Service account name to attach to the Gateway deployment | `""` |
| `commonLabels` | Labels to be applied to all ressources created by this chart | `{}` |
| `nodeSelector` | Container node selector | `{}` |
| `tolerations` | Container tolerations | `[]` |
Expand All @@ -118,4 +119,3 @@ Enable and configure chart dependencies if not available in your deployment
| Name | Description | Value |
| --------------- | ----------------------------------------------------------------------------- | ------- |
| `kafka.enabled` | Deploy a kafka along side gateway (This should only used for testing purpose) | `false` |

2 changes: 0 additions & 2 deletions charts/gateway/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -122,9 +122,7 @@ spec:
items:
- key: interceptors.json
path: interceptors.json
{{- if .Values.serviceAccount.create }}
serviceAccountName: {{ template "conduktor-gateway.serviceAccountName" . }}
{{- end }}
{{- with .Values.global.imagePullSecrets }}
imagePullSecrets: {{- toYaml . | nindent 8 }}
{{- end }}
Expand Down
2 changes: 2 additions & 0 deletions charts/gateway/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -187,6 +187,8 @@ serviceAccount:
# Specifies whether a service account should be created
# If AWS IAM is used, need to have create: false
create: false
## @param serviceAccount.name Service account name to attach to the Gateway deployment
name: ""

## @param commonLabels Labels to be applied to all ressources created by this chart
commonLabels: {}
Expand Down

0 comments on commit bd42523

Please sign in to comment.