Skip to content

fix #4 Support Report Naming to be multiproject compatible#5

Open
mbauhardt wants to merge 2 commits into
jenkinsci:masterfrom
mbauhardt:support-multiproject
Open

fix #4 Support Report Naming to be multiproject compatible#5
mbauhardt wants to merge 2 commits into
jenkinsci:masterfrom
mbauhardt:support-multiproject

Conversation

@mbauhardt
Copy link
Copy Markdown

@mbauhardt mbauhardt commented Nov 6, 2020

Why
I want to use the jmh report plugin on a multi project build.
At the moment, this plugin can handle only one report in a single project. In a multi project env the results will be overwritten by each other.

How
Introduce reportName attribute to be able to render a unique name for each report.

Furthermore,
with that change set, the project view doesn't make any sense now. Because the actual report (in case of a multi project) sticks in the module itself and not in the project anymore.
For that reason the whole project view stuff is deleted within this change set.

@mbauhardt mbauhardt changed the title #4 Support Report Naming to be multiproject compatible fix #4 Support Report Naming to be multiproject compatible Nov 6, 2020
@AlexChrbn
Copy link
Copy Markdown

Could you update the readme please?

@jzillmann
Copy link
Copy Markdown
Contributor

I feel the the project level functionality (compare all runs against each other), is essential since it allows to keep track on how the project is developing performance wise.
Thus I don't want to remove it. I understand the need of a multi-project support, but I think the way to go is to merge the module reports to a single report.

@AlexChrbn
Copy link
Copy Markdown

For me, it would be great to have categories. It'd be like implemented in this PR, but i would just create multiple files, each a different category with different links in the Jenkins view.

@jzillmann
Copy link
Copy Markdown
Contributor

@AlexKroeker So for a Jenkins run you would basically have one report per submodule ? And for the project view as well ?

@AlexChrbn
Copy link
Copy Markdown

@jzillmann That would pretty much be it.
Basically, it's enough for me if multiple reports can be added (e.g. via a regex) and each report is accessible via a standalone link in Jenkins.
Plus, an overall view would also be beneficial.

@jzillmann
Copy link
Copy Markdown
Contributor

Ok, sounds fine. Do it! :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants