From 33f67c0801a6c2925acc5a8777575de102fbcd27 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Tom=C3=A1=C5=A1=20Kukr=C3=A1l?= Date: Fri, 8 Mar 2024 16:43:40 +0100 Subject: [PATCH 1/8] configure traffic policy for svc This change add option to configure traffic policy for the service. It is backward compatible and will not change the default behavior. --- helm/oauth2-proxy/Chart.yaml | 2 +- helm/oauth2-proxy/templates/service.yaml | 6 ++++++ helm/oauth2-proxy/values.yaml | 5 +++++ 3 files changed, 12 insertions(+), 1 deletion(-) diff --git a/helm/oauth2-proxy/Chart.yaml b/helm/oauth2-proxy/Chart.yaml index 0682fbe4..6d31c370 100644 --- a/helm/oauth2-proxy/Chart.yaml +++ b/helm/oauth2-proxy/Chart.yaml @@ -1,5 +1,5 @@ name: oauth2-proxy -version: 7.1.0 +version: 7.2.0 apiVersion: v2 appVersion: 7.6.0 home: https://oauth2-proxy.github.io/oauth2-proxy/ diff --git a/helm/oauth2-proxy/templates/service.yaml b/helm/oauth2-proxy/templates/service.yaml index d16120ee..e2e8f9d9 100644 --- a/helm/oauth2-proxy/templates/service.yaml +++ b/helm/oauth2-proxy/templates/service.yaml @@ -27,6 +27,12 @@ spec: {{- end -}} {{- else }} type: {{ .Values.service.type }} +{{- end }} +{{- if .Values.service.externalTrafficPolicy }} + externalTrafficPolicy: {{ .Values.service.externalTrafficPolicy }} +{{- end }} +{{- if .Values.service.internalTrafficPolicy }} + internalTrafficPolicy: {{ .Values.service.internalTrafficPolicy }} {{- end }} ports: - port: {{ .Values.service.portNumber }} diff --git a/helm/oauth2-proxy/values.yaml b/helm/oauth2-proxy/values.yaml index 219f45ad..f59c74e3 100644 --- a/helm/oauth2-proxy/values.yaml +++ b/helm/oauth2-proxy/values.yaml @@ -128,6 +128,11 @@ service: appProtocol: http annotations: {} # foo.io/bar: "true" + # configure externalTrafficPolicy + externalTrafficPolicy: + # configure internalTrafficPolicy + internalTrafficPolicy: + ## Create or use ServiceAccount serviceAccount: From 4f2c4e04affdca2003cc02a63d227534dbf2d111 Mon Sep 17 00:00:00 2001 From: Pierluigi Lenoci Date: Mon, 8 Apr 2024 11:54:44 +0200 Subject: [PATCH 2/8] Update Chart.yaml --- helm/oauth2-proxy/Chart.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/helm/oauth2-proxy/Chart.yaml b/helm/oauth2-proxy/Chart.yaml index 736212fe..24bf98e8 100644 --- a/helm/oauth2-proxy/Chart.yaml +++ b/helm/oauth2-proxy/Chart.yaml @@ -1,5 +1,5 @@ name: oauth2-proxy -version: 7.2.0 +version: 7.3.0 apiVersion: v2 appVersion: 7.6.0 home: https://oauth2-proxy.github.io/oauth2-proxy/ From 650897b250940d995ffc9fd1ed95f072e000427d Mon Sep 17 00:00:00 2001 From: Pierluigi Lenoci Date: Mon, 8 Apr 2024 15:50:24 +0200 Subject: [PATCH 3/8] Update Chart.yaml --- helm/oauth2-proxy/Chart.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/helm/oauth2-proxy/Chart.yaml b/helm/oauth2-proxy/Chart.yaml index 24bf98e8..d5037c7c 100644 --- a/helm/oauth2-proxy/Chart.yaml +++ b/helm/oauth2-proxy/Chart.yaml @@ -1,5 +1,5 @@ name: oauth2-proxy -version: 7.3.0 +version: 7.4.0 apiVersion: v2 appVersion: 7.6.0 home: https://oauth2-proxy.github.io/oauth2-proxy/ From 64eb93be3518d70cead2eda41cc2d544969211d2 Mon Sep 17 00:00:00 2001 From: Pierluigi Lenoci Date: Fri, 31 May 2024 11:52:42 +0200 Subject: [PATCH 4/8] Update Chart.yaml --- helm/oauth2-proxy/Chart.yaml | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/helm/oauth2-proxy/Chart.yaml b/helm/oauth2-proxy/Chart.yaml index d567a5df..6b601e23 100644 --- a/helm/oauth2-proxy/Chart.yaml +++ b/helm/oauth2-proxy/Chart.yaml @@ -1,5 +1,5 @@ name: oauth2-proxy -version: 7.6.1 +version: 7.7.0 apiVersion: v2 appVersion: 7.6.0 home: https://oauth2-proxy.github.io/oauth2-proxy/ @@ -34,8 +34,8 @@ maintainers: kubeVersion: ">=1.9.0-0" annotations: artifacthub.io/changes: | - - kind: fixed - description: Fixed test for horizontal autoscaling feature + - kind: added + description: option to configure traffic policy for the service links: - name: Github PR - url: https://github.com/oauth2-proxy/manifests/pull/211 + url: https://github.com/oauth2-proxy/manifests/pull/190 From 36bc73deeaf80ee848026775a09c1c2de9196694 Mon Sep 17 00:00:00 2001 From: Pierluigi Lenoci Date: Fri, 31 May 2024 11:54:08 +0200 Subject: [PATCH 5/8] Cluster --- helm/oauth2-proxy/values.yaml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/helm/oauth2-proxy/values.yaml b/helm/oauth2-proxy/values.yaml index a46c0e56..e42efd27 100644 --- a/helm/oauth2-proxy/values.yaml +++ b/helm/oauth2-proxy/values.yaml @@ -142,9 +142,9 @@ service: annotations: {} # foo.io/bar: "true" # configure externalTrafficPolicy - externalTrafficPolicy: + externalTrafficPolicy: Cluster # configure internalTrafficPolicy - internalTrafficPolicy: + internalTrafficPolicy: Cluster ## Create or use ServiceAccount From caf711a25992d1aa7b2fe8817a3355ee54405dea Mon Sep 17 00:00:00 2001 From: Pierluigi Lenoci Date: Fri, 31 May 2024 11:55:40 +0200 Subject: [PATCH 6/8] Update values.yaml --- helm/oauth2-proxy/values.yaml | 1 - 1 file changed, 1 deletion(-) diff --git a/helm/oauth2-proxy/values.yaml b/helm/oauth2-proxy/values.yaml index e42efd27..34ed4ab5 100644 --- a/helm/oauth2-proxy/values.yaml +++ b/helm/oauth2-proxy/values.yaml @@ -146,7 +146,6 @@ service: # configure internalTrafficPolicy internalTrafficPolicy: Cluster - ## Create or use ServiceAccount serviceAccount: ## Specifies whether a ServiceAccount should be created From 97aba79c52013a7a0ff52442c8c16abb89594013 Mon Sep 17 00:00:00 2001 From: Pierluigi Lenoci Date: Fri, 31 May 2024 12:07:30 +0200 Subject: [PATCH 7/8] Update README.md --- helm/oauth2-proxy/README.md | 2 ++ 1 file changed, 2 insertions(+) diff --git a/helm/oauth2-proxy/README.md b/helm/oauth2-proxy/README.md index 98ba58ce..5fee9251 100644 --- a/helm/oauth2-proxy/README.md +++ b/helm/oauth2-proxy/README.md @@ -188,6 +188,8 @@ The following table lists the configurable parameters of the oauth2-proxy chart | `revisionHistoryLimit` | maximum number of revisions maintained | 10 | | `service.portNumber` | port number for the service | `80` | | `service.appProtocol` | application protocol on the port of the service | `http` | +| `service.externalTrafficPolicy` | denotes if the service desires to route external traffic to node-local or cluster-wide endpoints | `Cluster` | +| `service.internalTrafficPolicy` | denotes if the service desires to route internal traffic to node-local or cluster-wide endpoints | `Cluster` | | `service.type` | type of service | `ClusterIP` | | `service.clusterIP` | cluster ip address | `nil` | | `service.loadBalancerIP` | ip of load balancer | `nil` | From 0323b5e77c092881a341d8f77fcc5646caa75c74 Mon Sep 17 00:00:00 2001 From: Pierluigi Lenoci Date: Fri, 31 May 2024 12:17:23 +0200 Subject: [PATCH 8/8] Update values.yaml --- helm/oauth2-proxy/values.yaml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/helm/oauth2-proxy/values.yaml b/helm/oauth2-proxy/values.yaml index 34ed4ab5..21f80831 100644 --- a/helm/oauth2-proxy/values.yaml +++ b/helm/oauth2-proxy/values.yaml @@ -142,9 +142,9 @@ service: annotations: {} # foo.io/bar: "true" # configure externalTrafficPolicy - externalTrafficPolicy: Cluster + externalTrafficPolicy: "" # configure internalTrafficPolicy - internalTrafficPolicy: Cluster + internalTrafficPolicy: "" ## Create or use ServiceAccount serviceAccount: