Skip to content

[FIXED] Gateway: Possible loss of queue sub interest on restart#6607

Merged
derekcollison merged 1 commit intomainfrom
gw_sub_interest_fix
Mar 6, 2025
Merged

[FIXED] Gateway: Possible loss of queue sub interest on restart#6607
derekcollison merged 1 commit intomainfrom
gw_sub_interest_fix

Conversation

@kozlovic
Copy link
Copy Markdown
Member

@kozlovic kozlovic commented Mar 5, 2025

If a server has a connection from a route or a leaf and the remote has more than one member of the same queue group, when the connection is recreated (due to network issue or server restart), then count coming from the connection would be greater than 1 but the server would store the interest count for outbound gateways as 1.

When one of the remote queue subscriber unsubscribes, the count would go down to 0 and the server would incorrectly send a protocol to the gateway indicating that there is no longer interest for this queue group, which would stop the flow.

A new queue subscriber on that group would resume the flow.

Signed-off-by: Ivan Kozlovic ivan@synadia.com

If a server has a connection from a route or a leaf and the remote
has more than one member of the same queue group, when the connection
is recreated (due to network issue or server restart), then count
coming from the connection would be greater than 1 but the server
would store the interest count for outbound gateways as 1.

When one of the remote queue subscriber unsubscribes, the count would
go down to 0 and the server would incorrectly send a protocol to the
gateway indicating that there is no longer interest for this queue
group, which would stop the flow.

A new queue subscriber on that group would resume the flow.

Signed-off-by: Ivan Kozlovic <ivan@synadia.com>
@kozlovic kozlovic requested a review from a team as a code owner March 5, 2025 23:37
@kozlovic kozlovic requested a review from derekcollison March 6, 2025 00:07
Copy link
Copy Markdown
Member

@derekcollison derekcollison left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@derekcollison derekcollison merged commit dd8dbe7 into main Mar 6, 2025
5 checks passed
@derekcollison derekcollison deleted the gw_sub_interest_fix branch March 6, 2025 01:46
neilalexander added a commit that referenced this pull request Apr 17, 2025
Includes the following (already cherry-picked) PRs:

- #6587
- #6607
- #6612
- #6609
- #6620
- #6668
- #6674
- #6647
- #6684
- #6691
- #6697
- #6705
- #6706
- #6704
- #6714
- #6720
- #6727
- #6730
- #6726
- #6732
- #6759
- #6753
- #6685
- #6769
- #6777
- #6785
- #6786
- #6778
- #6790
- #6791
- #6798
- #6794
- #6801

Signed-off-by: Neil Twigg <neil@nats.io>

Signed-off-by: Neil Twigg <neil@nats.io>
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

Successfully merging this pull request may close these issues.

2 participants