search

intel / lkp-tests

Linux Kernel Performance tests
GNU General Public License v2.0
258 stars 156 forks source link

Move action of post-run after the stats are generated #315

Closed yolkfull closed 1 year ago

yolkfull commented 1 year ago

Currently the post-run action(s) will be called before stats are generated, user cannot do post-handling with the json result like converting json to HTML, upload json results to databases, etc. This patch simply improves that and brings many benifits.

yolkfull commented 1 year ago

Anybody can help review this patch? Thanks very much in advance.

rli9 commented 1 year ago

Thanks for the patch, and sorry for late. The idea is helpful. Refer to monitors/no-stdout/perf-profile, currently post-run is also used to generate real result which is used for generic stats. So post-run can't be directly moved after unite. For the purpose here, I think you can consider to add another stage, like post-unite.

yolkfull commented 1 year ago

Thanks for the patch, and sorry for late. The idea is helpful. Refer to monitors/no-stdout/perf-profile, currently post-run is also used to generate real result which is used for generic stats. So post-run can't be directly moved after unite. For the purpose here, I think you can consider to add another stage, like post-unite.

Oh, got it, thank you so much for pointing that out.