-
Notifications
You must be signed in to change notification settings - Fork 2
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
Feature/s3 utils 158/storage consumption metrics #312
Feature/s3 utils 158/storage consumption metrics #312
Conversation
Hello benzekrimaha,My role is to assist you with the merge of this Available options
Available commands
Status report is not available. |
892877a
to
6252082
Compare
Request integration branchesWaiting for integration branch creation to be requested by the user. To request integration branches, please comment on this pull request with the following command:
Alternatively, the |
6252082
to
905181b
Compare
7c957f3
to
5b7f15a
Compare
b34c8b0
to
a50c483
Compare
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.
Adding a comment as discussed: we need to set the counters/gauges to 0 at the beginning of a count-items run, otherwise the values will only increase
fb1a8f6
to
dba0c98
Compare
57e2d47
to
7a54697
Compare
98cc347
to
bc28c67
Compare
/create_integration_branches |
Integration data createdI have created the integration data for the additional destination branches.
The following branches will NOT be impacted:
You can set option
The following options are set: create_integration_branches |
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
The following options are set: create_integration_branches |
@bert-e create_pull_requests |
Integration data createdI have created the integration data for the additional destination branches.
The following branches will NOT be impacted:
Follow integration pull requests if you would like to be notified of The following options are set: create_pull_requests, create_integration_branches |
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
The following options are set: create_pull_requests, create_integration_branches |
/approve |
I have successfully merged the changeset of this pull request
The following branches have NOT changed:
Please check the status of the associated issue S3UTILS-158. Goodbye benzekrimaha. The following options are set: approve, create_pull_requests, create_integration_branches |
This PR aims to provide monitoring metrics for the count items cron-job.
Today the metrics that have been put in place are :
bucketCount
: Total number of bucketsbucketProcessingDuration
: Duration of processing a bucketconsolidationDuration
: Duration of metrics consolidation in secondsworkersCount
: a metric that shows success / errors during the workers countobjectsCount
: Number of objects based on their statesmetricsCount
: Number of buckets ( none empty) , number of accounts with buckets , and number of locationsIt also puts in place a job stop mecanism resumed with :
==> If the job completed and the scraping occured => immediate stop
==> Otherwise we wait for a duration ( threshold x frequency x 4 x 1000 ) if a scraping occurs during it we stop otherwise we wait for the whole duration then we stop the job
Note for reviewers : this is a first iteration as the job has never been monitored ( only alerts were put in place ) please let me know if you see that some metrics might make more sens / or others are not really needed
Linked issue : https://scality.atlassian.net/browse/S3UTILS-158