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)
Taskcluster
Services
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")
Reporter | ||
Updated•8 years ago
|
Flags: needinfo?(aki)
Comment 1•8 years ago
|
||
The audit log may be useful, especially if the task is broken due to cot issues. Otherwise I agree.
Flags: needinfo?(aki)
Comment 2•8 years ago
|
||
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
Reporter | ||
Comment 3•8 years ago
|
||
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"
Comment 4•8 years ago
|
||
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.
Comment 5•8 years ago
|
||
We're going to just upload the log https://github.com/mozilla-releng/scriptworker/issues/35 .
Reporter | ||
Comment 6•8 years ago
|
||
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
Assignee | ||
Updated•6 years ago
|
Component: Integration → Services
You need to log in
before you can comment on or make changes to this bug.
Description
•