New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 748116 link

Starred by 1 user

Issue metadata

Status: Archived
Owner:
Closed: Jul 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

cheets_GTS.GtsPlacementTestCases failing on eve-release

Project Member Reported by bhthompson@google.com, Jul 24 2017

Issue description

eve-release:759 failed

First failure was https://uberchromegw.corp.google.com/i/chromeos/builders/eve-release/builds/752
OS delta: https://crosland.corp.google.com/log/9766.0.0..9767.0.0
Android delta:  https://android-build.googleplex.com/builds/4203319/branches/git_nyc-mr1-arc/targets/aosp_cheets_arm-userdebug/cls?end=4203036
Chrome is identical.

https://pantheon.corp.google.com/storage/browser/chromeos-autotest-results/130080763-chromeos-test/chromeos2-row4-rack9-host4/
...
  cheets_GTS.GtsPlacementTestCases          [ FAILED ]
  cheets_GTS.GtsPlacementTestCases            FAIL: Error: Internal waiver bookkeeping has become inconsistent (tests=10, passed=4, failed=6, not_executed=0, waived=9)
  cheets_GTS.GtsPlacementTestCases            retry_count: 2
...

Builders failed on: 
- eve-release: 
  https://luci-milo.appspot.com/buildbot/chromeos/eve-release/759



 

Comment 1 by kinaba@chromium.org, Jul 24 2017

Cc: ihf@chromium.org
Owner: kinaba@chromium.org
Status: Assigned (was: Available)
The autotest wrapper for CTS/GTS runner is failing to recognized x86_64 abi and hence confused by the output. Fixing the regex below should be sufficient to handle that.

https://chromium.googlesource.com/chromiumos/third_party/autotest/+/master/server/cros/tradefed_test.py#205

Comment 2 by kinaba@chromium.org, Jul 25 2017

Status: Started (was: Assigned)

Comment 3 by kinaba@chromium.org, Jul 25 2017

Labels: -Pri-2 -M-62 M-61 Pri-1 Type-Bug
CL: https://chromium-review.googlesource.com/c/583871

The change triggered the issue is marked as M61: b/63841179
hence I believe this fix is also necessary for M61, too. Relabeling. 
Project Member

Comment 4 by sheriffbot@chromium.org, Jul 25 2017

Labels: Hotlist-Google
Project Member

Comment 5 by bugdroid1@chromium.org, Jul 26 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/49b90a0c14107ea68ad65c758864d4087c84f526

commit 49b90a0c14107ea68ad65c758864d4087c84f526
Author: Kazuhiro Inaba <kinaba@chromium.org>
Date: Wed Jul 26 09:07:32 2017

ARC: Let cts-tradefed parser recognize x86_64 ABI.

BUG= chromium:748116 
TEST=./utils/unittest_suite.py server.cros.tradefed_test_unittest

Change-Id: I18ba093de213d0055b305e9c2d711ad402d67d2e
Reviewed-on: https://chromium-review.googlesource.com/583871
Commit-Ready: Kazuhiro Inaba <kinaba@chromium.org>
Tested-by: Kazuhiro Inaba <kinaba@chromium.org>
Reviewed-by: Ilja H. Friedel <ihf@chromium.org>
Reviewed-by: Bernie Thompson <bhthompson@chromium.org>

[add] https://crrev.com/49b90a0c14107ea68ad65c758864d4087c84f526/server/cros/tradefed_test_unittest_data/GtsPlacementTestCases.txt
[modify] https://crrev.com/49b90a0c14107ea68ad65c758864d4087c84f526/server/cros/tradefed_test.py
[modify] https://crrev.com/49b90a0c14107ea68ad65c758864d4087c84f526/server/cros/tradefed_test_unittest.py

Comment 6 by kinaba@chromium.org, Jul 26 2017

Labels: Merge-Request-61
Fixed for M62. Merge-requesting for M61.
Labels: -Merge-Request-61 Merge-Approved-61
Project Member

Comment 8 by bugdroid1@chromium.org, Jul 26 2017

Labels: merge-merged-release-R61-9765.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/57fa78cc52bc64c7d5e04bf6d59082a3177a0c13

commit 57fa78cc52bc64c7d5e04bf6d59082a3177a0c13
Author: Kazuhiro Inaba <kinaba@chromium.org>
Date: Wed Jul 26 19:01:08 2017

ARC: Let cts-tradefed parser recognize x86_64 ABI.

BUG= chromium:748116 
TEST=./utils/unittest_suite.py server.cros.tradefed_test_unittest

Change-Id: I18ba093de213d0055b305e9c2d711ad402d67d2e
Reviewed-on: https://chromium-review.googlesource.com/583871
Commit-Ready: Kazuhiro Inaba <kinaba@chromium.org>
Tested-by: Kazuhiro Inaba <kinaba@chromium.org>
Reviewed-by: Ilja H. Friedel <ihf@chromium.org>
Reviewed-by: Bernie Thompson <bhthompson@chromium.org>
(cherry picked from commit 49b90a0c14107ea68ad65c758864d4087c84f526)
Reviewed-on: https://chromium-review.googlesource.com/586489
Reviewed-by: Kazuhiro Inaba <kinaba@chromium.org>
Commit-Queue: Kazuhiro Inaba <kinaba@chromium.org>
Trybot-Ready: Kazuhiro Inaba <kinaba@chromium.org>

[add] https://crrev.com/57fa78cc52bc64c7d5e04bf6d59082a3177a0c13/server/cros/tradefed_test_unittest_data/GtsPlacementTestCases.txt
[modify] https://crrev.com/57fa78cc52bc64c7d5e04bf6d59082a3177a0c13/server/cros/tradefed_test.py
[modify] https://crrev.com/57fa78cc52bc64c7d5e04bf6d59082a3177a0c13/server/cros/tradefed_test_unittest.py

Comment 9 by kinaba@chromium.org, Jul 26 2017

Status: Fixed (was: Started)
Project Member

Comment 10 by bugdroid1@chromium.org, Jul 27 2017

Labels: merge-merged-release-R60-9592.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/051b66eac2087c2a063982a894529fec00b73a59

commit 051b66eac2087c2a063982a894529fec00b73a59
Author: Kazuhiro Inaba <kinaba@chromium.org>
Date: Thu Jul 27 21:03:58 2017

ARC: Let cts-tradefed parser recognize x86_64 ABI.

BUG= chromium:748116 
TEST=./utils/unittest_suite.py server.cros.tradefed_test_unittest

Change-Id: I18ba093de213d0055b305e9c2d711ad402d67d2e
Reviewed-on: https://chromium-review.googlesource.com/583871
Commit-Ready: Kazuhiro Inaba <kinaba@chromium.org>
Tested-by: Kazuhiro Inaba <kinaba@chromium.org>
Reviewed-by: Ilja H. Friedel <ihf@chromium.org>
Reviewed-by: Bernie Thompson <bhthompson@chromium.org>
(cherry picked from commit 49b90a0c14107ea68ad65c758864d4087c84f526)
Reviewed-on: https://chromium-review.googlesource.com/586489
Reviewed-by: Kazuhiro Inaba <kinaba@chromium.org>
Commit-Queue: Kazuhiro Inaba <kinaba@chromium.org>
Trybot-Ready: Kazuhiro Inaba <kinaba@chromium.org>
(cherry picked from commit 57fa78cc52bc64c7d5e04bf6d59082a3177a0c13)
Reviewed-on: https://chromium-review.googlesource.com/587350
Tested-by: Ilja H. Friedel <ihf@chromium.org>

[add] https://crrev.com/051b66eac2087c2a063982a894529fec00b73a59/server/cros/tradefed_test_unittest_data/GtsPlacementTestCases.txt
[modify] https://crrev.com/051b66eac2087c2a063982a894529fec00b73a59/server/cros/tradefed_test.py
[modify] https://crrev.com/051b66eac2087c2a063982a894529fec00b73a59/server/cros/tradefed_test_unittest.py

Project Member

Comment 11 by sheriffbot@chromium.org, Jul 31 2017

This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: -Merge-Approved-61 Merge-Merged

Comment 13 by dchan@chromium.org, Jan 22 2018

Status: Archived (was: Fixed)

Sign in to add a comment