Merge lp://staging/~liuyq0307/lava-android-test/cts-4_1_r1 into lp://staging/lava-android-test

Proposed by Yongqin Liu
Status: Merged
Merged at revision: 212
Proposed branch: lp://staging/~liuyq0307/lava-android-test/cts-4_1_r1
Merge into: lp://staging/lava-android-test
Diff against target: 93 lines (+45/-5)
2 files modified
lava_android_test/test_definitions/cts/cts_prepare.sh (+3/-3)
lava_android_test/test_definitions/cts/cts_wrapper.py (+42/-2)
To merge this branch: bzr merge lp://staging/~liuyq0307/lava-android-test/cts-4_1_r1
Reviewer Review Type Date Requested Status
Linaro Validation Team Pending
Review via email: mp+128429@code.staging.launchpad.net

Description of the change

update to use the latest 4.1.r1 package that google provided

To post a comment you must log in.
Revision history for this message
Michael Hudson-Doyle (mwhudson) wrote :

I don't feel particularly qualified to review this, but it looks OK I
guess. We're not actually running CTS currently, right?

Revision history for this message
Yongqin Liu (liuyq0307) wrote :

> I don't feel particularly qualified to review this, but it looks OK I
> guess. We're not actually running CTS currently, right?

Yes, we are not running the CTS because the bug it consumes the CPU.
But this branch is not for that problem, it is just updated to use the latest cts-4_1_r1 CTS package.

And I want to investigate the CTS consumes CPU problem based on this branch with the latest CTS package

Revision history for this message
Yongqin Liu (liuyq0307) wrote :

Sorry, this is the CTS consumes CPU bug:
https://bugs.launchpad.net/linaro-android/+bug/1034218

Revision history for this message
Michael Hudson-Doyle (mwhudson) wrote :

Yongqin Liu <email address hidden> writes:

>> I don't feel particularly qualified to review this, but it looks OK I
>> guess. We're not actually running CTS currently, right?
>
> Yes, we are not running the CTS because the bug it consumes the CPU.
> But this branch is not for that problem, it is just updated to use the latest cts-4_1_r1 CTS package.
>
> And I want to investigate the CTS consumes CPU problem based on this branch with the latest CTS package

Fair enough. Merge away then :-)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches