New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: signer module adds metrics for each rpc interface #1237
Conversation
cd6962e
to
014c24f
Compare
metrics.ReqTime.WithLabelValues(SignerFailureCreateGVG).Observe(time.Since(startTime).Seconds()) | ||
} else { | ||
metrics.ReqCounter.WithLabelValues(SignerSuccessCreateGVG).Inc() | ||
metrics.ReqTime.WithLabelValues(SignerSuccessCreateGVG).Observe(time.Since(startTime).Seconds()) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Check other interfaces of the signer and add metrics together.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
base/gfspapp/app_options.go
Outdated
SignerFailure = "signer_failure" | ||
SignerSuccessBucketApproval = "signer_bucket_approval_success" | ||
SignerFailureBucketApproval = "signer_bucket_approval_failure" | ||
SignerSuccessMigrateBucketApproval = "signer_bucket_approval_success" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
migrate bucket.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
base/gfspapp/app_options.go
Outdated
SignerSuccessBucketApproval = "signer_bucket_approval_success" | ||
SignerFailureBucketApproval = "signer_bucket_approval_failure" | ||
SignerSuccessMigrateBucketApproval = "signer_bucket_approval_success" | ||
SignerFailureMigrateBucketApproval = "signer_bucket_approval_failure" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ditto.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
@@ -73,6 +71,8 @@ func (e *ExecuteModular) HandleReplicatePieceTask(ctx context.Context, task core | |||
sealErr := e.sealObject(ctx, task, sealMsg) | |||
metrics.PerfPutObjectTime.WithLabelValues("background_seal_object_cost").Observe(time.Since(sealTime).Seconds()) | |||
metrics.PerfPutObjectTime.WithLabelValues("background_task_seal_object_end").Observe(time.Since(startReplicateTime).Seconds()) | |||
metrics.PerfPutObjectTime.WithLabelValues("object_throughout_all_process").Observe(time.Since( | |||
time.Unix(task.GetObjectInfo().GetCreateAt(), 0)).Seconds()) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also need to add in HandleSealObjectTask.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
added
5ad57ae
to
7c98235
Compare
7c98235
to
cfc7f75
Compare
Description
Signer module adds metrics for each rpc interface.
Rationale
N/A
Example
N/A
Changes
Notable changes: