Merge pull request #220 from whezzel/master

add support for externalTrafficPolicy on service
jp-gouin · Dec 17, 2024 · 3dc8ba1 · 3dc8ba1
2 parents c22e62e + 843cc2c
commit 3dc8ba1
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 0 deletions.
diff --git a/README.md b/README.md
@@ -138,6 +138,7 @@ Parameters related to Kubernetes.
 | `service.type`                             | Service type can be ClusterIP, NodePort, LoadBalancer                                                             | `ClusterIP`         |
 | `service.sslLdapPortNodePort`              | Nodeport of External service port for SSL if service.type is NodePort                                             | `nil`               |
 | `service.ipFamilyPolicy`                   | Represents the dual-stack-ness requested or required by this Service.                                             | `SingleStack`       |
+| `service.externalTrafficPolicy`            | Sets the externalTrafficPolicy for this Service.                                                                  | `Cluster`           |
 | `serviceReadOnly.annotations`              | Annotations to add to the service                                                                                 | `{}`                |
 | `serviceReadOnly.externalIPs`              | Service external IP addresses                                                                                     | `[]`                |
 | `serviceReadOnly.enableLdapPort`           | Enable LDAP port on the service and headless service                                                              | `true`              |
@@ -149,6 +150,7 @@ Parameters related to Kubernetes.
 | `serviceReadOnly.sslLdapPortNodePort`      | Nodeport of External service port for SSL if service.type is NodePort                                             | `nil`               |
 | `serviceReadOnly.type`                     | Service type can be ClusterIP, NodePort, LoadBalancer                                                             | `ClusterIP`         |
 | `serviceReadOnly.ipFamilyPolicy`                   | Represents the dual-stack-ness requested or required by this Service.                                             | `SingleStack`       |
+| `serviceReadOnly.externalTrafficPolicy`    | Sets the externalTrafficPolicy for this Service.                                                                  | `Cluster`           |
 | `persistence.enabled`                      | Whether to use PersistentVolumes or not                                                                           | `false`             |
 | `persistence.storageClass`                 | Storage class for PersistentVolumes.                                                                              | `<unset>`           |
 | `persistence.existingClaim`                | Add existing Volumes Claim.                                                                                       | `<unset>`           |

diff --git a/templates/service-readonly.yaml b/templates/service-readonly.yaml
@@ -52,6 +52,7 @@ spec:
       {{- end }}
     {{- end }}
   sessionAffinity: {{ .Values.service.sessionAffinity }}
+  externalTrafficPolicy: {{ .Values.service.externalTrafficPolicy }}
   selector:
     app.kubernetes.io/component: {{ template "openldap.fullname" . }}-readonly
     release: {{ .Release.Name }}

diff --git a/templates/service.yaml b/templates/service.yaml
@@ -51,6 +51,7 @@ spec:
       {{- end }}
     {{- end }}
   sessionAffinity: {{ .Values.service.sessionAffinity }}
+  externalTrafficPolicy: {{ .Values.service.externalTrafficPolicy }}
   selector:
     app.kubernetes.io/component: {{ template "openldap.fullname" . }}
     release: {{ .Release.Name }}
diff --git a/values.yaml b/values.yaml
@@ -97,6 +97,7 @@ service:
   #loadBalancerSourceRanges: []
   type: ClusterIP
   sessionAffinity: None
+  externalTrafficPolicy: Cluster
 
   ## Represents the dual-stack-ness requested or required by this Service. Possible values are
   ## SingleStack, PreferDualStack or RequireDualStack.
@@ -130,6 +131,7 @@ serviceReadOnly:
   #loadBalancerSourceRanges: []
   type: ClusterIP
   sessionAffinity: None
+  externalTrafficPolicy: Cluster
 
   ## Represents the dual-stack-ness requested or required by this Service. Possible values are
   ## SingleStack, PreferDualStack or RequireDualStack.