Updating c6f711e..f1a61b1
Fast-forward
 swh/scrubber/storage_checker.py | 28 ++++++++++++++--------------
 1 file changed, 14 insertions(+), 14 deletions(-)

Changes applied before test

commit f1a61b1f501a5c3e342972c66688b08400dacea6
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Thu Aug 18 15:12:37 2022 +0200

    Split batch_duration_seconds metric in two
    
    It will probably be useful to know what part of the check
    takes the most time.

commit 0cc86b0e45f015429406112b16e8b2e5196512ba
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Thu Aug 18 15:07:19 2022 +0200

    Deduplicate statsd calls
    
    This will allow adding more tags easily in future commits

See https://jenkins.softwareheritage.org/job/DSCRUB/job/tests-on-diff/54/ for more details.

Harbormaster completed remote builds in B30862: Diff 29810.Aug 18 2022, 3:15 PM

vlorentz requested review of this revision.Aug 18 2022, 3:15 PM

Maybe use the same metric with an operation="check_hashes"/"check_references" tag?

olasd accepted this revision.Aug 18 2022, 4:38 PM

This revision is now accepted and ready to land.Aug 18 2022, 4:38 PM

good idea!

use tags

Build is green

Patch application report for D8253 (id=29821)

Could not rebase; Attempt merge onto c6f711ecbf...

Updating c6f711e..aaae867
Fast-forward
 swh/scrubber/storage_checker.py | 30 +++++++++++++++++-------------
 1 file changed, 17 insertions(+), 13 deletions(-)

Changes applied before test

commit aaae867e8d297d8f1aee26e4326209d69c3b392f
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Thu Aug 18 15:12:37 2022 +0200

    Split batch_duration_seconds metric in two
    
    It will probably be useful to know what part of the check
    takes the most time.

commit 0cc86b0e45f015429406112b16e8b2e5196512ba
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Thu Aug 18 15:07:19 2022 +0200

    Deduplicate statsd calls
    
    This will allow adding more tags easily in future commits

See https://jenkins.softwareheritage.org/job/DSCRUB/job/tests-on-diff/59/ for more details.

Harbormaster completed remote builds in B30873: Diff 29821.Aug 18 2022, 4:56 PM

Closed by commit rDSCRUBaaae867e8d29: Split batch_duration_seconds metric in two (authored by vlorentz). · Explain WhyAug 18 2022, 4:59 PM

This revision was automatically updated to reflect the committed changes.

vlorentz added a commit: rDSCRUBaaae867e8d29: Split batch_duration_seconds metric in two.