diff options
author | 2017-08-19 07:18:35 +0100 | |
---|---|---|
committer | 2017-08-19 07:18:35 +0100 | |
commit | 627687b11fe56d3235a2f6a5055f1c6749752096 (patch) | |
tree | e07143ff472de3243cf48fc365ab0c7d821c10ee /middleware/kubernetes/parse.go | |
parent | f96cf27193032120ba727316c4057dffed0cbe48 (diff) | |
download | coredns-627687b11fe56d3235a2f6a5055f1c6749752096.tar.gz coredns-627687b11fe56d3235a2f6a5055f1c6749752096.tar.zst coredns-627687b11fe56d3235a2f6a5055f1c6749752096.zip |
mw/kubernetes: remove zone from parseRequest (#938)
* mw/kubernetes: remove zone from parseRequest
State has the zone info as well, so don't need to have it in
parseRequest anymore.
* Fix up tests
* improve test coverage
Diffstat (limited to 'middleware/kubernetes/parse.go')
-rw-r--r-- | middleware/kubernetes/parse.go | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/middleware/kubernetes/parse.go b/middleware/kubernetes/parse.go index c7d614ec1..700dc654a 100644 --- a/middleware/kubernetes/parse.go +++ b/middleware/kubernetes/parse.go @@ -20,7 +20,6 @@ type recordRequest struct { // A each name can be for a pod or a service, here we track what we've seen. This value is true for // pods and false for services. If we ever need to extend this well use a typed value. podOrSvc string - zone string } // parseRequest parses the qname to find all the elements we need for querying k8s. @@ -34,8 +33,6 @@ func (k *Kubernetes) parseRequest(state request.Request) (r recordRequest, err e base, _ := dnsutil.TrimZone(state.Name(), state.Zone) segs := dns.SplitDomainName(base) - r.zone = state.Zone - offset := 0 if state.QType() == dns.TypeSRV { // The kubernetes peer-finder expects queries with empty port and service to resolve @@ -101,6 +98,5 @@ func (r recordRequest) String() string { s += "." + r.service s += "." + r.namespace s += "." + r.podOrSvc - s += "." + r.zone return s } |