mw/kubernetes: remove federation and cidr (#916)

* mw/kubernetes: remove federation and cidr

Remove both as we have a corefile syntax change that handles cidr and
remove federation because that is going to be its own middleware.

* backwards incompat changes

This PR:
* removes cidr from kubernetes (core Corefile feature now)
* removes federation from kubernets (comes back as new middleware)
* [remove autopath - which was already gone, so that already was
  backwards incompat]
* adds `fallthrough` to the *etcd* middleware and makes you enable it.
* Fail on unknown properties
* documentation
* Disable TestHealthCheck as it uses realtime and fails

1 files changed, 2 insertions, 10 deletions

diff --git a/middleware/kubernetes/handler.go b/middleware/kubernetes/handler.go
index 9e63ef387..1e41e9ebf 100644
--- a/middleware/kubernetes/handler.go
+++ b/middleware/kubernetes/handler.go
@@ -25,18 +25,10 @@ func (k Kubernetes) ServeDNS(ctx context.Context, w dns.ResponseWriter, r *dns.M
 	// otherwise delegate to the next in the pipeline.
 	zone := middleware.Zones(k.Zones).Matches(state.Name())
 	if zone == "" {
-		if state.QType() != dns.TypePTR {
-			return middleware.NextOrFailure(k.Name(), k.Next, ctx, w, r)
-		}
-		// If this is a PTR request, and the request is in a defined
-		// pod/service cidr range, process the request in this middleware,
-		// otherwise pass to next middleware.
-		if !k.isRequestInReverseRange(state.Name()) {
+		if k.Fallthrough {
 			return middleware.NextOrFailure(k.Name(), k.Next, ctx, w, r)
 		}
-
-		// Set the zone to this specific request, as we want to handle this reverse request.
-		zone = state.Name()
+		return dns.RcodeServerFailure, nil
 	}
 
 	state.Zone = zone