Disable storing bootstrap information by default

pkg/cli/cmds/server.go

@@ -22,6 +22,7 @@ type Server struct {
 	ExtraSchedulerArgs  cli.StringSlice
 	ExtraControllerArgs cli.StringSlice
 	Rootless            bool
+	StoreBootstrap      bool
 	StorageEndpoint     string
 	StorageCAFile       string
 	StorageCertFile     string
@@ -142,6 +143,12 @@ func NewServerCommand(action func(*cli.Context) error) cli.Command {
 				Usage:       "(experimental) Run rootless",
 				Destination: &ServerConfig.Rootless,
 			},
+			cli.BoolFlag{
+				Name:        "bootstrap-save",
+				Usage:       "(experimental) Save bootstrap information in the storage endpoint",
+				Hidden:      true,
+				Destination: &ServerConfig.StoreBootstrap,
+			},
 			cli.StringFlag{
 				Name:        "storage-endpoint",
 				Usage:       "Specify etcd, Mysql, Postgres, or Sqlite (default) data source name",

pkg/cli/server/server.go

@@ -87,6 +87,7 @@ func run(app *cli.Context, cfg *cmds.Server) error {
 	serverConfig.ControlConfig.Storage.KeyFile = cfg.StorageKeyFile
 	serverConfig.ControlConfig.AdvertiseIP = cfg.AdvertiseIP
 	serverConfig.ControlConfig.AdvertisePort = cfg.AdvertisePort
+	serverConfig.ControlConfig.BootstrapReadOnly = !cfg.StoreBootstrap
 
 	if cmds.AgentConfig.FlannelIface != "" && cmds.AgentConfig.NodeIP == "" {
 		cmds.AgentConfig.NodeIP = netutil.GetIPFromInterface(cmds.AgentConfig.FlannelIface)

pkg/daemons/config/types.go

@@ -68,28 +68,27 @@ type Agent struct {
 }
 
 type Control struct {
-	AdvertisePort           int
+	AdvertisePort         int
-	AdvertiseIP             string
+	AdvertiseIP           string
-	ListenPort              int
+	ListenPort            int
-	HTTPSPort               int
+	HTTPSPort             int
-	ClusterSecret           string
+	ClusterSecret         string
-	ClusterIPRange          *net.IPNet
+	ClusterIPRange        *net.IPNet
-	ServiceIPRange          *net.IPNet
+	ServiceIPRange        *net.IPNet
-	ClusterDNS              net.IP
+	ClusterDNS            net.IP
-	ClusterDomain           string
+	ClusterDomain         string
-	NoCoreDNS               bool
+	NoCoreDNS             bool
-	KubeConfigOutput        string
+	KubeConfigOutput      string
-	KubeConfigMode          string
+	KubeConfigMode        string
-	DataDir                 string
+	DataDir               string
-	Skips                   []string
+	Skips                 []string
-	BootstrapReadOnly       bool
+	BootstrapReadOnly     bool
-	BootstrapOverwriteLocal bool
+	Storage               endpoint.Config
-	Storage                 endpoint.Config
+	NoScheduler           bool
-	NoScheduler             bool
+	ExtraAPIArgs          []string
-	ExtraAPIArgs            []string
+	ExtraControllerArgs   []string
-	ExtraControllerArgs     []string
+	ExtraSchedulerAPIArgs []string
-	ExtraSchedulerAPIArgs   []string
+	NoLeaderElect         bool
-	NoLeaderElect           bool
 
 	Runtime *ControlRuntime `json:"-"`
 }

pkg/daemons/control/bootstrap.go

@@ -18,7 +18,7 @@ const (
 )
 
 // fetchBootstrapData copies the bootstrap data (certs, keys, passwords)
-// from etcd to inidividual files specified by cfg.Runtime.
+// from etcd to individual files specified by cfg.Runtime.
 func fetchBootstrapData(ctx context.Context, cfg *config.Control, c client.Client) error {
 	logrus.Info("Fetching bootstrap data from etcd")
 	gr, err := c.Get(ctx, k3sRuntimeEtcdPath)
@@ -45,12 +45,6 @@ func fetchBootstrapData(ctx context.Context, cfg *config.Control, c client.Clien
 			continue
 		}
 
-		if !cfg.BootstrapOverwriteLocal {
-			if _, err := os.Stat(path); err == nil {
-				continue
-			}
-		}
-
 		if err := os.MkdirAll(filepath.Dir(path), 0700); err != nil {
 			return errors.Wrapf(err, "failed to mkdir %s", filepath.Dir(path))
 		}