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

Shutdown joining member when initial cluster version is not valid #13390

Merged

Conversation

vbekiaris
Copy link
Collaborator

The local member should be validating the initial cluster version
as is already done each time cluster version changes.

@vbekiaris vbekiaris added this to the 3.10.3 milestone Jul 2, 2018
@vbekiaris vbekiaris self-assigned this Jul 2, 2018
@vbekiaris vbekiaris modified the milestones: 3.10.3, 3.10.4 Jul 6, 2018
@mmedenjak mmedenjak modified the milestones: 3.10.4, 3.10.5 Jul 23, 2018
@vbekiaris vbekiaris force-pushed the enhancements/3.10.3/ru-license branch from 1509b28 to c0f83b5 Compare August 29, 2018 10:50
The local member should be validating the initial cluster version
as is already done each time cluster version changes.
@vbekiaris vbekiaris force-pushed the enhancements/3.10.3/ru-license branch from c0f83b5 to b74efa7 Compare August 29, 2018 10:52
@mmedenjak mmedenjak modified the milestones: 3.10.5, 3.10.6 Sep 3, 2018
@vbekiaris vbekiaris changed the title [DO NOT MERGE] Shutdown joining member when initial cluster version is not valid Shutdown joining member when initial cluster version is not valid Sep 6, 2018
@mdogan
Copy link
Contributor

mdogan commented Sep 10, 2018

@vbekiaris: Are you going to add test for this scenario in EE repo?

@vbekiaris
Copy link
Collaborator Author

@mdogan yes, see https://github.com/hazelcast/hazelcast-enterprise/pull/2201/files#diff-52ad544b612ec90b4597ca2434da7100R55

@vbekiaris vbekiaris merged commit 54b54ba into hazelcast:maintenance-3.x Sep 10, 2018
@vbekiaris
Copy link
Collaborator Author

@mdogan @mmedenjak thanks for the reviews

@vbekiaris vbekiaris deleted the enhancements/3.10.3/ru-license branch September 10, 2018 09:52
@mmedenjak mmedenjak added the Source: Internal PR or issue was opened by an employee label Apr 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants