aboutsummaryrefslogtreecommitdiff
path: root/plugin/kubernetes/reverse.go
diff options
context:
space:
mode:
authorGravatar Chris O'Haver <cohaver@infoblox.com> 2018-10-10 15:28:45 -0400
committerGravatar John Belamaric <jbelamaric@google.com> 2018-10-10 12:28:45 -0700
commit974ed086f25ad45a01947e276e2eb8aa73d007a3 (patch)
tree2c1468f5a4e90634382a81904ef2736ca44198a7 /plugin/kubernetes/reverse.go
parent8432f1420732e61cb1a7ebb9d6446db6f43aa850 (diff)
downloadcoredns-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.go36
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}
}
}
}