Skip to content

Conversation

@AntProBasealt
Copy link
Collaborator

Add rights for bind-dns.

@AntProBasealt AntProBasealt requested a review from mastersin August 4, 2021 10:09
@AntProBasealt AntProBasealt self-assigned this Aug 4, 2021
dc.vm.network "private_network", ip: "10.64.6.%s" % [10+n], netmask: 24, virtualbox__intnet: "intnet"
dc.vm.box = "BaseALT/alt-server-8.2-x86_64"
dc.vm.box_version = "1.0.0"
dc.vm.box_download_checksum = "db0d0f26575bafd0bb4989dc00d85539e0fd9272c943b373ebbb6d978a3c15ee"
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why are you deleting lines with checksum?

shell: grep -q 'bind-dns' /etc/bind/named.conf || echo 'include "/var/lib/samba/bind-dns/named.conf";' >> /etc/bind/named.conf

- name: generate options.conf for bind
template:
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please specify file permissions while deploying file.

shell: grep -q KRB5RCACHETYPE /etc/sysconfig/bind || echo 'KRB5RCACHETYPE="none"' >> /etc/sysconfig/bind

- name: Add BIND9_DLZ plugin for bind
shell: grep -q 'bind-dns' /etc/bind/named.conf || echo 'include "/var/lib/samba/bind-dns/named.conf";' >> /etc/bind/named.conf
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I suggest looking at lineinfile Ansible module for this kind of tasks.

//listen-on { localnets; {{ samba_network }}; };
forwarders { {{ samba_dns_forward }}; };
allow-query { localnets; {{ samba_network }}; };
//allow-query-cache { localnets; {{ samba_network }}; };
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I suggest to remove commented lines.

@ghost ghost added the enhancement New feature or request label Aug 4, 2021
@AntProBasealt AntProBasealt force-pushed the dns_backend branch 2 times, most recently from 8d58404 to 1dea607 Compare August 31, 2021 14:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants