Fix quorum calculation logic

Author: alexvru

ydb/core/blobstorage/nodewarden/distconf_invoke_common.cpp

@@ -239,7 +239,7 @@ namespace NKikimr::NStorage {
     void TInvokeRequestHandlerActor::StartProposition(NKikimrBlobStorage::TStorageConfig *config, bool acceptLocalQuorum,
             bool requireScepter, bool mindPrev, const NKikimrBlobStorage::TStorageConfig *propositionBase,
             bool fromBootstrap) {
-        if (!Self->HasConnectedNodeQuorum(*config, acceptLocalQuorum)) {
+        if (!Self->HasConnectedNodeQuorum(mindPrev && config->HasPrevConfig() ? config->GetPrevConfig() : *config, acceptLocalQuorum)) {
             throw TExError() << "No quorum to start propose/commit configuration";
         } else if (requireScepter && !Self->Scepter) {
             throw TExError() << "No scepter";

ydb/core/blobstorage/nodewarden/distconf_quorum.cpp

@@ -187,7 +187,7 @@ namespace NKikimr::NStorage {
         return true;
     }
 
-    bool HasStorageQuorum(const NKikimrBlobStorage::TStorageConfig& config, std::span<TSuccessfulDisk> successful,
+    std::optional<bool> HasStorageQuorum(const NKikimrBlobStorage::TStorageConfig& config, std::span<TSuccessfulDisk> successful,
             const THashMap<TString, TBridgePileId>& /*bridgePileNameMap*/, TBridgePileId singleBridgePileId,
             const TNodeWardenConfig& nwConfig, bool allowUnformatted, IOutputStream *out, const char *name) {
         auto makeError = [&](TString error) -> bool {
@@ -323,17 +323,27 @@ namespace NKikimr::NStorage {
             }
         }
 
-        return true; // all group meet their quorums
+        return groups.empty()
+            ? std::nullopt // we don't know for sure about quorum -- no static groups defined
+            : std::make_optional(true); // all groups meet their quorums
     }
 
     bool HasConfigQuorum(const NKikimrBlobStorage::TStorageConfig& config, std::span<TSuccessfulDisk> successful,
             const THashMap<TString, TBridgePileId>& bridgePileNameMap, TBridgePileId singleBridgePileId,
             const TNodeWardenConfig& nwConfig, bool mindPrev, TStringStream *out) {
-        return HasDiskQuorum(config, successful, bridgePileNameMap, singleBridgePileId, out, "new") &&
-            HasStorageQuorum(config, successful, bridgePileNameMap, singleBridgePileId, nwConfig, true, out, "new") &&
-            (!mindPrev || !config.HasPrevConfig() ||
-                HasDiskQuorum(config.GetPrevConfig(), successful, bridgePileNameMap, singleBridgePileId, out, "prev") &&
-                HasStorageQuorum(config.GetPrevConfig(), successful, bridgePileNameMap, singleBridgePileId, nwConfig, false, out, "prev"));
+        auto getQuorum = [&](auto& config, const char *name, bool allowUnformatted) {
+            auto q = HasStorageQuorum(config, successful, bridgePileNameMap, singleBridgePileId, nwConfig,
+                allowUnformatted, out, name);
+            if (!q) {
+                q.emplace(HasDiskQuorum(config, successful, bridgePileNameMap, singleBridgePileId, out, name));
+            }
+            return *q;
+        };
+        bool q = getQuorum(config, "new", true);
+        if (q && mindPrev && config.HasPrevConfig()) {
+            q = getQuorum(config.GetPrevConfig(), "prev", false);
+        }
+        return q;
     }
 
 } // NKikimr::NStorage

ydb/core/blobstorage/nodewarden/distconf_quorum.h

@@ -17,7 +17,7 @@ namespace NKikimr::NStorage {
         const THashMap<TString, TBridgePileId>& bridgePileNameMap, TBridgePileId singleBridgePileId, IOutputStream *out,
         const char *name);
 
-    bool HasStorageQuorum(const NKikimrBlobStorage::TStorageConfig& config, std::span<TSuccessfulDisk> successful,
+    std::optional<bool> HasStorageQuorum(const NKikimrBlobStorage::TStorageConfig& config, std::span<TSuccessfulDisk> successful,
         const THashMap<TString, TBridgePileId>& bridgePileNameMap, TBridgePileId singleBridgePileId,
         const TNodeWardenConfig& nwConfig, bool allowUnformatted, IOutputStream *out, const char *name);