]> git.apps.os.sepia.ceph.com Git - ceph.git/commit
mgr/telemetry: fix waiting for mgr to warm up 45772/head
authorYaarit Hatuka <yaarit@redhat.com>
Tue, 9 Nov 2021 18:31:11 +0000 (18:31 +0000)
committerLaura Flores <lflores@redhat.com>
Tue, 5 Apr 2022 00:06:48 +0000 (19:06 -0500)
commitd3c094171598e42ae01800a6e06eb601b065b374
tree620fb0e1e042125a7e1714b96eb73da45bc0a06f
parentb51d866559c7cde15d3b75ee17bbba8fdf2584be
mgr/telemetry: fix waiting for mgr to warm up

1. The implementation of config_notify() in telemetry module sets the
flag for event, which is supposed to wake up the 'serve' thread whenever
a config option is changed. The problem is that we call config_notify()
at the beginning of serve(), before we enter its 'run' loop. This call
sets the event which cancels the 10 seconds wait for the mgr to warm up.
To fix this, we extract the logic of updating the config options to a
separate function (config_update_module_option()), and call it on
__init__, instead of calling config_notify() in serve().

2. We should always wait for the mgr to warm up here (10 seconds). In
case of a sporadic event (e.g. a config option change via CLI) the event
will be set, and wait will return immediately. We enforce this wait by
using time.sleep(10) instead of event.wait(10).

Fixes: https://tracker.ceph.com/issues/53204
Signed-off-by: Yaarit Hatuka <yaarit@redhat.com>
(cherry picked from commit fa5cc0ca081ca3cce552e0cb21a1e17273cf3482)

 Conflicts:
src/pybind/mgr/telemetry/module.py

- Several options under __init__ not present in Octopus
- No type checking in Octopus
src/pybind/mgr/telemetry/module.py