Skip to content

Enable Gitlab reply in ThroughputProfileHandler for all ci-test MRs

Christoph Hasse requested to merge gitlab_reply_throughputprofilehandler into master

Note that for testing reasons the MRs that implement this feature, mainly !178 (merged) and !181 (merged) are already merged.
This MR only includes a small change which essentially enables the feature for all throughput tests launched from a ci-test build.

In short: The ThroughputProfileHandler will now post a message that looks like Throughput Test Moore_hlt1_pp_default: 28853.9 Events/s -- change: 0.14% to the discussion which launched the ci-test.
Real example here.
This is for now enabled for the hlt1_pp_default and hlt2_reco_baseline. Thus, we can expect one hlt1 and one hlt2 comment for each ci-test.
If a change in throughput in hlt1 is > 0.5% or for hlt2 > 1% a hlt{1,2}-throughput-{in,de}creased label is automatically added to the MR.

Assigning this to @rmatev and @raaij as I'd like their 'go ahead' before we merge this :)

And big thank you again to @maszyman who helped a lot!! 🙏

Edited by Christoph Hasse

Merge request reports