You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@jmeter.apache.org by "mcgitty (via GitHub)" <gi...@apache.org> on 2023/07/10 18:35:39 UTC
[GitHub] [jmeter] mcgitty opened a new pull request, #6040: Add jmeter.reportgenerator.outputdir property
mcgitty opened a new pull request, #6040:
URL: https://github.com/apache/jmeter/pull/6040
## Description
This property is noted in jmeter.log and proven to be correct when referenced in test plan as ${__P(jmeter.reportgenerator.outputdir,)}. However, it not documented here and is unclear whether property "jmeter.reportgenerator.exporter.html.property.output_dir" was incorrectly named.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@jmeter.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [jmeter] vlsi commented on pull request #6040: Add jmeter.reportgenerator.outputdir property
Posted by "vlsi (via GitHub)" <gi...@apache.org>.
vlsi commented on PR #6040:
URL: https://github.com/apache/jmeter/pull/6040#issuecomment-1629560696
Judging by the code, they both should work: https://github.com/apache/jmeter/blob/12aefb0c70526a61f955ba1dad7fe3e99200b0b2/src/core/src/main/java/org/apache/jmeter/report/dashboard/HtmlTemplateExporter.java#L319-L325
`jmeter.reportgenerator.exporter.html.property.output_dir` is a property for HTML reporter, while `jmeter.reportgenerator.outputdir` is a global default for all reporters.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@jmeter.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [jmeter] vlsi commented on pull request #6040: Add jmeter.reportgenerator.outputdir property
Posted by "vlsi (via GitHub)" <gi...@apache.org>.
vlsi commented on PR #6040:
URL: https://github.com/apache/jmeter/pull/6040#issuecomment-1629614611
I think it should. However, I am not sure what sould be the documentation so it is right and non-confusing
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@jmeter.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [jmeter] mcgitty commented on pull request #6040: Add jmeter.reportgenerator.outputdir property
Posted by "mcgitty (via GitHub)" <gi...@apache.org>.
mcgitty commented on PR #6040:
URL: https://github.com/apache/jmeter/pull/6040#issuecomment-1629609911
The first issue, should "jmeter.reportgenerator.outputdir" be documented here?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@jmeter.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org