mon: make 'osd crush move ...' idempotent

If we don't need to move the item, return success.

Signed-off-by: Sage Weil <sage@inktank.com>

diff --git a/src/mon/OSDMonitor.cc b/src/mon/OSDMonitor.cc
index e8cf2c84280de18e89944a7060b42117628c6e93..a9d68075cd4cf192631d8111053431172695155f 100644 (file)
--- a/src/mon/OSDMonitor.cc
+++ b/src/mon/OSDMonitor.cc
@@ -2656,7 +2656,8 @@ bool OSDMonitor::prepare_command(MMonCommand *m)
            return true;
          }
        } else {
-           ss << "no need to move item id " << id << " name '" << name << "' to location " << loc << " in crush map";
+         ss << "no need to move item id " << id << " name '" << name << "' to location " << loc << " in crush map";
+         err = 0;
        }
       } while (false);
     }