SERVER-58926 ReshardingDonor should not try to set the stop time of the critical section if it the critical section was already released

(cherry picked from commit 07a6841c533193ec64190a870ee11672ba4d39e5)

1 files changed, 8 insertions, 2 deletions

diff --git a/src/mongo/db/s/resharding/resharding_metrics.cpp b/src/mongo/db/s/resharding/resharding_metrics.cpp
index a47cffc131b..4804ccb2bb7 100644
--- a/src/mongo/db/s/resharding/resharding_metrics.cpp
+++ b/src/mongo/db/s/resharding/resharding_metrics.cpp
@@ -105,13 +105,19 @@ static StringData serializeState(boost::optional<CoordinatorStateEnum> e) {
 class TimeInterval {
 public:
     void start(Date_t d) noexcept {
-        invariant(!_start, "Already started");
+        if (_start) {
+            LOGV2_WARNING(5892600, "Resharding metrics already started, start() is a no-op");
+            return;
+        }
         _start = d;
     }
 
     void end(Date_t d) noexcept {
         invariant(_start, "Not started");
-        invariant(!_end, "Already stopped");
+        if (_end) {
+            LOGV2_WARNING(5892601, "Resharding metrics already ended, end() is a no-op");
+            return;
+        }
         _end = d;
     }