Rbillon59 / jmeter-docker-compose-starterkit

37 stars 25 forks source link

Could you please upgrade jmeter version to latest? #4

Closed djmanju closed 12 months ago

djmanju commented 1 year ago

Hello. First of all, Thank you very much for this amazing starterkit. It is very useful!

Currently the latest version (rbillon59/jmeter) appears to be 5.2.1. Could you please make version 5.6.2 available (saw a note on your blog to contact you if we needed newer version)

Thank you very much in advance!

Rbillon59 commented 1 year ago

Hey @djmanju

I've refactored the project to use the work I've done for https://github.com/Rbillon59/jmeter-k8s-starterkit using the k8s jmeter docker image https://github.com/Rbillon59/jmeter-k8s-base

This image use the JMeter version as tags and is now the one used in the docker-compose starterkit as well.

You just need to change the JMETER_VERSION env var in the docker-compose.yaml file

Please try the refactor branch and let me know if it works well for you, so I can merge it.

Thanks

Rbillon59 commented 1 year ago

Also, if you're interested, the changes are available in https://github.com/Rbillon59/jmeter-docker-compose-starterkit/pull/5/commits/9ffb9d70e535570031015015da8ba540d70a60e7

djmanju commented 1 year ago

Thank you @Rbillon59! This is awesome.

Could you make this work for openjdk17 or openjdk11? Cause even sudo apt search openjdk doesn't show 16 anymore? I couldn't test it since I have 11 or 17.

Thanks in advance!

Rbillon59 commented 1 year ago

Hey, you're welcome!

What do you mean?

The docker image bundles JMeter and the Java version, you don't need to have Java installed locally.

What would be the use case to have multiple available Java versions ?

Btw, Java 17 causes a bug in the report generation.

Thanks

QAInsights commented 1 year ago

Btw, Java 17 causes a bug in the report generation.

Is there an open defect/issue raised for this in JMeter repo?

Rbillon59 commented 1 year ago

Btw, Java 17 causes a bug in the report generation.

Is there an open defect/issue raised for this in JMeter repo?

I did not check but got the following error on JDK 17+

Error generating the report: org.apache.jmeter.report.dashboard.GenerationException: Error while processing samples: Consumer failed with message :Consumer failed with message :Consumer failed with message :Consumer failed with message :Begin size 0 is not equal to fixed size 5 In resume

And in this stack overflow thread, it was discussed