Merge lp:~blake-rouse/maas/tgt-service-sudoers into lp:~maas-maintainers/maas/packaging
Proposed by
Blake Rouse
| Status: | Merged |
|---|---|
| Approved by: | Blake Rouse |
| Approved revision: | no longer in the source branch. |
| Merged at revision: | 393 |
| Proposed branch: | lp:~blake-rouse/maas/tgt-service-sudoers |
| Merge into: | lp:~maas-maintainers/maas/packaging |
| Diff against target: |
11 lines (+2/-0) 1 file modified
debian/extras/99-maas-sudoers (+2/-0) |
| To merge this branch: | bzr merge lp:~blake-rouse/maas/tgt-service-sudoers |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Raphaël Badin (community) | Approve | ||
| Mike Pontillo (community) | Approve | ||
|
Review via email:
|
|||
Commit message
Add "service tgt status" and "service tgt start" to the sudoers file.
This is done so the service monitor can check the status of 'tgt' and start the service when it is down.
Description of the change
This is needed for this final change in the series of changes.
https:/
To post a comment you must log in.
Revision history for this message
| Blake Rouse (blake-rouse) wrote | # |
Revision history for this message
| Mike Pontillo (mpontillo) wrote | # |
Revision history for this message
| Raphaël Badin (rvb) | # |
review:
Approve
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
Looks good.
One question: this seems to be upstart specific. On systemd, there is no /usr/sbin/service. So how does this work on systemd?
Most newer scripts I've seen are using 'invoke-rc.d' instead, which is available on both.