Closed
Bug 641276
Opened 14 years ago
Closed 14 years ago
scheduler master is not polling mobile repositories
Categories
(Release Engineering :: General, defect, P2)
Release Engineering
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: mozilla, Assigned: catlee)
References
Details
Attachments
(2 files)
475 bytes,
patch
|
mozilla
:
review+
bhearsum
:
review+
catlee
:
checked-in+
|
Details | Diff | Splinter Review |
2.19 KB,
patch
|
mozilla
:
review+
bhearsum
:
review+
catlee
:
checked-in+
|
Details | Diff | Splinter Review |
I can't find anything in the scheduler master.cfg or misc.generateMobileBranchObjects that would tell me why this wouldn't work.
My current theory involves the isHgPollerTriggered() function, though I don't know why it wouldn't poll at all.
Reporter | ||
Comment 1•14 years ago
|
||
When this is fixed, or when we merge m-b in, we can turn off the 0.7 nightlies again =P
Reporter | ||
Updated•14 years ago
|
Severity: normal → major
Reporter | ||
Updated•14 years ago
|
Assignee: nobody → aki
Assignee | ||
Updated•14 years ago
|
Assignee: aki → catlee
Priority: -- → P2
Assignee | ||
Comment 2•14 years ago
|
||
This will get duplicate pollers, which we'll clean up in master.cfg
Attachment #519236 -
Flags: review?(bhearsum)
Attachment #519236 -
Flags: review?(aki)
Reporter | ||
Updated•14 years ago
|
Attachment #519236 -
Flags: review?(aki) → review+
Updated•14 years ago
|
Attachment #519236 -
Flags: review?(bhearsum) → review+
Assignee | ||
Comment 3•14 years ago
|
||
This removes duplicate change sources. It works because change sources implement buildbot's ComparableMixin protocol, and so instances of different classes are unequal, and instance of the same class are compared by examining compare_attrs. HgPoller ends up comparing hgUrl, branch and pollInterval, which is fine for us.
dump_master.py on the scheduler master reports the addition of a single poller.
Attachment #519240 -
Flags: review?(bhearsum)
Attachment #519240 -
Flags: review?(aki)
Updated•14 years ago
|
Attachment #519240 -
Flags: review?(bhearsum) → review+
Reporter | ||
Updated•14 years ago
|
Attachment #519240 -
Flags: review?(aki) → review+
Assignee | ||
Comment 4•14 years ago
|
||
Comment on attachment 519236 [details] [diff] [review]
Turn mobile pollers on for non-try branches
http://hg.mozilla.org/build/buildbotcustom/rev/08a927eceb61
Attachment #519236 -
Flags: checked-in+
Assignee | ||
Comment 5•14 years ago
|
||
Comment on attachment 519240 [details] [diff] [review]
Clean up duplicate change sources
http://hg.mozilla.org/build/buildbot-configs/rev/7f0307929c1f
Attachment #519240 -
Flags: checked-in+
Reporter | ||
Comment 6•14 years ago
|
||
I've pushed this out, and see mobile-browser polling lines in scheduler master.
Thanks Catlee!
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Updated•12 years ago
|
Product: mozilla.org → Release Engineering
You need to log in
before you can comment on or make changes to this bug.
Description
•