Project

General

Profile

Bug #49280

mds/orch: bare/short hostname as a number is not supported

Added by David Casier 13 days ago. Updated 1 day ago.

Status:
Duplicate
Priority:
Normal
Category:
orchestrator
Target version:
% Done:

0%

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

Description

If the bare/short hostname is a simple id (example: 2.storage.domain):

----

root@1.storage.domain:~# ceph orch ls 
Error EINVAL: DaemonDescription: Cannot calculate service_id: daemon_id='23' hostname='2.storage.domain'

----

This part of the code will return an error : (https://github.com/ceph/ceph/blob/master/src/pybind/mgr/orchestrator/_interface.py#L1369)

----

            elif host in self.daemon_id:
                # daemon_id == "service_id.host" 
                # daemon_id == "service_id.host.random" 
                pre, post = self.daemon_id.rsplit(host, 1)
                if not pre.endswith('.'):
                    # '.' sep missing at front of host
                    raise err
                elif post and not post.startswith('.'):
                    # '.' sep missing at end of host
                    raise err
                return pre[:-1]



Related issues

Related to Orchestrator - Cleanup #46219: cephadm: remove DaemonDescription.service_id() New

History

#1 Updated by David Casier 13 days ago

root@1.storage.domain:~# ceph orch ls
Error EINVAL: DaemonDescription: Cannot calculate service_id: daemon_id='23' hostname='2.storage.domain'
            elif host in self.daemon_id:
                # daemon_id == "service_id.host" 
                # daemon_id == "service_id.host.random" 
                pre, post = self.daemon_id.rsplit(host, 1)
                if not pre.endswith('.'):
                    # '.' sep missing at front of host
                    raise err
                elif post and not post.startswith('.'):
                    # '.' sep missing at end of host
                    raise err
                return pre[:-1]

#2 Updated by Sebastian Wagner 9 days ago

  • Description updated (diff)

#3 Updated by Sebastian Wagner 9 days ago

  • Assignee set to Sebastian Wagner

#4 Updated by Sebastian Wagner 1 day ago

  • Related to Cleanup #46219: cephadm: remove DaemonDescription.service_id() added

#5 Updated by Sebastian Wagner 1 day ago

  • Status changed from New to Duplicate

will be fixed by #46219

Also available in: Atom PDF