Skip to content

Conversation

@Frannie-Ludmilla
Copy link
Contributor

@Frannie-Ludmilla Frannie-Ludmilla commented Oct 18, 2024

New signature for the quarantine procedure. Relates to https://github.com/neo-technology/neo4j/pull/27340

====
The `wipeMode` option is only enforced during the removal of a database from quarantine.
The possible values for `wipeMode` are: `DO_NOT_WIPE` (default value and previous behaviour), `WIPE_STATE_KEEP_STORE` and `WIPE_STATE_AND_STORE`.
Please keep in mind that the deletion of the cluster state and store (`WIPE_STATE_AND_STORE`) or even just the deletion of the state (`WIPE_STATE_KEEP_STORE`) could cause loss of data and should be carefully evaluated depending on the case.
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe it's worth explaining that wiping the cluster state means this server will join the cluster as a new member. And in order for the cluster to accept new members, they need to have a quorum of old members. And therefore it's dangerous to wipe the state for >=quorum of members. Sometimes the quarantine might be due to the store state, so deleting the store only might be sufficient. But I see now that we don't have an option for it, let's discuss it.

@NataliaIvakina
Copy link
Collaborator

Hi! @Frannie-Ludmilla, could you please rebase your PR with the latest version of the dev branch? You PR misses so many procedures and current changes.

@neo-technology-commit-status-publisher
Copy link
Collaborator

neo-technology-commit-status-publisher commented Oct 21, 2024

Thanks for the documentation updates.

The preview documentation has now been torn down - reopening this PR will republish it.

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.

4 participants