Re: Bug#653053: ldap2zone: Sending email every hour fill up the mail spool

[Petter Reinholdtsen <pere@hungry.com>]

Version: 0.2-2

[Cajus Pollmeier]
> It does not close the bug for squeeze ;-)

Sure, but BTS can track versions, and will know even if it is closed in
unstable.  Closing it in unstable to record the status.

> To have it fixed we need to comunicate with the release team. Which
> will not be possible for me within the remaining time in this year...

OK.  I initiate the contact, then.  cc to debian-release@.

The issue at hand is that the ldap2zone package in Squeeze run a cron
job that generate an email every hour even if nothing is wrong.  This
quickly fill up /var/ on Debian Edu main-server installations and causes
Debian Edu servers to stop working after a few days.

A fix in lpad2zone has been implemented in unstable, where printing to
stdout has been replaced with logging to syslog.  I would very much like
to have the same change in Squeeze, and thus ask for permission from the
release managers before uploading to proposed-updates.  What version
number do you recommend to use for this version?

This is a draft version of the patch to fix this issue.  I expect the
maintainer will do the upload if you approve the fix.

diff -u ldap2zone-0.1/debian/changelog ldap2zone-0.1/debian/changelog
--- ldap2zone-0.1/debian/changelog
+++ ldap2zone-0.1/debian/changelog
@@ -1,3 +1,10 @@
+ldap2zone (0.1-7squeeze1) proposed-updates; urgency=low
+
+  * Backport fix from 2.2-2: Made script log using logger by a
+    patch contributed by Petter Reinholdtsen (Closes: #653053).
+
+ -- Petter Reinholdtsen <pere@debian.org>  Sun, 25 Dec 2011 09:46:53 +0100
+
 ldap2zone (0.1-7) unstable; urgency=low
 
   * Corrected error in notify dns statement breaking the program
diff -u ldap2zone-0.1/debian/patches/05_correct_bashisms_ldap2bind.dpatch ldap2zone-0.1/debian/patches/05_correct_bashisms_ldap2bind.dpatch
--- ldap2zone-0.1/debian/patches/05_correct_bashisms_ldap2bind.dpatch
+++ ldap2zone-0.1/debian/patches/05_correct_bashisms_ldap2bind.dpatch
@@ -27,7 +27,7 @@
  	if [ $? -ne 0 ]; then
 -		echo -e "Reloading the zone '$domain' failed:\n$result" 1>&2
-+		printf "Reloading the zone '$domain' failed: $result\n" 1>&2
++		logger -t ldap2bind "Reloading the zone '$domain' failed: $result\n" 1>&2
 +	else
-+		printf "Reloading the zone '$domain' was successful\n" 1>&2
++		logger -t ldap2bind "Reloading the zone '$domain' was successful\n" 1>&2
  	fi
  done
-- 
Happy hacking
Petter Reinholdtsen