New issue
Advanced search Search tips

Issue 770716 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Oct 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 1
Type: Bug



Sign in to add a comment

update_scripts failing on build139-b4 (win_baremetal)

Project Member Reported by ehmaldonado@chromium.org, Oct 2 2017

Issue description

On build139-b4, the update_scripts step fails with:


Looking for .gclient starting from C:\b\build

Traceback (most recent call last):
  File "C:\b\depot_tools\gclient.py", line 2636, in <module>
    sys.exit(main(sys.argv[1:]))
  File "C:\b\depot_tools\gclient.py", line 2622, in main
    return dispatcher.execute(OptionParser(), argv)
  File "C:\b\depot_tools\subcommand.py", line 252, in execute
    return command(parser, args[1:])
  File "C:\b\depot_tools\gclient.py", line 2366, in CMDsync
    client = GClient.LoadCurrentConfig(options)
  File "C:\b\depot_tools\gclient.py", line 1357, in LoadCurrentConfig
    path = gclient_utils.FindGclientRoot(os.getcwd(), options.config_filename)
  File "C:\b\depot_tools\gclient_utils.py", line 627, in FindGclientRoot
    exec(FileRead(entries_filename), scope)
TypeError: expected string without null bytes

Full logs at https://build.chromium.org/p/tryserver.webrtc/builders/win_baremetal/builds/25275/steps/update_scripts/logs/stdio
 
Components: -Infra Infra>Labs
Labels: OS-Windows
Maybe re-imaging would fix it?
Project Member

Comment 3 by bugdroid1@chromium.org, Oct 3 2017

The following revision refers to this bug:
  https://webrtc.googlesource.com/src.git/+/1cfa95b8b84ccd2b3ecdaddbd7dfb27877fbd2ec

commit 1cfa95b8b84ccd2b3ecdaddbd7dfb27877fbd2ec
Author: Edward Lemur <ehmaldonado@webrtc.org>
Date: Tue Oct 03 16:02:03 2017

Disable win_baremetal.

No-Try: true
Bug:  chromium:770716 
Change-Id: I8a2a655a8ede2f1865f572b8c81d68c5028aae73
Reviewed-on: https://webrtc-review.googlesource.com/6100
Reviewed-by: Henrik Kjellander <kjellander@webrtc.org>
Commit-Queue: Edward Lemur <ehmaldonado@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#20119}
[modify] https://crrev.com/1cfa95b8b84ccd2b3ecdaddbd7dfb27877fbd2ec/infra/config/cq.cfg

Comment 4 by vhang@chromium.org, Oct 3 2017

Owner: jo...@chromium.org
Status: Assigned (was: Untriaged)

Comment 5 by jo...@chromium.org, Oct 4 2017

Status: Fixed (was: Assigned)
Clobbered the checkout, and it seems to have had a green run subsequently:

https://build.chromium.org/p/tryserver.webrtc/builders/win_baremetal/builds/25326
Project Member

Comment 6 by bugdroid1@chromium.org, Oct 5 2017

The following revision refers to this bug:
  https://webrtc.googlesource.com/src.git/+/69760535055f73255469290ae3e6cb199520d1e0

commit 69760535055f73255469290ae3e6cb199520d1e0
Author: Henrik Kjellander <kjellander@webrtc.org>
Date: Thu Oct 05 06:59:15 2017

Revert "Disable win_baremetal."

This reverts commit 1cfa95b8b84ccd2b3ecdaddbd7dfb27877fbd2ec.

Reason for revert: The faulty machine is now fixed: https://bugs.chromium.org/p/chromium/issues/detail?id=770716

Original change's description:
> Disable win_baremetal.
> 
> No-Try: true
> Bug:  chromium:770716 
> Change-Id: I8a2a655a8ede2f1865f572b8c81d68c5028aae73
> Reviewed-on: https://webrtc-review.googlesource.com/6100
> Reviewed-by: Henrik Kjellander <kjellander@webrtc.org>
> Commit-Queue: Edward Lemur <ehmaldonado@webrtc.org>
> Cr-Commit-Position: refs/heads/master@{#20119}

TBR=kjellander@webrtc.org,ehmaldonado@webrtc.org

Change-Id: I1bf0500af2b6372064a0a01f22613fb883159e07
Notry: True
Bug:  chromium:770716 
Reviewed-on: https://webrtc-review.googlesource.com/6760
Reviewed-by: Henrik Kjellander <kjellander@webrtc.org>
Commit-Queue: Henrik Kjellander <kjellander@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#20153}
[modify] https://crrev.com/69760535055f73255469290ae3e6cb199520d1e0/infra/config/cq.cfg

Sign in to add a comment