Commit 74539d6
committed
sample-app: Use hardcoded URL as db-insert trigger subscriber
This is needed because the node-server service listens on port 8080, and
the K8s ref resolver doesn't support custom ports [1], and referring to
the node-server-svc service yields:
http://node-server-svc.default.svc.cluster.local/insert
Instead of the correct URL:
http://node-server-svc.default.svc.cluster.local:8080/insert
Which results in hanging connections and eventually timeouts when adding
new comments that pass the trigger's filter.
[1] https://github.com/knative/pkg/blob/dcf159339de2fec9dde678a04cb7a354e912d2bf/resolver/addressable_resolver.go#L227
Signed-off-by: Guzman <guzman@guzman.fi>1 parent a4062d0 commit 74539d6
File tree
2 files changed
+2
-10
lines changed- code-samples/eventing/bookstore-sample-app/solution/node-server/config
- docs/bookstore/page-6
2 files changed
+2
-10
lines changedLines changed: 1 addition & 5 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
15 | 15 | | |
16 | 16 | | |
17 | 17 | | |
18 | | - | |
19 | | - | |
20 | | - | |
21 | | - | |
22 | | - | |
| 18 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
36 | 36 | | |
37 | 37 | | |
38 | 38 | | |
39 | | - | |
40 | | - | |
41 | | - | |
42 | | - | |
43 | | - | |
| 39 | + | |
44 | 40 | | |
45 | 41 | | |
46 | 42 | | |
| |||
0 commit comments