New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 764307 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Sep 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

download_from_google_storage fails to fetch file "Err: Unknown option -p"

Project Member Reported by athom@google.com, Sep 12 2017

Issue description

download_from_google_storage fails since depot_tools 509776ebbbef259a459a32235dac53998c5c12b8 ([gsutil] run through "vpython") on most of Dart's LUCI builders in our CQ.

Link to the log stream and error in the step gclient runhooks:
https://luci-logdog.appspot.com/v/?s=dart%2Fbuildbucket%2Fcr-buildbucket.appspot.com%2F8968692595798468624%2F%2B%2Fsteps%2Fgclient_runhooks%2F0%2Fstdout
________ running 'download_from_google_storage --no_auth --no_resume --bucket dart-dependencies --recursive --directory sdk/third_party/d8' in '/b/s/w/ir/kitchen-workdir'
0> Failed to fetch file gs://dart-dependencies/82c3a2f3e2b5edbd3defc1db07463a872c703f75 for sdk/third_party/d8/linux/d8, skipping. [Err: Unknown option: -p
usage: /b/s/w/ir/cache/vpython/2686c2/bin/python [option] ... [-c cmd | -m mod | file | -] [arg] ...
Try `python -h' for more information.
]

Task: https://chromium-swarm.appspot.com/task?id=388cd9a4abb59d10

Reverting depot_tools to https://chromium.googlesource.com/chromium/tools/depot_tools/+/aaea62b9c3bb6c9e065245d92c7471459d2b2967 makes it work.

Currently our CQ is broken because of this error.
 

Comment 1 by athom@google.com, Sep 12 2017

Description: Show this description

Comment 2 by athom@google.com, Sep 12 2017

Labels: OS-Linux OS-Mac OS-Windows

Comment 3 by whesse@google.com, Sep 12 2017

Labels: -Restrict-View-Google
Removing the restriction to Google chromium accounts, so our users can track this issue.  This issue is now publicly viewable.

Comment 4 by d...@chromium.org, Sep 12 2017

Cc: d...@chromium.org
Owner: ----
Please don't assign ownership... I am not going to be able to work on this today, even though it is P1. CC is fine, but assigning ownership implies that I am actually owning this, which I am not able to do at the moment.
Project Member

Comment 5 by bugdroid1@chromium.org, Sep 12 2017

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/infradata/config/+/44ebb7b315b78f758e018bda425d49952193b6ab

commit 44ebb7b315b78f758e018bda425d49952193b6ab
Author: Dan Jacques <dnj@google.com>
Date: Tue Sep 12 14:23:03 2017

Comment 6 by d...@chromium.org, Sep 12 2017

I went ahead and looked into it anyway. This works fine on my system, so I think the problem here is that the "vpython" version used in LUCI is really old and hasn't picked up this CL: https://chromium.googlesource.com/infra/luci/luci-go/+/a2088a053d6c49ac398d0346fd6a58d8f04596f0

I rolled into canary: https://chrome-internal-review.googlesource.com/c/infradata/config/+/453192

If things are stable, we can propagate that into production. CL is prepared: https://chrome-internal-review.googlesource.com/453212

Comment 7 by d...@chromium.org, Sep 12 2017

Owner: d...@chromium.org
Status: Fixed (was: Untriaged)
Things looked good so I submitted the CL. Everything should be working now.
Project Member

Comment 8 by bugdroid1@chromium.org, Sep 12 2017

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/infradata/config/+/7ef4cbffbf1e8c3e017a5196907b90aa180230b7

commit 7ef4cbffbf1e8c3e017a5196907b90aa180230b7
Author: Dan Jacques <dnj@google.com>
Date: Tue Sep 12 15:09:46 2017

Comment 9 by athom@google.com, Sep 12 2017

Status: Verified (was: Fixed)
Thanks! Our CQ is up and running again!

Sign in to add a comment