Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Address watch: incorrect behavior on edge routers #434

Closed
kgiusti opened this issue May 5, 2022 · 0 comments
Closed

Address watch: incorrect behavior on edge routers #434

kgiusti opened this issue May 5, 2022 · 0 comments
Assignees
Milestone

Comments

@kgiusti
Copy link
Contributor

kgiusti commented May 5, 2022

A watched address on an edge router reports incorrect consumer counts.

Problem description:

Three router linear deployment: edgeA - interior - edgeB, tcpListener provisioned on edgeA, tcpConnector provisioned on edgeB, same VAN address (of course). Relies on listener tcp socket lifecycle patch:

https://github.com/kgiusti/skupper-router/tree/ISSUE-120

The tcpListener watches the VAN address via the address watch feature.

Initially the tcpListener is notified of one local consumer (weird, but expected because remote consumers are not visible) corresponding to the remote tcpConnector. This is expected.

Issue a curl request against the listener - this succeeds but as soon as curl disconnects the address watch on the tcpListener updates the local consumer count to zero. Further attempts to connect to the listener are refused.

The problem appears to be in addr_proxy.c where the loss of the curl incoming link removes the corresponding out-link to the interior router. Likely due to the "no longer source" check not being aware that the given address also has an address watch associated with it.

@kgiusti kgiusti added this to the 2.1.0 milestone May 5, 2022
ted-ross added a commit to ted-ross/skupper-router that referenced this issue May 5, 2022
…stination in address watches. Fixed the bug found by this test.
kgiusti pushed a commit to kgiusti/skupper-router that referenced this issue May 26, 2022
…stination in address watches. Fixed the bug found by this test.

(cherry picked from commit 8794208)
kgiusti pushed a commit that referenced this issue May 26, 2022
…ddress watches. Fixed the bug found by this test.

(cherry picked from commit 8794208)
@kgiusti kgiusti modified the milestones: 2.1.0, 2.0.2 May 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants