Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

systemd's service file should use the 'process' KillMode option on Debian also #29295

Closed
douardda opened this issue Dec 1, 2015 · 3 comments · Fixed by #33792
Closed

systemd's service file should use the 'process' KillMode option on Debian also #29295

douardda opened this issue Dec 1, 2015 · 3 comments · Fixed by #33792

Comments

@douardda
Copy link

douardda commented Dec 1, 2015

so one can manage the salt-minion daemon from salt without sawing the branch on which we sit (means: killing the forked salt-minion responsible for handling the command).

See #7997 (comment)

@douardda
Copy link
Author

douardda commented Dec 1, 2015

using this setup, on debian jessie, I can run:

root@pw01:/srv/salt# salt pd01 pkg.install pkgs="['salt-common','salt-minion']" 
pd01:
    ----------
    python-croniter:
        ----------
        new:
            0.3.4-1
        old:
    python-tz:
        ----------
        new:
            2012c+dfsg-0.1
        old:
    salt-common:
        ----------
        new:
            2015.5.3+ds-1~bpo8+1
        old:
            2015.5.2+ds-1~bpo8+1
    salt-minion:
        ----------
        new:
            2015.5.3+ds-1~bpo8+1
        old:
            2015.5.2+ds-1~bpo8+1

root@pw01:/srv/salt# salt pd01 grains.item saltversion 
pd01:
    ----------
    saltversion:
        2015.5.3

@douardda
Copy link
Author

douardda commented Dec 1, 2015

Ok this has been included in 2014.7.6... but NOT for debian systems...

why on earth such a fix has been included for all distros bur debian???

@douardda douardda changed the title systemd's service file should use the 'process' KillMode option systemd's service file should use the 'process' KillMode option on Debian also Dec 1, 2015
douardda pushed a commit to douardda/salt that referenced this issue Dec 1, 2015
@douardda
Copy link
Author

douardda commented Dec 1, 2015

Debian packaging being handled directly by debian, I close this ticket.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant