Skip to content

Commit

Permalink
Need to set LoadbalancerIP in Service specification. Patch to copy al…
Browse files Browse the repository at this point in the history
…l specs over from Values files. (bokysan#72)

* copy other specs over

I want to copy over loadBalancerIP to grab a predefined static ip

* removed hardcoded LoadbalancerIP

* Set theme jekyll-theme-slate

* update README

* test service specs are copied over

* undo config changes

* remove config file
  • Loading branch information
komasoftware authored Jun 4, 2021
1 parent 02b1caa commit 9a444d4
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 0 deletions.
1 change: 1 addition & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -460,6 +460,7 @@ Chart configuration is as follows:
| `service.port` | `587` | SMTP submission port |
| `service.labels` | `{}` | Additional service labels |
| `service.annotations` | `{}` | Additional service annotations |
| `service.spec` | `{}` | Additional service specifications |
| `service.nodePort` | *empty* | Use a specific `nodePort` |
| `service.nodeIP` | *empty* | Use a specific `nodeIP` |
| `resources` | `{}` | [Pod resources](https://kubernetes.io/docs/concepts/configuration/manage-resources-containers/) |
Expand Down
1 change: 1 addition & 0 deletions helm/mail/templates/service.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@ metadata:
{{- with .Values.service.annotations }}{{ toYaml . | nindent 4 }}{{ end }}
spec:
type: {{ .Values.service.type }}
{{- with .Values.service.spec }}{{ toYaml . | nindent 2 }}{{ end }}
ports:
- port: {{ .Values.service.port }}
targetPort: smtp
Expand Down
6 changes: 6 additions & 0 deletions helm/test_6_service_spec.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
# copy over all values from the service.spc
service:
type: LoadBalancer
spec:
loadBalancerIP: 10.10.100.100
externalTrafficPolicy: Local

0 comments on commit 9a444d4

Please sign in to comment.