Skip to content

Commit a4062d0

Browse files
committed
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 <[email protected]>
1 parent 64fc93b commit a4062d0

File tree

3 files changed

+3
-3
lines changed

3 files changed

+3
-3
lines changed

code-samples/eventing/bookstore-sample-app/solution/node-server/config/100-deployment.yaml

+1-1
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,6 @@ spec:
2929
app: node-server
3030
ports:
3131
- protocol: TCP
32-
port: 80
32+
port: 8080
3333
targetPort: 8000
3434
type: LoadBalancer

code-samples/eventing/bookstore-sample-app/start/node-server/config/100-deployment.yaml

+1-1
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,6 @@ spec:
2929
app: node-server
3030
ports:
3131
- protocol: TCP
32-
port: 80
32+
port: 8080
3333
targetPort: 8000
3434
type: LoadBalancer

docs/bookstore/page-0.5/environment-setup.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -270,7 +270,7 @@ node-server-svc LoadBalancer 10.101.90.35 <pending> 80:31792/T
270270
If port forwarding is required, open a new terminal and run:
271271

272272
```shell
273-
kubectl port-forward svc/node-server-svc 8080:80
273+
kubectl port-forward svc/node-server-svc 8080:8080
274274
```
275275
You should see the following output:
276276

0 commit comments

Comments
 (0)