diff options
author | 2018-10-10 15:28:45 -0400 | |
---|---|---|
committer | 2018-10-10 12:28:45 -0700 | |
commit | 974ed086f25ad45a01947e276e2eb8aa73d007a3 (patch) | |
tree | 2c1468f5a4e90634382a81904ef2736ca44198a7 /plugin/kubernetes/reverse.go | |
parent | 8432f1420732e61cb1a7ebb9d6446db6f43aa850 (diff) | |
download | coredns-974ed086f25ad45a01947e276e2eb8aa73d007a3.tar.gz coredns-974ed086f25ad45a01947e276e2eb8aa73d007a3.tar.zst coredns-974ed086f25ad45a01947e276e2eb8aa73d007a3.zip |
use keys (#2167)
Diffstat (limited to 'plugin/kubernetes/reverse.go')
-rw-r--r-- | plugin/kubernetes/reverse.go | 36 |
1 files changed, 18 insertions, 18 deletions
diff --git a/plugin/kubernetes/reverse.go b/plugin/kubernetes/reverse.go index 5873bcbc8..1e060283c 100644 --- a/plugin/kubernetes/reverse.go +++ b/plugin/kubernetes/reverse.go @@ -18,35 +18,35 @@ func (k *Kubernetes) Reverse(state request.Request, exact bool, opt plugin.Optio return nil, e } - records := k.serviceRecordForIP(ip, state.Name()) - if len(records) == 0 { - return records, errNoItems + record := k.serviceRecordForIP(ip, state.Name()) + if record == nil { + return nil, errNoItems } - return records, nil + return []msg.Service{*record}, nil } // serviceRecordForIP gets a service record with a cluster ip matching the ip argument // If a service cluster ip does not match, it checks all endpoints -func (k *Kubernetes) serviceRecordForIP(ip, name string) []msg.Service { +func (k *Kubernetes) serviceRecordForIP(ip, name string) *msg.Service { // First check services with cluster ips - for _, service := range k.APIConn.SvcIndexReverse(ip) { + service := k.APIConn.SvcIndexReverse(ip) + if service != nil { if len(k.Namespaces) > 0 && !k.namespaceExposed(service.Namespace) { - continue + return nil } domain := strings.Join([]string{service.Name, service.Namespace, Svc, k.primaryZone()}, ".") - return []msg.Service{{Host: domain, TTL: k.ttl}} + return &msg.Service{Host: domain, TTL: k.ttl} } // If no cluster ips match, search endpoints - for _, ep := range k.APIConn.EpIndexReverse(ip) { - if len(k.Namespaces) > 0 && !k.namespaceExposed(ep.Namespace) { - continue - } - for _, eps := range ep.Subsets { - for _, addr := range eps.Addresses { - if addr.IP == ip { - domain := strings.Join([]string{endpointHostname(addr, k.endpointNameMode), ep.Name, ep.Namespace, Svc, k.primaryZone()}, ".") - return []msg.Service{{Host: domain, TTL: k.ttl}} - } + ep := k.APIConn.EpIndexReverse(ip) + if ep == nil || len(k.Namespaces) > 0 && !k.namespaceExposed(ep.Namespace) { + return nil + } + for _, eps := range ep.Subsets { + for _, addr := range eps.Addresses { + if addr.IP == ip { + domain := strings.Join([]string{endpointHostname(addr, k.endpointNameMode), ep.Name, ep.Namespace, Svc, k.primaryZone()}, ".") + return &msg.Service{Host: domain, TTL: k.ttl} } } } |