Issue metadata
Sign in to add a comment
|
Observing regression for video_YouTubeHTML5 test on ToT |
||||||||||||||||||||||
Issue descriptionhttps://wmatrix.googleplex.com/unfiltered?hide_missing=True&tests=video_YouTubeHTML5&days_back=30 Observing consistent failures from R58-9309.0.0 on all devices. Error: Video failed to load. Player expected status: Playing and current status: Unstarted.
,
Mar 3 2017
Actually, the video used in the test seems to not be available anymore. Rohit, would you be able to help triage please?
,
Mar 3 2017
Avinash, can you check if this is a YT api regression or something changed in YouTube on CrOS?
,
Mar 3 2017
Video using in testing is showing as "This video is private". Can we use this video ID(https://www.youtube.com/watch?v=Rykmwn0SMWU) for future testing. Thanks!
,
Mar 3 2017
Yes, if it passes the test :-)
,
Mar 4 2017
,
Mar 4 2017
,
Mar 4 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/third_party/autotest/+/6fe94f3bbb775f4fdb55a3f391ab92961f574985 commit 6fe94f3bbb775f4fdb55a3f391ab92961f574985 Author: avinash kodipelli <avkodipelli@chromium.org> Date: Sat Mar 04 04:56:45 2017 Changing existing youtube video Current video in test is failing because of permission issue, So we are adding new video for testing BUG= chromium:698101 TEST=manual Change-Id: I6a50d1590f5e4d0c349b2e4f9e35451997c9df38 Reviewed-on: https://chromium-review.googlesource.com/449881 Commit-Ready: Avinash Kodipelli <avkodipelli@chromium.org> Tested-by: Avinash Kodipelli <avkodipelli@chromium.org> Reviewed-by: Vinayak Suley <vsuley@chromium.org> [modify] https://crrev.com/6fe94f3bbb775f4fdb55a3f391ab92961f574985/client/site_tests/video_YouTubeHTML5/youtube5.html
,
Mar 6 2017
Can we merge this change into M-58 and M-57 for better youtube testing coverage in the lab.
,
Mar 7 2017
This bug requires manual review: Only 6 days from stable, we might already have a stable candidate build Please contact the milestone owner if you have questions. Owners: amineer@(clank), cmasso@(bling), ketakid@(cros), govind@(desktop) For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Mar 7 2017
,
Mar 7 2017
Your change meets the bar and is auto-approved for M58. Please go ahead and merge the CL to branch 3029 manually. Please contact milestone owner if you have questions. Owners: amineer@(clank), cmasso@(bling), bhthompson@(cros), govind@(desktop) For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Mar 7 2017
Please merge onto M58 before requesting a merge to M57.
,
Mar 8 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/third_party/autotest/+/2fbc37b50c91a9ebc2ff3192203912f84a7bdfec commit 2fbc37b50c91a9ebc2ff3192203912f84a7bdfec Author: avinash kodipelli <avkodipelli@chromium.org> Date: Wed Mar 08 01:01:07 2017 Changing existing youtube video Current video in test is failing because of permission issue, So we are adding new video for testing BUG= chromium:698101 TEST=manual Change-Id: I6a50d1590f5e4d0c349b2e4f9e35451997c9df38 Reviewed-on: https://chromium-review.googlesource.com/449881 Commit-Ready: Avinash Kodipelli <avkodipelli@chromium.org> Tested-by: Avinash Kodipelli <avkodipelli@chromium.org> Reviewed-by: Vinayak Suley <vsuley@chromium.org> (cherry picked from commit 6fe94f3bbb775f4fdb55a3f391ab92961f574985) Reviewed-on: https://chromium-review.googlesource.com/451597 Commit-Queue: Vinayak Suley <vsuley@chromium.org> Trybot-Ready: Avinash Kodipelli <avkodipelli@chromium.org> [modify] https://crrev.com/2fbc37b50c91a9ebc2ff3192203912f84a7bdfec/client/site_tests/video_YouTubeHTML5/youtube5.html
,
Mar 8 2017
Merge fixed this test in 58: https://wmatrix.googleplex.com/unfiltered?hide_missing=True&releases=58&tests=video_YouTubeHTML5. Fix is very low risk, can we merge back to 57 as well?
,
Mar 8 2017
This bug requires manual review: Only 5 days from stable, we might already have a stable candidate build Please contact the milestone owner if you have questions. Owners: amineer@(clank), cmasso@(bling), ketakid@(cros), govind@(desktop) For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Mar 10 2017
Removing release block stable label as this is a test only issue.
,
Mar 13 2017
This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible! If all merges have been completed, please remove any remaining Merge-Approved labels from this issue. Thanks for your time! To disable nags, add the Disable-Nags label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Mar 13 2017
Approving merge to M57 Chrome OS.
,
Mar 13 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/third_party/autotest/+/92f10c20226e973a74ec6d73e8b66793abc3229e commit 92f10c20226e973a74ec6d73e8b66793abc3229e Author: avinash kodipelli <avkodipelli@chromium.org> Date: Mon Mar 13 18:20:47 2017 Changing existing youtube video Current video in test is failing because of permission issue, So we are adding new video for testing BUG= chromium:698101 TEST=manual Change-Id: I6a50d1590f5e4d0c349b2e4f9e35451997c9df38 Reviewed-on: https://chromium-review.googlesource.com/449881 Commit-Ready: Avinash Kodipelli <avkodipelli@chromium.org> Tested-by: Avinash Kodipelli <avkodipelli@chromium.org> Reviewed-by: Vinayak Suley <vsuley@chromium.org> (cherry picked from commit 6fe94f3bbb775f4fdb55a3f391ab92961f574985) Reviewed-on: https://chromium-review.googlesource.com/453800 Commit-Queue: Avinash Kodipelli <avkodipelli@chromium.org> Trybot-Ready: Avinash Kodipelli <avkodipelli@chromium.org> [modify] https://crrev.com/92f10c20226e973a74ec6d73e8b66793abc3229e/client/site_tests/video_YouTubeHTML5/youtube5.html
,
Mar 13 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/third_party/autotest/+/92f10c20226e973a74ec6d73e8b66793abc3229e commit 92f10c20226e973a74ec6d73e8b66793abc3229e Author: avinash kodipelli <avkodipelli@chromium.org> Date: Mon Mar 13 18:20:47 2017 Changing existing youtube video Current video in test is failing because of permission issue, So we are adding new video for testing BUG= chromium:698101 TEST=manual Change-Id: I6a50d1590f5e4d0c349b2e4f9e35451997c9df38 Reviewed-on: https://chromium-review.googlesource.com/449881 Commit-Ready: Avinash Kodipelli <avkodipelli@chromium.org> Tested-by: Avinash Kodipelli <avkodipelli@chromium.org> Reviewed-by: Vinayak Suley <vsuley@chromium.org> (cherry picked from commit 6fe94f3bbb775f4fdb55a3f391ab92961f574985) Reviewed-on: https://chromium-review.googlesource.com/453800 Commit-Queue: Avinash Kodipelli <avkodipelli@chromium.org> Trybot-Ready: Avinash Kodipelli <avkodipelli@chromium.org> [modify] https://crrev.com/92f10c20226e973a74ec6d73e8b66793abc3229e/client/site_tests/video_YouTubeHTML5/youtube5.html
,
Mar 13 2017
,
Mar 17 2017
This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible! If all merges have been completed, please remove any remaining Merge-Approved labels from this issue. Thanks for your time! To disable nags, add the Disable-Nags label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Mar 17 2017
Verified using wmatrix results |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by posciak@chromium.org
, Mar 3 2017Labels: videoshortlist