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

Issue 707995 link

Starred by 3 users

Issue metadata

Status: Started
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug


Sign in to add a comment

Launch Chromium Status NG

Project Member Reported by seanmccullough@chromium.org, Apr 3 2017

Issue description

The current deployment model for the *-status.appspot.com apps is to iterate through each hostname and do a separate deploy. This makes rolling back a pain. It's also wasteful to run so many different appengine projects that share the exact same code.

We should move all of these hostnames to be served by the same GAE app instance.

There are a few ways to do this:
- write one-time reverse proxy code, and push to each of the current separate gae apps. Proxie fwds to shared instances. 
  - pro: quick fix
  - con: still have multiple gae projects to manage.
- write one-time 302 placeholder code, and push to each of the current separate gae apps. 302 redirects to shared instances.
  - pro: quick fix. 
  - con: still have multiple gae projects to manage, some scripts may not handle 302s cleanly (low risk, but nonzero).
- figure out how to serve multiple hostnames from single gae project. 
  - pro: single gae app.
  - con: still a mess to manage with different hostnames.
- migrate all clients to point to single gae hostname, preferably something under sheriff-o-matic.appspot.com/... so that all *-status 'apps' share the same release management and monitoring bits. 
  - pros: cleanest solution. can delete separate apps from src and cloud. centralized auth etc.
  - cons: potentially lots of clean-up work for client code to update hostnames.


 
Cc: -zhangtiff@chromium.org
Owner: zhangtiff@chromium.org
Status: Assigned (was: Available)
Design doc for the last option (which is the one we should do): https://docs.google.com/document/d/1BAhMcCH3_U9ZOPOg_FrG2YIkm1bD-ZNBqKx6Q9lhDmI/edit#

Over to you, zhangtiff!
Cc: dsansome@chromium.org
Project Member

Comment 3 by bugdroid1@chromium.org, Jun 28 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/infra/+/780a9efaf5d75aa1d991e4adbf06d42debf4f349

commit 780a9efaf5d75aa1d991e4adbf06d42debf4f349
Author: Tiff Zhang <zhangtiff@google.com>
Date: Wed Jun 28 20:02:34 2017

Chromium-status: Add Google Analytics tracking.

Bug:707995
Change-Id: Ia14b3d2887d24f2438b51454c12c0f798770a8b4
Reviewed-on: https://chromium-review.googlesource.com/552903
Reviewed-by: Sean McCullough <seanmccullough@chromium.org>
Commit-Queue: Tiffany Zhang <zhangtiff@chromium.org>

[modify] https://crrev.com/780a9efaf5d75aa1d991e4adbf06d42debf4f349/appengine/chromium_status/templates/base.html

Project Member

Comment 4 by bugdroid1@chromium.org, Jun 29 2017

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

commit a12a844a9fed6821094ea6eecf2c2ececfe13ce5
Author: Tiff Zhang <zhangtiff@google.com>
Date: Thu Jun 29 21:17:31 2017

SoM: Add page to view all tree statuses.

Bug:707995
Change-Id: Icf0bd26e97ca2bb63bdf3dcea746c932aee3c475
Reviewed-on: https://chromium-review.googlesource.com/549075
Reviewed-by: Sean McCullough <seanmccullough@chromium.org>
Commit-Queue: Tiffany Zhang <zhangtiff@chromium.org>

[modify] https://crrev.com/a12a844a9fed6821094ea6eecf2c2ececfe13ce5/go/src/infra/appengine/sheriff-o-matic/frontend/elements/som-app/som-app.html
[add] https://crrev.com/a12a844a9fed6821094ea6eecf2c2ececfe13ce5/go/src/infra/appengine/sheriff-o-matic/frontend/elements/tree-status/som-tree-status-styles.html
[add] https://crrev.com/a12a844a9fed6821094ea6eecf2c2ececfe13ce5/go/src/infra/appengine/sheriff-o-matic/frontend/elements/tree-status/som-all-status/som-all-status.js
[modify] https://crrev.com/a12a844a9fed6821094ea6eecf2c2ececfe13ce5/go/src/infra/appengine/sheriff-o-matic/frontend/elements/som-app/som-app.js
[add] https://crrev.com/a12a844a9fed6821094ea6eecf2c2ececfe13ce5/go/src/infra/appengine/sheriff-o-matic/frontend/elements/tree-status/som-all-status/som-all-status.html
[rename] https://crrev.com/a12a844a9fed6821094ea6eecf2c2ececfe13ce5/go/src/infra/appengine/sheriff-o-matic/frontend/test/tree-status/som-tree-status-test.html
[modify] https://crrev.com/a12a844a9fed6821094ea6eecf2c2ececfe13ce5/go/src/infra/appengine/sheriff-o-matic/frontend/elements/som-alert-view/som-alert-view.html
[add] https://crrev.com/a12a844a9fed6821094ea6eecf2c2ececfe13ce5/go/src/infra/appengine/sheriff-o-matic/frontend/elements/tree-status/som-tree-status-behavior.html
[rename] https://crrev.com/a12a844a9fed6821094ea6eecf2c2ececfe13ce5/go/src/infra/appengine/sheriff-o-matic/frontend/elements/tree-status/som-tree-status/som-tree-status.js
[rename] https://crrev.com/a12a844a9fed6821094ea6eecf2c2ececfe13ce5/go/src/infra/appengine/sheriff-o-matic/frontend/elements/tree-status/som-tree-status/som-tree-status.html
[modify] https://crrev.com/a12a844a9fed6821094ea6eecf2c2ececfe13ce5/go/src/infra/appengine/sheriff-o-matic/frontend/test/index.html

Labels: -Pri-3 Pri-2
Status: Started (was: Assigned)
Summary: Merge Chromium_Status into Sheriff-o-Matic (was: chromium_status app deployment is a mess)
Blocking: 760292
Blocking: -760292
Blockedon: 760292
Blockedon: 760295
Blockedon: 760297
Components: Infra>Sheriffing>SheriffOMatic
Labels: Milestone-TreeStatus
Blockedon: 760306
Blockedon: 760305
Blockedon: 664655
Summary: Launch Chromium Status NG (was: Merge Chromium_Status into Sheriff-o-Matic )
Blockedon: 21801
Note: This plan has pivoted to making a new verison of Chromium Status instead of merging Chromium-Status into Sheriff-o-Matic. Design doc here: https://docs.google.com/a/google.com/document/d/1kF7gsnDO8A7EIEWu9SbjrtZyFRIqNPkYpoS9Jz8GSis/edit?usp=sharing 
Project Member

Comment 18 by bugdroid1@chromium.org, Sep 26 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/infra/+/3b74a5d87237be10e7de000feb0cbc6021a5a3bb

commit 3b74a5d87237be10e7de000feb0cbc6021a5a3bb
Author: Tiff Zhang <zhangtiff@google.com>
Date: Tue Sep 26 13:25:18 2017

Chromium-Status: Remove unused endpoints.

Bug:707995
Change-Id: I097132012ad55607d65717a8ef797818626941cd
Reviewed-on: https://chromium-review.googlesource.com/679736
Reviewed-by: Marc-Antoine Ruel <maruel@chromium.org>
Commit-Queue: Marc-Antoine Ruel <maruel@chromium.org>

[modify] https://crrev.com/3b74a5d87237be10e7de000feb0cbc6021a5a3bb/appengine/chromium_status/appengine_module/chromium_status/main.py
[modify] https://crrev.com/3b74a5d87237be10e7de000feb0cbc6021a5a3bb/appengine/chromium_status/templates/base.html

Project Member

Comment 19 by bugdroid1@chromium.org, Oct 15 2017

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

commit b320fc7f3a6acfb669ac511ec0fcb8cea0098d02
Author: Tiff Zhang <zhangtiff@google.com>
Date: Fri Oct 13 18:38:40 2017

Tree-status: Create minimal Tree-Status app.


Bug:707995
Change-Id: I57650858d58ce737d5b29af359ceb854543b2063
Reviewed-on: https://chromium-review.googlesource.com/674324
Commit-Queue: Tiffany Zhang <zhangtiff@chromium.org>
Reviewed-by: Sean McCullough <seanmccullough@chromium.org>

[add] https://crrev.com/b320fc7f3a6acfb669ac511ec0fcb8cea0098d02/go/src/infra/appengine/tree-status/README.md
[add] https://crrev.com/b320fc7f3a6acfb669ac511ec0fcb8cea0098d02/go/src/infra/appengine/tree-status/gae.py
[add] https://crrev.com/b320fc7f3a6acfb669ac511ec0fcb8cea0098d02/go/src/infra/appengine/tree-status/elements/ts-tree-row/ts-tree-row.js
[add] https://crrev.com/b320fc7f3a6acfb669ac511ec0fcb8cea0098d02/go/src/infra/appengine/tree-status/images/favicon.ico
[add] https://crrev.com/b320fc7f3a6acfb669ac511ec0fcb8cea0098d02/go/src/infra/appengine/tree-status/elements/ts-tree-view/ts-tree-view.html
[add] https://crrev.com/b320fc7f3a6acfb669ac511ec0fcb8cea0098d02/go/src/infra/appengine/tree-status/elements/tree-status-styles.html
[add] https://crrev.com/b320fc7f3a6acfb669ac511ec0fcb8cea0098d02/go/src/infra/appengine/tree-status/.gitignore
[add] https://crrev.com/b320fc7f3a6acfb669ac511ec0fcb8cea0098d02/go/src/infra/appengine/tree-status/elements/ts-app/ts-app.html
[add] https://crrev.com/b320fc7f3a6acfb669ac511ec0fcb8cea0098d02/go/src/infra/appengine/tree-status/package.json
[add] https://crrev.com/b320fc7f3a6acfb669ac511ec0fcb8cea0098d02/go/src/infra/appengine/tree-status/elements/ts-app/ts-app.js
[add] https://crrev.com/b320fc7f3a6acfb669ac511ec0fcb8cea0098d02/go/src/infra/appengine/tree-status/elements/ts-tree-view/ts-tree-view.js
[add] https://crrev.com/b320fc7f3a6acfb669ac511ec0fcb8cea0098d02/go/src/infra/appengine/tree-status/wct.conf.json
[add] https://crrev.com/b320fc7f3a6acfb669ac511ec0fcb8cea0098d02/go/src/infra/appengine/tree-status/images/chromium.png
[add] https://crrev.com/b320fc7f3a6acfb669ac511ec0fcb8cea0098d02/go/src/infra/appengine/tree-status/OWNERS
[add] https://crrev.com/b320fc7f3a6acfb669ac511ec0fcb8cea0098d02/go/src/infra/appengine/tree-status/Makefile
[add] https://crrev.com/b320fc7f3a6acfb669ac511ec0fcb8cea0098d02/go/src/infra/appengine/tree-status/bower.json
[add] https://crrev.com/b320fc7f3a6acfb669ac511ec0fcb8cea0098d02/go/src/infra/appengine/tree-status/main.go
[add] https://crrev.com/b320fc7f3a6acfb669ac511ec0fcb8cea0098d02/go/src/infra/appengine/tree-status/elements/ts-tree-row/ts-tree-row.html
[add] https://crrev.com/b320fc7f3a6acfb669ac511ec0fcb8cea0098d02/go/src/infra/appengine/tree-status/tree-status.infra_testing
[add] https://crrev.com/b320fc7f3a6acfb669ac511ec0fcb8cea0098d02/go/src/infra/appengine/tree-status/app.yaml
[add] https://crrev.com/b320fc7f3a6acfb669ac511ec0fcb8cea0098d02/go/src/infra/appengine/tree-status/index.html

Components: -Infra>Sheriffing>SheriffOMatic

Sign in to add a comment