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