Skip to content

Commit

Permalink
Merge #113
Browse files Browse the repository at this point in the history
113: Add loadBalancerIP and externalTrafficPolicy support for service r=alallema a=agodard

# Pull Request

## What does this PR do?
Fixes #112 : Add `service.loadBalancerIP` and `service.externalTrafficPolicy` optional chart values to configure meilisearch kubernetes service with custom IP address from IaaS.

Co-authored-by: agodard <[email protected]>
  • Loading branch information
bors[bot] and agodard authored May 19, 2022
2 parents 0c9c9e7 + 1e8f546 commit 8031b21
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 1 deletion.
2 changes: 1 addition & 1 deletion charts/meilisearch/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ apiVersion: v1
appVersion: "v0.27.0"
description: A Helm chart for the Meilisearch search engine
name: meilisearch
version: 0.1.30
version: 0.1.31
icon: https://res.cloudinary.com/meilisearch/image/upload/v1597822872/Logo/logo_img.svg
home: https://github.com/meilisearch/meilisearch-kubernetes/charts
maintainers:
Expand Down
4 changes: 4 additions & 0 deletions charts/meilisearch/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -82,6 +82,10 @@ helm uninstall <your-service-name>
| | |
| `service.type` | Kubernetes Service type | `ClusterIP`
| | |
| `service.externalTrafficPolicy` | Service external traffic policy | `-` (No external traffic policy)
| | |
| `service.loadBalancerIP` | Service load balancer IP | `-` (No load balancer IP)
| | |
| `service.annotations` | Additional annotations for service | `{}`
| | |
| `persistence.enabled` | Enable persistence using PVC | `false`
Expand Down
6 changes: 6 additions & 0 deletions charts/meilisearch/templates/service.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,13 @@ metadata:
{{ toYaml . | nindent 4 }}
{{- end }}
spec:
{{- if .Values.service.externalTrafficPolicy }}
externalTrafficPolicy: {{.Values.service.externalTrafficPolicy}}
{{- end }}
type: {{ .Values.service.type }}
{{- if .Values.service.loadBalancerIP }}
loadBalancerIP: {{.Values.service.loadBalancerIP}}
{{- end }}
ports:
- port: {{ .Values.service.port }}
targetPort: http
Expand Down

0 comments on commit 8031b21

Please sign in to comment.