Merge lp://staging/~blake-rouse/maas/fix-lease-notifier-packaging into lp://staging/~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: 461
Proposed branch: lp://staging/~blake-rouse/maas/fix-lease-notifier-packaging
Merge into: lp://staging/~maas-maintainers/maas/packaging
Diff against target: 99 lines (+22/-16)
5 files modified
debian/extras/99-maas-sudoers (+0/-1)
debian/extras/maas-rack (+21/-7)
debian/maas-dhcp.apparmor (+1/-0)
debian/maas-dhcp.maas-dhcpd.service (+0/-4)
debian/maas-dhcp.maas-dhcpd6.service (+0/-4)
To merge this branch: bzr merge lp://staging/~blake-rouse/maas/fix-lease-notifier-packaging
Reviewer Review Type Date Requested Status
Mike Pontillo (community) Approve
Review via email: mp+289413@code.staging.launchpad.net

Commit message

Add to apparmor profile to allow dhcpd to call maas-rackd. Fix maas-rack command to allow dhcpd user to call dhcp-notify only. Remove KillSignal from maas-dhcpd and maas-dhcpd6 as the fixed isc-dhcp-server no longer requires that kill signal to stop.

To post a comment you must log in.
Revision history for this message
Mike Pontillo (mpontillo) wrote :

Looks good.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches