Bug 1468128 - script to send unsent mail stops on any bad message
Summary: script to send unsent mail stops on any bad message
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Bugzilla
Classification: Community
Component: Email Notifications
Version: 4.4
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: 4.4
Assignee: Jeff Fearn 🐞
QA Contact: tools-bugs
URL:
Whiteboard:
Depends On:
Blocks: 1285621
TreeView+ depends on / blocked
 
Reported: 2017-07-06 06:54 UTC by Jeff Fearn 🐞
Modified: 2018-12-09 06:29 UTC (History)
3 users (show)

Fixed In Version: 5.0.3-rh31
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2017-07-14 03:07:42 UTC
Embargoed:


Attachments (Terms of Use)

Description Jeff Fearn 🐞 2017-07-06 06:54:34 UTC
Description of problem:
The script backing the cron job to send unsent mail terminates if a bad email is encountered.

./contrib/sendunsentbugmail.pl

Version-Release number of selected component (if applicable):
4.4

How reproducible:
Easy

Steps to Reproduce:
1. run the script

Actual results:
It will die because there are bad change sets in the DB.

Expected results:
All pending email is sent.

Additional info:
It should eval the call to Bugzilla::BugMail::Send and blurt out a message if it catches an error.

Comment 3 Jeff Fearn 🐞 2017-07-12 23:03:48 UTC
Sending mail for bug 934000...
        blew up, skipping because: Invalid parameter passed to Bugzilla::Bug::new_from_list: It must be
numeric.

Comment 4 Rony Gong 🔥 2017-07-13 06:33:17 UTC
@Jfearn, it work smoothly in QE server except with killer, Do you think it is a problem?

Sending mail for bug 1104574...
3 mails sent.
Took 1 seconds.

Sending mail for bug 1104598...
Killed
You have new mail in /var/spool/mail/root


Then I rerun this script, it could send out all email smoothly.

Comment 5 Jeff Fearn 🐞 2017-07-13 23:52:53 UTC
Yeah that should be OK.


Note You need to log in before you can comment on or make changes to this bug.