Closed Bug 548054 Opened 12 years ago Closed 12 years ago

send email on mobile build failures

Categories

(Release Engineering :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: aki, Assigned: aki)

References

Details

(Whiteboard: [limit builders/branches])

Attachments

(3 files, 1 obsolete file)

to the list created in bug 548051.
Attachment #439932 - Flags: review?(jhford) → review+
Should be fixed.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Attached patch m.com ->m.orgSplinter Review
Attachment #440560 - Flags: review?(jhford)
Attachment #440560 - Flags: review?(jhford) → review+
pm02 needs a restart.
Now sending emails, after a pm02 restart on Saturday.
Status: REOPENED → RESOLVED
Closed: 12 years ago12 years ago
Resolution: --- → FIXED
blassey now specifies m-c and m-1.9.2 only.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Whiteboard: [limit builders/branches]
Comment on attachment 442878 [details] [diff] [review]
limit mails to mobile-trunk and mobile-1.9.2


>-    mailNotifyBuilders.extend(builders + nightlyBuilders)
>+    if branch['mail_on_failure']:
>+        mailNotifyBuilders.extend(builders + nightlyBuilders)


Instead, could we use branch.get('mail_on_failure', False) and remove the extra lines in the config dictionary?
Attachment #442878 - Flags: review?(jhford) → review-
Attachment #442878 - Attachment is obsolete: true
Attachment #443432 - Flags: review?(jhford)
Attachment #443432 - Flags: review?(jhford) → review+
Status: REOPENED → RESOLVED
Closed: 12 years ago12 years ago
Resolution: --- → FIXED
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.