aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authoralexvru <alexvru@ydb.tech>2022-10-20 13:51:12 +0300
committeralexvru <alexvru@ydb.tech>2022-10-20 13:51:12 +0300
commita69941ccfb9d3d477b5aacf9800e7d59a3463366 (patch)
tree4aa0e94ec0c5a3367187147414426d5a9aaabcd8
parentf2cf13594bce31096267fb1cb2918394c668ca75 (diff)
downloadydb-a69941ccfb9d3d477b5aacf9800e7d59a3463366.tar.gz
Actualize controller audit logs
-rw-r--r--ydb/core/mind/bscontroller/cmds_drive_status.cpp13
-rw-r--r--ydb/core/mind/bscontroller/config_cmd.cpp3
2 files changed, 9 insertions, 7 deletions
diff --git a/ydb/core/mind/bscontroller/cmds_drive_status.cpp b/ydb/core/mind/bscontroller/cmds_drive_status.cpp
index c41f85b5a8..55ca69fb81 100644
--- a/ydb/core/mind/bscontroller/cmds_drive_status.cpp
+++ b/ydb/core/mind/bscontroller/cmds_drive_status.cpp
@@ -127,8 +127,8 @@ namespace NKikimr::NBsController {
if (pdiskInfo->BoxId != boxId) {
throw TExError() << "Drive is defind in host configs, but placed in another box# " << pdiskInfo->BoxId;
}
- STLOG(PRI_NOTICE, BS_CONTROLLER_AUDIT, BSCA06, "Set new ExpectedSerial for HostConfigs drive",
- (Serial, newSerial), (BoxId, boxId), (PDiskId, *updatePDiskId), (Path, path));
+ STLOG(PRI_INFO, BS_CONTROLLER_AUDIT, BSCA06, "Set new ExpectedSerial for HostConfigs drive",
+ (UniqueId, UniqueId), (Serial, newSerial), (BoxId, boxId), (PDiskId, *updatePDiskId), (Path, path));
return;
}
}
@@ -174,7 +174,8 @@ namespace NKikimr::NBsController {
Y_VERIFY(success);
driveInfoNew->PDiskConfig = config;
- STLOG(PRI_NOTICE, BS_CONTROLLER_AUDIT, BSCA00, "AddDriveSerial", (Serial, newSerial), (BoxId, boxId));
+ STLOG(PRI_INFO, BS_CONTROLLER_AUDIT, BSCA00, "AddDriveSerial", (UniqueId, UniqueId), (Serial, newSerial),
+ (BoxId, boxId));
}
void TBlobStorageController::TConfigState::ExecuteStep(const NKikimrBlobStorage::TRemoveDriveSerial& cmd,
@@ -225,8 +226,8 @@ namespace NKikimr::NBsController {
}
auto* pdiskUpdate = PDisks.FindForUpdate(*removePDiskId);
pdiskUpdate->ExpectedSerial = {};
- STLOG(PRI_NOTICE, BS_CONTROLLER_AUDIT, BSCA08, "Reset ExpectedSerial for HostConfig drive",
- (Serial, serial), (PDiskId, *removePDiskId));
+ STLOG(PRI_INFO, BS_CONTROLLER_AUDIT, BSCA08, "Reset ExpectedSerial for HostConfig drive",
+ (UniqueId, UniqueId), (Serial, serial), (PDiskId, *removePDiskId));
// create fictional row in DrivesSerials to be able to reply kAlready for already removed disk
// even if they are defined through HostConfig
@@ -257,7 +258,7 @@ namespace NKikimr::NBsController {
driveInfoMutable->PDiskId.Clear();
driveInfoMutable->LifeStage = NKikimrBlobStorage::TDriveLifeStage::REMOVED;
- STLOG(PRI_NOTICE, BS_CONTROLLER_AUDIT, BSCA07, "RemoveDriveSerial", (Serial, serial));
+ STLOG(PRI_INFO, BS_CONTROLLER_AUDIT, BSCA07, "RemoveDriveSerial", (UniqueId, UniqueId), (Serial, serial));
}
}
diff --git a/ydb/core/mind/bscontroller/config_cmd.cpp b/ydb/core/mind/bscontroller/config_cmd.cpp
index 02aa1828b0..d4c3c4a331 100644
--- a/ydb/core/mind/bscontroller/config_cmd.cpp
+++ b/ydb/core/mind/bscontroller/config_cmd.cpp
@@ -253,7 +253,7 @@ namespace NKikimr::NBsController {
LogCommand(txc, TDuration::Seconds(timer.Passed()));
}
- STLOG(PRI_INFO, BS_CONTROLLER_AUDIT, BSCA03, "Transaction complete",
+ STLOG(PRI_INFO, BS_CONTROLLER_AUDIT, BSCA03, "Transaction ended",
(UniqueId, State->UniqueId),
(Status, Success ? "commit" : "rollback"),
(Error, Error));
@@ -345,6 +345,7 @@ namespace NKikimr::NBsController {
void Complete(const TActorContext&) override {
if (auto state = std::exchange(State, std::nullopt)) {
+ STLOG(PRI_INFO, BS_CONTROLLER_AUDIT, BSCA09, "Transaction complete", (UniqueId, State->UniqueId));
state->ApplyConfigUpdates();
}
TActivationContext::Send(new IEventHandle(NotifyId, Self->SelfId(), Ev.Release(), 0, Cookie));