New issue
Advanced search Search tips

Issue 593426 link

Starred by 1 user

Issue metadata

Status: Archived
Owner: ----
Closed: May 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug

Blocking:
issue 408424



Sign in to add a comment

Unify googleapiclient and oauth2client between AppEngine and CLI tools

Project Member Reported by sergeybe...@chromium.org, Mar 9 2016

Issue description

There is hope that the latest AppEngine releases might support more modern versions of these libraries. In particular, this would allow us to have single copies of these libraries in repos that vendor ts_mon, such as luci-py.
 

Comment 1 by benhenry@google.com, Apr 27 2016

Components: Infra>Monitoring
Labels: -Infra-Monitoring

Comment 2 by mar...@chromium.org, Jun 21 2016

Can you comment on the incompatibilities?
I don't remember the exact details right now (will need to investigate again), but basically AppEngine-side code (e.g. in dev_appserver) referred to a specific module attribute which is no longer present in the new versions of googleapiclient or oauth2client.

You'll get a crash in a handler if you try to update any of these libraries and run an app in dev_appserver. The same crash also happens in the cloud.

This was a few months ago, I didn't have time to look into it recently. Things might have changed - they recently unified the default credentials workflow, and this bug might have been fixed.
Labels: Type-Bug
Cc: sergeybe...@chromium.org
Owner: ----
Status: Available (was: Assigned)
Not really working on this bug right now, making available.
Cc: vadimsh@chromium.org
FTR, the CL doing the library update is https://codereview.chromium.org/1768993002.

Comment 7 by efoo@chromium.org, May 1 2017

Components: -Infra>Monitoring Infra>SDK
Moving to Infra>SDK component since this is not a Infra>Monitoring specific issue. 
Project Member

Comment 8 by sheriffbot@chromium.org, May 2 2018

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Status: Archived (was: Untriaged)
We should be using vpython everywhere these days.

Sign in to add a comment