From b7d44394a8cbc7e3e83694610a8a39b10832dd1b Mon Sep 17 00:00:00 2001 From: StekPerepolnen Date: Mon, 6 May 2024 14:20:05 +0000 Subject: [PATCH] hc vulnerabilities --- ydb/core/health_check/health_check.cpp | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/ydb/core/health_check/health_check.cpp b/ydb/core/health_check/health_check.cpp index e96246170869..6a91f7fad1bf 100644 --- a/ydb/core/health_check/health_check.cpp +++ b/ydb/core/health_check/health_check.cpp @@ -1487,12 +1487,16 @@ class TSelfCheckRequest : public TActorBootstrapped { void FillVDiskStatus(const TString& vSlotId, Ydb::Monitoring::StorageVDiskStatus& storageVDiskStatus, TSelfCheckContext context) { auto itVSlot = BSConfigVSlots.find(vSlotId); const TEvInterconnect::TNodeInfo* nodeInfo = nullptr; + + context.Location.mutable_storage()->mutable_pool()->mutable_group()->mutable_vdisk()->mutable_id()->Clear(); + context.Location.mutable_storage()->mutable_pool()->mutable_group()->clear_id(); // you can see VDisks Group Id in vSlotId field if (itVSlot != BSConfigVSlots.end()) { TNodeId nodeId = itVSlot->second->vslotid().nodeid(); auto itNodeInfo = MergedNodeInfo.find(nodeId); if (itNodeInfo != MergedNodeInfo.end()) { nodeInfo = itNodeInfo->second; } + context.Location.mutable_storage()->mutable_pool()->mutable_group()->mutable_vdisk()->add_id(GetVDiskId(*itVSlot->second)); context.Location.mutable_storage()->mutable_node()->set_id(nodeId); } else { context.Location.mutable_storage()->mutable_node()->clear_id(); @@ -1505,10 +1509,6 @@ class TSelfCheckRequest : public TActorBootstrapped { context.Location.mutable_storage()->mutable_node()->clear_port(); } - context.Location.mutable_storage()->mutable_pool()->mutable_group()->mutable_vdisk()->mutable_id()->Clear(); - context.Location.mutable_storage()->mutable_pool()->mutable_group()->mutable_vdisk()->add_id(GetVDiskId(*itVSlot->second)); - context.Location.mutable_storage()->mutable_pool()->mutable_group()->clear_id(); // you can see VDisks Group Id in vSlotId field - storageVDiskStatus.set_id(vSlotId); if (itVSlot == BSConfigVSlots.end()) { // this report, in theory, can't happen because there was slot mention in bsc group info. this slot info have to exists in bsc too