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

Issue 874763 link

Starred by 1 user

Issue metadata

Status: Untriaged
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Move telemetry/internal/bin out of telemetry python package

Project Member Reported by perezju@chromium.org, Aug 16

Issue description

The folder:

  //telemetry/telemetry/internal/bin

Does not include any checked in files but is used by the DependencyManager to download binary files into it.

So the folder has to be excluded from:
https://cs.chromium.org/chromium/src/tools/perf/chrome_telemetry_build/BUILD.gn?rcl=9ea9b5ee08b766efcf2ea8188f2444b52db93696&l=190

to prevent from copying files in it to isolate outputs.

This makes it awkward to maintain, since adding new files or folders to Telemetry (in catapult) require changes to that list before they can roll into chromium.

Instead, we should just move that path to:

  //telemetry/bin

Which would allow all of:

  //telemetry/telemetry/

to be included in the BUILD.gn rules.
 
Cc: zmo@chromium.org kbr@chromium.org
Components: Tests>Telemetry
Cc: -nedngu...@google.com nednguyen@chromium.org

Comment 3 by benhenry@google.com, Jan 16 (6 days ago)

Components: Test>Telemetry

Comment 4 by benhenry@google.com, Jan 16 (6 days ago)

Components: -Tests>Telemetry

Sign in to add a comment