Fix monitoring page registration races in tests (ydb-platform#8770)

GrigoriyPA · Sep 6, 2024 · 154f927 · 154f927
1 parent 96dfeb9
commit 154f927
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 6 deletions.
diff --git a/ydb/core/mon/sync_http_mon.cpp b/ydb/core/mon/sync_http_mon.cpp
@@ -54,11 +54,8 @@ namespace NActors {
     }
 
     void TSyncHttpMon::Stop() {
-        if (IndexMonPage) {
-            IndexMonPage->ClearPages(); // it's required to avoid loop-reference
-            TBase::Stop();
-            IndexMonPage.Drop();
-        }
+        IndexMonPage->ClearPages(); // it's required to avoid loop-reference
+        TBase::Stop();
     }
 
     void TSyncHttpMon::Register(NMonitoring::IMonPage* page) {

diff --git a/ydb/core/testlib/actors/test_runtime.cpp b/ydb/core/testlib/actors/test_runtime.cpp
@@ -27,7 +27,6 @@ namespace NActors {
     void TTestActorRuntime::TNodeData::Stop() {
         if (Mon) {
             Mon->Stop();
-            GetAppData<NKikimr::TAppData>()->Mon = nullptr;
         }
         TNodeDataBase::Stop();
     }