Decommission the ionmonkey tree

RESOLVED FIXED

Status

Release Engineering
General Automation
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: philor, Assigned: Callek)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [capacity])

Attachments

(3 attachments, 1 obsolete attachment)

(Reporter)

Description

3 years ago
Too bad it's still the 2nd of April, not the 23rd, when I could say "the Ionmonkey tree has had exactly one push in the last year." Still, it' been nearly a year since it was used, a year during which Try became a perfectly useful place to push spidermonkey experiments, since it now does all the shell builds and hazard builds you could want (and I don't really know whether or not Ionmonkey actually does a current full set).

Last week, we switched the default for nightlies from building on the last good rev, which also switched them from looking at whether or not a nightly had been built on the last rev, to just building on the tip.

Since then, we've built six pointless nightlies on the Ionmonkey tree, and triggered six full sets of pointless mostly-unrunnable tests on it. The fact that we're out $60 for that is minimal, compared to the fact that we're out the use of those slaves during a critical time of the day.

Best choice: decide that Ionmonkey has done its job, and that now Try and the occasional use of a twig is a better use of our resources, and retire it.

Okay choice: decide that what's really a barely-used twig doesn't actually need nightlies, and shut them off.

Really not so great a choice: set enable_nightly_lastgood to True on it, so that it won't build another nightly on a rev that has already had one (and also won't build a nightly at all if any platform is permared, or just doesn't happen to successfully build on the tip push).

Your call, bhackett, reassign to nobody@ once you've picked one.

Comment 1

3 years ago
(In reply to Phil Ringnalda (:philor) from comment #0)
> Last week, we switched the default for nightlies from building on the last
> good rev, which also switched them from looking at whether or not a nightly
> had been built on the last rev, to just building on the tip.

(I was about to file a bug for this but then found bug 968075, for anyone else who was thinking of doing the same.)

Comment 2

3 years ago
(In reply to Ed Morley [:edmorley UTC+0] from comment #1)
> (I was about to file a bug for this but then found bug 968075, for anyone
> else who was thinking of doing the same.)

Now bug 991707
It's fine to retire the Ionmonkey branch.

Comment 4

3 years ago
Great, thank you :-)
Assignee: bhackett1024 → nobody
Summary: Either remove the Ionmonkey tree, or shut off nightlies on it, or make it use lastgood → Decommission the ionmonkey tree
(Assignee)

Comment 5

3 years ago
Created attachment 8401360 [details] [diff] [review]
[configs] v1
Assignee: nobody → bugspam.Callek
Status: NEW → ASSIGNED
Attachment #8401360 - Flags: review?(catlee)
(Assignee)

Comment 6

3 years ago
Created attachment 8401361 [details] [diff] [review]
[configs] v1.1

somehow previous patch lost changes to 1 file
Attachment #8401360 - Attachment is obsolete: true
Attachment #8401360 - Flags: review?(catlee)
Attachment #8401361 - Flags: review?(catlee)
(Assignee)

Comment 7

3 years ago
Created attachment 8401369 [details] [diff] [review]
[custom] v1
Attachment #8401369 - Flags: review?(catlee)
(Assignee)

Comment 8

3 years ago
Created attachment 8401372 [details] [diff] [review]
[mozharness] v1
Attachment #8401372 - Flags: review?(catlee)

Updated

3 years ago
Attachment #8401361 - Flags: review?(catlee) → review+

Updated

3 years ago
Attachment #8401369 - Flags: review?(catlee) → review+

Updated

3 years ago
Attachment #8401372 - Flags: review?(catlee) → review+
(Assignee)

Comment 9

3 years ago
https://hg.mozilla.org/build/buildbot-configs/rev/ccddeced8b8a
https://hg.mozilla.org/build/buildbotcustom/rev/aafbdac2c9f7
https://hg.mozilla.org/build/mozharness/rev/b4245fa9628a
in production
(Reporter)

Updated

3 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.