rgw/lc: LCOpAction_CurrentExpiration checks mtime for delete markers

the unconditional 'return true' for delete markers was causing check()
to ignore the lifecycle policy's "Days" field, causing delete markers to
be expired and removed prematurely

instead of the early return, fall through to the mtime/size comparisons
used for normal objects

Fixes: https://tracker.ceph.com/issues/73240
Signed-off-by: Casey Bodley <cbodley@redhat.com>

diff --git a/src/rgw/rgw_lc.cc b/src/rgw/rgw_lc.cc
index 1fbca2f0968df1d9c775031ebc734cb043cea13e..fd60f271f51d79163d534f86b5191b21deba08d6 100644 (file)
--- a/src/rgw/rgw_lc.cc
+++ b/src/rgw/rgw_lc.cc
@@ -1175,8 +1175,8 @@ public:
 
       ldpp_dout(dpp, 7) << __func__ << "(): dm-check DELE: key=" << o.key
                         << " " << oc.wq->thr_name() << dendl;
-      *exp_time = real_clock::now();
-      return true;
+
+      // go on to compare mtime, size, etc
     }
 
     auto& mtime = o.meta.mtime;