From 25a999ba329bff39bf5be8e984871a0ba252fac6 Mon Sep 17 00:00:00 2001 From: Guzman Date: Fri, 24 Jan 2025 17:47:59 +0100 Subject: [PATCH] sample-app: Fix port of node-server-svc (80 to 8080) Without this, following the instructions that don't require port forwarding (e.g. using "minikube tunnel") fails: the service is exposed on port 80, which is not what the frontend expects. After this commit the setup mimics that of the frontend service, which works both with and without port forwarding. Signed-off-by: Guzman --- .../start/node-server/config/100-deployment.yaml | 2 +- docs/bookstore/page-0.5/environment-setup.md | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/code-samples/eventing/bookstore-sample-app/start/node-server/config/100-deployment.yaml b/code-samples/eventing/bookstore-sample-app/start/node-server/config/100-deployment.yaml index 9a56b883383..647320fec91 100644 --- a/code-samples/eventing/bookstore-sample-app/start/node-server/config/100-deployment.yaml +++ b/code-samples/eventing/bookstore-sample-app/start/node-server/config/100-deployment.yaml @@ -29,6 +29,6 @@ spec: app: node-server ports: - protocol: TCP - port: 80 + port: 8080 targetPort: 8000 type: LoadBalancer \ No newline at end of file diff --git a/docs/bookstore/page-0.5/environment-setup.md b/docs/bookstore/page-0.5/environment-setup.md index 6bb18215278..94f1387b82d 100644 --- a/docs/bookstore/page-0.5/environment-setup.md +++ b/docs/bookstore/page-0.5/environment-setup.md @@ -270,7 +270,7 @@ node-server-svc LoadBalancer 10.101.90.35 80:31792/T If port forwarding is required, open a new terminal and run: ```shell -kubectl port-forward svc/node-server-svc 8080:80 +kubectl port-forward svc/node-server-svc 8080:8080 ``` You should see the following output: