Bug 397711 - [RFE] [PATCH] make yum-updatesd use sendmail
[RFE] [PATCH] make yum-updatesd use sendmail
Status: CLOSED CURRENTRELEASE
Product: Fedora
Classification: Fedora
Component: yum-updatesd (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Jeremy Katz
Fedora Extras Quality Assurance
: FutureFeature
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2007-11-24 06:44 EST by Pierre Ossman
Modified: 2008-04-23 17:29 EDT (History)
2 users (show)

See Also:
Fixed In Version: 0.9
Doc Type: Enhancement
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2008-04-23 17:29:10 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)
yum-updatesd-sendmail.patch (3.40 KB, patch)
2007-11-24 06:44 EST, Pierre Ossman
no flags Details | Diff
yum-email-compat.patch (683 bytes, patch)
2007-12-09 12:43 EST, Pierre Ossman
no flags Details | Diff
yum-email-crash.patch (883 bytes, patch)
2007-12-09 12:43 EST, Pierre Ossman
no flags Details | Diff

  None (edit)
Description Pierre Ossman 2007-11-24 06:44:26 EST
Almost every other program sends reports by calling /usr/bin/sendmail to the
address "root". yum-updatesd, however, requires a SMTP server to submit its
mail, something that usually also requires domain-decorated names.

This patch allows yum-updatesd to submit mail reports using /usr/bin/sendmail
like any other system monitoring program. The old code path is still there and
is controlled by the value "use_sendmail" in the configuration.

The patch in bug 387181 is assumed to be present when applying this one.
Comment 1 Pierre Ossman 2007-11-24 06:44:26 EST
Created attachment 268001 [details]
yum-updatesd-sendmail.patch
Comment 2 Jeremy Katz 2007-11-26 11:16:57 EST
Thanks, applied in git
Comment 3 Pierre Ossman 2007-12-05 02:10:17 EST
When will we see it in F8 updates/testing?
Comment 4 Ville Skyttä 2007-12-05 17:55:56 EST
The patch looks buggy, it uses /usr/bin/sendmail which doesn't exist.  Should be
using /usr/sbin/sendmail instead.
Comment 5 Fedora Update System 2007-12-06 15:54:23 EST
yum-updatesd-0.8-2.fc8 has been pushed to the Fedora 8 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update yum-updatesd'
Comment 6 Peter Bieringer 2007-12-09 07:54:34 EST
Has this been ever tested using postfix?

After fixing the use of email_to according to
https://bugzilla.redhat.com/show_bug.cgi?id=387051

I get now:

sendmail: fatal: Recipient addresses must be specified on the command line or
via the -t option
Comment 7 Pierre Ossman 2007-12-09 12:43:15 EST
Created attachment 282291 [details]
yum-email-compat.patch

Adds the -t flag to be more compatible with other /usr/sbin/sendmail
implementations.
Comment 8 Pierre Ossman 2007-12-09 12:43:39 EST
Created attachment 282301 [details]
yum-email-crash.patch

Avoid a traceback on some packages.
Comment 9 Pierre Ossman 2007-12-09 12:44:10 EST
Other than those two patches, the RPM in testing works just fine.
Comment 10 Jeremy Katz 2007-12-10 13:39:54 EST
Committed to master.  Will give a few more days before doing another build 
Comment 11 Fedora Update System 2007-12-20 14:52:17 EST
yum-updatesd-0.9-1.fc8 has been pushed to the Fedora 8 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update yum-updatesd'
Comment 12 Pierre Ossman 2007-12-26 14:07:50 EST
Seems to work nicely now.
Comment 13 Fedora Update System 2007-12-28 12:18:57 EST
yum-updatesd-0.9-1.fc8 has been pushed to the Fedora 8 stable repository.  If problems still persist, please make note of it in this bug report.
Comment 14 Jon Stanley 2008-04-23 16:30:29 EDT
Adding FutureFeature keyword to RFE's.

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