From 2b1fc858c127e391449b2f2dfeb021900358a2cc Mon Sep 17 00:00:00 2001 From: Mateusz Gozdek Date: Fri, 8 Nov 2019 08:42:44 +0100 Subject: [PATCH] [stable/jenkins] Change default serviceType to ClusterIP (#18103) To avoid accidental costs when chart is deployed for testing without any values. I think if user wants to expose their Jenkins directly into internet, that should be explicit decision. Signed-off-by: Mateusz Gozdek --- stable/jenkins/Chart.yaml | 2 +- stable/jenkins/README.md | 2 +- stable/jenkins/values.yaml | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/stable/jenkins/Chart.yaml b/stable/jenkins/Chart.yaml index 705e347e16..8ebc3f098e 100755 --- a/stable/jenkins/Chart.yaml +++ b/stable/jenkins/Chart.yaml @@ -1,7 +1,7 @@ apiVersion: v1 name: jenkins home: https://jenkins.io/ -version: 1.8.2 +version: 1.9.0 appVersion: lts description: Open source continuous integration server. It supports multiple SCM tools including CVS, Subversion and Git. It can execute Apache Ant and Apache Maven-based diff --git a/stable/jenkins/README.md b/stable/jenkins/README.md index 4c097842c7..d212f87cad 100644 --- a/stable/jenkins/README.md +++ b/stable/jenkins/README.md @@ -87,7 +87,7 @@ The following tables list the configurable parameters of the Jenkins chart and t | `master.fsGroup` | uid that will be used for persistent volume | `0` | | `master.hostAliases` | Aliases for IPs in `/etc/hosts` | `[]` | | `master.serviceAnnotations` | Service annotations | `{}` | -| `master.serviceType` | k8s service type | `LoadBalancer` | +| `master.serviceType` | k8s service type | `ClusterIP` | | `master.servicePort` | k8s service port | `8080` | | `master.targetPort` | k8s target port | `8080` | | `master.nodePort` | k8s node port | Not set | diff --git a/stable/jenkins/values.yaml b/stable/jenkins/values.yaml index ec4f89d821..02da33067d 100644 --- a/stable/jenkins/values.yaml +++ b/stable/jenkins/values.yaml @@ -90,7 +90,7 @@ master: targetPort: 8080 # For minikube, set this to NodePort, elsewhere use LoadBalancer # Use ClusterIP if your setup includes ingress controller - serviceType: LoadBalancer + serviceType: ClusterIP # Jenkins master service annotations serviceAnnotations: {} # Jenkins master custom labels