Skip to content

Wip/systemd daemon

clayton craft requested to merge dcbaker:wip/systemd-daemon into master

Created by: dcbaker

Despite the diff that the white space is generating, this is not a huge patch series. It converts the poll_branches daemon from using the python daemon.py to relying on systemd to run it as a daemon. This has practical advantages of moving the complexity out of our quickly written and little maintained scripts into systemd, which is well maintained and tested, as well as buying features, like restart and proper dependencies.

The changes to the code themselves are pretty minimal, and are mainly meant to integrate features better into systemd and it's components, especially journald.

The only problem I have is that I don't have a good way to test the changes I'm making, as they're deeply ingrained in the system jenkins uses itself. It should be noted that once patch 3 is applied, until the service is setup the daemon will be non-functional. Since this must be copied into /etc/init.d currently I don't feel that this is a regression or break in behavior, the original file will still function just fine.

Merge request reports