Great, this should work fine. I'll give you a feedback if not after merging the changes.
Regards, Axel
On Tue, 27 Nov 2018 at 11:47, Milosz Wasilewski < milosz.wasilewski@linaro.org> wrote:
On Tue, 27 Nov 2018 at 10:44, Milosz Wasilewski milosz.wasilewski@linaro.org wrote:
On Tue, 27 Nov 2018 at 10:42, Axel Lebourhis axel.lebourhis@linaro.org
wrote:
Hi Milosz,
I gave a try to the new tradefed job. The JAVA_OPTIONS is not
overriden correctly.
I can see in tradefed.yaml you didn't modified the following line :
- sudo -u testuser ./tradefed.sh -o "${TIMEOUT}" -c "${TEST_URL}" -t
"${TEST_PARAMS}" -p "${TEST_PATH}" -r "${RESULTS_FORMAT}" -n "${ANDROID_SERIAL}" -f "${FAILURES_PRINTED}" -a "${AP_SSID}" -k "${AP_KEY}"
I think you forgot to add "-j $(JAVA_OPTIONS)" in order to pass the
new parameter.
Ouch, you're totally right. I'll fix it in a moment.
https://review.linaro.org/#/c/qa/test-definitions/+/29228/
Hope this helps. milosz