Merge lp://staging/~mpontillo/maas/beacon-monitor--packaging into lp://staging/~maas-maintainers/maas/packaging

Proposed by Mike Pontillo
Status: Merged
Approved by: Mike Pontillo
Approved revision: 567
Merged at revision: 562
Proposed branch: lp://staging/~mpontillo/maas/beacon-monitor--packaging
Merge into: lp://staging/~maas-maintainers/maas/packaging
Diff against target: 73 lines (+18/-10)
5 files modified
debian/changelog (+13/-9)
debian/extras/99-maas-sudoers (+1/-0)
debian/extras/maas-rack (+2/-0)
debian/maas-rack-controller.install (+1/-0)
debian/maas-rack-controller.lintian-overrides (+1/-1)
To merge this branch: bzr merge lp://staging/~mpontillo/maas/beacon-monitor--packaging
Reviewer Review Type Date Requested Status
Andres Rodriguez (community) Approve
Данило Шеган (community) Approve
Review via email: mp+325493@code.staging.launchpad.net

Commit message

Packaging changes necessary to monitor beacons.

Description of the change

This corresponds to the changes here:

https://code.launchpad.net/~mpontillo/maas/beaconing-monitor/+merge/325199

In order to use the beaconing monitor inside the .deb, these packaging changes are needed.

Even if we're not maintaining the .deb moving forward in favor of the snap, I think it's still valuable to land this for comparison purposes, while we ramp up testing of the snap. I'm still using the .debs in some local environments where I'm not ready to move to the snap yet.

To post a comment you must log in.
Revision history for this message
Данило Шеган (danilo) wrote :

Pretty simple, looks good.

review: Approve
Revision history for this message
Andres Rodriguez (andreserl) wrote :

No debian/changelog entry saying you are adding/installing this file.

review: Needs Fixing
561. By Mike Pontillo

Add sudoers entry for maas-beacon-monitor.

562. By Mike Pontillo

Update changelog.

563. By Mike Pontillo

Merge packaging trunk. Fix changelog conflict.

Revision history for this message
Mike Pontillo (mpontillo) wrote :

Thanks; this is ready for another look.

564. By Mike Pontillo

Merge latest packaging branch.

565. By Mike Pontillo

Update changelog version.

566. By Mike Pontillo

Fix changelog merge conflict.

567. By Mike Pontillo

Separate changelog entries.

Revision history for this message
Andres Rodriguez (andreserl) :
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