Skip to content
This repository has been archived by the owner on Dec 3, 2019. It is now read-only.

Commit

Permalink
[Telemetry] Disable DISABLE_CLOUD_STORAGE_IO before each test process…
Browse files Browse the repository at this point in the history
… run

BUG=catapult:#2192
TBR=sullivan@chromium.org, aiolos@chromium.org

Review URL: https://codereview.chromium.org/1843963002
  • Loading branch information
nedn authored and Commit bot committed Mar 30, 2016
1 parent 1cd7f08 commit d114748
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions telemetry/telemetry/testing/run_tests.py
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@
# Use of this source code is governed by a BSD-style license that can be
# found in the LICENSE file.

import os
import logging
import sys

Expand All @@ -19,7 +18,6 @@
from telemetry.testing import browser_test_case
from telemetry.testing import options_for_unittests

from catapult_base import cloud_storage
from catapult_base import xvfb

import typ
Expand Down Expand Up @@ -236,7 +234,10 @@ def _SetUpProcess(child, context): # pylint: disable=unused-argument
ps_util.EnableListingStrayProcessesUponExitHook()
# Make sure that we don't invokes cloud storage I/Os when we run the tests in
# parallel.
os.environ[cloud_storage.DISABLE_CLOUD_STORAGE_IO] = '1'
# TODO(nednguyen): Enabled this once telemetry tests in Chromium is updated
# to prefetch files.
# (https://github.com/catapult-project/catapult/issues/2192)
# os.environ[cloud_storage.DISABLE_CLOUD_STORAGE_IO] = '1'
if binary_manager.NeedsInit():
# Typ doesn't keep the DependencyManager initialization in the child
# processes.
Expand Down

0 comments on commit d114748

Please sign in to comment.