Skip to content

Commit

Permalink
feat: make enableServiceLinks configurable (#118)
Browse files Browse the repository at this point in the history
Signed-off-by: Lester Guerzon <[email protected]>
  • Loading branch information
guerzon authored Sep 7, 2024
1 parent 527440a commit 46c7e41
Show file tree
Hide file tree
Showing 5 changed files with 12 additions and 1 deletion.
2 changes: 1 addition & 1 deletion charts/vaultwarden/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -13,5 +13,5 @@ maintainers:
- name: guerzon
email: [email protected]
url: https://github.com/guerzon
version: 0.27.0
version: 0.28.0
kubeVersion: ">=1.12.0-0"
1 change: 1 addition & 0 deletions charts/vaultwarden/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -338,6 +338,7 @@ helm -n $NAMESPACE uninstall $RELEASE_NAME
| `podSecurityContext` | Pod security options | `{}` |
| `securityContext` | Default security options to run vault as read only container without privilege escalation | `{}` |
| `dnsConfig` | Pod DNS options | `{}` |
| `enableServiceLinks` | Enable service links, Kubernetes default is true | `true` |

### Reliability configuration

Expand Down
2 changes: 2 additions & 0 deletions charts/vaultwarden/ci/test-values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -24,3 +24,5 @@ duo:
iKey: "999888"
sKey:
value: "HELLO"

enableServiceLinks: false
3 changes: 3 additions & 0 deletions charts/vaultwarden/templates/_podSpec.tpl
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,9 @@ securityContext:
initContainers:
{{- toYaml . | nindent 2 }}
{{- end }}
{{- if not .Values.enableServiceLinks }}
enableServiceLinks: false
{{- end }}
containers:
- image: {{ .Values.image.registry }}/{{ .Values.image.repository }}:{{ .Values.image.tag }}
imagePullPolicy: {{ .Values.image.pullPolicy }}
Expand Down
5 changes: 5 additions & 0 deletions charts/vaultwarden/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -124,6 +124,11 @@ securityContext:
## Ref: https://kubernetes.io/docs/concepts/services-networking/dns-pod-service/#pod-dns-config
dnsConfig: {}

## @param enableServiceLinks Enable service links, Kubernetes default is true
## Ref: https://kubernetes.io/docs/tutorials/services/connect-applications-service/#accessing-the-service
##
enableServiceLinks: true

## @section Reliability configuration
##

Expand Down

0 comments on commit 46c7e41

Please sign in to comment.