DevFW-CICD/ingress-nginx-helm
12
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

deploy: add protocol to all Container/ServicePorts

Browse source 
kubectl apply --server-side currently doesn't work with Port specs that
are missing protocol:
https://github.com/kubernetes-sigs/structured-merge-diff/issues/130 so
we should always specify it.
This commit is contained in:
Bouke van der Bijl 2019-11-28 12:41:48 +00:00
parent d523b4a96e
commit 5b918e2d95
7 changed files with 14 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
deploy/cloud-generic/deployment.yaml
View file

@ -44,8 +44,10 @@ spec:
ports:
- name: http
containerPort: 80
protocol: TCP
- name: https
containerPort: 443
protocol: TCP
livenessProbe:
failureThreshold: 3
httpGet:

2
deploy/cloud-generic/service.yaml
View file

@ -8,7 +8,9 @@ spec:
ports:
- name: http
port: 80
protocol: TCP
targetPort: http
- name: https
port: 443
protocol: TCP
targetPort: https

2
deploy/static/mandatory.yaml
View file

@ -246,8 +246,10 @@ spec:
ports:
- name: http
containerPort: 80
protocol: TCP
- name: https
containerPort: 443
protocol: TCP
livenessProbe:
failureThreshold: 3
httpGet:

2
deploy/static/provider/aws/service-l4.yaml
View file

@ -21,9 +21,11 @@ spec:
ports:
- name: http
port: 80
protocol: TCP
targetPort: http
- name: https
port: 443
protocol: TCP
targetPort: https
---

2
deploy/static/provider/aws/service-l7.yaml
View file

@ -25,9 +25,11 @@ spec:
ports:
- name: http
port: 80
protocol: TCP
targetPort: http
- name: https
port: 443
protocol: TCP
targetPort: http
---

2
deploy/static/provider/aws/service-nlb.yaml
View file

@ -19,9 +19,11 @@ spec:
ports:
- name: http
port: 80
protocol: TCP
targetPort: http
- name: https
port: 443
protocol: TCP
targetPort: https
---

2
deploy/static/provider/cloud-generic.yaml
View file

@ -15,9 +15,11 @@ spec:
ports:
- name: http
port: 80
protocol: TCP
targetPort: http
- name: https
port: 443
protocol: TCP
targetPort: https
---