diff options
| author | Sage Weil <sage@inktank.com> | 2013-08-09 18:02:32 -0700 |
|---|---|---|
| committer | Sage Weil <sage@inktank.com> | 2013-08-09 21:24:08 -0700 |
| commit | cb50b5a7f1ab2d4e7fdad623a0e7769000755a70 (patch) | |
| tree | a99935ab17a6fc66fbe28e459e7281ac3581c9ec | |
| parent | b66a3d6029810c354c46b739fce1bac5dbd857fa (diff) | |
| download | ceph-cb50b5a7f1ab2d4e7fdad623a0e7769000755a70.tar.gz | |
ceph-disk: fix mount options passed to move_mount
Commit 6cbe0f021f62b3ebd5f68fcc01a12fde6f08cff5 added a mount_options but
in certain cases it may be blank. Fill in with the defaults, just as we
do in mount().
Backport: cuttlefish
Reviewed-by: Dan Mick <dan.mick@inktank.com>
Signed-off-by: Sage Weil <sage@inktank.com>
| -rwxr-xr-x | src/ceph-disk | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/src/ceph-disk b/src/ceph-disk index e34faaf0689..0ecfdf02073 100755 --- a/src/ceph-disk +++ b/src/ceph-disk @@ -1345,6 +1345,7 @@ def move_mount( path, cluster, osd_id, + fstype, mount_options, ): LOG.debug('Moving mount to final location...') @@ -1355,6 +1356,10 @@ def move_mount( ) maybe_mkdir(osd_data) + # pick best-of-breed mount options based on fs type + if mount_options is None: + mount_options = MOUNT_OPTIONS.get(fstype, '') + # we really want to mount --move, but that is not supported when # the parent mount is shared, as it is by default on RH, Fedora, # and probably others. Also, --bind doesn't properly manipulate @@ -1528,6 +1533,7 @@ def mount_activate( path=path, cluster=cluster, osd_id=osd_id, + fstype=fstype, mount_options=mount_options, ) return (cluster, osd_id) |