| Age | Commit message (Collapse) | Author | Files | Lines |
|---|
|
* Added minimal-responses plugin
Signed-off-by: Soumya Ghosh Dastidar <gdsoumya@gmail.com>
* Removed unnecessary comments
* Updated tests
Signed-off-by: Soumya Ghosh Dastidar <gdsoumya@gmail.com>
* Reformated imports
Signed-off-by: Soumya Ghosh Dastidar <gdsoumya@gmail.com>
* Updated package name
Signed-off-by: Soumya Ghosh Dastidar <gdsoumya@gmail.com>
* Removed unnecessary comments
Co-authored-by: Miek Gieben <miek@miek.nl>
* Added changes
Signed-off-by: Soumya Ghosh Dastidar <gdsoumya@gmail.com>
* updated
Signed-off-by: Soumya Ghosh Dastidar <gdsoumya@gmail.com>
* Updated comment for NextOrFailure
Signed-off-by: Soumya Ghosh Dastidar <gdsoumya@gmail.com>
* Updated to test.Case for testing
Signed-off-by: Soumya Ghosh Dastidar <gdsoumya@gmail.com>
* Formated imports using goimports
Signed-off-by: Soumya Ghosh Dastidar <gdsoumya@gmail.com>
Co-authored-by: Miek Gieben <miek@miek.nl>
|
|
* spelling fixes
* its/it's
|
|
|
|
* more
Signed-off-by: Miek Gieben <miek@miek.nl>
* Remove server addr from the context
This was added twice, just leave the server which also holds the
address.
Conflicts with #2719 but should be easy to fix.
Signed-off-by: Miek Gieben <miek@miek.nl>
* doesn't need server context
Signed-off-by: Miek Gieben <miek@miek.nl>
|
|
|
|
Got a bit messed up with stb lib "context" usage.
|
|
* global: move to context
Move from golang.org/x/net/context to std lib's context.
Change done with:
for i in $(grep -l '/context' **/*.go); do sed -e 's|golang.org/x/net/context|context|' -i $i; echo $i; done
for i in **/*.go; do goimports -w $i; done
* drop from dns.pb.go as well
|
|
* plugin/metrics: set server address in context
Allow cross server block metrics to co-exist; for this we should label
each metric with the server label. Put this information in the context
and provide a helper function to get it out.
Abstracting with entirely away with difficult as the release client_go
(0.8.0) doesn't have the CurryWith functions yet. So current use is like
so:
define metric, with server label:
RcodeCount = prometheus.NewCounterVec(prometheus.CounterOpts{
Namespace: plugin.Namespace,
Subsystem: "forward",
Name: "response_rcode_count_total",
Help: "Counter of requests made per upstream.",
}, []string{"server", "rcode", "to"})
And report ith with the helper function metrics.WithServer:
RcodeCount.WithLabelValues(metrics.WithServer(ctx), rc, p.addr).Add(1)
|
|
* plugins: Return error for multiple use of some
Return plugin.ErrOnce when a plugin that doesn't support it, is called
mutliple times.
This now adds it for: cache, dnssec, errors, forward, hosts, nsid.
And changes it slightly in kubernetes, pprof, reload, root.
* more tests
|
|
* Add pkg/fall for Fallthrough
Move this into it's own package to facilitate tests. Important bug
was fixed: make the names fully qualified.
Add fall package to hosts, reverse, etcd, and fix kubernetes and any
tests. The k8s tests are still as-is, might need a future cleanup.
|
|
* Add zone filtering to fallthrough
* Doh. gofmt
* Update documentation
|
|
|
|
* metrics: use exponentional buckets
See https://github.com/kubernetes/dns/pull/168
|
|
* Use seconds for bucketing.
* Add template for high accuracy timing buckets to avoid copy-pasta.
|
|
* Use gometalinter and enforcing go fmt/lint/vet
Before this PR go fmt is enabled, go lint is suggest only.
From time to time we have to manually check for go lint and go vet
for any issues.
This fix uses gometalinter and enforcing go fmt/lint/vet.
Several reasons:
- gometalinter could handle multiple linters concurrently
- gometalinter supports suppression with `// nolint[: <linter>]`
Previously one reason we didn't enable go lint was due to the
```
warning: context.Context should be the first parameter of a function (golint)
```
this is now possible with gometalinter and `// nolint: golint` (See changes).
This fix also discovered several go vet issues and fixes it.
Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
* Fix several issues reported by gometalinter (go vet)
This commit fixes several issues reported by gometalinter (go vet).
Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
* Increase deadline
Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
|
|
* Rename middleware to plugin
first pass; mostly used 'sed', few spots where I manually changed
text.
This still builds a coredns binary.
* fmt error
* Rename AddMiddleware to AddPlugin
* Readd AddMiddleware to remain backwards compat
|
Soumya Ghosh Dastidar
Chris O'Haver
Miek Gieben
moredhel
John Belamaric
Rene Treffer
Ben Kochie
Yong Tang
