[Merge] lp:~xnox/upstart/reload-signal into lp:upstart

Dmitrijs Ledkovs launchpad at surgut.co.uk
Wed Aug 21 11:30:44 UTC 2013


You have been requested to review the proposed merge of lp:~xnox/upstart/reload-signal into lp:upstart.

For more details, see:
https://code.launchpad.net/~xnox/upstart/reload-signal/+merge/176099

This branch adds "reload signal" stanza.

Instead of initctl calling kill on the main process, it now calls an instance dbus method Reload.

Reload method, looks up reload_signal and calls kill on the main process with it.

Tests added:
* serialisation/deserialisation
* job configuration parsing
* job_reload() function in init/job.c

Broken tests:
* test_initctl reload_action test currently hangs, and thus is commented out. Help with testing it is appreciated, I guess it now needs to simply test that dbus calls are done in the right order without actually testing that signals were sent to a process, as this is now done in the job_reload() test.

-- 
https://code.launchpad.net/~xnox/upstart/reload-signal/+merge/176099
Your team Upstart Reviewers is requested to review the proposed merge of lp:~xnox/upstart/reload-signal into lp:upstart.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: review-diff.txt
Type: text/x-diff
Size: 25486 bytes
Desc: not available
URL: <https://lists.ubuntu.com/archives/upstart-devel/attachments/20130821/ef25368b/attachment-0001.diff>


More information about the upstart-devel mailing list