Use %d SOA serial when logging about the SOA serial.
Signed-off-by: Miek Gieben <miek@miek.nl>

Wanted this *before* 1.6.1 but I forgot; ah well...
Signed-off-by: Miek Gieben <miek@miek.nl>

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

Various fixes to make things less flaky:

* kubernetes: put klog.SetOutput in the setup function, not in the init
  function to see if that helps
* file: make z.Expired a boolean instead of a pointer to a boolean
* test: fix TestSecondaryZoneTransfer test, which wasn't actually
  testing in the right way. It's more right now, but may still be racy
  (race introduced because a file's lazy loading of zones)
Signed-off-by: Miek Gieben <miek@miek.nl>

Add the new (short) notes for the 1.6.1 release.
Signed-off-by: Miek Gieben <miek@miek.nl>

* plugin/hosts: create inline map in setup

The inline map wasn't create in the setup.go fuction leading to a crash,
which is masked by a recover (but leads to a SERVFAIL, and not logging
the request).

Various other simplifications.

host plugin could use some test that actually cover these edgecases.
Signed-off-by: Miek Gieben <miek@miek.nl>

* PR review changes
Signed-off-by: Miek Gieben <miek@miek.nl>

without the paging parameter set, it will default to 100 records per request. with large enough zones and
potentially multiple coredns daemons configured to pull from route 53, this can quickly add up and reach
aws global api rate limits (5/sec per root account). increasing paging to max can help reduce the number
of requests needed to pull records for a zone without no down side that i am aware of.

this helps issue #2353, but probably is not a complete fix.
Signed-off-by: Matt Kulka <mkulka@parchment.com>

These are too hidden now. They increase the issue-load, because people
don't see them.

Add log.Errorf in the core/dnsserver recover routine.
Signed-off-by: Miek Gieben <miek@miek.nl>

Signed-off-by: alan <zg.zhu@daocloud.io>

Move github.com/openzipkin/zipkin-go-opentracing => github.com/openzipkin-contrib/zipkin-go-opentracing (#3070)

Move github.com/openzipkin/zipkin-go-opentracing => github.com/openzipkin-contrib/zipkin-go-opentracing
and run `go mod tidy`

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

Automatically submitted.

I don't believe this is actually needed (anymore). The:

os.Stderr = os.Stdout

is a crazy hack that def. needs to go.
Signed-off-by: Miek Gieben <miek@miek.nl>

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

Signed-off-by: zhengguang zhu <zhengguang.zhu@daocloud.io>

Bumps [github.com/aws/aws-sdk-go](https://github.com/aws/aws-sdk-go) from 1.21.2 to 1.21.6.
- [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.21.2...v1.21.6)
Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

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

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

* fix the notes and set release date to today
* run a 'make -f Makefile.doc' to generate the manal pages.
Signed-off-by: Miek Gieben <miek@miek.nl>

* contrib: add words on adding new plugins

New plugins are usually dropped in one giant ball of code, add some
words on how this can be approached and made to work faster.

(will at least give us something to point at)
Signed-off-by: Miek Gieben <miek@miek.nl>

* Update CONTRIBUTING.md
Co-Authored-By: Michael Grosser <development@stp-ip.net>

* Update CONTRIBUTING.md
Co-Authored-By: Michael Grosser <development@stp-ip.net>

* Update CONTRIBUTING.md
Co-Authored-By: Michael Grosser <development@stp-ip.net>

* Update CONTRIBUTING.md

* Update CONTRIBUTING.md
Co-Authored-By: Michael Grosser <development@stp-ip.net>

Read lock before reading the Expired field of a zone.

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

* Fix corefile usage

* plugin/file: allow README.md testing

Allow readme testing for the file plugin and fix bugs that where found:

* the reader wasn't reset when re-reading the same io.reader for a
  different origin.
Signed-off-by: Miek Gieben <miek@miek.nl>

* Update test/example_test.go
Co-Authored-By: Michael Grosser <development@stp-ip.net>

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

* Update CONTRIBUTING.md for dependency update methods (go dep -> go mod)
Signed-off-by: Yong Tang <yong.tang.github@outlook.com>

* Small fix for reveiw comment
Signed-off-by: Yong Tang <yong.tang.github@outlook.com>

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

The host plugin kept on adding entries instead of overwriting. Split the
inline cache off from the /etc/hosts file cache and clear /etc/hosts
file cache and re-parsing.

A bunch of other cleanup as well. Use functions defined in the plugin
package, don't re-parse strings if you don't have to and use To4() to
check the family for IP addresses. Fix all test cases a carried entries
are always fqdn-ed. Various smaller cleanup in unnessacry constants.

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

* Add 1.6.0 release notes

Preliminary notes for the 1.6.0 release
Signed-off-by: Miek Gieben <miek@miek.nl>

* Update notes/coredns-1.6.0.md
Co-Authored-By: Michael Grosser <development@stp-ip.net>

* Update notes/coredns-1.6.0.md
Co-Authored-By: Michael Grosser <development@stp-ip.net>

* plugin/file: simplify locking

Simplify the locking, remove the reloadMu and just piggyback on the
other lock for accessing content, which assumes things can be move
underneath.

Copy the Apex and Zone to new vars to make sure the pointer isn't
updated from under us.

The releadMu isn't need at all, the time.Ticker firing while we're
reading means we will just miss that tick and get it on the next go.

Add rrutil subpackage and put some more generic functions in there, that
are now used from file and the tree package. This removes some
duplication.

Rename additionalProcessing that didn't actually do that to
externalLookup, because that's what being done at some point.
Signed-off-by: Miek Gieben <miek@miek.nl>

* Update plugin/file/lookup.go
Co-Authored-By: Michael Grosser <development@stp-ip.net>

Bumps [github.com/aws/aws-sdk-go](https://github.com/aws/aws-sdk-go) from 1.21.1 to 1.21.2.
- [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.21.1...v1.21.2)
Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Bumps [github.com/aws/aws-sdk-go](https://github.com/aws/aws-sdk-go) from 1.21.0 to 1.21.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.21.0...v1.21.1)
Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

After calling NewZone the number of records should be zero, but due to
how zone.All() was implemented so empty RRs would be added. This then
fails the == 0 check in xfr.go and put nil in the slice, this then
subsequently panics on the Len().

Fix this making All() smarter when adding records. Added little test to
enfore this.
Signed-off-by: Miek Gieben <miek@miek.nl>

Bumps [github.com/aws/aws-sdk-go](https://github.com/aws/aws-sdk-go) from 1.20.20 to 1.21.0.
- [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.20.20...v1.21.0)
Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

* pkg/cache: fix race in Add() and Evict()

This fixes a race in Add() when the shard is at max capacity and the key
being added is already stored.  Previously, the shard would evict a
random value - when all it needed to do was replace an existing value.
There was a race in how Evict() picked which key to remove, which would
cause concurrent calls to Evict() to remove the same key.

Additionally, this commit removes a lot of the lock contention and a
race around Add() and Evict() by changing them to immediately hold the
write lock.  Previously, they would check conditions with the read lock
held and not re-check those conditions once the write lock was acquired
(this is a race).

* pkg/cache: code review comments

* pkg/cache: simplify Add() logic

Just a nitpicky typo fix.

* plugin/file: Rename do to walk, cleanup and document

* This renames Do to Walk to be more inline with Go standards. Also make
  it return an error instead of a bool.
  Also give give walk access to rrs. Alternatively e.m could be
  exported, but just access the map of rrs should work as well. Another
  alternative would be adding a whole bunch of helper functions, but
  those need grab and return the data. Just having access to the rrs
  should be easiest for most Walks.
* It adds Type and TypeForWildcard to show the different functions
* *Removes* the identical RR check when inserting; this was only done
  for A, AAAA and MX and not finished; removed under the mantra garbage
  in garbage out.
* Reuses Types to return all the types in an *tree.Elem
Signed-off-by: Miek Gieben <miek@miek.nl>

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