Closed Bug 1321558 Opened 8 years ago Closed 8 years ago

Don't show "public/cot/*" in treeherder "Artifact Uploaded" view

Categories

(Taskcluster :: Services, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: Callek, Unassigned)

Details

For taskcluster jobs, the artifact-uploaded entries in treeherder, having things from "public/cot/*" is unlikely to be necessary/wanted. I'm n-i to :aki to confirm he agrees with me here. (You can see an example on https://treeherder.mozilla.org/#/jobs?repo=date&selectedJob=140032 with the "Job Details")
Flags: needinfo?(aki)
The audit log may be useful, especially if the task is broken due to cot issues. Otherwise I agree.
Flags: needinfo?(aki)
These entries come from what taskcluster submits with the job (they're not extracted from the log), so I think they should be filtered out on the taskcluster side rather than after they've already been submitted to Treeherder?
Component: Treeherder → Integration
Product: Tree Management → Taskcluster
Version: --- → unspecified
They should be available in the task artifacts list, of course. The problem arises that the `path` is not shown in treeherder itself, so we get 'target.apk' and 'target.apk (1)' in the treeherder view, one of which is signed the other not, and no way to know which is the "artifact CoT grabbed from dependant tasks" and which is "artifact this task generated"
Yep, we're just providing a list of artifacts. I suspect that the thing translating those into links on TH is taskcluster-treeherder. Maybe it should just provide artifacts to TH with their full names? Or can TH not handle '/'? Note: if you want to get fancy, we could add task.extra.treeherder.showArtifacts = [..] with a list of regexes for artifacts that should be shown in treeherder.
the PR is fixed, marking THIS bug wontfix since it wasn't changed in any code relevant to this component.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WONTFIX
Component: Integration → Services
You need to log in before you can comment on or make changes to this bug.