Merge pull request #41484 from liewegas/cephadm-progress

mgr/cephadm: progress item for service apply

Reviewed-by: Sebastian Wagner <sewagner@redhat.com>
Reviewed-by: Juan Miguel Olmo <jolmomar@redhat.com>

diff --cc src/pybind/mgr/cephadm/serve.py
index 652d9c0c0f1e2c998814fd789fa4a9b71fd13c83,56cc842b042f8e3c67d1587092b94d105ee803e4..3a2128e4958045ca6e1b9b43a9e8edd6c848887e
--- 1/src/pybind/mgr/cephadm/serve.py
--- 2/src/pybind/mgr/cephadm/serve.py
+++ b/src/pybind/mgr/cephadm/serve.py
@@@ -781,11 -769,13 +805,13 @@@ class CephadmServe
                  daemon_spec = svc.prepare_create(daemon_spec)
                  self._create_daemon(daemon_spec)
                  r = True
+                 progress_done += 1
+                 update_progress()
              except (RuntimeError, OrchestratorError) as e:
 -                self.mgr.events.for_service(
 -                    spec, 'ERROR',
 -                    f"Failed while placing {slot.daemon_type}.{daemon_id} "
 -                    f"on {slot.hostname}: {e}")
 +                msg = (f"Failed while placing {slot.daemon_type}.{daemon_id} "
 +                       f"on {slot.hostname}: {e}")
 +                self.mgr.events.for_service(spec, 'ERROR', msg)
 +                self.mgr.log.error(msg)
                  # only return "no change" if no one else has already succeeded.
                  # later successes will also change to True
                  if r is None: