Merge lp:~mpontillo/maas/dns-template-changes-trunk into lp:~maas-committers/maas/trunk
Status: | Merged |
---|---|
Approved by: | Mike Pontillo |
Approved revision: | no longer in the source branch. |
Merged at revision: | 4082 |
Proposed branch: | lp:~mpontillo/maas/dns-template-changes-trunk |
Merge into: | lp:~maas-committers/maas/trunk |
Diff against target: |
351 lines (+176/-6) 8 files modified
etc/maas/templates/dns/named.conf.options.inside.maas.template (+7/-0) src/maasserver/management/commands/edit_named_options.py (+1/-1) src/maasserver/tests/test_commands_edit_named_options.py (+2/-2) src/provisioningserver/dns/config.py (+38/-0) src/provisioningserver/dns/testing.py (+2/-0) src/provisioningserver/dns/tests/test_config.py (+117/-0) src/provisioningserver/utils/isc.py (+3/-1) src/provisioningserver/utils/tests/test_isc.py (+6/-2) |
To merge this branch: | bzr merge lp:~mpontillo/maas/dns-template-changes-trunk |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Raphaël Badin (community) | Approve | ||
Andres Rodriguez (community) | Approve | ||
Review via email: mp+264216@code.launchpad.net |
Commit message
Allow users to define allow-* parameters in named.conf.options; prevent writing them to named.conf.
Description of the change
* Add code to parse /etc/bind/
* Move ISC parsing utilities to provisioningserver so they can be used within the DNS writing code (which seems like it doesn't belong in provisioningserver in the first place, but moving that seemed more risky than moving a single utility, and the ISC parsing utilities might one day be used on the cluster.)
lgtm! Although, I'd like someone else to review as well.