Merge lp:~ted/ubuntu-app-launch/process-group-kill into lp:ubuntu-app-launch/14.04
Proposed by
Ted Gould
Status: | Merged |
---|---|
Approved by: | Oliver Grawert |
Approved revision: | 148 |
Merged at revision: | 143 |
Proposed branch: | lp:~ted/ubuntu-app-launch/process-group-kill |
Merge into: | lp:ubuntu-app-launch/14.04 |
Diff against target: |
26 lines (+10/-2) 1 file modified
upstart-jobs/application-click.conf.in (+10/-2) |
To merge this branch: | bzr merge lp:~ted/ubuntu-app-launch/process-group-kill |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Oliver Grawert | Approve | ||
PS Jenkins bot (community) | continuous-integration | Approve | |
Review via email: mp+215475@code.launchpad.net |
Commit message
Kill all jobs in process group on exit
Description of the change
After starting record the PID so that we can ensure the PID's process group gets destroyed using pkill after the job stops. A big stick, but insurance.
To post a comment you must log in.
PASSED: Continuous integration, rev:147 jenkins. qa.ubuntu. com/job/ upstart- app-launch- ci/253/ jenkins. qa.ubuntu. com/job/ upstart- app-launch- trusty- amd64-ci/ 112 jenkins. qa.ubuntu. com/job/ upstart- app-launch- trusty- armhf-ci/ 112 jenkins. qa.ubuntu. com/job/ upstart- app-launch- trusty- i386-ci/ 113
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
Click here to trigger a rebuild: s-jenkins. ubuntu- ci:8080/ job/upstart- app-launch- ci/253/ rebuild
http://