Automatically submitted.

Caught my eye, we name things directive still, esp when talking about
the prometheus *plugin*. Rename everything that needs to be plugin to
'plugin'. Also make sure Metrics is a H2 section (not H1).
Signed-off-by: Miek Gieben <miek@miek.nl>

Other latency metrics have `_duration` in the name change this metric
to be in sync with the other ones.
Signed-off-by: Miek Gieben <miek@miek.nl>

* circle CI: run with latest Go

We compile coredns with latest Go on each release, do this same in
Circle-ci
Signed-off-by: Miek Gieben <miek@miek.nl>

This upgrade etcd to v3.4.1:

    go get go.etcd.io/etcd@a14579fbfb1a000439a40abf71862df51b0a2136

this is the commit latest on the release page.
Signed-off-by: Miek Gieben <miek@miek.nl>

Signed-off-by: Miek Gieben <miek@miek.nl>

For now metric is measure only for headless services. Informer has been slighlty
refactored, so the code can measure latency without storing extra fields on
Endpoint struct.
Signed-off-by: Janek Łukaszewicz <janluk@google.com>

Suggestions from code review
Co-Authored-By: Chris O'Haver <cohaver@infoblox.com>

* Make request.Request smaller

This makes the request struct smaller and removes the pointer to the do
boolean (tri-bool) as size == 0 will indicate if we have cached it.

Family can be a int8 because it only carries 3 values, Size itself is
just a uint16 under the covers.

This is a more comprehensive fix than #3292

Closes #3292
Signed-off-by: Miek Gieben <miek@miek.nl>

* cache: fix test

this now needs a valid response writter
Signed-off-by: Miek Gieben <miek@miek.nl>

The goroutine closure was causing objects to be heap allocated.  Moving
it to a separate function fixes that.

```benchmark                                old ns/op     new ns/op     delta
BenchmarkCacheResponse/NoPrefetch-12     773           713           -7.76%
BenchmarkCacheResponse/Prefetch-12       878           837           -4.67%
BenchmarkHash-12                         9.17          9.18          +0.11%

benchmark                                old allocs     new allocs     delta
BenchmarkCacheResponse/NoPrefetch-12     9              8              -11.11%
BenchmarkCacheResponse/Prefetch-12       9              8              -11.11%
BenchmarkHash-12                         0              0              +0.00%

benchmark                                old bytes     new bytes     delta
BenchmarkCacheResponse/NoPrefetch-12     471           327           -30.57%
BenchmarkCacheResponse/Prefetch-12       471           327           -30.57%
BenchmarkHash-12                         0             0             +0.00%
```
Signed-off-by: Charlie Vieth <charlie.vieth@gmail.com>
Signed-off-by: Miek Gieben <miek@miek.nl>

Don't perform this code in the init, this allocated 1 megabyte of memory
even if you don't use the plugin. Looks to be only there for testing,
adding a comment to reflect that.

Fixes #3342
Signed-off-by: Miek Gieben <miek@miek.nl>

* plugin/clouddns: remove initialization from init

Init should just call the plugin.Register with a setup function.

Fixes: #3343
Signed-off-by: Miek Gieben <miek@miek.nl>

* Fix placement for var f
Signed-off-by: Miek Gieben <miek@miek.nl>

Signed-off-by: Hauke Löffler <hloeffler@users.noreply.github.com>

Signed-off-by: Yong Tang <yong.tang.github@outlook.com>

Fix the documentation, remove autopath entry and fix the transfer by
copying some bits from the file plugin.
Signed-off-by: Miek Gieben <miek@miek.nl>

This fixes the vet warning: the cancel function returned by
context.WithTimeout should be called, not discarded, to avoid a context
leak.
Signed-off-by: Ingo Gottwald <in.gottwald@gmail.com>

* Move map to array

The map was not needed move to an array, see #1941 for the original
idea. That of course didn't apply anymore; make a super minimal change
to implements the idea from #1941
Signed-off-by: Miek Gieben <miek@miek.nl>

* Add total count
Signed-off-by: Miek Gieben <miek@miek.nl>

* plugin/forward: may Yield not block

Yield may block when we're super busy with creating (and looking) for
connection. Set a small timeout on Yield, to skip putting the connection
back in the queue.

Use persistentConn troughout the socket handling code to be more
consistent.
Signed-off-by: Miek Gieben <miek@miek.nl>

Dont do
Signed-off-by: Miek Gieben <miek@miek.nl>

* Set used in Yield

This gives one central place where we update used in the persistConns
Signed-off-by: Miek Gieben <miek@miek.nl>

Signed-off-by: utrehubenka <utrehubenka@infoblox.com>

* Use session.NewSession instead of session.New
Signed-off-by: Erfan Besharat <erbesharat@gmail.com>

* Use grpc.DialContext instead of grpc.WithTimeout
Signed-off-by: Erfan Besharat <erbesharat@gmail.com>

* Pass non-nil context to context.WithTimeout
Signed-off-by: Erfan Besharat <erbesharat@gmail.com>

* Return the error directly in route53 setup
Co-Authored-By: Miek Gieben <miek@miek.nl>
Signed-off-by: Erfan Besharat <erbesharat@gmail.com>

* Run gostaticheck

Run gostaticcheck on the codebase and fix almost all flagged items.

Only keep

* coremain/run.go:192:2: var appVersion is unused (U1000)
* plugin/chaos/setup.go:54:3: the surrounding loop is unconditionally terminated (SA4004)
* plugin/etcd/setup.go:103:3: the surrounding loop is unconditionally terminated (SA4004)
* plugin/pkg/replacer/replacer.go:274:13: argument should be pointer-like to avoid allocations (SA6002)
* plugin/route53/setup.go:124:28: session.New is deprecated: Use NewSession functions to create sessions instead. NewSession has the same functionality as New except an error can be returned when the func is called instead of waiting to receive an error until a request is made.  (SA1019)
* test/grpc_test.go:25:69: grpc.WithTimeout is deprecated: use DialContext and context.WithTimeout instead.  Will be supported throughout 1.x.  (SA1019)

The first one isn't true, as this is set via ldflags. The rest is
minor. The deprecation should be fixed at some point; I'll file some
issues.
Signed-off-by: Miek Gieben <miek@miek.nl>

* Make sure to plug in the plugins

import the plugins, that file that did this was removed, put it in the
reload test as this requires an almost complete coredns server.
Signed-off-by: Miek Gieben <miek@miek.nl>

Bumps [github.com/aws/aws-sdk-go](https://github.com/aws/aws-sdk-go) from 1.24.3 to 1.25.1.
- [Release notes](https://github.com/aws/aws-sdk-go/releases)
- [Changelog](https://github.com/aws/aws-sdk-go/blob/master/CHANGELOG.md)
- [Commits](https://github.com/aws/aws-sdk-go/compare/v1.24.3...v1.25.1)
Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Bumps [google.golang.org/grpc](https://github.com/grpc/grpc-go) from 1.23.0 to 1.24.0.
- [Release notes](https://github.com/grpc/grpc-go/releases)
- [Commits](https://github.com/grpc/grpc-go/compare/v1.23.0...v1.24.0)
Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

* Makefiles: remove GO111MODULE

This is the default in 1.13 it can be removed; also make the test out
log less, so failure are more pronounced in the travis out, hide
coverage stats for instance.
Signed-off-by: Miek Gieben <miek@miek.nl>

* Kill -v as wel
Signed-off-by: Miek Gieben <miek@miek.nl>

Remove all these uses and just make them work on caddy.Controller. Also
don't export parsing functions as their should be private to the plugin.
Signed-off-by: Miek Gieben <miek@miek.nl>

These plugins where missed in #3287 because their setup is done in a file
other than setup.go
Signed-off-by: Miek Gieben <miek@miek.nl>

Go gen updated the deps this is why 1.6.4 has that suffix. Do a go gen
and a go mod tidy.

Fixes: #3318
Signed-off-by: Miek Gieben <miek@miek.nl>

Signed-off-by: Miek Gieben <miek@miek.nl>

Automatically submitted.

add the acl manual page; mechanical change otherwise.
Signed-off-by: Miek Gieben <miek@miek.nl>

* plugin/reload: reflow documentation

For some reason these we're all bullets points, which made for awkward
reading.
Signed-off-by: Miek Gieben <miek@miek.nl>

* typo
Signed-off-by: Miek Gieben <miek@miek.nl>

* Update plugin/reload/README.md
Co-Authored-By: Michael Grosser <development@stp-ip.net>

Signed-off-by: Guangming Wang <guangming.wang@daocloud.io>

Remove talk about labels that are not added.
Signed-off-by: Miek Gieben <miek@miek.nl>

Signed-off-by: xieyanker <xjsisnice@gmail.com>

Automatically submitted.

TrimPrefix re-assign to former variable
Signed-off-by: Guangming Wang <guangming.wang@daocloud.io>

Bumps [github.com/aws/aws-sdk-go](https://github.com/aws/aws-sdk-go) from 1.23.21 to 1.24.3.
- [Release notes](https://github.com/aws/aws-sdk-go/releases)
- [Changelog](https://github.com/aws/aws-sdk-go/blob/master/CHANGELOG.md)
- [Commits](https://github.com/aws/aws-sdk-go/compare/v1.23.21...v1.24.3)
Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Bumps [gopkg.in/DataDog/dd-trace-go.v1](https://github.com/DataDog/dd-trace-go) from 1.17.0 to 1.18.0.
- [Release notes](https://github.com/DataDog/dd-trace-go/releases)
- [Commits](https://github.com/DataDog/dd-trace-go/compare/v1.17.0...v1.18.0)
Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Signed-off-by: hwdef <hwdef97@gmail.com>

Signed-off-by: Guangming Wang <guangming.wang@daocloud.io>

Abstract the caddy call and make it simpler.

See #3261 for some part of the discussion.

Go from:

~~~ go
func init() {
       caddy.RegisterPlugin("any", caddy.Plugin{
               ServerType: "dns",
               Action:     setup,
       })
}
~~~

To:

~~~ go
func init() { plugin.Register("any", setup) }
~~~

This requires some external documents in coredns.io to be updated as
well; the old way still works, so it's backwards compatible.
Signed-off-by: Miek Gieben <miek@miek.nl>