WIP: Reload rather than restart nscd #56
Conversation
traylenator
force-pushed
the
reload
branch
2 times, most recently
from
5d704e4 to
eb624d1
Compare
traylenator
changed the title
|
Dear @traylenator, thanks for the PR! This is Vox Pupuli Tasks, your friendly Vox Pupuli Github Bot. I noticed that your pull request has CI failures. Can you please have a look at the failing CI jobs? |
|
Dear @traylenator, thanks for the PR! This is Vox Pupuli Tasks, your friendly Vox Pupuli Github Bot. I noticed that your pull request has CI failures. Can you please have a look at the failing CI jobs? |
|
Dear @traylenator, thanks for the PR! This is Vox Pupuli Tasks, your friendly Vox Pupuli GitHub Bot. I noticed that your pull request contains merge conflict. Can you please rebase? You can find my sourcecode at voxpupuli/vox-pupuli-tasks |
data/Ubuntu-16.04.yaml
Outdated
| @@ -0,0 +1,2 @@ | |||
| --- | |||
| nscd::service_restart: /usr/sbin/service nscd reload | |||
There was a problem hiding this comment.
is this valid? I never saw a 16.04 without systemd and in all other modules we assume 16.04 and newer ubuntus are running with systemd
There was a problem hiding this comment.
/bin/systemctl just to be annoying.
data/Debian-8.yaml
Outdated
| @@ -0,0 +1,2 @@ | |||
| --- | |||
| nscd::service_restart: /usr/sbin/service nscd reload | |||
There was a problem hiding this comment.
I know that Debian 8 could be used with systemd and sysv, but in all other modules we have we assume it's running systemd. (also I think we should not support debian 8 anymore because it has very limited support and debian 9/10 are available)
Pull Request (PR) description
On configuration change nscd should be reloaded and not restarted
will be called as appropriate.