kubeadm: Add omitempty tags to nullable values and use metav1.Duration

cmd/kubeadm/app/apis/kubeadm/types.go

@@ -17,8 +17,6 @@ limitations under the License.
 package kubeadm
 
 import (
-	"time"
-
 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
 )
 
@@ -36,7 +34,7 @@ type MasterConfiguration struct {
 	AuthorizationModes []string
 
 	Token    string
-	TokenTTL time.Duration
+	TokenTTL metav1.Duration
 
 	APIServerExtraArgs         map[string]string
 	ControllerManagerExtraArgs map[string]string

cmd/kubeadm/app/apis/kubeadm/v1alpha1/defaults.go

@@ -20,6 +20,7 @@ import (
 	"net/url"
 	"strings"
 
+	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
 	"k8s.io/apimachinery/pkg/runtime"
 	"k8s.io/kubernetes/cmd/kubeadm/app/constants"
 )
@@ -65,8 +66,10 @@ func SetDefaults_MasterConfiguration(obj *MasterConfiguration) {
 		obj.CertificatesDir = DefaultCertificatesDir
 	}
 
-	if obj.TokenTTL == 0 {
-		obj.TokenTTL = constants.DefaultTokenDuration
+	if obj.TokenTTL.Duration == 0 {
+		obj.TokenTTL = metav1.Duration{
+			Duration: constants.DefaultTokenDuration,
+		}
 	}
 
 	if obj.ImageRepository == "" {

cmd/kubeadm/app/apis/kubeadm/v1alpha1/types.go

@@ -17,8 +17,6 @@ limitations under the License.
 package v1alpha1
 
 import (
-	"time"
-
 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
 )
 
@@ -33,17 +31,17 @@ type MasterConfiguration struct {
 	KubernetesVersion  string     `json:"kubernetesVersion"`
 	CloudProvider      string     `json:"cloudProvider"`
 	NodeName           string     `json:"nodeName"`
-	AuthorizationModes []string   `json:"authorizationModes"`
+	AuthorizationModes []string   `json:"authorizationModes,omitempty"`
 
-	Token    string        `json:"token"`
-	TokenTTL time.Duration `json:"tokenTTL"`
+	Token    string          `json:"token"`
+	TokenTTL metav1.Duration `json:"tokenTTL"`
 
-	APIServerExtraArgs         map[string]string `json:"apiServerExtraArgs"`
-	ControllerManagerExtraArgs map[string]string `json:"controllerManagerExtraArgs"`
-	SchedulerExtraArgs         map[string]string `json:"schedulerExtraArgs"`
+	APIServerExtraArgs         map[string]string `json:"apiServerExtraArgs,omitempty"`
+	ControllerManagerExtraArgs map[string]string `json:"controllerManagerExtraArgs,omitempty"`
+	SchedulerExtraArgs         map[string]string `json:"schedulerExtraArgs,omitempty"`
 
 	// APIServerCertSANs sets extra Subject Alternative Names for the API Server signing cert
-	APIServerCertSANs []string `json:"apiServerCertSANs"`
+	APIServerCertSANs []string `json:"apiServerCertSANs,omitempty"`
 	// CertificatesDir specifies where to store or look for all required certificates
 	CertificatesDir string `json:"certificatesDir"`
 
@@ -53,7 +51,7 @@ type MasterConfiguration struct {
 	UnifiedControlPlaneImage string `json:"unifiedControlPlaneImage"`
 
 	// FeatureGates enabled by the user
-	FeatureGates map[string]bool `json:"featureGates"`
+	FeatureGates map[string]bool `json:"featureGates,omitempty"`
 }
 
 type API struct {
@@ -81,7 +79,7 @@ type Etcd struct {
 	CertFile  string            `json:"certFile"`
 	KeyFile   string            `json:"keyFile"`
 	DataDir   string            `json:"dataDir"`
-	ExtraArgs map[string]string `json:"extraArgs"`
+	ExtraArgs map[string]string `json:"extraArgs,omitempty"`
 	// Image specifies which container image to use for running etcd. If empty, automatically populated by kubeadm using the image repository and default etcd version
 	Image string `json:"image"`
 }
@@ -94,7 +92,7 @@ type NodeConfiguration struct {
 	CACertPath               string   `json:"caCertPath"`
 	DiscoveryFile            string   `json:"discoveryFile"`
 	DiscoveryToken           string   `json:"discoveryToken"`
-	DiscoveryTokenAPIServers []string `json:"discoveryTokenAPIServers"`
+	DiscoveryTokenAPIServers []string `json:"discoveryTokenAPIServers,omitempty"`
 	NodeName                 string   `json:"nodeName"`
 	TLSBootstrapToken        string   `json:"tlsBootstrapToken"`
 	Token                    string   `json:"token"`
@@ -107,7 +105,7 @@ type NodeConfiguration struct {
 	// SHA-256 hash of the Subject Public Key Info (SPKI) object in DER-encoded
 	// ASN.1. These hashes can be calculated using, for example, OpenSSL:
 	// openssl x509 -pubkey -in ca.crt openssl rsa -pubin -outform der 2>&/dev/null | openssl dgst -sha256 -hex
-	DiscoveryTokenCACertHashes []string `json:"discoveryTokenCACertHashes"`
+	DiscoveryTokenCACertHashes []string `json:"discoveryTokenCACertHashes,omitempty"`
 
 	// DiscoveryTokenUnsafeSkipCAVerification allows token-based discovery
 	// without CA verification via DiscoveryTokenCACertHashes. This can weaken

cmd/kubeadm/app/cmd/init.go

@@ -169,7 +169,7 @@ func AddInitConfigFlags(flagSet *flag.FlagSet, cfg *kubeadmapiext.MasterConfigur
 		"The token to use for establishing bidirectional trust between nodes and masters.",
 	)
 	flagSet.DurationVar(
-		&cfg.TokenTTL, "token-ttl", cfg.TokenTTL,
+		&cfg.TokenTTL.Duration, "token-ttl", cfg.TokenTTL.Duration,
 		"The duration before the bootstrap token is automatically deleted. 0 means 'never expires'.",
 	)
 	flagSet.StringVar(featureGatesString, "feature-gates", *featureGatesString, "A set of key=value pairs that describe feature gates for various features. "+
@@ -352,7 +352,7 @@ func (i *Init) Run(out io.Writer) error {
 
 	// Create the default node bootstrap token
 	tokenDescription := "The default bootstrap token generated by 'kubeadm init'."
-	if err := nodebootstraptokenphase.UpdateOrCreateToken(client, i.cfg.Token, false, i.cfg.TokenTTL, kubeadmconstants.DefaultTokenUsages, []string{}, tokenDescription); err != nil {
+	if err := nodebootstraptokenphase.UpdateOrCreateToken(client, i.cfg.Token, false, i.cfg.TokenTTL.Duration, kubeadmconstants.DefaultTokenUsages, []string{}, tokenDescription); err != nil {
 		return err
 	}
 	// Create RBAC rules that makes the bootstrap tokens able to post CSRs

cmd/kubeadm/app/cmd/upgrade/common_test.go

@@ -41,21 +41,15 @@ func TestPrintConfiguration(t *testing.T) {
 	api:
 	  advertiseAddress: ""
 	  bindPort: 0
-	apiServerCertSANs: null
-	apiServerExtraArgs: null
-	authorizationModes: null
 	certificatesDir: ""
 	cloudProvider: ""
-	controllerManagerExtraArgs: null
 	etcd:
 	  caFile: ""
 	  certFile: ""
 	  dataDir: ""
 	  endpoints: null
-	  extraArgs: null
 	  image: ""
 	  keyFile: ""
-	featureGates: null
 	imageRepository: ""
 	kubernetesVersion: v1.7.1
 	networking:
@@ -63,9 +57,8 @@ func TestPrintConfiguration(t *testing.T) {
 	  podSubnet: ""
 	  serviceSubnet: ""
 	nodeName: ""
-	schedulerExtraArgs: null
 	token: ""
-	tokenTTL: 0
+	tokenTTL: 0s
 	unifiedControlPlaneImage: ""
 `),
 		},
@@ -80,21 +73,15 @@ func TestPrintConfiguration(t *testing.T) {
 	api:
 	  advertiseAddress: ""
 	  bindPort: 0
-	apiServerCertSANs: null
-	apiServerExtraArgs: null
-	authorizationModes: null
 	certificatesDir: ""
 	cloudProvider: ""
-	controllerManagerExtraArgs: null
 	etcd:
 	  caFile: ""
 	  certFile: ""
 	  dataDir: ""
 	  endpoints: null
-	  extraArgs: null
 	  image: ""
 	  keyFile: ""
-	featureGates: null
 	imageRepository: ""
 	kubernetesVersion: v1.7.1
 	networking:
@@ -102,9 +89,8 @@ func TestPrintConfiguration(t *testing.T) {
 	  podSubnet: ""
 	  serviceSubnet: 10.96.0.1/12
 	nodeName: ""
-	schedulerExtraArgs: null
 	token: ""
-	tokenTTL: 0
+	tokenTTL: 0s
 	unifiedControlPlaneImage: ""
 `),
 		},

cmd/kubeadm/app/phases/upgrade/staticpods_test.go

@@ -69,7 +69,7 @@ networking:
 nodeName: thegopher
 schedulerExtraArgs: null
 token: ce3aa5.5ec8455bb76b379f
-tokenTTL: 86400000000000
+tokenTTL: 24h
 unifiedControlPlaneImage: ""
 `
 )