[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 <mateusz@kinvolk.io>
This commit is contained in:
Mateusz Gozdek
2019-11-07 23:42:44 -08:00
committed by Kubernetes Prow Robot
parent a5aaea81ce
commit 2b1fc858c1
3 changed files with 3 additions and 3 deletions
+1 -1
View File
@@ -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
+1 -1
View File
@@ -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 |
+1 -1
View File
@@ -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