diff options
| author | alexbogo <alexbogo@ydb.tech> | 2023-03-22 13:40:52 +0300 |
|---|---|---|
| committer | alexbogo <alexbogo@ydb.tech> | 2023-03-22 13:40:52 +0300 |
| commit | a01ef539b97d8eb4028a31320431266b063607d8 (patch) | |
| tree | 039a825adec111fdb60969df317965e65fce5c9a | |
| parent | 558102e9d11075acd943599473439d7961a8830d (diff) | |
| download | ydb-a01ef539b97d8eb4028a31320431266b063607d8.tar.gz | |
[ymq] fix authorize duration metric: fill start time and use old names & remove unused code
init
| -rw-r--r-- | ydb/core/ymq/base/counters.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/ydb/core/ymq/base/counters.cpp b/ydb/core/ymq/base/counters.cpp index 7fe6cdd0ab3..87061d8d737 100644 --- a/ydb/core/ymq/base/counters.cpp +++ b/ydb/core/ymq/base/counters.cpp @@ -987,8 +987,8 @@ void TCloudAuthCounters::InitCounters(TIntrusivePtr<::NMonitoring::TDynamicCount const auto actionAndCredentialCounters = actionCounters->GetSubgroup("credential_type", credentialTypeStr); if (actionType == NCloudAuth::EActionType::Authorize) { - INIT_COUNTER_WITH_NAME(actionAndCredentialCounters, AuthorizeSuccess[credentialType], "success", ELifetime::Persistent, EValueType::Derivative, Lazy(*Cfg)); - INIT_COUNTER_WITH_NAME(actionAndCredentialCounters, AuthorizeError[credentialType], "error", ELifetime::Persistent, EValueType::Derivative, Lazy(*Cfg)); + INIT_COUNTER_WITH_NAME(actionAndCredentialCounters, AuthorizeSuccess[credentialType], "Ok", ELifetime::Persistent, EValueType::Derivative, Lazy(*Cfg)); + INIT_COUNTER_WITH_NAME(actionAndCredentialCounters, AuthorizeError[credentialType], "PermissionDenied", ELifetime::Persistent, EValueType::Derivative, Lazy(*Cfg)); } else { for (size_t grpcStatus = 0; grpcStatus < GRPC_STATUSES_COUNT; ++grpcStatus) { INIT_COUNTER_WITH_NAME(actionAndCredentialCounters, CloudAuthCounters[actionType][credentialType][grpcStatus], StringifyGrpcStatus(grpcStatus), ELifetime::Persistent, EValueType::Derivative, Lazy(*Cfg)); |