Treat rename across pools as an error

Fixes: ceph#2370 Reviewed-by: Samuel Just <[email protected]>
dano · May 2, 2012 · ee26c5d · ee26c5d
1 parent 2a93258
commit ee26c5d
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/man/rbd.8 b/man/rbd.8
@@ -116,7 +116,7 @@ Creates a new image and imports its data from path.
 Copies the content of a src\-image into the newly created dest\-image.
 .TP
 .B \fBmv\fP [\fIsrc\-image\fP] [\fIdest\-image\fP]
-Renames an image.
+Renames an image.  Note: rename across pools is unsupported.
 .TP
 .B \fBsnap\fP ls [\fIimage\-name\fP]
 Dumps the list of snapshots inside a specific image.

diff --git a/src/rbd.cc b/src/rbd.cc
@@ -1090,6 +1090,13 @@ int main(int argc, const char **argv)
     usage_exit();
   }
 
+  if ((opt_cmd == OPT_RENAME) && (strcmp(poolname, dest_poolname) != 0)) {
+    cerr << "error: mv/rename across pools not supported" << std::endl;
+    cerr << "source pool: " << poolname << " dest pool: " << dest_poolname
+      << std::endl;
+    exit(EXIT_FAILURE);
+  }
+
   bool talk_to_cluster = (opt_cmd != OPT_MAP &&
 			  opt_cmd != OPT_UNMAP &&
 			  opt_cmd != OPT_SHOWMAPPED);