27 Nov
2018
27 Nov
'18
11:47 p.m.
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