search

jenkinsci / peass-ci-plugin

Jenkins plugin for peass to support performance measurement in CI
https://plugins.jenkins.io/peass-ci/
GNU Affero General Public License v3.0
4 stars 13 forks source link

--timeout option doesn't change the waiting period #213

Closed alex-iotiq closed 1 year ago

alex-iotiq commented 1 year ago

The timeout is always 5 minutes regardless of the -timeout option. It may be related to -useAnbox option.

And log says "Waiting for 300 minutes", shouldn't it be "Waiting for 300 seconds" ?

DaGeRe commented 1 year ago

Thanks for the hint with the minutes, this is fixed: https://github.com/DaGeRe/peass/commit/b8f8aaf551d39a723771296acba62cf73c242cdc

Could you further specify when this happens? On RTS, Measurement, RCA, everytime...?

alex-iotiq commented 1 year ago

This happens in ProcessSuccessTester probably on RTS

DaGeRe commented 1 year ago

Currently, I cannot reproduce this issue. Could you set breakpoints at all constructor calls to ExecutionConfig and tell me the stack trace where this happens?

alex-iotiq commented 1 year ago

It looks like it's working now. It was probably a configuration error on my side.