New issue
Advanced search Search tips

Issue 782569 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Nov 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug



Sign in to add a comment

Fix 404 error in crosvideo.appspot.com.

Project Member Reported by hiroh@chromium.org, Nov 8 2017

Issue description

vsuley@ is working around the cite for crbug.com/721630.
crosvideo.appspot.com is broken down now somehow (see  crbug.com/782509 ).
This causes video_ChromeHWDecodeUsed mse tests failure.
As a temporal solution to fix them, crosvideo2.appspot.com is used now (crosreview.com/757890).

Please do as follows.
1. fix crosvideo.appspot.com and it can be accessed.
2. submit the CL to replace crosvideo2.appspot.com with crosvideo.appspot.com in control.*.mse. Here, please be careful because, if crosvideo.appspot.com is still broken down, CQ HW test is failed.

 
Apologies for the interruption! crosvideo.appspot.com is back online. I think we should replace it permanently with crosvideo2.appspot.com. I'll check if any other code needs updating. 

Comment 2 Deleted

Status: Assigned (was: WontFix)
Mistakenly closed in comment #2. Reopening. 

Comment 4 by hiroh@chromium.org, Nov 10 2017

Looks like crosvideo.appspot.com works well now.
May I please do step 2.
Thanks

Comment 5 by vsu...@chromium.org, Nov 14 2017

Changelist to revert back to crosvideo.appspot.com is in the CQ.
Project Member

Comment 6 by bugdroid1@chromium.org, Nov 15 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/18c6c70b2727ed93165e50964b7684e60db4fc89

commit 18c6c70b2727ed93165e50964b7684e60db4fc89
Author: Vinayak Suley <vsuley@chromium.org>
Date: Wed Nov 15 08:39:54 2017

Directing URLs to go back to Crosvideo.appspot.com

I had temporarily broken crosvideo so we had to redirect to
crosvideo2, moving back to crosvideo since it is fixed. This is
not a simple 'revert' of the original changelist because it had
some other changes also that we need to keep around.

BUG= chromium:782569 
TEST=Tested locally

Change-Id: Ic492db5a2dabbac423f32e7ff789e26aa3f848c7
Reviewed-on: https://chromium-review.googlesource.com/768250
Commit-Ready: Hirokazu Honda <hiroh@chromium.org>
Tested-by: Hirokazu Honda <hiroh@chromium.org>
Tested-by: Vinayak Suley <vsuley@chromium.org>
Reviewed-by: Hirokazu Honda <hiroh@chromium.org>

[modify] https://crrev.com/18c6c70b2727ed93165e50964b7684e60db4fc89/client/site_tests/video_ChromeHWDecodeUsed/control.vp9.mse
[modify] https://crrev.com/18c6c70b2727ed93165e50964b7684e60db4fc89/client/site_tests/video_ChromeHWDecodeUsed/control.vp8.mse
[modify] https://crrev.com/18c6c70b2727ed93165e50964b7684e60db4fc89/client/site_tests/video_ChromeHWDecodeUsed/control.h264.mse

Comment 7 by vsu...@chromium.org, Nov 15 2017

Status: Fixed (was: Assigned)

Sign in to add a comment