Add a default cluster dns name config

It is very handy to be able to store the default cluster dns name so
that I can fire up and teardown clusters easily without losing that
setting. You can think of this as a default value for the
`start --dns-domain <val>` flag, but only when starting a new cluster.

In my particular case, I would like to set my default cluster dna domain
to a subdomain under a domain that I control. E.g.
`cluster.wt.user.dev.example.com`. I want to do this to make it easier
to manage TLS certs for my dev cluster.

For context, the `start --dns-domain <val>` flag works like this:
  If you fire up a new cluster with that flag, you will get a new cluster
  with the domain name base. If you start an existing cluster, the
  config will be updated to the new domain name base and then started.

This config will work a little differently. The change implements a
config that will only affect newly started cluster. Here are some
examples to show that difference:

Newly started cluster example:
```
$ minikube config set DefaultClusterDNSDomain cluster2.local
$ minikube start
...
```

Stop and starting the cluster with `--dns-domain` flag:
```
$ minikube stop
...
$ minikub start --dns-domain cluster3.local
...
```

Stopping and starting a new cluster with a non-default domain name and
without the `--dns-domain` flag:
```
$ minikube stop
...
$ minikub start
...
```

The reason for this behavior is that I am not configuring the name of a
cluster. That is a cluster configuration option. I am setting the
default for that.

If I manually overrode the name of a cluster when started previously, I
don't want a default option to override my cluster configuration.

Author: wt

cmd/minikube/cmd/config/config.go

@@ -172,6 +172,10 @@ var settings = []Setting{
 		name: config.MaxAuditEntries,
 		set:  SetInt,
 	},
+	{
+		name: config.DefaultClusterDNSDomain,
+		set:  SetString,
+	},
 }
 
 // ConfigCmd represents the config command

cmd/minikube/cmd/start.go

@@ -161,6 +161,8 @@ func runStart(cmd *cobra.Command, _ []string) {
 	}
 	defer pkgtrace.Cleanup()
 
+	viper.SetDefault(config.DefaultClusterDNSDomain, constants.DefaultClusterDNSDomain)
+
 	displayVersion(version.GetVersion())
 	go download.CleanUpOlderPreloads()
 

cmd/minikube/cmd/start_flags.go

@@ -221,7 +221,7 @@ func initKubernetesFlags() {
 		Valid components are: kubelet, kubeadm, apiserver, controller-manager, etcd, proxy, scheduler
 		Valid kubeadm parameters: `+fmt.Sprintf("%s, %s", strings.Join(bsutil.KubeadmExtraArgsAllowed[bsutil.KubeadmCmdParam], ", "), strings.Join(bsutil.KubeadmExtraArgsAllowed[bsutil.KubeadmConfigParam], ",")))
 	startCmd.Flags().String(featureGates, "", "A set of key=value pairs that describe feature gates for alpha/experimental features.")
-	startCmd.Flags().String(dnsDomain, constants.ClusterDNSDomain, "The cluster dns domain name used in the Kubernetes cluster")
+	startCmd.Flags().String(dnsDomain, constants.DefaultClusterDNSDomain, "The cluster dns domain name used in the Kubernetes cluster")
 	startCmd.Flags().Int(apiServerPort, constants.APIServerPort, "The apiserver listening port")
 	startCmd.Flags().String(apiServerName, constants.APIServerName, "The authoritative apiserver hostname for apiserver certificates and connectivity. This can be used if you want to make the apiserver available from outside the machine")
 	startCmd.Flags().StringSliceVar(&apiServerNames, "apiserver-names", nil, "A set of apiserver names which are used in the generated certificate for kubernetes.  This can be used if you want to make the apiserver available from outside the machine")
@@ -560,6 +560,11 @@ func generateNewConfigFromFlags(cmd *cobra.Command, k8sVersion string, rtime str
 		out.WarningT("--network flag is only valid with the docker/podman, qemu, kvm, and vfkit drivers, it will be ignored")
 	}
 
+	defaultClusterDNSDomain := viper.GetString(dnsDomain)
+	if defaultClusterDNSDomain == constants.DefaultClusterDNSDomain {
+		defaultClusterDNSDomain = viper.GetString(config.DefaultClusterDNSDomain)
+	}
+
 	validateHANodeCount(cmd)
 
 	checkNumaCount(k8sVersion)
@@ -636,7 +641,7 @@ func generateNewConfigFromFlags(cmd *cobra.Command, k8sVersion string, rtime str
 			APIServerName:          viper.GetString(apiServerName),
 			APIServerNames:         apiServerNames,
 			APIServerIPs:           apiServerIPs,
-			DNSDomain:              viper.GetString(dnsDomain),
+			DNSDomain:              defaultClusterDNSDomain,
 			FeatureGates:           viper.GetString(featureGates),
 			ContainerRuntime:       rtime,
 			CRISocket:              viper.GetString(criSocket),

go.work.sum

@@ -36,7 +36,7 @@ func TestSetupCerts(t *testing.T) {
 		CertExpiration: constants.DefaultCertExpiration,
 		KubernetesConfig: config.KubernetesConfig{
 			APIServerName: constants.APIServerName,
-			DNSDomain:     constants.ClusterDNSDomain,
+			DNSDomain:     constants.DefaultClusterDNSDomain,
 			ServiceCIDR:   constants.DefaultServiceCIDR,
 		},
 	}

pkg/minikube/bootstrapper/certs_test.go

@@ -58,6 +58,9 @@ const (
 	EmbedCerts = "EmbedCerts"
 	// MaxAuditEntries is the maximum number of audit entries to retain
 	MaxAuditEntries = "MaxAuditEntries"
+	// DefaultDnsDomain is the key for the default dns domain name when creating
+	// a cluster
+	DefaultClusterDNSDomain = "DefaultClusterDNSDomain"
 )
 
 var (

pkg/minikube/config/config.go

@@ -75,7 +75,7 @@ const (
 	// APIServerName is the default API server name
 	APIServerName = "minikubeCA"
 	// ClusterDNSDomain is the default DNS domain
-	ClusterDNSDomain = "cluster.local"
+	DefaultClusterDNSDomain = "cluster.local"
 	// DefaultServiceCIDR is The CIDR to be used for service cluster IPs
 	DefaultServiceCIDR = "10.96.0.0/12"
 	// HostAlias is a DNS alias to the container/VM host IP

pkg/minikube/constants/constants.go

@@ -41,6 +41,7 @@ Configurable fields:
  * native-ssh
  * rootless
  * MaxAuditEntries
+ * DefaultClusterDNSDomain
 
 ```shell
 minikube config SUBCOMMAND [flags]