Merge lp://qastaging/~andreserl/maas/maas_systemd_1.7 into lp://qastaging/~maas-maintainers/maas/packaging.utopic

Proposed by Andres Rodriguez
Status: Work in progress
Proposed branch: lp://qastaging/~andreserl/maas/maas_systemd_1.7
Merge into: lp://qastaging/~maas-maintainers/maas/packaging.utopic
Diff against target: 140 lines (+92/-1)
6 files modified
debian/control (+1/-0)
debian/maas-cluster-controller.maas-clusterd.service (+19/-0)
debian/maas-dhcp.maas-dhcpd.service (+23/-0)
debian/maas-dhcp.maas-dhcpd6.service (+23/-0)
debian/maas-proxy.maas-proxy.service (+13/-0)
debian/rules (+13/-1)
To merge this branch: bzr merge lp://qastaging/~andreserl/maas/maas_systemd_1.7
Reviewer Review Type Date Requested Status
Martin Pitt (community) Approve
MAAS Maintainers Pending
Review via email: mp+252511@code.qastaging.launchpad.net
To post a comment you must log in.
Revision history for this message
Martin Pitt (pitti) wrote :

Thanks! This addresses most of my remarks in https://code.launchpad.net/~maas-maintainers/maas/packaging_add_systemd_support/+merge/251829 , except one which I repeated below. The units look good to me now. (Note that I didn't test any of this, I haven't worked with setting up MAAS yet. But with cloud images and vivid being systemd by default now I suppose you can test this easily in CI even?)

review: Approve

Unmerged revisions

358. By Andres Rodriguez

Add systemd support

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

to all changes: