search

systemli / ansible-role-needrestart

Install and maintain needrestart on debian-like systems
GNU General Public License v3.0
10 stars 11 forks source link

Move config to conf.d folder #49

Closed steps128 closed 1 year ago

steps128 commented 1 year ago

Move the custom configuration to conf.d folder so there is no need to maintain the needrestart.conf file in this role.

t2d commented 1 year ago

Thank you for your PR. Can you do it without the yaml inheritance? I don't think it's a feature officially supported by ansible and I don't see it's specific value here.

steps128 commented 1 year ago

Thank you for your PR. Can you do it without the yaml inheritance? I don't think it's a feature officially supported by ansible and I don't see it's specific value here.

Sure thing, I just wanted to use it as practice. It is officially supported though.

t2d commented 1 year ago

I still believe we need to add a debian standard needrestart.conf to override old settings made by this role. It can be with a switch, but it should be turned on by default.

t2d commented 1 year ago

I did the necessary steps myself in #50 and created a new release 3.0.0. I hope, this fits your needs. Thanks for your contribution.