Skip to content

Commit

Permalink
FSMap: when upgrading from MDSMap, do not initialize from a non-exist…
Browse files Browse the repository at this point in the history
…ent FS

Signed-off-by: Greg Farnum <gfarnum@redhat.com>
  • Loading branch information
gregsfortytwo committed Mar 11, 2016
1 parent 09fce2d commit 1634827
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions src/mds/FSMap.cc
Expand Up @@ -332,8 +332,10 @@ void FSMap::decode(bufferlist::iterator& p)
*migrate_fs = legacy_fs;
migrate_fs->fscid = FS_CLUSTER_ID_ANONYMOUS;
migrate_fs->mds_map.fs_name = "default";
legacy_client_fscid = migrate_fs->fscid;
filesystems[migrate_fs->fscid] = migrate_fs;
if (migrate_fs->mds_map.enabled) {

This comment has been minimized.

Copy link
@jcsp

jcsp Mar 12, 2016

This change should be superseded by ceph#8073 (a more general conditional on whether to construct migrate_fs at all if enabled was not set)

legacy_client_fscid = migrate_fs->fscid;
filesystems[migrate_fs->fscid] = migrate_fs;
}
compat = migrate_fs->mds_map.compat;
enable_multiple = false;
} else {
Expand Down

0 comments on commit 1634827

Please sign in to comment.