better

Author: vporyadke

ydb/core/mind/hive/hive_ut.cpp

@@ -2828,6 +2828,25 @@ Y_UNIT_TEST_SUITE(THiveTest) {
                 tabletB = tablets[1];
             }
         }
+
+        // If assured space is not set, usage is always set to 1
+        auto groupMetricsExchange = MakeHolder<TEvBlobStorage::TEvControllerGroupMetricsExchange>();
+        for (const auto& [group, tablets] : groupToTablets) {
+            NKikimrBlobStorage::TGroupMetrics* metrics = groupMetricsExchange->Record.AddGroupMetrics();
+
+            metrics->SetGroupId(group);
+            metrics->MutableGroupParameters()->SetGroupID(group);
+            metrics->MutableGroupParameters()->SetStoragePoolName("def1");
+            metrics->MutableGroupParameters()->MutableAssuredResources()->SetSpace(300'000'000);
+        }
+
+        runtime.SendToPipe(MakeBSControllerID(0), sender, groupMetricsExchange.Release(), 0, GetPipeConfigWithRetries());
+        {
+            TDispatchOptions options;
+            options.FinalEvents.push_back(TDispatchOptions::TFinalEventCondition(TEvBlobStorage::EvControllerGroupMetricsExchange));
+            runtime.DispatchEvents(options);
+        }
+
         TChannelsBindings channels = BINDED_CHANNELS;
         for (auto& bind : channels) {
             bind.SetSize(200'000'000);

ydb/core/mind/hive/tx__update_tablet_groups.cpp

@@ -162,8 +162,6 @@ class TTxUpdateTabletGroups : public TTransactionBase<THive> {
                 fromGeneration = 0;
             } else if (channel->History.back().GroupID == group->GetGroupID()) {
                 // We decided to keep the group the same
-                // Do nothing, except set new timestamp
-                channel->History.back().Timestamp = ctx.Now();
                 continue;
             } else {
                 needToIncreaseGeneration = true;