Closed Bug 1299078 Opened 9 years ago Closed 8 years ago

Nightlies not being generated

Categories

(SeaMonkey :: Release Engineering, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: ewong, Assigned: ewong)

Details

Attachments

(1 file, 5 obsolete files)

There are currently no nightlies. Basically due to a sequence of events, our nightlies have failed to run. Event #1: http://hg.mozilla.org/build/buildbot-configs/rev/3bcc86136b6b was pushed Event #2: master had a hiccup that required restart After 'fixing' some db items, the scheduler came and went and still didn't run the builds. So.. once again, thanks to nthomas' hint, I came across the following code: http://hg.mozilla.org/build/buildbotcustom/file/seamonkey-production/misc.py#l1979 which is : http://hg.mozilla.org/build/buildbotcustom/file/seamonkey-production/misc_scheduler.py#l252 lastGoodRev() from http://hg.mozilla.org/build/buildbotcustom/file/seamonkey-production/misc_scheduler.py#l269 returns None because there hasn't been a good Nightly build in 24hrs. So it continues to: http://hg.mozilla.org/build/buildbotcustom/file/seamonkey-production/misc_scheduler.py#l288 which checks if l10nBranch. since enable_l10n has been disabled, this is false. Therefore, lastGoodFunc() returns None. So, my current idea is to actually remove lines #292 and 293 so that it goes to line #297. 288 if not triggerBuildIfNoChanges: 289 if l10nBranch: 290 if (start-lastL10nChange) > (24*3600): 291 return None 292 else: 293 return None 294 295 # Couldn't find a good revision. Fall back to using the latest 296 # revision on this branch 297 c = lastChange(db, t, branch) 298 if c: 299 rev = c.revision 300 log.msg("lastChange returned %s" % (rev)) and looking through the ssFunc, I realized that given "triggerBuildIfNoChanges=False", the system goes to
Attached patch [buildbotcustom] proposed patch (obsolete) — Splinter Review
*n-i myself to remind me to ask Callek for review
Assignee: nobody → ewong
Status: NEW → ASSIGNED
Flags: needinfo?(ewong)
over irc, :nthomas also suggested we use lastRevFunc instead of lastGoodFunc. at least it will avoid this problem.
was looking at production branch and thought we had lastRevFunc. We don't, so adding to misc.py
Attachment #8786259 - Attachment is obsolete: true
Flags: needinfo?(ewong)
Summary: Nightlies scheduling failing. → Nightlies not being generated
didn't finish comment #0 apparently :( "...the system goes to http://hg.mozilla.org/build/buildbotcustom/file/seamonkey-production/misc_scheduler.py#l292 since l10nBranch is false. so it returns None.
Attached patch [buildbotcustom] (wip) patch (obsolete) — Splinter Review
this patch actually unhorks the nightly. but it's still a wip.
Attachment #8786262 - Attachment is obsolete: true
Attachment #8786676 - Flags: review?(bugspam.Callek)
tl;dr the most recent patch forces the nightly to run. however, it still needs some fixes as within the not forceBuildLastChange condition, it has: later_rev = getLatestRev(db, t, branch, rev, last_built_revs[0]) I just set it to an arbitrary value (i.e. the 1st item in the list). I took the lastRevFunc() and getLastBuiltRevisions() from moco's buildbotcustom code; but adapted it to our code. Though all in all, I'm still unhappy that it's just a hack to get things going. I'm wondering if we should just use the lastChange revision and build against that for the nightly?
Attached patch [buildbotcustom] wip patch (obsolete) — Splinter Review
Attachment #8786676 - Attachment is obsolete: true
Attachment #8786676 - Flags: review?(bugspam.Callek)
Attachment #8787085 - Flags: feedback?(bugspam.Callek)
Will go for a post-land-review and will push fixes after. Currently the manually-applied-patches-to-fix-misc-builds needs to be applied properly.
Attachment #8786255 - Attachment is obsolete: true
Attachment #8787085 - Attachment is obsolete: true
Attachment #8787085 - Flags: feedback?(bugspam.Callek)
Attachment #8798292 - Flags: review?(bugspam.Callek)
(In reply to Edmund Wong (:ewong) from comment #9) > https://hg.mozilla.org/build/buildbotcustom/rev/ > ac4f92670349e34455ac9ee6fcdf7b795c2505e7 > Bug 1299078 - Use lastRevFunc instead of lastGoodFunc. backing this out.
Attachment #8798292 - Attachment description: [buildbotcustom] proposed patch (v4) → [buildbotcustom] proposed patch (v4) [checked-in]
Comment on attachment 8798292 [details] [diff] [review] [buildbotcustom] proposed patch (v4) [checked-in] considered fixed.
Attachment #8798292 - Flags: review?(bugspam.Callek)
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: