#15065: Add standby_for_fscid in MMDSBeacon and handle it #8257
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
added 15 commits
March 22, 2016 12:08
Not used for anything currently, but let's make this class pass through the features to FSMap's encode method for the benefit of the future generations. Signed-off-by: John Spray <john.spray@redhat.com>
Signed-off-by: John Spray <john.spray@redhat.com>
Signed-off-by: John Spray <john.spray@redhat.com>
Coverity: CID 1355578: Performance inefficiencies (PASS_BY_VALUE) Signed-off-by: John Spray <john.spray@redhat.com>
This got mangled somehow. Signed-off-by: John Spray <john.spray@redhat.com>
This broke with 4ad8f72 ('features' gets output as an item in the 'export_targets' list), which wasn't the fsmap branch, but fix it while I'm tidying. Signed-off-by: John Spray <john.spray@redhat.com>
...from Filesystem. This will prepare us for if MDSMap gets any more feature-sensitive encoding in the future. Fixes: #15062 Signed-off-by: John Spray <john.spray@redhat.com>
Only for use in extremis, where we have encountered a bug that generates an invalid FSMap, but the way to resolve it might be to proceed anyway and stay up long enough to use commands to fix it. Fixes: #15063 Signed-off-by: John Spray <john.spray@redhat.com>
These were just dumping JSON as a placeholder. Not strictly necessary to have the extra non-machine-readable variants of these outputs, but some users will probably appreciate it, so why not. Signed-off-by: John Spray <john.spray@redhat.com>
Signed-off-by: John Spray <john.spray@redhat.com>
This is handy if you want to quickly create a cluster with lots of filesystems, for playing with e.g. what the ceph status output looks like when multiple filesystems are in play. This changes the default filesystem/pool names to be e.g. cephfs_data_a instead of just `cephfs_data`, but anything who relied on those names in a vstart environment houldn't have been (this won't affect teuthology tests, of course, where we still have a few annoying hardcoded pool names). Signed-off-by: John Spray <john.spray@redhat.com>
Name dates from earlier version of multi-fs where I was referring to filesystem IDs as namespace IDs. Signed-off-by: John Spray <john.spray@redhat.com>
For the MDS daemon to tell the mon which filesystem it wants to a be a standby for, when it is expressing a standby_for_rank. Signed-off-by: John Spray <john.spray@redhat.com>
...and pass it through into MMDSBeacon. Signed-off-by: John Spray <john.spray@redhat.com>
...so that an MDS can specify that it wants to act within a particular filesystem without having to be bound to a particular rank. Signed-off-by: John Spray <john.spray@redhat.com>
|
The tests for this ceph/ceph-qa-suite#905 |
gregsfortytwo
added a commit
that referenced
this pull request
Mar 24, 2016
Conflicts: src/mds/FSMap.cc #8257
|
Looks good to me. I forget, did you want your standby-replay logic changes to go in here, or separately? (Perhaps post-jewel?) |
|
The standby replay cleanup will be a separate PR |
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.
This PR depends on #7998, the new stuff here starts at the "s/standby_for_ns/standby_for_fscid/" commit