[puppet] Apply partial fix for upstream #9167
Todd Zullinger
tmz at fedoraproject.org
Mon Oct 24 19:20:37 UTC 2011
commit 47fcf33655b9457878ee0dfca6d2b6c3478affc0
Author: Todd Zullinger <tmz at pobox.com>
Date: Fri Oct 21 14:54:12 2011 -0400
Apply partial fix for upstream #9167
Tagmail report sends email when nothing happens. This is a partial fix
because it doesn't handle --noop mode.
...Do-not-sent-tagmail-reports-if-no-changes.patch | 46 ++++++++++++++++++++
puppet.spec | 5 ++
2 files changed, 51 insertions(+), 0 deletions(-)
---
diff --git a/0001-9167-Do-not-sent-tagmail-reports-if-no-changes.patch b/0001-9167-Do-not-sent-tagmail-reports-if-no-changes.patch
new file mode 100644
index 0000000..5ba1da1
--- /dev/null
+++ b/0001-9167-Do-not-sent-tagmail-reports-if-no-changes.patch
@@ -0,0 +1,46 @@
+From e8c0c799e6f648a90028ce2248e14247f021f1c7 Mon Sep 17 00:00:00 2001
+From: Kelsey Hightower <kelsey.hightower at gmail.com>
+Date: Wed, 24 Aug 2011 21:51:45 -0400
+Subject: [PATCH/puppet] (#9167) Do not sent tagmail reports if no changes
+
+Prevent Puppet Master from sending tagmail reports if the status of the
+puppet run is "unchanged".
+---
+ lib/puppet/reports/tagmail.rb | 6 ++++++
+ spec/unit/reports/tagmail_spec.rb | 5 +++++
+ 2 files changed, 11 insertions(+), 0 deletions(-)
+
+diff --git a/lib/puppet/reports/tagmail.rb b/lib/puppet/reports/tagmail.rb
+index c37341e..cf33cfa 100644
+--- a/lib/puppet/reports/tagmail.rb
++++ b/lib/puppet/reports/tagmail.rb
+@@ -108,6 +108,12 @@ Puppet::Reports.register_report(:tagmail) do
+
+ # Process the report. This just calls the other associated messages.
+ def process
++ # Do not send the email report if nothing changed.
++ if self.status == "unchanged"
++ Puppet.info "Not sending tagmail report; no changes"
++ return
++ end
++
+ unless FileTest.exists?(Puppet[:tagmap])
+ Puppet.notice "Cannot send tagmail report; no tagmap file #{Puppet[:tagmap]}"
+ return
+diff --git a/spec/unit/reports/tagmail_spec.rb b/spec/unit/reports/tagmail_spec.rb
+index a53d119..0df67ec 100755
+--- a/spec/unit/reports/tagmail_spec.rb
++++ b/spec/unit/reports/tagmail_spec.rb
+@@ -88,4 +88,9 @@ describe tagmail do
+ results.should be_nil
+ end
+ end
++
++ describe "when there are no changes" do
++ it "should not send an email"
++ end
++
+ end
+--
+1.7.6
+
diff --git a/puppet.spec b/puppet.spec
index 9803813..9a752d7 100644
--- a/puppet.spec
+++ b/puppet.spec
@@ -14,6 +14,8 @@ Source0: http://downloads.puppetlabs.com/%{name}/%{name}-%{version}.tar.g
Source1: http://downloads.puppetlabs.com/%{name}/%{name}-%{version}.tar.gz.asc
# https://projects.puppetlabs.com/issues/10244
Patch0: 0001-10244-Restore-Mongrel-XMLRPC-functionality.patch
+# https://projects.puppetlabs.com/issues/9167
+Patch1: 0001-9167-Do-not-sent-tagmail-reports-if-no-changes.patch
Group: System Environment/Base
@@ -69,6 +71,7 @@ The server can also function as a certificate authority and file server.
%prep
%setup -q
%patch0 -p1
+%patch1 -p1
patch -s -p1 < conf/redhat/rundir-perms.patch
%build
@@ -258,6 +261,8 @@ rm -rf %{buildroot}
* Sun Oct 23 2011 Todd Zullinger <tmz at pobox.com> - 2.6.12-1
- Update to 2.6.12, fixes CVE-2011-3872
- Add upstream patch to restore Mongrel XMLRPC functionality (upstream #10244)
+- Apply partial fix for upstream #9167 (tagmail report sends email when nothing
+ happens)
* Thu Sep 29 2011 Todd Zullinger <tmz at pobox.com> - 2.6.6-3
- Apply upstream patches for CVE-2011-3869, CVE-2011-3870, CVE-2011-3871, and
More information about the scm-commits
mailing list