Skip to content

[flagday2020] Add instructions for CoreDNS - #148

Open
chantra wants to merge 2 commits into
dns-violations:masterfrom
chantra:patch-1
Open

[flagday2020] Add instructions for CoreDNS#148
chantra wants to merge 2 commits into
dns-violations:masterfrom
chantra:patch-1

Conversation

@chantra

@chantra chantra commented Jan 29, 2021

Copy link
Copy Markdown
Contributor

CoreDNS has a bufsize plugin that can be used to force the bufsize used. This PR adds instruction to set it to 1232 in CoreDNS config.

CoreDNS has a [bufsize plugin](https://coredns.io/plugins/bufsize/) that can be used to force the bufsize used. This PR adds instruction to set it to 1232 in CoreDNS config.
@chantra

chantra commented Jan 29, 2021

Copy link
Copy Markdown
Contributor Author

Happy to add to each individual localized index files if @miekg could confirm this is indeed the correct way to set it.

@chantra chantra changed the title Add instructions for CoreDNS [flagday2020] Add instructions for CoreDNS Jan 29, 2021
@miekg

miekg commented Jan 30, 2021

Copy link
Copy Markdown

yep that's correct

@chantra

chantra commented Jan 30, 2021

Copy link
Copy Markdown
Contributor Author

Great. I will update the rest of the files whenever I get close to a computer.

Each index file was duplicating the same config stanzas, this is prone
to getting out of sync. As I was adding the `CoreDNS` stanza to
dnsflagday, I updated each index file to use 1 source of truth instead
of just adding the CoreDNS specific config.

* Move server configs for 2020 flagday in `_includes/2020_server_configs.md`
* update all index files to include this file instead of duplicating
  configs in each files.
@chantra

chantra commented Feb 1, 2021

Copy link
Copy Markdown
Contributor Author

ok, I think this is good for review.

Instead of copy/pasting the same config in all index files, I moved the configs into a separate .md file and included it from each index files.
Ran jekyll serve on my box, pages are still showing the configs, and all include the CoreDNS one.
The CSS seems to be messed up because the default layout is not applied, but that is unrelated to this change.

@miekg

miekg commented Feb 1, 2021

Copy link
Copy Markdown

/lgtm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants