Skip to content

Commit 369426c

Browse files
authored
remove port 8080 exposure from envoy proxy (#67)
* remove port 8080 exposure from envoy proxy Signed-off-by: don2112e <darren.oneill@ericsson.com> * update envoy proxy version to 1.36.2 --------- Signed-off-by: don2112e <darren.oneill@ericsson.com>
1 parent 43a0410 commit 369426c

7 files changed

Lines changed: 6 additions & 38 deletions

File tree

‎.gitignore‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -158,3 +158,6 @@ cython_debug/
158158
# and can be added to the global gitignore or merged into this file. For a more nuclear
159159
# option (not recommended) you can uncomment the following to ignore the entire idea folder.
160160
#.idea/
161+
162+
helloworldAppPackage/
163+
csar-output/

‎charts/eric-oss-hello-world-python-app/templates/configmap/envoy-configmap.yaml‎

Lines changed: 0 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -57,32 +57,6 @@ data:
5757
validation_context:
5858
trusted_ca:
5959
filename: {{ printf "%s/%s" (default $.Values.instantiationDefaults.proxyCaCertMountPath $.Values.proxyCaCertMountPath) (default $.Values.instantiationDefaults.platformCaCertFileName $.Values.platformCaCertFileName) | quote }}
60-
# Plain HTTP listener for other endpoints
61-
- name: listener_http
62-
address:
63-
socket_address:
64-
address: 0.0.0.0
65-
port_value: 8080
66-
filter_chains:
67-
- filters:
68-
- name: envoy.filters.network.http_connection_manager
69-
typed_config:
70-
"@type": type.googleapis.com/envoy.extensions.filters.network.http_connection_manager.v3.HttpConnectionManager
71-
stat_prefix: ingress_http_plain
72-
route_config:
73-
virtual_hosts:
74-
- name: plain_service
75-
domains: ["*"]
76-
routes:
77-
- match:
78-
path: "/sample-app/python/health"
79-
route:
80-
cluster: eric-oss-hello-world-python-app-cluster
81-
http_filters:
82-
- name: envoy.filters.http.router
83-
typed_config:
84-
"@type": type.googleapis.com/envoy.extensions.filters.http.router.v3.Router
85-
8660
clusters:
8761
- name: eric-oss-hello-world-python-app-cluster
8862
type: STATIC

‎charts/eric-oss-hello-world-python-app/templates/deployment/deployment.yaml‎

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -156,7 +156,7 @@ spec:
156156
value: "/etc/adp/logcontrol.json"
157157
{{- include "eric-oss-hello-world-python-app.jaegerEnv" . | indent 12 }}
158158
ports:
159-
- name: http-metrics
159+
- name: http
160160
containerPort: 8050
161161
protocol: TCP
162162
livenessProbe:
@@ -215,9 +215,6 @@ spec:
215215
command: ["envoy"]
216216
args: ["-c", "/etc/envoy/envoy.yaml", "--base-id", "1"]
217217
ports:
218-
- name: envoy-http
219-
containerPort: 8080
220-
protocol: TCP
221218
- name: envoy-https
222219
containerPort: 8443
223220
protocol: TCP

‎charts/eric-oss-hello-world-python-app/templates/service/service.yaml‎

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -15,10 +15,6 @@ spec:
1515
ipFamilies: [{{ .Values.global.internalIPFamily }}]
1616
{{- end }}
1717
ports:
18-
- port: {{ index .Values.service "http-port" }}
19-
targetPort: {{ .Values.service.httpTargetPort | default 8080 }}
20-
protocol: TCP
21-
name: envoy-http
2218
- port: {{ index .Values.service "https-port" }}
2319
targetPort: {{ .Values.service.httpsTargetPort | default 8443 }}
2420
protocol: TCP

‎charts/eric-oss-hello-world-python-app/values.yaml‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,6 @@ seccompProfile:
4343

4444
service:
4545
type: ClusterIP
46-
http-port: 8080
4746
https-port: 8443
4847

4948
resources:

‎docker-compose.yaml‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,5 +17,4 @@ services:
1717
depends_on:
1818
- app
1919
ports:
20-
- "8080:8080"
2120
- "8443:8443"

‎envoy.Dockerfile‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
1-
FROM envoyproxy/envoy:distroless-v1.35.0
1+
FROM envoyproxy/envoy:distroless-v1.36.2
22

33
USER 60577:60577
4-
EXPOSE 8080 8443
4+
EXPOSE 8443
55

66
CMD ["envoy", "-c", "/etc/envoy/envoy.yaml"]

0 commit comments

Comments
 (0)