diff options
author | 2018-09-22 15:12:02 +0100 | |
---|---|---|
committer | 2018-09-22 15:12:02 +0100 | |
commit | 9546b606cb58a372361b39604c1ccf54ed91c7b9 (patch) | |
tree | dd218e883da9d27e34a76dc75407bccb3b32c938 /plugin/kubernetes/federation.go | |
parent | b3d69f1c7f226805e7f70cfd0e88b488dc2eab1b (diff) | |
download | coredns-9546b606cb58a372361b39604c1ccf54ed91c7b9.tar.gz coredns-9546b606cb58a372361b39604c1ccf54ed91c7b9.tar.zst coredns-9546b606cb58a372361b39604c1ccf54ed91c7b9.zip |
K8s remove string ops (#2119)
* plugin/kubernetes: remove bunch a string ops
This removes a bunch of appends to where not needed, makes dnsutil.Join
take variadic args which removes the need to wrap in a new string slice.
Signed-off-by: Miek Gieben <miek@miek.nl>
* Fix calls to dnsutil.Join
Signed-off-by: Miek Gieben <miek@miek.nl>
* Revert these
Signed-off-by: Miek Gieben <miek@miek.nl>
Diffstat (limited to 'plugin/kubernetes/federation.go')
-rw-r--r-- | plugin/kubernetes/federation.go | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/plugin/kubernetes/federation.go b/plugin/kubernetes/federation.go index 78e31668b..bf169b911 100644 --- a/plugin/kubernetes/federation.go +++ b/plugin/kubernetes/federation.go @@ -44,8 +44,8 @@ func (k *Kubernetes) Federations(state request.Request, fname, fzone string) (ms } if r.endpoint == "" { - return msg.Service{Host: dnsutil.Join([]string{r.service, r.namespace, fname, r.podOrSvc, lz, lr, fzone})}, nil + return msg.Service{Host: dnsutil.Join(r.service, r.namespace, fname, r.podOrSvc, lz, lr, fzone)}, nil } - return msg.Service{Host: dnsutil.Join([]string{r.endpoint, r.service, r.namespace, fname, r.podOrSvc, lz, lr, fzone})}, nil + return msg.Service{Host: dnsutil.Join(r.endpoint, r.service, r.namespace, fname, r.podOrSvc, lz, lr, fzone)}, nil } |