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

Issue 692278 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Last visit > 30 days ago
Closed: Feb 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

Both telemetry informational pfq bots failed for telemetry_UnitTests.perf test

Project Member Reported by jen...@chromium.org, Feb 14 2017

Issue description

Cc: bccheng@chromium.org achuith@chromium.org nedngu...@google.com
Components: -Infra>Platform>Buildbot Tests>Telemetry
Labels: -Pri-3 OS-Chrome Pri-2
Owner: rnep...@chromium.org
Status: Assigned (was: Untriaged)
Looks like testFetchWPRs failed:

02/15 01:47:51.549 INFO |run_chromeos_tests:0052| [266/318] fetch_benchmark_deps_unittest.FetchBenchmarkDepsUnittest.testFetchWPRs failed unexpectedly 1.3164s:
02/15 01:47:51.549 INFO |run_chromeos_tests:0052|   Traceback (most recent call last):
02/15 01:47:51.550 INFO |run_chromeos_tests:0052|     File "/usr/local/telemetry/src/tools/perf/fetch_benchmark_deps_unittest.py", line 63, in testFetchWPRs
02/15 01:47:51.550 INFO |run_chromeos_tests:0052|       self._RunFetchBenchmarkDepsTest('smoothness.top_25_smooth')
02/15 01:47:51.551 INFO |run_chromeos_tests:0052|     File "/usr/local/telemetry/src/tools/perf/fetch_benchmark_deps_unittest.py", line 53, in _RunFetchBenchmarkDepsTest
02/15 01:47:51.551 INFO |run_chromeos_tests:0052|       fetch_benchmark_deps.main(output)
02/15 01:47:51.552 INFO |run_chromeos_tests:0052|     File "/usr/local/telemetry/src/tools/perf/fetch_benchmark_deps.py", line 78, in main
02/15 01:47:51.552 INFO |run_chromeos_tests:0052|       _FetchDependenciesIfNeeded(story_set)
02/15 01:47:51.552 INFO |run_chromeos_tests:0052|     File "/usr/local/telemetry/src/tools/perf/fetch_benchmark_deps.py", line 31, in _FetchDependenciesIfNeeded
02/15 01:47:51.553 INFO |run_chromeos_tests:0052|       story_set.wpr_archive_info.DownloadArchivesIfNeeded()
02/15 01:47:51.553 INFO |run_chromeos_tests:0052|     File "/usr/local/telemetry/src/third_party/catapult/telemetry/telemetry/wpr/archive_info2.py", line 110, in DownloadArchivesIfNeeded
02/15 01:47:51.553 INFO |run_chromeos_tests:0052|       download_if_needed(archive_path)
02/15 01:47:51.554 INFO |run_chromeos_tests:0052|     File "/usr/local/telemetry/src/third_party/catapult/telemetry/telemetry/wpr/archive_info2.py", line 89, in download_if_needed
02/15 01:47:51.554 INFO |run_chromeos_tests:0052|       cloud_storage.GetIfChanged(path, self._bucket)
02/15 01:47:51.555 INFO |run_chromeos_tests:0052|     File "/usr/local/telemetry/src/third_party/catapult/common/py_utils/py_utils/cloud_storage.py", line 415, in GetIfChanged
02/15 01:47:51.555 INFO |run_chromeos_tests:0052|       _GetLocked(bucket, expected_hash, file_path)
02/15 01:47:51.556 INFO |run_chromeos_tests:0052|     File "/usr/local/telemetry/src/third_party/catapult/common/py_utils/py_utils/cloud_storage.py", line 341, in _GetLocked
02/15 01:47:51.556 INFO |run_chromeos_tests:0052|       _RunCommand(['cp', url, partial_download_path.name])
02/15 01:47:51.556 INFO |run_chromeos_tests:0052|     File "/usr/local/telemetry/src/third_party/catapult/common/py_utils/py_utils/cloud_storage.py", line 164, in _RunCommand
02/15 01:47:51.557 INFO |run_chromeos_tests:0052|       raise CloudStorageError(stderr)
02/15 01:47:51.558 INFO |run_chromeos_tests:0052|   CloudStorageError: ServiceException: 401 Anonymous users does not have storage.objects.get access to object chrome-partner-telemetry/14033b38b9be98cbcafa9cbb49636a4b93256f46.


First failing build:
https://uberchromegw.corp.google.com/i/chromiumos.chromium/builders/amd64-generic-telemetry/builds/10792

This seems to be the CL causing the failure:
https://codereview.chromium.org/2688903002

This is likely because of changes I made to archive_info to support platform specific WPR recordings. I commented on the CL how to fix the test.
This should be fixed now.
https://build.chromium.org/p/chromiumos.chromium/builders/x86-generic-telemetry/builds/11739

I'll let it cycle green a few more times before closing out.

Comment 6 by jen...@chromium.org, Feb 15 2017

Cool, thanks for the fix!
Status: Fixed (was: Assigned)
Closing as fixed. Let me know if it starts failing again.

Comment 9 by dchan@google.com, Apr 17 2017

Labels: VerifyIn-59

Comment 10 by dchan@google.com, May 30 2017

Labels: VerifyIn-60
Labels: VerifyIn-61
Status: Verified (was: Fixed)
Closing. Please reopen it if its not fixed. Thanks!

Sign in to add a comment