Closed
Bug 802593
Opened 12 years ago
Closed 12 years ago
ChangeNotifier mail broken in multimaster
Categories
(Release Engineering :: Release Automation: Other, defect)
Release Engineering
Release Automation: Other
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: bhearsum, Assigned: bhearsum)
Details
Attachments
(1 file)
1.23 KB,
patch
|
rail
:
review+
bhearsum
:
checked-in+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Comment 1•12 years ago
|
||
I think this is because the ChangeNotifier isn't present on the scheduler master, and that's where I started the release. Possible solutions: * Start releases on a build master instead * Put ChangeNotifier's on the scheduler master
Assignee | ||
Comment 2•12 years ago
|
||
Attachment #672514 -
Flags: review?(rail)
Comment 3•12 years ago
|
||
Comment on attachment 672514 [details] [diff] [review] include release ChangeNotifier's on scheduler masters It looks good in case if we have only 1 scheduler master. In release.py we have 2 change notifiers, one for the initial sendchange and one for "android signing" triggered by UrlPoller. If we have more than 1 scheduler master we may end up with multiple UrlPollers and as result multiple emails for the second one. Since we are going to remove android notifier soon (once we sign it on demand), let's proceed with this.
Attachment #672514 -
Flags: review?(rail) → review+
Assignee | ||
Updated•12 years ago
|
Attachment #672514 -
Flags: checked-in+
Assignee | ||
Comment 4•12 years ago
|
||
Updating summary because this affects android signing mail too.
Summary: "tagging started" mail broken in multimaster → ChangeNotifier mail broken in multimaster
Assignee | ||
Comment 5•12 years ago
|
||
https://wiki.mozilla.org/ReleaseEngineering:Maintenance says that this is in production now.
Status: ASSIGNED → RESOLVED
Closed: 12 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
•