Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[flagday2020] Add instructions for CoreDNS #148

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

chantra
Copy link
Contributor

@chantra chantra commented Jan 29, 2021

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
Copy link
Contributor Author

chantra commented Jan 29, 2021

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
Copy link

miekg commented Jan 30, 2021

yep that's correct

@chantra
Copy link
Contributor Author

chantra commented Jan 30, 2021

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
Copy link
Contributor Author

chantra commented Feb 1, 2021

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
Copy link

miekg commented Feb 1, 2021

/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