Project

General

Profile

Actions

Bug #46782

open

cephadm bootstrap demands --mon-ip or --mon-addrv option even when Mon IPs are specified in the placement YAML

Added by Nathan Cutler over 3 years ago. Updated almost 2 years ago.

Status:
New
Priority:
Low
Assignee:
-
Category:
cephadm (binary)
Target version:
-
% Done:

0%

Source:
Tags:
Backport:
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

Scenario:

I run "cephadm bootstrap" sending it the following YAML via --apply-spec:

service_type: mgr
service_name: mgr
placement:
    hosts:
        - master
---
service_type: mon
service_name: mon
placement:
    hosts:
        - master:10.20.119.200

Now, in my mind it doesn't makes sense to provide the same information twice, so I omit "--mon-ip 10.20.119.200". But "cephadm bootstrap" doesn't like this:

master:~ # cat /var/log/ceph/cephadm.log 
INFO:cephadm:Skip prepare_host
INFO:root:Cluster fsid: 51cdb752-d30d-11ea-9858-5254003561d1
DEBUG:cephadm:Acquiring lock 140384369371064 on /run/cephadm/51cdb752-d30d-11ea-9858-5254003561d1.lock
DEBUG:cephadm:Lock 140384369371064 acquired on /run/cephadm/51cdb752-d30d-11ea-9858-5254003561d1.lock
Traceback (most recent call last):
  File "/usr/sbin/cephadm", line 5046, in <module>
    r = args.func()
  File "/usr/sbin/cephadm", line 1214, in _default_image
    return func()
  File "/usr/sbin/cephadm", line 2537, in command_bootstrap
    raise Error('must specify --mon-ip or --mon-addrv')
__main__.Error: must specify --mon-ip or --mon-addrv
DEBUG:cephadm:Releasing lock 140384369371064 on /run/cephadm/51cdb752-d30d-11ea-9858-5254003561d1.lock
DEBUG:cephadm:Lock 140384369371064 released on /run/cephadm/51cdb752-d30d-11ea-9858-5254003561d1.lock
Actions #1

Updated by Sebastian Wagner over 3 years ago

  • Status changed from New to Triaged
  • Priority changed from Normal to Low

getting the info from the yaml spec without any hard dependencies to pyyaml is going to be painful.

Actions #2

Updated by Sebastian Wagner almost 3 years ago

  • Category set to cephadm (binary)
  • Status changed from Triaged to Need More Info

This requires:

  • the ability to parse YAML within the
  • import python-common from the cephadm binary.
Actions #3

Updated by Redouane Kachach Elhichou about 2 years ago

  • Status changed from Need More Info to New
Actions #4

Updated by Redouane Kachach Elhichou almost 2 years ago

Probably it could be addressed once https://github.com/ceph/ceph/pull/41855 is merged

Actions

Also available in: Atom PDF