aboutsummaryrefslogtreecommitdiff
AgeCommit message (Collapse)AuthorFilesLines
2019-05-31make sure client CA and auth type are set if CA is explicitly specified. (#2825)Gravatar JINMEI Tatuya 6-1/+160
* make sure client CA and auth type are set if CA is explicitly specified. added some simple tests to confirm the effect. * test certificates (forgot to add them in the previous commit) * made client auth policy configurable with new client_auth option. README has been updated accordingly. * fix editorial in README
2019-05-31kubernetes: bump to client-go 11.0.0 (#2854)Gravatar Chris O'Haver 4-22/+27
* client-go 11.0.0 * Core moved to CoreV1
2019-05-29Fix for #2842, instead of returning the first Pod, return the one whi… (#2846)Gravatar Andras Spitzer 2-0/+8
* Fix for #2842, instead of returning the first Pod, return the one which is Running * a more memory efficient version of the fix, string -> bool * fix with no extra fields in struct, return nil at Pod conversion if Pod is not Running * let Kuberneretes filter for Running Pods using FieldSelector * filter for Pods that are Running and Pending (implicit)
2019-05-27Add render.com to adopters list (#2847)Gravatar Anurag Goel 1-0/+1
2019-05-27Add server instance to the context in ServerTLS and ServerHTTPS (#2840)Gravatar Andrey Meshkov 3-3/+5
* Add server instance to the context in ServerTLS and ServerHTTPS The problem with the current code is that there's no way to get the server instance inside a plugin. Because of that "metrics" plugin sets empty "server" label for requests served over TLS or HTTPS. * use s.Server instead of s * Added server to the context of grpc requests
2019-05-27Update dns to v1.1.12, and remove duplicate dependencies (#2845)Gravatar Yong Tang 2-21/+52
This fix: 1. Update dns to v1.1.12 2. Redirect dns v1.1.3 in caddy to v1.1.12 so that there is only one version of dns. Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
2019-05-27Update shebangs for environmentally finding Bash -- Compatibility update to ↵Gravatar Alyx 5-5/+5
provide support for platforms that do not ship Bash in their /bin (#2841)
2019-05-27Update prometheus library to 0.9.3 (#2844)Gravatar Yong Tang 2-35/+81
Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
2019-05-23pkg/log: fix data race on d (#2698)Gravatar Miek Gieben 6-26/+47
* pkg/log: fix data race on d Wrap d in a mutex to prevent data race. This makes is slower, but this is a debugging aid anyway. It's not used normally. Signed-off-by: Miek Gieben <miek@miek.nl> * Fix tests compilation Signed-off-by: Miek Gieben <miek@miek.nl> * Fix test compile Signed-off-by: Miek Gieben <miek@miek.nl>
2019-05-18plugin/metrcs: fix datarace on listeners (#2835)Gravatar Miek Gieben 6-30/+62
This fixes a data race on the listener(s) that get started in the metrics plugins. It also restore pkg/uniq to its former glory and removes and state being carried in there; this means for metrics that registry.go was to replicate that behavior *with* locking (as pkg/uniq doesn't do, or need that). Also renamed uniqAddr to just u, to make it slightly shorter. Signed-off-by: Miek Gieben <miek@miek.nl>
2019-05-18pkg/log: reset the buffer in the tests (#2828)Gravatar Miek Gieben 1-0/+4
Reset the buf otherwise we're not checking the new value. Signed-off-by: Miek Gieben <miek@miek.nl>
2019-05-13plugin/metrics: fix failed reload (#2816)Gravatar Miek Gieben 3-30/+25
Fix metrics endpoint on a failed reload, follows the same lines as the previous PRs, see for e.g. 076b8d4f. Test with a Corefile with 2 server blocks and metrics enabled and then introducing a syntax error: ~~~ [ERROR] Restart failed: Corefile:5 - Error during parsing: Unknown directive 'jfkdjk' [ERROR] SIGUSR1: starting with listener file descriptors: Corefile:5 - Error during parsing: Unknown directive 'jfkdjk' ~~~ And then curl-ing the metrics endpoint. See #2659 and as this is the last one. Fixes: #2659 Getting this all right turns out to be tricky, also it's not easy testable which is something I should fix. Signed-off-by: Miek Gieben <miek@miek.nl>
2019-05-07make ignore empty work with ext svc types (#2823)Gravatar Chris O'Haver 2-1/+11
2019-05-05Rename middleware -> plugin in comment (#2818)Gravatar Yong Tang 1-1/+1
Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
2019-05-05plugin/autopath docs: remove last use of middleware (#2817)Gravatar Miek Gieben 1-1/+1
This has somehow survived; the docs still used Middleware instead of Plugins. Signed-off-by: Miek Gieben <miek@miek.nl>
2019-05-05docs: plugin.md (#2813)Gravatar Miek Gieben 1-13/+8
Update this file, give Readiness its own section and remove the talk about the *reverse* plugin as it does not exist in the main tree anymore. Signed-off-by: Miek Gieben <miek@miek.nl>
2019-05-04plugin/health: add OnRestartFailed (#2812)Gravatar Miek Gieben 3-10/+3
Add OnReStartFailed which makes the health plugin stay up if the Corefile is corrupt and we revert to the previous version. Also needs a fix for the channel handling See #2659 Testing it will log the following when restarting with a corrupted Corefile ~~~ 2019-05-04T18:01:59.431Z [INFO] linux/amd64, go1.12.4, CoreDNS-1.5.0 linux/amd64, go1.12.4, [INFO] SIGUSR1: Reloading [INFO] Reloading [ERROR] Restart failed: Corefile:5 - Error during parsing: Unknown directive 'bdhfhdhj' [ERROR] SIGUSR1: starting with listener file descriptors: Corefile:5 - Error during parsing: Unknown directive 'bdhfhdhj' ~~~ After which the curl still works. This also needed a change to reset the channel used for the metrics go-routine which gets closed on shutdown, otherwise you'll see: ~~~ ^C[INFO] SIGINT: Shutting down panic: close of closed channel goroutine 90 [running]: github.com/coredns/coredns/plugin/health.(*health).OnFinalShutdown(0xc000089bc0, 0xc000063d88, 0x4afe6d) ~~~ Signed-off-by: Miek Gieben <miek@miek.nl>
2019-05-04plugin/health: cleanups (#2811)Gravatar Miek Gieben 6-28/+14
Small, trivial cleanup: got triggered because I saw a comment on how health plugins polls other plugins which isn't true. * Remove useless newHealth function * healthParse -> parse * Remove useless constants Net deletion of code. Signed-off-by: Miek Gieben <miek@miek.nl>
2019-05-01kubernetes: never respond with NXDOMAIN for authority label (#2769)Gravatar Billie Cleek 2-3/+66
* kubernetes: never respond with NXDOMAIN for authority label Return a nodata response when trying to resolve the authority's label for a record type that doesn't match the record type of the authority. This guards against poisoning the authority record by requesting the wrong record type for the authority label. For instance, given an authoritative resolver that uses subdomain delegation for Kubernetes services of a cluster that's configured to use IPv4, the parent may be poisoned by querying it for the authority label of the cluster subdomain with a AAAA record type, which would otherwise (i.e. without this change) return an NXDOMAIN. That is, given cluster.example.com NS 10800 ns.dns.cluster.example.com ns.dns.cluster.example.com A 10800 10.0.1.2 The parent may be poisoned for the SOA TTL by querying it for a AAAA record of ns.dns.cluster.example.com, causing the parent to fail delegate properly until the SOA TTL lapses. * kubernetes: add tests for authority queries
2019-04-30Update README.md (#2808)Gravatar Cricket Liu 1-9/+9
Minor grammatical fixes.
2019-04-30plugin/loop: Update troubleshooting step (#2804)Gravatar Chris O'Haver 1-1/+1
* Update README.md
2019-04-29Remove trailing whitespaces (#2806)Gravatar Yong Tang 2-3/+3
While running make notices the following: ``` ** presubmit/trailing-whitespace plugin/grpc/README.md:Multiple upstreams are randomized (see `policy`) on first use. When a proxy returns an error plugin/pprof/setup_test.go: {`pprof :1234 { plugin/pprof/setup_test.go: {`pprof { ** presubmit/trailing-whitespace: please remove any trailing white space ``` This fix removes the whitespaces Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
2019-04-28Add any plugin (#2801)Gravatar Miek Gieben 8-0/+129
* Add any plugin This adds the any plugin, a plain copy of coredns/any documented here https://coredns.io/explugins/any/ as an external plugin. Fixes: #2785 Signed-off-by: Miek Gieben <miek@miek.nl> * Stickler bot nit Signed-off-by: Miek Gieben <miek@miek.nl>
2019-04-27Upgrade to Caddy v1 (#2800)Gravatar Michael Grosser 2-18/+53
Automatically submitted.
2019-04-24Update README.md (#2799)Gravatar Cricket Liu 1-1/+1
s/from/form
2019-04-23plugin/chaos: randomize author list (#2794)Gravatar Miek Gieben 1-2/+6
Randomize the author list on request; keep the zowners.go file stable so a 'go generate' remain stable. chaos.Owners could potentially be a map and be randomized by ranging over it, but this seems simpler and fewer lines of code. Bit of Easter hacking; seems more fair to randomize this list. Signed-off-by: Miek Gieben <miek@miek.nl>
2019-04-22Doc update (#2792)Gravatar Miek Gieben 10-124/+112
* Fix spelling in coredns.1 and corefile.7 md files * Run make -f Makefile.doc Signed-off-by: Miek Gieben <miek@miek.nl>
2019-04-18plugin/health: clarify use a bit (#2791)Gravatar Miek Gieben 1-4/+6
Make clearer how health works and that is it process wide. Signed-off-by: Miek Gieben <miek@miek.nl>
2019-04-11Remove varyoo from OWNERSGravatar varyoo 1-1/+0
Bye for now :-)
2019-04-08upgrade miekg/dns to 1.1.8 (#2774)Gravatar Miek Gieben 2-1/+3
Upgrade miekg/dns to 1.1.8 Signed-off-by: Miek Gieben <miek@miek.nl>
2019-04-08plugins: remove `upstream [ADDRESS]` from docs (#2772)Gravatar Miek Gieben 4-26/+13
Various plugins still had this documented, the setup functions still allow it (which is fine), but update the docs that this arguments isn't there any more. Signed-off-by: Miek Gieben <miek@miek.nl>
2019-04-08[plugin/cache] cache failures (#2720)Gravatar Kun Chang 3-8/+46
* cache failures * use ServerError
2019-04-08Run gofmt -w -s on codebase (#2773)Gravatar Miek Gieben 8-71/+71
This formats and simplifies all code by running gofmt -w -s on all Go files. Signed-off-by: Miek Gieben <miek@miek.nl>
2019-04-06WIP: travis changes (#2766)v1.5.0Gravatar Miek Gieben 1-9/+3
* WIP: travis changes Simplify Travis so it fails less often. We don't need docker any more, let alone trusty and sudo, so simplifies this, to just os: linux. Signed-off-by: Miek Gieben <miek@miek.nl> * This? Signed-off-by: Miek Gieben <miek@miek.nl>
2019-04-06Bump version to 1.5.0 (#2765)Gravatar Miek Gieben 1-1/+1
Signed-off-by: Miek Gieben <miek@miek.nl>
2019-04-06manpages: chop off first header (#2764)Gravatar Miek Gieben 39-48/+1
each readme starts with the plugin's name as a header, this needs to be chopped off to provide a good manual page. Signed-off-by: Miek Gieben <miek@miek.nl>
2019-04-06Doc tweaks (#2763)Gravatar Miek Gieben 2-3/+3
* fix date on 150 release notes * fix syntax in route53 plugin Signed-off-by: Miek Gieben <miek@miek.nl>
2019-04-06docs: Regenerate all manpages using mmark (#2762)Gravatar Miek Gieben 42-4186/+4657
Mmark recently became able to create manual pages. This removed the dependency on 'ronn' and just uses mmark (Go program). Re-hookup Makefile.doc to generate the correct header mmark needs to see and regenate them all. Spot checking a few pages suggest they look good and actually better than rendered with ronn, esp. lists in lists. Fixes #2757 Signed-off-by: Miek Gieben <miek@miek.nl>
2019-04-04plugin/forward: remove proxy comparison (#2760)Gravatar Miek Gieben 1-4/+0
*proxy* is removed, so this is moot Signed-off-by: Miek Gieben <miek@miek.nl>
2019-04-04Update coredns-1.5.0.md (#2759)Gravatar Chris O'Haver 1-11/+11
2019-04-03Update k8s line and add deprecations (#2758)Gravatar John Belamaric 1-2/+7
2019-04-03ore release notes: k8s plugin and chaos (#2755)Gravatar Miek Gieben 1-0/+3
Signed-off-by: Miek Gieben <miek@miek.nl>
2019-04-03Fix coredns.1 manpage (#2756)Gravatar Miek Gieben 1-4/+4
Signed-off-by: Miek Gieben <miek@miek.nl>
2019-04-03manifest-tool doesn't support mips (#2753)Gravatar Chris O'Haver 1-1/+5
* remove mips from manifest * derp * add comment
2019-04-02plugin/kubernetes: Disable resync by default (#2752)Gravatar John Belamaric 2-2/+3
* Disable resync by default * Document default resync
2019-04-02Add all policies to forward plugin docs (#2751)Gravatar Fernando Ripoll 1-0/+3
2019-04-02notes/150: spell check and expand it a little (#2749)Gravatar Miek Gieben 1-4/+14
Signed-off-by: Miek Gieben <miek@miek.nl>
2019-04-01Bump the resync period to five hours (#2748)Gravatar John Belamaric 1-1/+1
2019-04-01release: add 1.5.0 release notes (#2741)Gravatar Miek Gieben 1-0/+44
* release: add 1.5.0 release notes Add the release notes to the notes/ directory to prepare for a release. Signed-off-by: Miek Gieben <miek@miek.nl> * Update notes/coredns-1.5.0.md Co-Authored-By: miekg <miek@miek.nl> * More Signed-off-by: Miek Gieben <miek@miek.nl> * Update notes/coredns-1.5.0.md Co-Authored-By: miekg <miek@miek.nl>
2019-04-01Update README.md (#2747)Gravatar Chris O'Haver 1-1/+3