Closed
Bug 1154434
Opened 10 years ago
Closed 10 years ago
Runslave made talos read from wrong path
Categories
(Release Engineering :: General, defect)
Tracking
(firefox37 fixed, firefox38 fixed, firefox39 fixed, firefox40 fixed)
People
(Reporter: Callek, Assigned: Callek)
References
Details
Attachments
(2 files)
1.54 KB,
patch
|
coop
:
review+
|
Details | Diff | Splinter Review |
1.05 KB,
patch
|
massimo
:
review+
massimo
:
checked-in+
|
Details | Diff | Splinter Review |
[15:32:39] jmaher Callek|Builddduty: I believe we have a mismatch of document root and where we copy talos files to
[15:33:42] jmaher Callek|Builddduty: if the doc root is /builds/slave/talos-slave/talos-data/talos, we have an issue because the script copies the talos files to /builds/slave/talos-data/talos, and i don't see symlinks to map it
Attachment #8592423 -
Flags: review?(coop)
Updated•10 years ago
|
Attachment #8592423 -
Flags: review?(coop) → review+
Assignee | ||
Comment 1•10 years ago
|
||
https://hg.mozilla.org/build/puppet/rev/11db8d8daec1
https://hg.mozilla.org/build/puppet/rev/b110cf27147c
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 2•10 years ago
|
||
massimo, if this passes muster can you land, and ask Tomcat to apply it as a pin to at least inbound please?
Attachment #8592622 -
Flags: review?(mgervasini)
Assignee | ||
Comment 3•10 years ago
|
||
CC'ing Tomcat and RyanVM for a heads up.
Assignee | ||
Comment 4•10 years ago
|
||
for bug history:
[00:43:25] philor Callek: I've got good news, and bad news
[00:43:44] Callek good news the world is burning, bad news its only the part of the world above ground?
[00:44:14] philor good news is you fixed talos on OS X, bad news is you broke hell out of it on Linux
[00:44:41] Callek ooo ffs
[00:44:51] Callek log link?
[00:45:18] philor https://treeherder.mozilla.org/logviewer.html#?job_id=8849609&repo=mozilla-inbound
[00:52:58] Callek philor: if I patch mozharness are you willing/able to land to inbound at the least?
[00:53:06] Callek this is a mozharness bug afaict
[00:53:29] philor Callek: landing on inbound is utterly painless and without responsibility
[00:54:02] philor witness all the people who've landed on top of 160+ unstarred failures all evening
[00:54:17] Callek I just don't have a tree locally with which to bump th emozharness rev
[00:59:22] Callek philor: r=you for https://callek.pastebin.mozilla.org/8830118 ?
[00:59:24] philor well, it's busted at least as far down as aurora, I assume beta, so I'll just hide all Linux talos everywhere
[00:59:35] philor thanks, mozharness pinning!
[01:00:14] philor Callek: it would be utterly pointless to have me review it
Assignee | ||
Updated•10 years ago
|
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment 5•10 years ago
|
||
Comment on attachment 8592622 [details] [diff] [review]
[mozharness] fix for linux talos
lgtm
landed: https://hg.mozilla.org/build/mozharness/rev/92aedcef4127
Attachment #8592622 -
Flags: review?(mgervasini)
Attachment #8592622 -
Flags: review+
Attachment #8592622 -
Flags: checked-in+
Comment 6•10 years ago
|
||
So what kinds of cases does it address? Does it improve perma/intermittent/both failures?
Comment 7•10 years ago
|
||
osx was busted for a week- perma on 10.6 which this fixes. The fix caused perma fail on linux. Now to see if 10.10 is fixed as well.
Comment 8•10 years ago
|
||
status-firefox37:
--- → fixed
Comment 9•10 years ago
|
||
Status: REOPENED → RESOLVED
Closed: 10 years ago → 10 years ago
status-firefox40:
--- → fixed
Resolution: --- → FIXED
Comment 10•10 years ago
|
||
status-firefox39:
--- → fixed
Comment 11•10 years ago
|
||
status-firefox38:
--- → fixed
Updated•7 years ago
|
Component: General Automation → General
You need to log in
before you can comment on or make changes to this bug.
Description
•