Fix fetching kubeadm join command from invalid masters (#77)

* Fixes wrong masters used for initializing another

* Adds a degenerate test case for node management

cmd/hope/utils/nodes_test.go

@@ -57,12 +57,12 @@ var testNodes []hope.Node = []hope.Node{
 		Memory:     2048,
 	},
 	{
-		Name:       "test-master-02",
-		Role:       hope.NodeRoleMaster.String(),
-		User:       "packer",
-		Host:       "192.168.1.10",
-		Cpu:        2,
-		Memory:     2048,
+		Name:   "test-master-02",
+		Role:   hope.NodeRoleMaster.String(),
+		User:   "packer",
+		Host:   "192.168.1.10",
+		Cpu:    2,
+		Memory: 2048,
 	},
 	{
 		Name:       "test-master-03",

pkg/hope/node_management.go

@@ -154,7 +154,7 @@ func CreateClusterMaster(log *logrus.Entry, node *Node, podNetworkCidr string, l
 		return err
 	}
 
-	existingMasters := (*masters)[1:]
+	existingMasters := lbMasters[1:]
 	joinCommand, err := KubeadmGetClusterJoinCommandFromAnyMaster(&existingMasters)
 	if err != nil {
 		return err

pkg/hope/node_management_test.go

@@ -0,0 +1,18 @@
+package hope
+
+import (
+	"testing"
+)
+
+import (
+	log "github.com/sirupsen/logrus"
+	"github.com/stretchr/testify/assert"
+)
+
+func TestSetupCommonNodeRequirementsNotKubernetesNode(t *testing.T) {
+	node := Node{
+		Role: "load-balancer",
+	}
+	err := setupCommonNodeRequirements(log.WithFields(log.Fields{}), &node)
+	assert.Error(t, err, "Node has role load-balancer, should not prepare as Kubernetes node")
+}