Samba logrotate script uses invalid argument to /etc/init.d/nmdb

[Josh Holland]

* Impact
the logrotate script uses an invalid reload syntax

* Test case
install samba, rotate the log, look at errors emails

* Impact
the command used now is the debian one, it should work, still check it reloads the service as it should

--------------------

I received the following in an email from Anacron:

/etc/cron.daily/logrotate:
Usage: /etc/init.d/nmbd {start|stop|restart|force-reload|status}
error: error running non-shared postrotate script for /var/log/samba/log.nmbd of '/var/log/samba/log.nmbd '
run-parts: /etc/cron.daily/logrotate exited with return code 1

On further investigation I found that the problem lies in the /etc/logrotate.d/samba file:

/var/log/samba/log.nmbd {
 weekly
 missingok
 rotate 7
 postrotate
  service nmbd reload 2>/dev/null
 endscript
 compress
 notifempty
}

where the postrotate script specifies a "reload", which the init script does not recognise.

ProblemType: Bug
DistroRelease: Ubuntu 14.10
Package: samba 2:4.1.11+dfsg-1ubuntu2
ProcVersionSignature: Ubuntu 3.16.0-23.31-generic 3.16.4
Uname: Linux 3.16.0-23-generic x86_64
NonfreeKernelModules: nvidia
ApportVersion: 2.14.7-0ubuntu8
Architecture: amd64
CurrentDesktop: Unity
Date: Sun Oct 26 12:36:21 2014
InstallationDate: Installed on 2014-05-30 (148 days ago)
InstallationMedia: Ubuntu 14.04 LTS "Trusty Tahr" - Release amd64 (20140417)
OtherFailedConnect: Yes
SambaServerRegression: No
SmbConfIncluded: No
SourcePackage: samba
UpgradeStatus: Upgraded to utopic on 2014-10-18 (8 days ago)

[Launchpad Janitor (janitor)]

Status changed to 'Confirmed' because the bug affects multiple users.

[Dominique Ramaekers (dominique-ramaekers)]

It's still there in Vivid...

[jimav (james-avera)]

Also confirm it's happening on vivid (15.04) - get an email from Anacron cron.daily after rebooting.

[GuyR (guy-rouger)]

Same problem in Xubuntu 15.04
/etc/cron.daily/logrotate:
Usage: /etc/init.d/nmbd {start|stop|restart|force-reload|status}
error: error running non-shared postrotate script for /var/log/samba/log.nmbd of '/var/log/samba/log.nmbd '
run-parts: /etc/cron.daily/logrotate exited with return code 1

[Simon Arlott (sa.me.uk)]

This can be fixed in the debian/samba.logrotate script by doing smbd and nmbd at the same time and using "smbcontrol all reload-config". This command doesn't return an error if the services aren't running.

[Ubuntu Foundations Team Bug Bot (crichton)]

The attachment "0001-Use-smbcontrol-for-log-rotation.patch" seems to be a patch. If it isn't, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are a member of the ~ubuntu-reviewers, unsubscribe the team.

[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issues please contact him.]

[elatllat (elatllat)]

Ubuntu seems more sloppy and ess responsive than debian... maybe it's time they accepted more community help.

[Robie Basak (racb)]

@elatllat

We'd love more community help! Please see https://wiki.ubuntu.com/SponsorshipProcess for details on getting this into the sponsorship queue. Keep in mind that submitted patches may not necessarily be suitable - sometimes they aren't the correct fix, might regress other users, or should be sent to Debian instead rather than increase the maintenance burden in Ubuntu. But if it's appropriate, a patch pilot processing the sponsorship queue will apply it.

Thanks again for volunteering!

[Sebastien Bacher (seb128)]

Thanks, indeed that's an issue introduced when switching away from upstart, I've changed the package back to use the Debian version of the script, which works, and forwarded them your suggested change on https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=804705

[Launchpad Janitor (janitor)]

This bug was fixed in the package samba - 2:4.1.20+dfsg-1ubuntu2

---------------
samba (2:4.1.20+dfsg-1ubuntu2) xenial; urgency=medium

  * debian/samba.logrotate:
    - revert to Debian version of the logrotate reload command, fix an
      invalid syntax introduced in the upstart->systemd transition
      (lp: #1385868)

 -- Sebastien Bacher <email address hidden> Tue, 10 Nov 2015 19:01:06 +0100

[Brian Murray (brian-murray)]

Hello Josh, or anyone else affected,

Accepted samba into wily-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/samba/2:4.1.17+dfsg-4ubuntu3 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-needed to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

[Sebastien Bacher (seb128)]

seems to done the job

[Launchpad Janitor (janitor)]

This bug was fixed in the package samba - 2:4.1.17+dfsg-4ubuntu3

---------------
samba (2:4.1.17+dfsg-4ubuntu3) wily; urgency=medium

  * debian/samba.logrotate:
    - revert to Debian version of the logrotate reload command, fix an
      invalid syntax introduced in the upstart->systemd transition
      (lp: #1385868)

 -- Sebastien Bacher <email address hidden> Tue, 10 Nov 2015 19:04:30 +0100

[Chris J Arges (arges)]

The verification of the Stable Release Update for samba has completed successfully and the package has now been released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regressions.