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

Cherry Pick 862 #864

Merged
merged 3 commits into from
Sep 23, 2024
Merged

Cherry Pick 862 #864

merged 3 commits into from
Sep 23, 2024

Conversation

tiagolobocastro
Copy link
Contributor

feat: re-shutdown nexus when node is online

When nexus shutdown fails, if the node comes back online let's
attempt the shutdown again.

Signed-off-by: Tiago Castro <[email protected]>

fix: don't disown replica from unshutdown nexus

In case nexus shutdown failed we used to disown the replica from
both the volume and the nexus.
This can be a problem if the nexus is still running as the io-engine
does not handle it gracefully, leading into pool lock issues.
Instead, simply disown the replica from the volume and not the nexus.

Signed-off-by: Tiago Castro <[email protected]>

In case nexus shutdown failed we used to disown the replica from
both the volume and the nexus.
This can be a problem if the nexus is still running as the io-engine
does not handle it gracefully, leading into pool lock issues.
Instead, simply disown the replica from the volume and not the nexus.

Signed-off-by: Tiago Castro <[email protected]>
When nexus shutdown fails, if the node comes back online let's
attempt the shutdown again.

Signed-off-by: Tiago Castro <[email protected]>
Pulls submodule update which doesn't hide errors on nix eval.

Signed-off-by: Tiago Castro <[email protected]>
@tiagolobocastro
Copy link
Contributor Author

bors merge

@bors-openebs-mayastor
Copy link

🕐 Waiting for PR status (GitHub check) to be set, probably by CI. Bors will automatically try to run when all required PR statuses are set.

@bors-openebs-mayastor
Copy link

Build succeeded:

@bors-openebs-mayastor bors-openebs-mayastor bot merged commit defcd62 into develop Sep 23, 2024
4 checks passed
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