Closed
Bug 704200
Opened 13 years ago
Closed 13 years ago
android dep builds permared after bug 701864; sometimes causing nightlies not to trigger
Categories
(Release Engineering :: General, defect)
Release Engineering
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: bhearsum, Assigned: bhearsum)
References
Details
Attachments
(2 files)
(deleted),
patch
|
catlee
:
review+
bhearsum
:
checked-in+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
Details | Diff | Splinter Review |
Because we require a working set of dep builds to trigger nightlies, these burning Android dep builds can cause us not to. In some cases, we end up triggering nightlies solely based on l10n changes, but this isn't always the case, such as bug 704122. https://bugzilla.mozilla.org/show_bug.cgi?id=704122#c7 has a better description of the algorithm used, but we should really fix this. Some options:
1) turn off the permared builds
2) disable android nightlies completely
3) stop looking at android dep build status when decided whether to trigger nightlies or not
Comment 1•13 years ago
|
||
I intentionally turned these on because we don't know mobile's timing and I didn't want to have us be a blocker to that. This bug changes that story quite a bit.
* If we turn off the 'android' platform (dep or nightly) we need to keep it on birch.
* If we only turn off 'android' nightlies, will the other nightlies trigger? This would be the best solution.
** set android enable_nightly to False
** set android enable_nightly for Birch to True
* Not sure how ugly (3) will be.
* Any way we resolve this, we should leave 'linux-android' and 'android-xul' alone.
Assignee | ||
Comment 2•13 years ago
|
||
(In reply to Aki Sasaki [:aki] (back nov 28) from comment #1)
> I intentionally turned these on because we don't know mobile's timing and I
> didn't want to have us be a blocker to that. This bug changes that story
> quite a bit.
>
> * If we turn off the 'android' platform (dep or nightly) we need to keep it
> on birch.
Yup. Can we drop android from the default list of platforms here: http://hg.mozilla.org/build/buildbot-configs/file/tip/mozilla/config.py#l70. It's already in birch's list (http://hg.mozilla.org/build/buildbot-configs/file/tip/mozilla/project_branches.py#l191), so we shouldn't have to do anything else except back out this change when we're ready.
> * If we only turn off 'android' nightlies, will the other nightlies trigger?
I don't think so. My reading of the code says that we look at *all* of the dep builders, not just the ones with nightlies enabled.
> This would be the best solution.
> ** set android enable_nightly to False
> ** set android enable_nightly for Birch to True
Therefore, this won't help.
> * Any way we resolve this, we should leave 'linux-android' and 'android-xul'
> alone.
Makes sense to me.
Assignee: nobody → bhearsum
Comment 3•13 years ago
|
||
Much as I don't like this solution, if mobile is really going to land something to stop the permared tomorrow, then we only have one more nightly that can fail to trigger, and it will only fail to trigger if there are no l10n changes all day today. There's still the risk (which we've been living with since last week) that we'll build a nightly on a busted rev, since using the l10n fallback puts us back in the old-skool "whatever's tip at 3am ships if it happens to compile" style, but maybe living with that risk is less painful than fixing this so that a single nightly can get a properly chosen rev.
Assignee | ||
Comment 4•13 years ago
|
||
That's a good point. I'm going to get a patch ready, but I'll hold off landing anything until we have a better idea when these builds are going to stop burning.
Comment 5•13 years ago
|
||
Mobile is working hard to get native builds working on m-c, but we won't be ready in the next week or so. Let's disable the native builds on m-c until mobile gives releng a thumbs up.
Assignee | ||
Comment 6•13 years ago
|
||
Not sure if Aki is going to be around to review this or not, but this should get us going again. I changed the default for the "android" platform to disabled, which turns it off for everything except birch, which has it explicitly enabled: http://hg.mozilla.org/build/buildbot-configs/file/tip/mozilla/project_branches.py#l191.
My before and after diff of config.py output looked sane, I'll attach it here as well.
Attachment #575917 -
Flags: review?(catlee)
Attachment #575917 -
Flags: review?(aki)
Assignee | ||
Comment 7•13 years ago
|
||
Updated•13 years ago
|
Attachment #575917 -
Flags: review?(catlee) → review+
Assignee | ||
Updated•13 years ago
|
Attachment #575917 -
Flags: review?(aki) → checked-in+
Assignee | ||
Comment 8•13 years ago
|
||
This patch is landed in production. The permared builders are now off everywhere, so nightlies should be back to normal.
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
Product: mozilla.org → Release Engineering
You need to log in
before you can comment on or make changes to this bug.
Description
•