New issue
Advanced search Search tips

Issue 804512 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

Some error messages from unibuilds in non-unibuilds

Project Member Reported by nsanders@chromium.org, Jan 22 2018

Issue description

It does seem the build worked ok otherwise. 

This build is from an older pre-unibuild chromebox variant


20:06:22: INFO: Running cidb query on pid 4142, repr(query) starts with <sqlalchemy.sql.expression.Update object at 0x7f2e528e0ed0>
20:06:22: INFO: RunCommand: /b/c/cbuild/repository/chromite/bin/cros_sdk -- /usr/bin/cros_config_host_py -c /build/guado-macrophage/usr/share/chromeos-config/config.dtb list-models in /b/c/cbuild/repository
20:06:26: INFO: (stderr):
Traceback (most recent call last):
  File "/usr/lib/python-exec/python2.7/cros_config_host_py", line 9, in <module>
    load_entry_point('cros-config-host==1', 'console_scripts', 'cros_config_host_py')()
  File "/usr/lib64/python2.7/site-packages/cros_config_host/cros_config_host.py", line 295, in main
    config = CrosConfig(opts.config, opts.yaml and FORMAT_YAML or None)
  File "/usr/lib64/python2.7/site-packages/cros_config_host/libcros_config_host.py", line 1034, in CrosConfig
    infile = open(fname)
IOError: [Errno 2] No such file or directory: '/build/guado-macrophage/usr/share/chromeos-config/config.dtb'
20:06:26: INFO: Running cidb query on pid 4142, repr(query) starts with <sqlalchemy.sql.expression.Update object at 0x7f2e528e0fd0>
************************************************************
** Finished Stage BuildPackages [afdo_use] - Wed, 10 Jan 2018 20:06:26 -0800 (PST)
************************************************************


https://luci-milo.appspot.com/buildbot/chromeos/guado-macrophage-release/372


 
Project Member

Comment 1 by bugdroid1@chromium.org, Feb 23 2018

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

commit 3d3eab7d1f1181379368e6e5d677cc356ef75596
Author: C Shapiro <shapiroc@chromium.org>
Date: Fri Feb 23 22:15:58 2018

Add support for yaml in cbuildbot

Also fixed error that prints to stderr when a non-unibuild is run since
this now just avoids the invocation in the first place.

BUG= chromium:804512 
TEST=unit tests

Change-Id: I21b17b640fee9c4f223fff4362b2344883709f16
Reviewed-on: https://chromium-review.googlesource.com/919564
Commit-Ready: C Shapiro <shapiroc@google.com>
Tested-by: C Shapiro <shapiroc@google.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Jason Clinton <jclinton@chromium.org>

[modify] https://crrev.com/3d3eab7d1f1181379368e6e5d677cc356ef75596/cbuildbot/stages/build_stages_unittest.py
[modify] https://crrev.com/3d3eab7d1f1181379368e6e5d677cc356ef75596/cbuildbot/stages/build_stages.py
[modify] https://crrev.com/3d3eab7d1f1181379368e6e5d677cc356ef75596/cbuildbot/commands.py
[modify] https://crrev.com/3d3eab7d1f1181379368e6e5d677cc356ef75596/cbuildbot/commands_unittest.py

Status: Fixed (was: Untriaged)
Project Member

Comment 3 by bugdroid1@chromium.org, Mar 2 2018

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

commit 900cff8c1a8d4d9acd9621f4ce599994aebd69d4
Author: C Shapiro <shapiroc@google.com>
Date: Fri Mar 02 23:47:17 2018

Revert "Add support for yaml in cbuildbot"

This reverts commit 3d3eab7d1f1181379368e6e5d677cc356ef75596.

Reason for revert: Seems to have broken unibuild detection right now.

Original change's description:
> Add support for yaml in cbuildbot
>
> Also fixed error that prints to stderr when a non-unibuild is run since
> this now just avoids the invocation in the first place.
>
> BUG= chromium:804512 
> TEST=unit tests
>
> Change-Id: I21b17b640fee9c4f223fff4362b2344883709f16
> Reviewed-on: https://chromium-review.googlesource.com/919564
> Commit-Ready: C Shapiro <shapiroc@google.com>
> Tested-by: C Shapiro <shapiroc@google.com>
> Reviewed-by: Simon Glass <sjg@chromium.org>
> Reviewed-by: Jason Clinton <jclinton@chromium.org>

Bug:  chromium:804512 
Change-Id: I5e193ffb2dcd658f0d3e13f0e21bb5a6f1ead947
Reviewed-on: https://chromium-review.googlesource.com/946526
Commit-Ready: C Shapiro <shapiroc@google.com>
Tested-by: C Shapiro <shapiroc@google.com>
Reviewed-by: Jason Clinton <jclinton@chromium.org>

[modify] https://crrev.com/900cff8c1a8d4d9acd9621f4ce599994aebd69d4/cbuildbot/stages/build_stages_unittest.py
[modify] https://crrev.com/900cff8c1a8d4d9acd9621f4ce599994aebd69d4/cbuildbot/stages/build_stages.py
[modify] https://crrev.com/900cff8c1a8d4d9acd9621f4ce599994aebd69d4/cbuildbot/commands.py
[modify] https://crrev.com/900cff8c1a8d4d9acd9621f4ce599994aebd69d4/cbuildbot/commands_unittest.py

Project Member

Comment 4 by bugdroid1@chromium.org, Mar 3 2018

Labels: merge-merged-release-R66-10452.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/3915db268a1a9ff979edfbac1bd5c0cd132d163a

commit 3915db268a1a9ff979edfbac1bd5c0cd132d163a
Author: C Shapiro <shapiroc@google.com>
Date: Sat Mar 03 12:48:57 2018

Revert "Add support for yaml in cbuildbot"

This reverts commit 3d3eab7d1f1181379368e6e5d677cc356ef75596.

Reason for revert: Seems to have broken unibuild detection right now.

Original change's description:
> Add support for yaml in cbuildbot
>
> Also fixed error that prints to stderr when a non-unibuild is run since
> this now just avoids the invocation in the first place.
>
> BUG= chromium:804512 
> TEST=unit tests
>
> Change-Id: I21b17b640fee9c4f223fff4362b2344883709f16
> Reviewed-on: https://chromium-review.googlesource.com/919564
> Commit-Ready: C Shapiro <shapiroc@google.com>
> Tested-by: C Shapiro <shapiroc@google.com>
> Reviewed-by: Simon Glass <sjg@chromium.org>
> Reviewed-by: Jason Clinton <jclinton@chromium.org>

Bug:  chromium:804512 
Change-Id: I5e193ffb2dcd658f0d3e13f0e21bb5a6f1ead947
Reviewed-on: https://chromium-review.googlesource.com/946526
Commit-Ready: C Shapiro <shapiroc@google.com>
Tested-by: C Shapiro <shapiroc@google.com>
Reviewed-by: Jason Clinton <jclinton@chromium.org>
(cherry picked from commit 900cff8c1a8d4d9acd9621f4ce599994aebd69d4)
Reviewed-on: https://chromium-review.googlesource.com/947982
Reviewed-by: C Shapiro <shapiroc@google.com>
Commit-Queue: C Shapiro <shapiroc@google.com>
Trybot-Ready: C Shapiro <shapiroc@google.com>

[modify] https://crrev.com/3915db268a1a9ff979edfbac1bd5c0cd132d163a/cbuildbot/stages/build_stages_unittest.py
[modify] https://crrev.com/3915db268a1a9ff979edfbac1bd5c0cd132d163a/cbuildbot/stages/build_stages.py
[modify] https://crrev.com/3915db268a1a9ff979edfbac1bd5c0cd132d163a/cbuildbot/commands.py
[modify] https://crrev.com/3915db268a1a9ff979edfbac1bd5c0cd132d163a/cbuildbot/commands_unittest.py

Project Member

Comment 5 by bugdroid1@chromium.org, Mar 3 2018

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

commit 500e89da3e361ff5b6fe22f75907f6a48c3f39c6
Author: C Shapiro <shapiroc@google.com>
Date: Sat Mar 03 20:22:43 2018

Reland "Revert "Add support for yaml in cbuildbot""

This is a reland of 3915db268a1a9ff979edfbac1bd5c0cd132d163a

Original change's description:
> Revert "Add support for yaml in cbuildbot"
> 
> This reverts commit 3d3eab7d1f1181379368e6e5d677cc356ef75596.
> 
> Reason for revert: Seems to have broken unibuild detection right now.
> 
> Original change's description:
> > Add support for yaml in cbuildbot
> >
> > Also fixed error that prints to stderr when a non-unibuild is run since
> > this now just avoids the invocation in the first place.
> >
> > BUG= chromium:804512 
> > TEST=unit tests
> >
> > Change-Id: I21b17b640fee9c4f223fff4362b2344883709f16
> > Reviewed-on: https://chromium-review.googlesource.com/919564
> > Commit-Ready: C Shapiro <shapiroc@google.com>
> > Tested-by: C Shapiro <shapiroc@google.com>
> > Reviewed-by: Simon Glass <sjg@chromium.org>
> > Reviewed-by: Jason Clinton <jclinton@chromium.org>
> 
> Bug:  chromium:804512 
> Change-Id: I5e193ffb2dcd658f0d3e13f0e21bb5a6f1ead947
> Reviewed-on: https://chromium-review.googlesource.com/946526
> Commit-Ready: C Shapiro <shapiroc@google.com>
> Tested-by: C Shapiro <shapiroc@google.com>
> Reviewed-by: Jason Clinton <jclinton@chromium.org>
> (cherry picked from commit 900cff8c1a8d4d9acd9621f4ce599994aebd69d4)
> Reviewed-on: https://chromium-review.googlesource.com/947982
> Reviewed-by: C Shapiro <shapiroc@google.com>
> Commit-Queue: C Shapiro <shapiroc@google.com>
> Trybot-Ready: C Shapiro <shapiroc@google.com>

Bug:  chromium:804512 
Change-Id: Ia9794e70f97f4548ffe4b9f00e37a4aa84d845e1
Reviewed-on: https://chromium-review.googlesource.com/947984
Reviewed-by: C Shapiro <shapiroc@google.com>
Commit-Queue: C Shapiro <shapiroc@google.com>
Tested-by: C Shapiro <shapiroc@google.com>
Trybot-Ready: C Shapiro <shapiroc@google.com>

[modify] https://crrev.com/500e89da3e361ff5b6fe22f75907f6a48c3f39c6/cbuildbot/stages/build_stages_unittest.py
[modify] https://crrev.com/500e89da3e361ff5b6fe22f75907f6a48c3f39c6/cbuildbot/stages/build_stages.py
[modify] https://crrev.com/500e89da3e361ff5b6fe22f75907f6a48c3f39c6/cbuildbot/commands.py
[modify] https://crrev.com/500e89da3e361ff5b6fe22f75907f6a48c3f39c6/cbuildbot/commands_unittest.py

Sign in to add a comment