Project: chromium Issues People Development process History Sign in
New issue
Advanced search Search tips
Starred by 1 user
Status: Verified
Owner:
Closed: Feb 2017
Cc:
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 sbasi@chromium.org, Feb 10 2017 Back to list
https://uberchromegw.corp.google.com/i/chromeos/builders/cyan-release/builds/843/steps/SyncChrome/logs/stdio

[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/gclient.py", line 2136, in <module>
    sys.exit(main(sys.argv[1:]))
  File "/b/depot_tools/gclient.py", line 2122, in main
    return dispatcher.execute(OptionParser(), argv)
  File "/b/depot_tools/subcommand.py", line 252, in execute
    return command(parser, args[1:])
  File "/b/depot_tools/gclient.py", line 1884, in CMDsync
    ret = client.RunOnDeps('update', args)
  File "/b/depot_tools/gclient.py", line 1371, in RunOnDeps
    work_queue.flush(revision_overrides, command, args, options=self._options)
  File "/b/depot_tools/gclient_utils.py", line 1048, in run
    self.item.run(*self.args, **self.kwargs)
  File "/b/depot_tools/gclient.py", line 798, in run
    self.ParseDepsFile()
  File "/b/depot_tools/gclient.py", 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>
    DoMain()
  File "/b/cbuild/internal_master/chromite/bin/sync_chrome", line 164, in DoMain
    commandline.ScriptWrapperMain(FindTarget)
  File "/b/cbuild/internal_master/chromite/lib/commandline.py", line 834, in ScriptWrapperMain
    ret = target(argv[1:])
  File "/b/cbuild/internal_master/chromite/scripts/sync_chrome.py", line 72, in main
    SyncChrome(gclient_path, options)
  File "/b/cbuild/internal_master/chromite/scripts/sync_chrome.py", line 48, in SyncChrome
    gclient.Sync(gclient_path, options.chrome_root, reset=options.reset)
  File "/b/cbuild/internal_master/chromite/lib/gclient.py", line 247, in Sync
    cros_build_lib.RunCommand(cmd, cwd=cwd)
  File "/b/cbuild/internal_master/chromite/lib/cros_build_lib.py", 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 sbasi@chromium.org, Feb 10 2017
Cc: michae...@chromium.org
Cc: dpranke@chromium.org steve...@chromium.org glevin@chromium.org abodenha@chromium.org
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)
Owner: dpranke@chromium.org
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. https://chromium.googlesource.com/chromium/tools/depot_tools.git/+/b1550ec8b36f49eb67f41ed8e4582aedd1e0ade7
Labels: VerifyIn-59
Labels: VerifyIn-60
Labels: VerifyIn-61
Status: Verified
Closing. Please reopen it if its not fixed. Thanks!
Sign in to add a comment