Merge pull request kubernetes#92964 from vponomaryov/issue-88752-in-master

agnhost image: use actual DNS domain instead of hardcoded cluster.local

Author: k8s-ci-robot

build/dependencies.yaml

@@ -1,7 +1,7 @@
 dependencies:
   # agnhost: bump this one first
   - name: "agnhost"
-    version: "2.23"
+    version: "2.24"
     refPaths:
     - path: test/images/agnhost/VERSION
       match: \d.\d

test/images/agnhost/VERSION

@@ -1 +1 @@
-2.23
+2.24

test/images/agnhost/agnhost.go

@@ -51,7 +51,7 @@ import (
 func main() {
 	rootCmd := &cobra.Command{
 		Use:     "app",
-		Version: "2.23",
+		Version: "2.24",
 	}
 
 	rootCmd.AddCommand(auditproxy.CmdAuditProxy)

test/images/agnhost/dns/common.go

@@ -52,7 +52,7 @@ var CmdEtcHosts = &cobra.Command{
 }
 
 func printDNSSuffixList(cmd *cobra.Command, args []string) {
-	dnsSuffixList := getDNSSuffixList()
+	dnsSuffixList := GetDNSSuffixList()
 	fmt.Println(strings.Join(dnsSuffixList, ","))
 }
 

test/images/agnhost/dns/dns.go

@@ -28,7 +28,9 @@ const etcHostsFile = "/etc/hosts"
 // nameserver DNS_CLUSTER_IP
 // search test-dns.svc.cluster.local svc.cluster.local cluster.local q53aahaikqaehcai3ylfqdtc5b.bx.internal.cloudapp.net
 // options ndots:5
-func getDNSSuffixList() []string {
+
+// GetDNSSuffixList reads DNS config file and returns the list of configured DNS suffixes
+func GetDNSSuffixList() []string {
 	fileData := readFile("/etc/resolv.conf")
 	lines := strings.Split(fileData, "\n")
 	for _, line := range lines {

test/images/agnhost/dns/dns_windows.go

@@ -80,8 +80,8 @@ func getRegistryValue(reg, key string) string {
 	return regValue
 }
 
-// getDNSSuffixList reads DNS config file and returns the list of configured DNS suffixes
-func getDNSSuffixList() []string {
+// GetDNSSuffixList reads DNS config file and returns the list of configured DNS suffixes
+func GetDNSSuffixList() []string {
 	// We start with the general suffix list that apply to all network connections.
 	allSuffixes := []string{}
 	suffixes := getRegistryValue(netRegistry, "SearchList")

test/images/agnhost/resource-consumer-controller/BUILD

@@ -10,6 +10,7 @@ go_library(
     srcs = ["controller.go"],
     importpath = "k8s.io/kubernetes/test/images/agnhost/resource-consumer-controller",
     deps = [
+        "//test/images/agnhost/dns:go_default_library",
         "//test/images/resource-consumer/common:go_default_library",
         "//vendor/github.com/spf13/cobra:go_default_library",
     ],

test/images/agnhost/resource-consumer-controller/controller.go

@@ -21,11 +21,13 @@ import (
 	"log"
 	"net/http"
 	"net/url"
+	"regexp"
 	"strconv"
 	"sync"
 
 	"github.com/spf13/cobra"
 
+	"k8s.io/kubernetes/test/images/agnhost/dns"
 	"k8s.io/kubernetes/test/images/resource-consumer/common"
 )
 
@@ -43,8 +45,31 @@ var (
 	consumerPort             int
 	consumerServiceName      string
 	consumerServiceNamespace string
+	dnsDomain                string
 )
 
+// getDNSDomain walks through DNS configuration and looks for "svc.foo" record
+// where "foo" is currently configured DNS suffix. Then picks that 'foo' part up
+// and returns to a caller.
+func getDNSDomain() string {
+	if dnsDomain != "" {
+		return dnsDomain
+	}
+	dnsSuffixList := dns.GetDNSSuffixList()
+	r, _ := regexp.Compile("^svc.")
+	for _, currentDNSSuffix := range dnsSuffixList {
+		if r.MatchString(currentDNSSuffix) {
+			// Save DNS suffix without the 'svc.' part
+			dnsDomain = currentDNSSuffix[4:]
+			break
+		}
+	}
+	if dnsDomain == "" {
+		panic("Could not find DNS suffix starting with 'svc.' substring")
+	}
+	return dnsDomain
+}
+
 func init() {
 	CmdResourceConsumerController.Flags().IntVar(&port, "port", 8080, "Port number.")
 	CmdResourceConsumerController.Flags().IntVar(&consumerPort, "consumer-port", 8080, "Port number of consumers.")
@@ -214,7 +239,8 @@ func (c *controller) sendConsumeCustomMetric(w http.ResponseWriter, metric strin
 }
 
 func createConsumerURL(suffix string) string {
-	return fmt.Sprintf("http://%s.%s.svc.cluster.local:%d%s", consumerServiceName, consumerServiceNamespace, consumerPort, suffix)
+	// NOTE: full DNS name is used due to the Windows platform restriction where PQDNs are not supported.
+	return fmt.Sprintf("http://%s.%s.svc.%s:%d%s", consumerServiceName, consumerServiceNamespace, getDNSDomain(), consumerPort, suffix)
 }
 
 // sendOneConsumeCPURequest sends POST request for cpu consumption