WIP: Parserequest2 cutback (#868)

* middleware/kubernetes: pull TXT out of parseRequest

Put the TXT handling one layer higher and remove it from parseRequest.
Also rename the podsvc field in there to podOrSvc. Now that it isn't
used anymore for TXT record (dns-version) that was put in there. We can
make this a boolean (in a future PR).

Make parseRequest get an optional Zone that is from state.Zone and use
that instead of its own code. Removed some tests and other smaller
cleanups.

Fixes #836

* add this reverse

* another check

* readd

* Rename to kPod and kService for some clarity

1 files changed, 4 insertions, 4 deletions

diff --git a/test/kubernetes_test.go b/test/kubernetes_test.go
index ac5414743..1d00be3b8 100644
--- a/test/kubernetes_test.go
+++ b/test/kubernetes_test.go
@@ -543,7 +543,7 @@ func TestKubernetesIntegrationCidrReverseZone(t *testing.T) {
     kubernetes cluster.local {
                 endpoint http://localhost:8080
                 namespaces test-1
-				cidrs 10.0.0.0/24				
+				cidrs 10.0.0.0/24
     }
 	erratic . {
 		drop 0
@@ -555,10 +555,10 @@ func TestKubernetesIntegrationCidrReverseZone(t *testing.T) {
 func TestKubernetesIntegrationPartialCidrReverseZone(t *testing.T) {
 	corefile :=
 		`.:0 {
-    kubernetes cluster.local {
+    kubernetes  cluster.local {
                 endpoint http://localhost:8080
                 namespaces test-1
-				cidrs 10.0.0.96/28 10.0.0.120/32
+		cidrs 10.0.0.96/28 10.0.0.120/32
     }
 	erratic . {
 		drop 0
@@ -572,7 +572,7 @@ func TestKubernetesIntegrationAllNSExposed(t *testing.T) {
 		`.:0 {
     kubernetes cluster.local {
                 endpoint http://localhost:8080
-				cidrs 10.0.0.0/24
+		cidrs 10.0.0.0/24
     }
 `
 	doIntegrationTests(t, corefile, dnsTestCasesAllNSExposed)