[BZ-1231203] throws if multiple mon are passed to "mon add"

Signed-off-by: Kefu Chai <kchai@redhat.com>

diff --git a/ceph_deploy/mon.py b/ceph_deploy/mon.py
index 9b20944db2c335c77715b721d6aecc51f499d8ab..bb6549aa33c7c81a71bbe831adcbb3679f8f76aa 100644 (file)
--- a/ceph_deploy/mon.py
+++ b/ceph_deploy/mon.py
@@ -175,6 +175,8 @@ def mon_add(args):
 
     if not args.mon:
         raise exc.NeedHostError()
+    elif len(args.mon) > 1:
+        raise exc.GenericError('Only one node can be added at a time')
     mon_host = args.mon[0]
 
     try:
@@ -482,7 +484,8 @@ def make(parser):
 
       If the section for the monitor exists and defines a `mon addr` that
       will be used, otherwise it will fallback by resolving the hostname to an
-      IP. If `--address` is used it will override all other options.
+      IP. If `--address` is used it will override all other options. Please
+      note that only one node can be added at a time.
 
     destroy
       Completely remove monitors on a remote host. Requires hostname(s) as

diff --git a/docs/source/mon.rst b/docs/source/mon.rst
index 075a4994afcc3fc7b01f726d6753a6d7e95d5fc6..5caca6edbd631326bf42f01559306eee56719347 100644 (file)
--- a/docs/source/mon.rst
+++ b/docs/source/mon.rst
@@ -62,7 +62,8 @@ nor overridden in the command-line it will fall-back to resolving the address
 of the provided host.
 
 .. warning:: If the monitor host has multiple addresses you should specify
-             the address directly to ensure the right IP is used.
+             the address directly to ensure the right IP is used. Please
+             note, only one node can be added at a time.
 
 .. versionadded:: 1.4.0