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.

Sign up for GitHub

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

Jump to bottom

Upgrade to Raft 1.0.0 #131

Closed
hsanjuan opened this issue Jul 25, 2017 · 2 comments
Closed

Upgrade to Raft 1.0.0 #131

hsanjuan opened this issue Jul 25, 2017 · 2 comments
Assignees
@hsanjuan
Labels
exp/wizard Extensive knowledge (implications, ramifications) required kind/enhancement A net-new feature or improvement to an existing feature P1 High: Likely tackled by core team if no one steps up

Comments

@hsanjuan
Copy link
Collaborator

We should start exploring how hard this is going to be, and if it works better or not.
@hsanjuan hsanjuan added exp/wizard Extensive knowledge (implications, ramifications) required status/ready Ready to be worked labels Jul 25, 2017
@hsanjuan hsanjuan added kind/enhancement A net-new feature or improvement to an existing feature P2 Medium: Good to have, but can wait until someone steps up labels Oct 10, 2017
@hsanjuan hsanjuan added this to the User feedback and bugs [Q4O3] milestone Oct 10, 2017
@hsanjuan
Copy link
Collaborator Author

See #112 #160 for problems with dynamic peer sets.

@hsanjuan
Copy link
Collaborator Author

the v2 branch has been merged to master and tagged as v1.0.0

See https://github.com/hashicorp/raft#tagged-releases

@hsanjuan hsanjuan self-assigned this Oct 11, 2017
@hsanjuan hsanjuan added P1 High: Likely tackled by core team if no one steps up status/in-progress In progress and removed P2 Medium: Good to have, but can wait until someone steps up status/ready Ready to be worked labels Oct 13, 2017
@hsanjuan hsanjuan changed the title Upgrade to Raft v2 branch Upgrade to Raft 1.0.0 Oct 24, 2017
hsanjuan added a commit that referenced this issue Oct 24, 2017
@hsanjuan
Issue #131: Bubble up some options to the configuration
69f468e 
CommitRetries and WaitForLeader timeout.

Avoid some redundant error checking

License: MIT
Signed-off-by: Hector Sanjuan <hector@protocol.ai>
@hsanjuan hsanjuan mentioned this issue Oct 25, 2017
Closed
hsanjuan added a commit that referenced this issue Oct 27, 2017
@hsanjuan
Issue #131: Rewrite commit/retry logic.
278a739 
License: MIT
Signed-off-by: Hector Sanjuan <hector@protocol.ai>
hsanjuan added a commit that referenced this issue Oct 27, 2017
@hsanjuan
Issue #131: Rewrite commit/retry logic.
b5126c8 
License: MIT
Signed-off-by: Hector Sanjuan <hector@protocol.ai>
hsanjuan added a commit that referenced this issue Oct 30, 2017
@hsanjuan
Issue #131: Do not crush when shutting down after consensus start error
0538d29 
License: MIT
Signed-off-by: Hector Sanjuan <hector@protocol.ai>
hsanjuan added a commit that referenced this issue Oct 30, 2017
@hsanjuan
Issue #131: Remove some left-over logging
fc65e00 
License: MIT
Signed-off-by: Hector Sanjuan <hector@protocol.ai>
hsanjuan added a commit that referenced this issue Oct 30, 2017
@hsanjuan
Issue #131: Be more verbose when raft initialization fails
7f4604d 
License: MIT
Signed-off-by: Hector Sanjuan <hector@protocol.ai>
hsanjuan added a commit that referenced this issue Oct 30, 2017
@hsanjuan
Issue #131: Do not abort on bad peerset
e97c9db 
Print warning instead.

Shutdown raft on peerRm.

License: MIT
Signed-off-by: Hector Sanjuan <hector@protocol.ai>
hsanjuan added a commit that referenced this issue Oct 30, 2017
@hsanjuan
Issue #131: Do not crash when shutting down after consensus start error
bd9811d 
License: MIT
Signed-off-by: Hector Sanjuan <hector@protocol.ai>
hsanjuan added a commit that referenced this issue Oct 30, 2017
@hsanjuan
Issue #131: Remove some left-over logging
cfa1275 
License: MIT
Signed-off-by: Hector Sanjuan <hector@protocol.ai>
hsanjuan added a commit that referenced this issue Oct 30, 2017
@hsanjuan
Issue #131: Be more verbose when raft initialization fails
21eea44 
License: MIT
Signed-off-by: Hector Sanjuan <hector@protocol.ai>
hsanjuan added a commit that referenced this issue Oct 30, 2017
@hsanjuan
Issue #131: Do not abort on bad peerset
1d95824 
Print warning instead.

Shutdown raft on peerRm.

License: MIT
Signed-off-by: Hector Sanjuan <hector@protocol.ai>
hsanjuan added a commit that referenced this issue Nov 1, 2017
@hsanjuan
Issue #131: Do not crash when shutting down after consensus start error
0980a5d 
License: MIT
Signed-off-by: Hector Sanjuan <hector@protocol.ai>
hsanjuan added a commit that referenced this issue Nov 1, 2017
@hsanjuan
Issue #131: Remove some left-over logging
7bfb3c4 
License: MIT
Signed-off-by: Hector Sanjuan <hector@protocol.ai>
hsanjuan added a commit that referenced this issue Nov 1, 2017
@hsanjuan
Issue #131: Be more verbose when raft initialization fails
b7ed9e5 
License: MIT
Signed-off-by: Hector Sanjuan <hector@protocol.ai>
hsanjuan added a commit that referenced this issue Nov 1, 2017
@hsanjuan
Issue #131: Do not abort on bad peerset
18dbf1a 
Print warning instead.

Shutdown raft on peerRm.

License: MIT
Signed-off-by: Hector Sanjuan <hector@protocol.ai>
hsanjuan added a commit that referenced this issue Nov 1, 2017
@hsanjuan
Issue #131: Improvements adding and removing
7a5f8f1 
This works on remove+shutdown procedure and fixes a few small
issues.

License: MIT
Signed-off-by: Hector Sanjuan <hector@protocol.ai>
hsanjuan added a commit that referenced this issue Nov 1, 2017
@hsanjuan
Issue #131: Destroy raft data when the peer has been removed
c912cfd 
License: MIT
Signed-off-by: Hector Sanjuan <hector@protocol.ai>
hsanjuan added a commit that referenced this issue Nov 1, 2017
@hsanjuan
Issue #131: Make sure peers are moved to bootstrap when leaving
45ac2a1 
Also, do not shutdown when seeing our own departure during bootstrap.

License: MIT
Signed-off-by: Hector Sanjuan <hector@protocol.ai>
hsanjuan added a commit that referenced this issue Nov 1, 2017
@hsanjuan
Issue #131: Make sure there are no double attempts to reset peers on …
ad42c96 
…remove

License: MIT
Signed-off-by: Hector Sanjuan <hector@protocol.ai>
hsanjuan added a commit that referenced this issue Nov 1, 2017
@hsanjuan
Issue #131: Make sure peers are moved to bootstrap when leaving
073c43e 
Also, do not shutdown when seeing our own departure during bootstrap.

License: MIT
Signed-off-by: Hector Sanjuan <hector@protocol.ai>
hsanjuan added a commit that referenced this issue Nov 1, 2017
@hsanjuan
Issue #131: rename addFromMultiaddrs to setFromMultiaddrs
bff1ec3 
License: MIT
Signed-off-by: Hector Sanjuan <hector@protocol.ai>
This was referenced Nov 1, 2017
Closed
Closed
@hsanjuan hsanjuan closed this as completed Nov 2, 2017
@ghost ghost removed the status/in-progress In progress label Nov 2, 2017
@hsanjuan hsanjuan mentioned this issue Nov 2, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hsanjuan hsanjuan

Labels
exp/wizard Extensive knowledge (implications, ramifications) required kind/enhancement A net-new feature or improvement to an existing feature P1 High: Likely tackled by core team if no one steps up
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@hsanjuan