Merge pull request #56 from xiaods/dev

tuning snapshort save case, handle etcd correctly

.github/workflows/go.yml

@@ -7,10 +7,11 @@ on:
 jobs:
 
   build:
-    name: Build
+    name: Build & Release
     runs-on: ubuntu-latest
+    env:
+      DRONE_TAG: v1.19.8+k8e3
     steps:
-
     - name: Set up Go 1.x
       uses: actions/setup-go@v2
       with:

go.mod

@@ -91,7 +91,6 @@ require (
 	github.com/pierrec/lz4 v2.5.2+incompatible
 	github.com/pkg/errors v0.9.1
 	github.com/rancher/dynamiclistener v0.2.3
-	github.com/rancher/kine v0.5.1
 	github.com/rancher/remotedialer v0.2.0
 	github.com/rancher/wrangler v0.6.1
 	github.com/rancher/wrangler-api v0.6.0

pkg/cluster/cluster.go

@@ -96,13 +96,18 @@ func (c *Cluster) Start(ctx context.Context) (<-chan struct{}, error) {
 		return nil, err
 	}
 
-	// at this point, if etcd is in use, it's up, ready,
-	// and bootstrapping is complete so save the bootstrap
-	// data
+	// at this point, if etcd is in use, it's bootstrapping is complete
+	// so save the bootstrap data. We will need for etcd to be up. If
+	// the save call returns an error, we panic since subsequent etcd
+	// snapshots will be empty.
 	if c.managedDB != nil {
-		if err := c.save(ctx); err != nil {
-			return nil, err
-		}
+		go func() {
+			for range ready {
+				if err := c.save(ctx); err != nil {
+					panic(err)
+				}
+			}
+		}()
 	}
 
 	return ready, nil