rgw: Don't print spurious error on `radosgw-admin` exit

When debugging crashes related to `RGWDataChangesLog` shutdown, I
fixed the crashes, but broke the logic that decides whether to print
the error.

Fixes: https://tracker.ceph.com/issues/70864
Signed-off-by: Adam C. Emerson <aemerson@redhat.com>

diff --git a/src/rgw/driver/rados/rgw_datalog.cc b/src/rgw/driver/rados/rgw_datalog.cc
index 438430634649a61934b8aa8b46ac4c99511556e3..d6ed19f20b462a999b48b5fee22bf126ddf14e57 100644 (file)
--- a/src/rgw/driver/rados/rgw_datalog.cc
+++ b/src/rgw/driver/rados/rgw_datalog.cc
@@ -1357,10 +1357,13 @@ bool RGWDataChangesLog::going_down() const
 
 asio::awaitable<void> RGWDataChangesLog::shutdown() {
   DoutPrefix dp{cct, ceph_subsys_rgw, "Datalog Shutdown"};
-  if (down_flag || !ran_background) {
+  if (down_flag) {
     co_return;
   }
   down_flag = true;
+  if (!ran_background)  {
+    co_return;
+  }
   renew_stop();
   // Revisit this later
   if (renew_signal)