Report recovery status for MDT #27
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ofaaland
force-pushed
the
toss-3086-protocol_changes
branch
from
d03867c
to
3ae683d
Compare
Update the protocol to include MDT recovery status. Define a version 2 of the lmt_mdt message, which includes the recovery status field. Obtain the recovery status for MDTs from the procfile recovery_status, as is already done for OSTs. Move code to parse the procfile out of ost.c to enable it to be used by mdt.c as well. Define a parser for the lmt_mdt version 2 message, and modify ltop to use the appropriate version. Similarly update the cerebrod monitor, lmt_mysql. When an MDT reports it is in RECOVERY, display a message in ltop at the top of the window to the right of the filesystem name. If more than one MDT is in recovery, the full status of the first one encountered will be displayed. This allows the user to determine whether the count of re-connected clients is increasing or not. Update the tests to reflect the protocol change.
ofaaland
force-pushed
the
toss-3086-protocol_changes
branch
from
3ae683d
to
87ce470
Compare
|
Landed! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Update the MDT cerebro metric to report the recovery status, from
the procfile recovery_status, as is already done for OSTs.
Move code to parse the procfile out of ost.c to enable it to be
used by mdt.c as well.
Define a version 2 of the lmt_mdt message, which includes the
recovery status field.
Define a parser for the lmt_mdt version 2 message, and modify ltop
to use the appropriate version.
When an MDT reports it is in RECOVERY, display a message in ltop
at the top of the window to the right of the filesystem name. If
more than one MDT is in recovery, the full status of the first one
encountered will be displayed. This allows the user to determine
whether the count of re-connected clients is increasing or not.