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

Issue 821207 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Apr 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug



Sign in to add a comment

ControlFileNotFound: dummy_Pass is not unique.

Project Member Reported by ayatane@chromium.org, Mar 12 2018

Issue description

https://uberchromegw.corp.google.com/i/chromeos/builders/master-paladin/builds/17987

Devserver call: ssh 100.115.219.133 'curl "http://100.115.219.133:8082/controlfiles?suite_name=&build=wolf-paladin/R67-10482.0.0-rc1"'


Relevant output

client/config/client/site_tests/dummy_Pass/control
client/config/client/site_tests/dummy_Pass/control.actionable
client/config/client/site_tests/dummy_Pass/control.bluetooth
client/config/client/site_tests/dummy_Pass/control.experimental
client/config/client/site_tests/dummy_Pass/control.wifichaos
client/config/server/site_tests/dummy_PassServer/control
client/config/server/site_tests/dummy_PassServer/control.nossp
client/config/server/site_tests/dummy_PassServer/control.sanity
client/config/server/site_tests/dummy_PassServer/control.ssp
client/site_tests/dummy_Pass/control
client/site_tests/dummy_Pass/control.actionable
client/site_tests/dummy_Pass/control.bluetooth
client/site_tests/dummy_Pass/control.experimental
client/site_tests/dummy_Pass/control.wifichaos
server/site_tests/dummy_PassServer/control
server/site_tests/dummy_PassServer/control.nossp
server/site_tests/dummy_PassServer/control.sanity
server/site_tests/dummy_PassServer/control.ssp


Looks like some symlink shenanigans
 
Cc: za...@chromium.org reve...@chromium.org jacoblin@chromium.org
Labels: -Pri-2 Pri-0
Owner: ayatane@chromium.org
Status: Assigned (was: Untriaged)
This is killing multiple cq runs.
The control_files.tar also shows these "client/config" directory entries.

https://storage.cloud.google.com/chromeos-image-archive/veyron_mighty-paladin/R67-10482.0.0-rc1/control_files.tar?_ga=2.57112874.-1030099716.1510283896

No idea what client/config is, and sounds fishy.
Cc: ayatane@chromium.org vapier@chromium.org
Owner: nya@chromium.org
Potentially related to https://chromium-review.googlesource.com/#/c/chromiumos/overlays/chromiumos-overlay/+/845472/


Comment 4 by nya@chromium.org, Mar 13 2018

Yes, my CL mentions client/config and looks suspicious. Let's revert it speculatively.

reverted
Project Member

Comment 6 by bugdroid1@chromium.org, Mar 13 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/4da61e9c13fc98bf89e4077c3015dd8b76773e40

commit 4da61e9c13fc98bf89e4077c3015dd8b76773e40
Author: Aviv Keshet <akeshet@chromium.org>
Date: Tue Mar 13 04:07:30 2018

Revert "autotest: Use CROS_WORKON_SUBTREE."

BUG=chromium:791888,  chromium:821207 
TEST=None

This reverts commit ec53f418f7a305c09efa0c4795530a25410e90bc.

Change-Id: I432d0ab848d98cd021a89f941d99619037d0a47d
Reviewed-on: https://chromium-review.googlesource.com/959752
Reviewed-by: Aviv Keshet <akeshet@chromium.org>
Tested-by: Aviv Keshet <akeshet@chromium.org>

[modify] https://crrev.com/4da61e9c13fc98bf89e4077c3015dd8b76773e40/eclass/autotest.eclass

Comment 7 by nya@chromium.org, Mar 13 2018

Thanks for revert. I will take a look what caused the failure.

The autotest eclass and ebuilds are really a mess.

It might be necessary to force an uprev of the autotest.ebuild or some other related ebuilds, as it is possible that the incorrect files are now included in some prebuilts.

Comment 9 by nya@chromium.org, Mar 13 2018

I agree autotest eclass/ebuilds are poorly written...

It's a good point we may need to force uprevs. I prepared a whitespace change:
https://chromium-review.googlesource.com/#/c/chromiumos/third_party/autotest/+/958306

Project Member

Comment 10 by bugdroid1@chromium.org, Mar 13 2018

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

commit 905ca396c59ef434c02ce2ed2b516cd1198f2ab3
Author: Shuhei Takahashi <nya@chromium.org>
Date: Tue Mar 13 05:01:22 2018

autotest: Whitespace change to trigger uprevs.

BUG= chromium:821207 
TEST=None

Change-Id: If2a22835e78ae06372c6f568fdab35726b26d5c5
Reviewed-on: https://chromium-review.googlesource.com/958306
Tested-by: Shuhei Takahashi <nya@chromium.org>
Reviewed-by: Aviv Keshet <akeshet@chromium.org>

[modify] https://crrev.com/905ca396c59ef434c02ce2ed2b516cd1198f2ab3/README.md

Oh duh, I didn't think to look in the overlays repo, thanks.
Labels: -Pri-0 Pri-1

Comment 13 by nya@chromium.org, Apr 17 2018

Owner: akes...@chromium.org
Status: Verified (was: Assigned)
This issue was fixed by revert.

Sign in to add a comment