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

[FIXED] Cluster: Handling of deleted channels #1013

Merged
merged 3 commits into from
Feb 4, 2020

Conversation

kozlovic
Copy link
Member

A channel could be deleted and then later recreated (same name).
This is not handled properly in some cases such as restore from
snapshot.
This PR attempts to fix the problem by storing in the RAFT db
a UID when a channel is created. This UID is added to some of
the replicated operations.

Resolves #1008
Resolves #1010

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

@coveralls
Copy link

coveralls commented Jan 23, 2020

Coverage Status

Coverage decreased (-0.02%) to 92.03% when pulling 26b046f on fix_snapshot_and_channel_delete into d7aed78 on master.

A channel could be deleted and then later recreated (same name).
This is not handled properly in some cases such as restore from
snapshot.
This PR attempts to fix the problem by storing in the RAFT db
a ID when a channel is created. This ID is added to some of
the replicated operations.

Resolves #1008
Resolves #1010

Signed-off-by: Ivan Kozlovic <ivan@synadia.com>
@kozlovic kozlovic force-pushed the fix_snapshot_and_channel_delete branch from bff9895 to 86e0432 Compare January 23, 2020 17:56
server/clustering.go Outdated Show resolved Hide resolved
server/clustering.go Show resolved Hide resolved
server/clustering.go Show resolved Hide resolved
server/server.go Outdated Show resolved Hide resolved
server/server.go Outdated Show resolved Hide resolved
server/server.go Outdated Show resolved Hide resolved
server/snapshot.go Outdated Show resolved Hide resolved
server/snapshot.go Outdated Show resolved Hide resolved
Signed-off-by: Ivan Kozlovic <ivan@synadia.com>
Copy link
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

Signed-off-by: Ivan Kozlovic <ivan@synadia.com>
@kozlovic kozlovic merged commit 515e026 into master Feb 4, 2020
@kozlovic kozlovic deleted the fix_snapshot_and_channel_delete branch February 4, 2020 00:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants