Hi everyone,
I'm currently running CTS tests on imx8mq on Android 9. My test job is based on Linaro's test definition located here : https://git.linaro.org/qa/test-definitions.git/tree/automated/android/tradef...
I had issue concerning the Java heap size, causing the test to abort and skip a lot of tests. So, I cloned this git tree and made modifications in "tradefed.sh" :
export _JAVA_OPTIONS="-Xmx350M"
replaced by :
export _JAVA_OPTIONS="-Xmx700M"
Since this modification, I'm able to run the full CTS 9. So, I think you should consider modify this script, so other users can benefit from it.
Best regards,
Axel
On Mon, 26 Nov 2018 at 09:37, Axel Lebourhis axel.lebourhis@linaro.org wrote:
Hi everyone,
I'm currently running CTS tests on imx8mq on Android 9. My test job is based on Linaro's test definition located here : https://git.linaro.org/qa/test-definitions.git/tree/automated/android/tradef...
I had issue concerning the Java heap size, causing the test to abort and skip a lot of tests. So, I cloned this git tree and made modifications in "tradefed.sh" :
export _JAVA_OPTIONS="-Xmx350M"
replaced by :
export _JAVA_OPTIONS="-Xmx700M"
Since this modification, I'm able to run the full CTS 9. So, I think you should consider modify this script, so other users can benefit from it.
Axel,
Do you mind sending me a patch? I can merge it to the repository.
milosz
Best regards,
Axel
Lava-users mailing list Lava-users@lists.lavasoftware.org https://lists.lavasoftware.org/mailman/listinfo/lava-users
Hi Milosz,
You will find the patch attached.
Regards, Axel
On Mon, 26 Nov 2018 at 09:39, Milosz Wasilewski < milosz.wasilewski@linaro.org> wrote:
On Mon, 26 Nov 2018 at 09:37, Axel Lebourhis axel.lebourhis@linaro.org wrote:
Hi everyone,
I'm currently running CTS tests on imx8mq on Android 9. My test job is based on Linaro's test definition located here :
https://git.linaro.org/qa/test-definitions.git/tree/automated/android/tradef...
I had issue concerning the Java heap size, causing the test to abort and skip a lot of tests. So, I cloned this git tree and made modifications in "tradefed.sh" :
export _JAVA_OPTIONS="-Xmx350M"
replaced by :
export _JAVA_OPTIONS="-Xmx700M"
Since this modification, I'm able to run the full CTS 9. So, I think you should consider modify this script, so other users can benefit from it.
Axel,
Do you mind sending me a patch? I can merge it to the repository.
milosz
Best regards,
Axel
Lava-users mailing list Lava-users@lists.lavasoftware.org https://lists.lavasoftware.org/mailman/listinfo/lava-users
On Mon, 26 Nov 2018 at 10:07, Axel Lebourhis axel.lebourhis@linaro.org wrote:
Hi Milosz,
You will find the patch attached.
Thanks. I tried applying it, but I think I have a more flexible solution. Please check here: https://review.linaro.org/#/c/qa/test-definitions/+/29209/
To use it add JAVA_OPTIONS to your lava job definition. Example below
- test: definitions: - from: git name: cts-part2-arm64-v8a params: ANDROID_VERSION: aosp-master FAILURES_PRINTED: '300' TEST_PARAMS: cts --abi arm64-v8a --disable-reboot --skip-preconditions --skip-device-info TEST_URL: <some_url_to_download_cts.zip> JAVA_OPTIONS: "-Xmx700M" path: automated/android/noninteractive-tradefed/tradefed.yaml repository: https://git.linaro.org/qa/test-definitions.git namespace: tlxc timeout: {minutes: 505}
milosz
Regards, Axel
On Mon, 26 Nov 2018 at 09:39, Milosz Wasilewski milosz.wasilewski@linaro.org wrote:
On Mon, 26 Nov 2018 at 09:37, Axel Lebourhis axel.lebourhis@linaro.org wrote:
Hi everyone,
I'm currently running CTS tests on imx8mq on Android 9. My test job is based on Linaro's test definition located here : https://git.linaro.org/qa/test-definitions.git/tree/automated/android/tradef...
I had issue concerning the Java heap size, causing the test to abort and skip a lot of tests. So, I cloned this git tree and made modifications in "tradefed.sh" :
export _JAVA_OPTIONS="-Xmx350M"
replaced by :
export _JAVA_OPTIONS="-Xmx700M"
Since this modification, I'm able to run the full CTS 9. So, I think you should consider modify this script, so other users can benefit from it.
Axel,
Do you mind sending me a patch? I can merge it to the repository.
milosz
Best regards,
Axel
Lava-users mailing list Lava-users@lists.lavasoftware.org https://lists.lavasoftware.org/mailman/listinfo/lava-users
Yes you are completely right that's way better. Thank you Milosz
Axel
On Mon, 26 Nov 2018 at 11:35, Milosz Wasilewski < milosz.wasilewski@linaro.org> wrote:
On Mon, 26 Nov 2018 at 10:07, Axel Lebourhis axel.lebourhis@linaro.org wrote:
Hi Milosz,
You will find the patch attached.
Thanks. I tried applying it, but I think I have a more flexible solution. Please check here: https://review.linaro.org/#/c/qa/test-definitions/+/29209/
To use it add JAVA_OPTIONS to your lava job definition. Example below
- test: definitions:
- from: git name: cts-part2-arm64-v8a params: ANDROID_VERSION: aosp-master FAILURES_PRINTED: '300' TEST_PARAMS: cts --abi arm64-v8a --disable-reboot
--skip-preconditions --skip-device-info TEST_URL: <some_url_to_download_cts.zip> JAVA_OPTIONS: "-Xmx700M" path: automated/android/noninteractive-tradefed/tradefed.yaml repository: https://git.linaro.org/qa/test-definitions.git namespace: tlxc timeout: {minutes: 505}
milosz
Regards, Axel
On Mon, 26 Nov 2018 at 09:39, Milosz Wasilewski <
milosz.wasilewski@linaro.org> wrote:
On Mon, 26 Nov 2018 at 09:37, Axel Lebourhis axel.lebourhis@linaro.org
wrote:
Hi everyone,
I'm currently running CTS tests on imx8mq on Android 9. My test job is based on Linaro's test definition located here :
https://git.linaro.org/qa/test-definitions.git/tree/automated/android/tradef...
I had issue concerning the Java heap size, causing the test to abort
and
skip a lot of tests. So, I cloned this git tree and made modifications in "tradefed.sh" :
export _JAVA_OPTIONS="-Xmx350M"
replaced by :
export _JAVA_OPTIONS="-Xmx700M"
Since this modification, I'm able to run the full CTS 9. So, I think you should consider modify this script, so other users can benefit from it.
Axel,
Do you mind sending me a patch? I can merge it to the repository.
milosz
Best regards,
Axel
Lava-users mailing list Lava-users@lists.lavasoftware.org https://lists.lavasoftware.org/mailman/listinfo/lava-users
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.
Regards, Axel
On Mon, 26 Nov 2018 at 11:49, Axel Lebourhis axel.lebourhis@linaro.org wrote:
Yes you are completely right that's way better. Thank you Milosz
Axel
On Mon, 26 Nov 2018 at 11:35, Milosz Wasilewski < milosz.wasilewski@linaro.org> wrote:
On Mon, 26 Nov 2018 at 10:07, Axel Lebourhis axel.lebourhis@linaro.org wrote:
Hi Milosz,
You will find the patch attached.
Thanks. I tried applying it, but I think I have a more flexible solution. Please check here: https://review.linaro.org/#/c/qa/test-definitions/+/29209/
To use it add JAVA_OPTIONS to your lava job definition. Example below
- test: definitions:
- from: git name: cts-part2-arm64-v8a params: ANDROID_VERSION: aosp-master FAILURES_PRINTED: '300' TEST_PARAMS: cts --abi arm64-v8a --disable-reboot
--skip-preconditions --skip-device-info TEST_URL: <some_url_to_download_cts.zip> JAVA_OPTIONS: "-Xmx700M" path: automated/android/noninteractive-tradefed/tradefed.yaml repository: https://git.linaro.org/qa/test-definitions.git namespace: tlxc timeout: {minutes: 505}
milosz
Regards, Axel
On Mon, 26 Nov 2018 at 09:39, Milosz Wasilewski <
milosz.wasilewski@linaro.org> wrote:
On Mon, 26 Nov 2018 at 09:37, Axel Lebourhis <
axel.lebourhis@linaro.org> wrote:
Hi everyone,
I'm currently running CTS tests on imx8mq on Android 9. My test job is based on Linaro's test definition located here :
https://git.linaro.org/qa/test-definitions.git/tree/automated/android/tradef...
I had issue concerning the Java heap size, causing the test to abort
and
skip a lot of tests. So, I cloned this git tree and made modifications in "tradefed.sh" :
export _JAVA_OPTIONS="-Xmx350M"
replaced by :
export _JAVA_OPTIONS="-Xmx700M"
Since this modification, I'm able to run the full CTS 9. So, I think you should consider modify this script, so other users can benefit from it.
Axel,
Do you mind sending me a patch? I can merge it to the repository.
milosz
Best regards,
Axel
Lava-users mailing list Lava-users@lists.lavasoftware.org https://lists.lavasoftware.org/mailman/listinfo/lava-users
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.
milosz
Regards, Axel
On Mon, 26 Nov 2018 at 11:49, Axel Lebourhis axel.lebourhis@linaro.org wrote:
Yes you are completely right that's way better. Thank you Milosz
Axel
On Mon, 26 Nov 2018 at 11:35, Milosz Wasilewski milosz.wasilewski@linaro.org wrote:
On Mon, 26 Nov 2018 at 10:07, Axel Lebourhis axel.lebourhis@linaro.org wrote:
Hi Milosz,
You will find the patch attached.
Thanks. I tried applying it, but I think I have a more flexible solution. Please check here: https://review.linaro.org/#/c/qa/test-definitions/+/29209/
To use it add JAVA_OPTIONS to your lava job definition. Example below
- test: definitions:
- from: git name: cts-part2-arm64-v8a params: ANDROID_VERSION: aosp-master FAILURES_PRINTED: '300' TEST_PARAMS: cts --abi arm64-v8a --disable-reboot
--skip-preconditions --skip-device-info TEST_URL: <some_url_to_download_cts.zip> JAVA_OPTIONS: "-Xmx700M" path: automated/android/noninteractive-tradefed/tradefed.yaml repository: https://git.linaro.org/qa/test-definitions.git namespace: tlxc timeout: {minutes: 505}
milosz
Regards, Axel
On Mon, 26 Nov 2018 at 09:39, Milosz Wasilewski milosz.wasilewski@linaro.org wrote:
On Mon, 26 Nov 2018 at 09:37, Axel Lebourhis axel.lebourhis@linaro.org wrote:
Hi everyone,
I'm currently running CTS tests on imx8mq on Android 9. My test job is based on Linaro's test definition located here : https://git.linaro.org/qa/test-definitions.git/tree/automated/android/tradef...
I had issue concerning the Java heap size, causing the test to abort and skip a lot of tests. So, I cloned this git tree and made modifications in "tradefed.sh" :
export _JAVA_OPTIONS="-Xmx350M"
replaced by :
export _JAVA_OPTIONS="-Xmx700M"
Since this modification, I'm able to run the full CTS 9. So, I think you should consider modify this script, so other users can benefit from it.
Axel,
Do you mind sending me a patch? I can merge it to the repository.
milosz
Best regards,
Axel
Lava-users mailing list Lava-users@lists.lavasoftware.org https://lists.lavasoftware.org/mailman/listinfo/lava-users
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
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
lava-users@lists.lavasoftware.org
-
Axel Lebourhis -
Milosz Wasilewski