diff options
author | alexvru <alexvru@ydb.tech> | 2022-10-20 13:51:12 +0300 |
---|---|---|
committer | alexvru <alexvru@ydb.tech> | 2022-10-20 13:51:12 +0300 |
commit | a69941ccfb9d3d477b5aacf9800e7d59a3463366 (patch) | |
tree | 4aa0e94ec0c5a3367187147414426d5a9aaabcd8 | |
parent | f2cf13594bce31096267fb1cb2918394c668ca75 (diff) | |
download | ydb-a69941ccfb9d3d477b5aacf9800e7d59a3463366.tar.gz |
Actualize controller audit logs
-rw-r--r-- | ydb/core/mind/bscontroller/cmds_drive_status.cpp | 13 | ||||
-rw-r--r-- | ydb/core/mind/bscontroller/config_cmd.cpp | 3 |
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)); |