New issue
Advanced search Search tips

Issue 799626 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jan 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Make Paygen Payload test suites Async.

Project Member Reported by dgarr...@chromium.org, Jan 5 2018

Issue description

We have a desire to make all hwtest suites in release builds asynchronous. This will make it impossible to generate the FINISHED flags.

GE has already adjusted their behavior to check for test suite results instead of FINISHED flags.
 
Cc: abutzier@chromium.org
Cc: pprabhu@chromium.org
I have sign off from abutzier@ that GE is ready for this change.
Project Member

Comment 3 by bugdroid1@chromium.org, Jan 9 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/2f194846565de315c249ef2572037da87f4fca83

commit 2f194846565de315c249ef2572037da87f4fca83
Author: Don Garrett <dgarrett@google.com>
Date: Tue Jan 09 01:22:44 2018

paygen: Stop generating FINISHED flag.

We plan to stop waiting for payload testing results. This means we
will no longer be able to correctly generate FINISHED flags after
tests complete. So.... stop generating them.

BUG= chromium:799626 
TEST=run_tests

Change-Id: Ic6de6ca89d8c56c80070b39041882bab2d0ce6d5
Reviewed-on: https://chromium-review.googlesource.com/853206
Commit-Ready: Don Garrett <dgarrett@chromium.org>
Tested-by: Don Garrett <dgarrett@chromium.org>
Reviewed-by: Alex Butzier <abutzier@chromium.org>
Reviewed-by: Don Garrett <dgarrett@chromium.org>

[modify] https://crrev.com/2f194846565de315c249ef2572037da87f4fca83/cbuildbot/stages/release_stages.py
[modify] https://crrev.com/2f194846565de315c249ef2572037da87f4fca83/lib/paygen/paygen_build_lib_unittest.py
[modify] https://crrev.com/2f194846565de315c249ef2572037da87f4fca83/lib/paygen/gspaths_unittest.py
[modify] https://crrev.com/2f194846565de315c249ef2572037da87f4fca83/lib/paygen/gspaths.py
[modify] https://crrev.com/2f194846565de315c249ef2572037da87f4fca83/cbuildbot/stages/release_stages_unittest.py
[modify] https://crrev.com/2f194846565de315c249ef2572037da87f4fca83/lib/paygen/paygen_build_lib.py

Project Member

Comment 4 by bugdroid1@chromium.org, Jan 9 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/8d9a7b743fdfa21ea5857d99d78a3223b6ccefe4

commit 8d9a7b743fdfa21ea5857d99d78a3223b6ccefe4
Author: Don Garrett <dgarrett@google.com>
Date: Tue Jan 09 01:22:44 2018

paygen: Make payload tests async.

Stop waiting on payload tests to complete.

BUG= chromium:799626 
TEST=run_tests

Change-Id: I9bb03fa5f221fcd277ab044549ca1eed4d8442d4
Reviewed-on: https://chromium-review.googlesource.com/853258
Commit-Ready: Don Garrett <dgarrett@chromium.org>
Tested-by: Don Garrett <dgarrett@chromium.org>
Reviewed-by: Alex Butzier <abutzier@chromium.org>
Reviewed-by: Xixuan Wu <xixuan@chromium.org>

[modify] https://crrev.com/8d9a7b743fdfa21ea5857d99d78a3223b6ccefe4/lib/paygen/paygen_build_lib.py

Status: Fixed (was: Started)

Sign in to add a comment