New issue
Advanced search Search tips

Issue 648358 link

Starred by 6 users

Issue metadata

Status: Fixed
Owner:
Closed: Sep 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 0
Type: Bug



Sign in to add a comment

AppEngine SDK URL changed.

Project Member Reported by d...@chromium.org, Sep 19 2016

Issue description

The AppEngine SDK page layout and scheme changed. Update "infra"'s "get_appengine.py" script to accommodate the new layout.

It looks like the Google Storage bucket backing the downloads is now listable, so we can take advantage of that.
 

Comment 1 by d...@chromium.org, Sep 19 2016

Components: Infra
Labels: -Pri-3 Infra-Troopers Pri-0
Status: Started (was: stated)
 Issue 648340  has been merged into this issue.
Sheriff-o-matic data is slso stale, last updated 2016-09-19T18:47:48. No new bug data has come in since then.
This may also be affecting the CQ: There are no pending builds, and no commits have landed in the git repo since this began.

It's already at P0, though, so no further action.
This also causes all gnumbds to be down: https://build.chromium.org/p/chromium.infra.cron/console
Also closed V8 tree. Please reopen when this is resolved:
https://v8-status.appspot.com/
Cc: tandrii@chromium.org
 Issue 648380  has been merged into this issue.

Comment 8 by d...@chromium.org, Sep 19 2016

Bumped "refs/heads/deployed" on infra/infra to 376c28d7f6cb33eced1f11b394970e6baca68f30.
Project Member

Comment 9 by bugdroid1@chromium.org, Sep 19 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/infra.git/+/376c28d7f6cb33eced1f11b394970e6baca68f30

commit 376c28d7f6cb33eced1f11b394970e6baca68f30
Author: Dan Jacques <dnj@chromium.org>
Date: Mon Sep 19 20:41:30 2016

Update "get_appengine.py" to use GS.

AppEngine's Google Storage bucket is now readable and listable! Update
our "get_appengine.py" script to use that bucket to get AppEngine
bootstraps.

If the declared VERSION is not valid, it will scan the bucket to
identify other potential versions and load the latest.

BUG= chromium:648358 
TEST=local
  - Tested locally on Linux.

Change-Id: Iae879b5450ea43111beacdec2238efdedb95dbfe
Reviewed-on: https://chromium-review.googlesource.com/386314
Commit-Queue: Daniel Jacques <dnj@chromium.org>
Reviewed-by: Robbie Iannucci <iannucci@chromium.org>
Reviewed-by: Daniel Jacques <dnj@chromium.org>

[modify] https://crrev.com/376c28d7f6cb33eced1f11b394970e6baca68f30/bootstrap/get_appengine.py

I opened chromium and v8 trees.

Comment 11 by d...@chromium.org, Sep 19 2016

Status: Fixed (was: Started)
This appears to be fixed.
Thanks a lot! Looks fixed to me.
Project Member

Comment 13 by bugdroid1@chromium.org, Sep 19 2016

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/infra/infra_internal.git/+/db9912cbe0c28c31747245293db65ba2a15bf692

commit db9912cbe0c28c31747245293db65ba2a15bf692
Author: Dan Jacques <dnj@chromium.org>
Date: Mon Sep 19 22:22:16 2016

Labels: cit-pm

Sign in to add a comment