crimson/osd: PGAdvanceMap ops complete in order

For the analysis refer to the bug tracker.

Dropping the `handle.exit()` looks like the simplest solution,
far simpler than e.g. introducing extra future-promise.

Fixes: https://tracker.ceph.com/issues/57536
Signed-off-by: Radoslaw Zarzynski <rzarzyns@redhat.com>

diff --git a/src/crimson/osd/osd_operations/pg_advance_map.cc b/src/crimson/osd/osd_operations/pg_advance_map.cc
index 51b279c7edb8b7957d74f29b1ba1c9e2c89bec95..c7fcacb315de205c407340709703fa3825bb91d9 100644 (file)
--- a/src/crimson/osd/osd_operations/pg_advance_map.cc
+++ b/src/crimson/osd/osd_operations/pg_advance_map.cc
@@ -74,7 +74,6 @@ seastar::future<> PGAdvanceMap::start()
           });
       }).then([this] {
         pg->handle_activate_map(rctx);
-        handle.exit();
         if (do_init) {
           shard_manager.pg_created(pg->get_pgid(), pg);
           shard_manager.get_shard_services().inc_pg_num();