Merge lp:~mterry/snappy/systemd-restart into lp:~snappy-dev/snappy/snappy-moved-to-github
Proposed by
Michael Terry
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Michael Terry | ||||
Approved revision: | 487 | ||||
Merged at revision: | 492 | ||||
Proposed branch: | lp:~mterry/snappy/systemd-restart | ||||
Merge into: | lp:~snappy-dev/snappy/snappy-moved-to-github | ||||
Diff against target: |
35 lines (+3/-0) 3 files modified
snappy/click_test.go (+1/-0) systemd/systemd.go (+1/-0) systemd/systemd_test.go (+1/-0) |
||||
To merge this branch: | bzr merge lp:~mterry/snappy/systemd-restart | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Martin Pitt (community) | Approve | ||
Michael Vogt (community) | Approve | ||
Review via email: mp+261266@code.launchpad.net |
Commit message
Set all app services to restart on failure.
Description of the change
Set all app services to restart on failure.
See http://
There was talk on the snappy-app-devel mailing list about not using this key for dbus-activated services. But we don't support those right now.
(We do use the Type=dbus property, but that doesn't mean it's an dbus-activated service -- for that, we'd need to drop the [Install] section and add SystemdService= to the dbus service file. I believe... I'm not an expert with systemd.)
To post a comment you must log in.
Thanks a bunch! This looks good.