From: dparmar18 Date: Wed, 6 Apr 2022 14:41:06 +0000 (+0530) Subject: cephfs-shell: get command should never create a directory on its own X-Git-Tag: v18.0.0~909^2~5 X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=64df8c2cdf352ff93404a895f2ce3a395341a6ed;p=ceph.git cephfs-shell: get command should never create a directory on its own Descrption: When using the get command on a single file, it would append the remote path to source path and create directories that shouldn't be created. For instance, file 'foo.txt' resides at `/dir1/dir2/` and `get` command is used to copy it to /tmp/foo then it would do `/tmp/dir1/dir2/foo.txt` which is not the expected behavior. Therefore this PR intends to correct this behavior. Fixes: https://tracker.ceph.com/issues/55112 Signed-off-by: Dhairya Parmar --- diff --git a/src/tools/cephfs/cephfs-shell b/src/tools/cephfs/cephfs-shell index 83be25c528db4..8ab719422dbdd 100755 --- a/src/tools/cephfs/cephfs-shell +++ b/src/tools/cephfs/cephfs-shell @@ -743,8 +743,7 @@ class CephFSShell(Cmd): return copy_to_local(root_src_dir, b'-') elif is_file_exists(args.remote_path): - copy_to_local(root_src_dir, - root_dst_dir + b'/' + root_src_dir) + copy_to_local(root_src_dir, root_dst_dir) elif b'/' in root_src_dir and is_file_exists(fname[1], fname[0]): copy_to_local(root_src_dir, root_dst_dir) else: