Add a trigger to reload rsyslog when a new configuration file is dropped in /etc/rsyslog.d

The attachment "rsyslog-yakkety.debdiff" seems to be a debdiff. The ubuntu-sponsors team has been subscribed to the bug report so that they can review and hopefully sponsor the debdiff. If the attachment isn't a patch, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are member of the ~ubuntu-sponsors, unsubscribe the team.

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

This doesn't yet seem to be fixed in Zesty, although debian fixed it in package version 8.24.0-1.

Too late now for sponsorship to Zesty. Since it will come with the next rsyslog merge in 8.24.0-1, well go forward with the SRU post-merge of the next rsyslog

Looks like this is still missing one additional commit from when it was included in Debian; namely the addition of init-system-helpers in the Depends: field for rsyslog (this makes sure we have a new-enough init-system-helpers so try-restart works).

I'll add it for you in this case. It feels counterproductive for me to reject the change again when you already did go back on it before.

By the way, this means that for a xenial SRU, this would need to be adjusted to the right version number, or that feature will need to be backported...

Hi Frode,

Now that cyphermox sponsored it for Artful ... Could you prepare the debdiff(s) for the affected supported releases ? and I'll sponsored them.

Regards,
Eric

This bug was fixed in the package rsyslog - 8.16.0-1ubuntu7

---------------
rsyslog (8.16.0-1ubuntu7) artful; urgency=medium

  * Cherry pick restart on configuration changes fix from Debian (LP: #1668639)
    - Trigger restart on configuration changes.
      Register a dpkg trigger on /etc/rsyslog.d that calls restart on
      configuration changes. (Closes: #791337)
    - Update dpkg trigger to use try-restart.
      Add try-restart action to SysV initscript for that, systemd supports
      this natively.
    - debian/control: Add Depends on init-system-helpers (>= 1.47~) to rsyslog.

 -- Frode Nordahl <email address hidden> Mon, 26 Jun 2017 06:29:30 +0000

@cyphermox Thank you!

@slashd Thank you! I will start work on debdiffs for the SRU's ASAP.

Unsubscribing ~ubuntu-sponsors as there's nothing left to do. Please feel free to resubscribe once patches are ready to upload (or mark your diff as a patch and let the bot do it for you) or if this was done in error.

Thank you!

Marking Won't fix for Yakkety as it is EOL.

Removing sts-sru-needed tag until patches are ready.

It is fixed in Ubuntu Bionic and late but this LP bug have never been mentioned in the changelog, which explains why the status didn't change until now (after a manual status change from my part)

Trusty and Xenial hasn't been fix, I did some quick verifications and base on the fix, which add a Depends as follow:

debian/control: init-system-helpers (>= 1.47~)

and considering rmadison:
 init-system-helpers | 1.29ubuntu1 | xenial | source, all
 init-system-helpers | 1.29ubuntu4 | xenial-updates | source, all

 init-system-helpers | 1.14 | trusty | source, all
 init-system-helpers | 1.14ubuntu1 | trusty-updates | source, all

I'm afraid the fix won't work for Trusty and Xenial.
Trusty is also in EOL.

With that being said, I will set Trusty/Xenial to "Won't fix".

Please feel free to re-open it if you judge Xenial need the fix by providing good justification/rational.

- Eric