launchpad-reviewers team mailing list archive

["Jeroen T. Vermeulen" <jtv@xxxxxxxxxxxxx>]

Jeroen T. Vermeulen has proposed merging lp:~jtv/maas/pkg-move-dns-templates-to-etc into lp:~maas-maintainers/maas/packaging.

Commit message:
Packaging change: install DNS templates to /etc/maas/templates/dns/.

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~jtv/maas/pkg-move-dns-templates-to-etc/+merge/167181

This accompanies the trunk change in https://code.launchpad.net/~jtv/maas/move-dns-templates-to-etc/+merge/167178 and follows the same coupled-branches pattern as I used for the PXE templates.

Unlike the other templates I've been moving to /etc, these will be installed by the maas-region-controller package.  Why not maas-dns?  Because the region controller will still generate DNS configs (and thus need the templates) even when maas-dns is not installed.

Once both branches have been approved, I'll land that trunk change and in this packaging branch, update debian/changelog to match its revision.


Jeroen
-- 
https://code.launchpad.net/~jtv/maas/pkg-move-dns-templates-to-etc/+merge/167181
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~jtv/maas/pkg-move-dns-templates-to-etc into lp:~maas-maintainers/maas/packaging.

=== modified file 'debian/maas-region-controller.install'
--- debian/maas-region-controller.install	2013-06-03 21:41:44 +0000
+++ debian/maas-region-controller.install	2013-06-03 23:55:29 +0000
@@ -25,6 +25,9 @@
 # Install new preseed files
 debian/tmp/etc/maas/preseeds
 
+# Install templates
+debian/tmp/etc/maas/templates/dns
+
 # Install all other stuff
 debian/extras/maas usr/sbin
 debian/extras/maas-region-celeryd usr/sbin