Skip to content

Commit

Permalink
[RM-36583] align with create by using short hostname
Browse files Browse the repository at this point in the history
If using fully qualified domain name to check monitor
status after "mon add", it will hit the error that can't
find the node file under /var/run/ceph/ which result
in aboring add monitor

Signed-off-by: Changcheng Liu <changcheng.liu@intel.com>
  • Loading branch information
LiucIntel committed Oct 24, 2018
1 parent bb0948e commit 420bdcc
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions ceph_deploy/mon.py
Original file line number Diff line number Diff line change
Expand Up @@ -193,6 +193,7 @@ def mon_add(args):
)

mon_section = 'mon.%s' % mon_host
(name, host) = next(mon_hosts(args.mon))
cfg_mon_addr = cfg.safe_get(mon_section, 'mon addr')

if args.address:
Expand Down Expand Up @@ -223,8 +224,8 @@ def mon_add(args):

# tell me the status of the deployed mon
time.sleep(2) # give some room to start
catch_mon_errors(distro.conn, rlogger, mon_host, cfg, args)
mon_status(distro.conn, rlogger, mon_host, args)
mon_status(distro.conn, rlogger, name, args)
catch_mon_errors(distro.conn, rlogger, name, cfg, args)
distro.conn.exit()

except RuntimeError as e:
Expand Down

1 comment on commit 420bdcc

@dmick
Copy link
Member

@dmick dmick commented on 420bdcc Oct 24, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This does more than just change the name used. Please explain all changes.

Please sign in to comment.