New issue
Advanced search Search tips

Issue 758612 link

Starred by 1 user

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

chromite: GetPackageDependencies: dump output in case of error

Project Member Reported by hychao@chromium.org, Aug 24 2017

Issue description

https://luci-milo.appspot.com/buildbot/chromiumos.tryserver/pre_cq/51488
https://luci-milo.appspot.com/buildbot/chromiumos.tryserver/pre_cq/51507
https://luci-milo.appspot.com/buildbot/chromiumos.tryserver/pre_cq/51527

20:31:49: ERROR: <type 'exceptions.IndexError'>: list index out of range
Traceback (most recent call last):
  File "/b/c/cbuild/repository/chromite/lib/failures_lib.py", line 228, in wrapped_functor
    return functor(*args, **kwargs)
  File "/b/c/cbuild/repository/chromite/cbuildbot/stages/android_stages.py", line 160, in PerformStage
    versions, branches = self._UpdateBoardDictsForAndroidBuildInfo()
  File "/b/c/cbuild/repository/chromite/cbuildbot/stages/android_stages.py", line 134, in _UpdateBoardDictsForAndroidBuildInfo
    version = self._run.DetermineAndroidVersion(boards=[board])
  File "/b/c/cbuild/repository/chromite/cbuildbot/cbuildbot_run.py", line 937, in FuncWrapper
    return result(*args, **kwargs)
  File "/b/c/cbuild/repository/chromite/cbuildbot/cbuildbot_run.py", line 852, in DetermineAndroidVersion
    package = self.DetermineAndroidPackage(board)
  File "/b/c/cbuild/repository/chromite/cbuildbot/cbuildbot_run.py", line 825, in DetermineAndroidPackage
    packages = portage_util.GetPackageDependencies(board, 'virtual/target-os')
  File "/b/c/cbuild/repository/chromite/lib/portage_util.py", line 1758, in GetPackageDependencies
    package = columns[1] + '-' + columns[2]
IndexError: list index out of range

The pre-cq was running with
https://chromium-review.googlesource.com/#/c/620431/
https://chrome-internal-review.googlesource.com/#/c/437612/
https://chrome-internal-review.googlesource.com/#/c/437613/
 

Comment 1 by hychao@chromium.org, Aug 25 2017

Cc: akes...@chromium.org

Comment 2 by hychao@chromium.org, Aug 25 2017

Cc: vapier@chromium.org drinkcat@chromium.org
I have run "emerge-eve -p --cols --quiet --root /var/empty -e virtual/target-os " and the output looks very fine.
And I can reproduce this by cbuildbot locally. Any instruction how I can add debug logs in chromite scripts and then run them locally?

Comment 3 by hychao@chromium.org, Aug 25 2017

Status: WontFix (was: Untriaged)
The cause turns out to be a stupid debug 'echo' left in ebuild file I forget to remove. And that adds a redundant line to the RunCommand output:

18:48:35: ERROR: ==GetPackageDependencies===
18:48:35: ERROR: ['kbl-rt5514-hotword-support-0.0.1-r3.tbz2', ' N     virtual/pkgconfig 0-r1 to /var/empty/', ' N     sys-apps/baselayout 2.2 to /var/empty/', ' N     sys-libs/zlib 1.2.11 to /var/empty/', ' N     app-arch/xz-utils 5.2.3 to /var/empty/', ...

Thanks drinkcat@ helped me looking at this.
Might be nice to fix chromite to catch any parsing error and dump the output in that case, to make this sort of issue easier to debug...

Comment 5 by vapier@chromium.org, Aug 25 2017

agreed.  seems like that should be an easy "except" block to add to the chromite code (like have GetPackageDependencies dump the output and raise a new exception).
Status: Available (was: WontFix)
Summary: chromite: GetPackageDependencies: dump output in case of error (was: eve-pre-cq failing AndroidMetadata)
Repurposing this issue, then.

Comment 7 Deleted

Comment 8 Deleted

Components: Infra>Client>ChromeOS>CI
Components: -Infra>Client>ChromeOS

Sign in to add a comment