justb4 / docker-jmeter

Docker image for Apache JMeter
MIT License
278 stars 310 forks source link

adding jmeter plugins - not working as expected #44

Closed imnot-abot closed 2 years ago

imnot-abot commented 3 years ago

Hi, Any chance this should be using JMETER_HOME instead of (pwd) ?

https://github.com/justb4/docker-jmeter/blob/bdf60f7557a8c62b456d24b987fd24b5beda907a/entrypoint.sh#L12

It works fine as long as I don't set the -w argument while spinning off the docker instance. Once adding the -w argument, as in the examples, it copies the plugins to the wrong folder.

Or else I'm getting the usage wrong.

Vikrant87-lego commented 3 years ago

I am also facing the same issue, did you by any chance happen to fix this?

justb4 commented 2 years ago

Hmm, the -w sets the working directory but that should normally not be changed. Yes, ${JMETER_HOME} is a better option. Will look into..

kareemshahin commented 2 years ago

Had the same problem! Opened up a PR to make the change.

justb4 commented 2 years ago

Should be solved by #56, closing for now.

kareemshahin commented 2 years ago

Sorry @justb4 i actually needed to patch this fix: https://github.com/justb4/docker-jmeter/pull/58