Skip to content

Conversation

hsanjuan
Copy link
Collaborator

This patch modifies the RPC protocol tag to use Major and Minor parts of the
version and not all of it.

This means all peers on the 0.5.x can run in the same cluster.

As cluster has become more mature and I see less risks in letting peers from
similar versions run together. This is useful when upgrading too.

License: MIT
Signed-off-by: Hector Sanjuan code@hector.link

@hsanjuan hsanjuan self-assigned this Sep 26, 2018
@hsanjuan hsanjuan requested a review from lanzafame September 26, 2018 15:50
@ghost ghost added the status/in-progress In progress label Sep 26, 2018
@coveralls
Copy link

coveralls commented Sep 26, 2018

Coverage Status

Coverage decreased (-0.02%) to 65.154% when pulling 59714f6 on fix/minor-version-compat into e941d6b on master.

Copy link
Contributor

@lanzafame lanzafame left a comment

Choose a reason for hiding this comment

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

@hsanjuan I like this a lot but I think a having the Commit variable is useful for debugging, especially development builds.

This patch modifies the RPC protocol tag to use Major and Minor parts of the
version and not all of it.

This means all peers on the 0.5.x can run in the same cluster.

As cluster has become more mature and I see less risks in letting peers from
similar versions run together. This is useful when upgrading too.

License: MIT
Signed-off-by: Hector Sanjuan <code@hector.link>
@hsanjuan hsanjuan force-pushed the fix/minor-version-compat branch from 22006b6 to 59714f6 Compare September 27, 2018 05:36
@hsanjuan
Copy link
Collaborator Author

@lanzafame what do you mean? The Commit information is part of the Version object now (as .Build). UI-wise things should be very similar (?).

@lanzafame
Copy link
Contributor

@hsanjuan Ahh sorry, I missed that.

@hsanjuan hsanjuan merged commit cbdd075 into master Sep 27, 2018
@hsanjuan hsanjuan deleted the fix/minor-version-compat branch September 27, 2018 06:13
@ghost ghost removed the status/in-progress In progress label Sep 27, 2018
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.

3 participants