]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commit
mgr/cephadm: make default command timeout field an int 51963/head
authorAdam King <adking@redhat.com>
Wed, 3 Jan 2024 13:35:05 +0000 (08:35 -0500)
committerAdam King <adking@redhat.com>
Wed, 3 Jan 2024 13:42:38 +0000 (08:42 -0500)
commite917d66838b2ea52538f4460d89d633ce7786067
treeec03634fb037e51945206a7b42f389d4193b76c3
parentefb69ee350d1c20bdbfc88db8a01ec88761bd99f
mgr/cephadm: make default command timeout field an int

When the fields was "secs" instead, we could hit

Traceback (most recent call last):
  File "/usr/share/ceph/mgr/cephadm/serve.py", line 1380, in _run_cephadm_json
    out, err, code = await self._run_cephadm(
  File "/usr/share/ceph/mgr/cephadm/serve.py", line 1525, in _run_cephadm
    raise OrchestratorError(
orchestrator._interface.OrchestratorError: cephadm exited with an error code: 2, stderr: usage: cephadm
       [-h] [--image IMAGE] [--docker] [--data-dir DATA_DIR]
       [--log-dir LOG_DIR] [--logrotate-dir LOGROTATE_DIR]
       [--sysctl-dir SYSCTL_DIR] [--unit-dir UNIT_DIR] [--verbose]
       [--timeout TIMEOUT] [--retry RETRY] [--env ENV] [--no-container-init]
       [--no-cgroups-split]
       {version,pull,inspect-image, . . .
       ...
cephadm: error: argument --timeout: invalid int value: '295.0'

where the value ends up as a floating point value
after converting to a string (which is necessary to actually
pass it to the binary). By setting the field to be an
int, we should be able to avoid this.

Signed-off-by: Adam King <adking@redhat.com>
src/pybind/mgr/cephadm/module.py