Skip to content

Dns backend#25

Open
AntProBasealt wants to merge 5 commits intomasterfrom
dns_backend
Open

Dns backend#25
AntProBasealt wants to merge 5 commits intomasterfrom
dns_backend

Conversation

@AntProBasealt
Copy link
Copy Markdown
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
Comment thread Vagrantfile
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
Copy Markdown

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?

Comment thread tasks/bind.yml
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
Copy Markdown

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.

Comment thread tasks/bind.yml
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
Copy Markdown

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.

Comment thread templates/options.conf.j2
//listen-on { localnets; {{ samba_network }}; };
forwarders { {{ samba_dns_forward }}; };
allow-query { localnets; {{ samba_network }}; };
//allow-query-cache { localnets; {{ samba_network }}; };
Copy link
Copy Markdown

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.

1 participant