2 files changed, 41 insertions, 15 deletions

diff --git a/src/mongo/logv2/log_util.cpp b/src/mongo/logv2/log_util.cpp
index ded9e390e89..c44ff5988af 100644
--- a/src/mongo/logv2/log_util.cpp
+++ b/src/mongo/logv2/log_util.cpp
@@ -49,9 +49,19 @@ void addLogRotator(StringData logType, LogRotateCallback cb) {
     logRotateCallbacks.emplace(logType, std::move(cb));
 }
 
-bool rotateLogs(bool renameFiles,
-                boost::optional<StringData> logType,
-                std::function<void(Status)> onMinorError) {
+void LogRotateErrorAppender::append(const Status& err) {
+    if (_combined.isOK()) {
+        _combined = err;
+    } else if (!err.isOK()) {
+        // if there are multiple, distinct error codes, use OperationFailed instead
+        auto newCode = (_combined.code() == err.code()) ? err.code() : ErrorCodes::OperationFailed;
+        _combined = Status(newCode, str::stream() << _combined.reason() << ", " << err.reason());
+    }
+}
+
+Status rotateLogs(bool renameFiles,
+                  boost::optional<StringData> logType,
+                  std::function<void(Status)> onMinorError) {
     std::string suffix = "." + terseCurrentTimeForFilename();
 
     LOGV2(23166, "Log rotation initiated", "suffix"_attr = suffix, "logType"_attr = logType);
@@ -59,19 +69,17 @@ bool rotateLogs(bool renameFiles,
     if (logType) {
         auto it = logRotateCallbacks.find(logType.get());
         if (it == logRotateCallbacks.end()) {
-            LOGV2_WARNING(
-                ErrorCodes::NoSuchKey, "Unknown log type for rotate", "logType"_attr = logType);
-            return false;
+            LOGV2_WARNING(6221500, "Unknown log type for rotate", "logType"_attr = logType);
+            return Status(ErrorCodes::NoSuchKey, "Unknown log type for rotate");
         }
         auto status = it->second(renameFiles, suffix, onMinorError);
         if (!status.isOK()) {
             LOGV2_WARNING(
                 1947001, "Log rotation failed", "reason"_attr = status, "logType"_attr = logType);
-            return false;
         }
-        return true;
+        return status;
     } else {
-        bool ret = true;
+        LogRotateErrorAppender errors;
         for (const auto& entry : logRotateCallbacks) {
             auto status = entry.second(renameFiles, suffix, onMinorError);
             if (!status.isOK()) {
@@ -79,10 +87,10 @@ bool rotateLogs(bool renameFiles,
                               "Log rotation failed",
                               "reason"_attr = status,
                               "logType"_attr = entry.first);
-                ret = false;
+                errors.append(status);
             }
         }
-        return ret;
+        return errors.getCombinedStatus();
     }
 }
 
diff --git a/src/mongo/logv2/log_util.h b/src/mongo/logv2/log_util.h
index a60e5f1dd67..e1b6c2c34e0 100644
--- a/src/mongo/logv2/log_util.h
+++ b/src/mongo/logv2/log_util.h
@@ -49,7 +49,25 @@ using LogRotateCallback = std::function<Status(bool, StringData, std::function<v
 void addLogRotator(StringData logType, LogRotateCallback cb);
 
 /**
- * Rotates the log files.  Returns true if all logs rotate successfully.
+ * Class that combines error Status objects into a single Status object.
+ */
+class LogRotateErrorAppender {
+public:
+    LogRotateErrorAppender() : _combined(Status::OK()) {}
+    LogRotateErrorAppender(const Status& init) : _combined(init) {}
+
+    const Status& getCombinedStatus() const {
+        return _combined;
+    }
+
+    void append(const Status& err);
+
+private:
+    Status _combined;
+};
+
+/**
+ * Rotates the log files.  Returns Status::OK() if all logs rotate successfully.
  *
  * renameFiles - true means we rename files, false means we expect the file to be renamed
  *               externally
@@ -59,9 +77,9 @@ void addLogRotator(StringData logType, LogRotateCallback cb);
  * We expect logrotate to rename the existing file before we rotate, and so the next open
  * we do should result in a file create.
  */
-bool rotateLogs(bool renameFiles,
-                boost::optional<StringData> logType,
-                std::function<void(Status)> onMinorError);
+Status rotateLogs(bool renameFiles,
+                  boost::optional<StringData> logType,
+                  std::function<void(Status)> onMinorError);
 
 /**
  * Returns true if system logs should be redacted.