New issue
Advanced search Search tips
Starred by 1 user

Issue metadata

Status: Verified
Closed: Feb 2017
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug

Sign in to add a comment

Canaries and PFQ all fail with SyncChrome errors

Project Member Reported by, Feb 10 2017 Back to list

Issue description

[0:00:55] Checked out 0e09de60e479fe2ee105fb731b403c729fce3ac1 to a detached HEAD. Before making any commits
in this repo, you should use 'git checkout <branch>' to switch to
an existing branch or use 'git checkout origin -b <branch>' to
create a new branch for your work.
Traceback (most recent call last):
  File "/b/depot_tools/", line 2136, in <module>
  File "/b/depot_tools/", line 2122, in main
    return dispatcher.execute(OptionParser(), argv)
  File "/b/depot_tools/", line 252, in execute
    return command(parser, args[1:])
  File "/b/depot_tools/", line 1884, in CMDsync
    ret = client.RunOnDeps('update', args)
  File "/b/depot_tools/", line 1371, in RunOnDeps
    work_queue.flush(revision_overrides, command, args, options=self._options)
  File "/b/depot_tools/", line 1048, in run*self.args, **self.kwargs)
  File "/b/depot_tools/", line 798, in run
  File "/b/depot_tools/", line 697, in ParseDepsFile
    'that is not referenced in deps: %s' % name)
TypeError: not enough arguments for format string
sync_chrome: Unhandled exception:
Traceback (most recent call last):
  File "/b/cbuild/internal_master/chromite/bin/sync_chrome", line 168, in <module>
  File "/b/cbuild/internal_master/chromite/bin/sync_chrome", line 164, in DoMain
  File "/b/cbuild/internal_master/chromite/lib/", line 834, in ScriptWrapperMain
    ret = target(argv[1:])
  File "/b/cbuild/internal_master/chromite/scripts/", line 72, in main
    SyncChrome(gclient_path, options)
  File "/b/cbuild/internal_master/chromite/scripts/", line 48, in SyncChrome
    gclient.Sync(gclient_path, options.chrome_root, reset=options.reset)
  File "/b/cbuild/internal_master/chromite/lib/", line 247, in Sync
    cros_build_lib.RunCommand(cmd, cwd=cwd)
  File "/b/cbuild/internal_master/chromite/lib/", line 625, in RunCommand
    raise RunCommandError(msg, cmd_result)
chromite.lib.cros_build_lib.RunCommandError: return code: 1; command: /b/depot_tools/gclient sync --verbose --nohooks --with_branch_heads --with_tags --reset --force --delete_unversioned_trees
cmd=['/b/depot_tools/gclient', 'sync', '--verbose', '--nohooks', '--with_branch_heads', '--with_tags', '--reset', '--force', '--delete_unversioned_trees'], cwd=/b/cbuild/internal_master/.cache/distfiles/target-master/chrome-src-internal
10:27:20: WARNING: Command failed with retriable error.

Comment 1 by, Feb 10 2017

Labels: -Pri-3 Build-PFQ-Failures OS-Chrome Pri-1
PFQ has the same failure. Only error is:

Error: 5> ParseDepsFile(src): recursedeps contains entry, that is not referenced in deps: src/third_party/android_tools
10:55:01: WARNING: Chrome checkout appears corrupt. Clobbering.

(Weird that the canary has the TypeError, but it *looks* like it's trying to print the same error.)

android_tools is in fact in DEPS (for android only), but also in recursedeps (not sure what that is yet).
Summary: Canaries and PFQ all fail with SyncChrome errors (was: Canaries all failed with SyncChrome errors)
Status: Fixed
That was a bug in a change that landed in depot_tools that has been reverted. It should work now, let me know if it doesn't.
Yeah, looks like the offending CL has been reverted. Thanks.

Comment 6 by, Apr 17 2017

Labels: VerifyIn-59

Comment 7 by, May 30 2017

Labels: VerifyIn-60

Comment 8 by, Aug 1 2017

Labels: VerifyIn-61
Status: Verified
Closing. Please reopen it if its not fixed. Thanks!

Sign in to add a comment