Skip to content

Commit a529ff0

Browse files
minikube_node -> minikube_cluster
1 parent ebf3017 commit a529ff0

File tree

3 files changed

+25
-19
lines changed

3 files changed

+25
-19
lines changed

hack/main.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -115,7 +115,7 @@ func main() {
115115
IsoUrls: []string{"https://github.com/kubernetes/minikube/releases/download/v1.26.1/minikube-v1.26.1-amd64.iso"},
116116
DeleteOnFailure: true},
117117
service.MinikubeClientDeps{
118-
Node: service.NewMinikubeNode(),
118+
Node: service.NewMinikubeCluster(),
119119
Downloader: service.NewMinikubeDownloader(),
120120
})
121121

minikube/resource_cluster.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -172,7 +172,7 @@ func setClusterState(d *schema.ResourceData, config *config.ClusterConfig, ports
172172
d.Set("vm_driver", config.VMDriver)
173173
}
174174

175-
//getClusterOutputs return the cluster key, certificate and certificate authority from the provided kubeconfig
175+
// getClusterOutputs return the cluster key, certificate and certificate authority from the provided kubeconfig
176176
func getClusterOutputs(kc *kubeconfig.Settings) (string, string, string, string, error) {
177177
key, err := state_utils.ReadContents(kc.ClientKey)
178178
if err != nil {
@@ -325,7 +325,7 @@ func initialiseMinikubeClient(d *schema.ResourceData, m interface{}) (service.Cl
325325
})
326326

327327
clusterClient.SetDependencies(service.MinikubeClientDeps{
328-
Node: service.NewMinikubeNode(),
328+
Node: service.NewMinikubeCluster(),
329329
Downloader: service.NewMinikubeDownloader(),
330330
})
331331

minikube/service/minikube_node.go renamed to minikube/service/minikube_cluster.go

Lines changed: 22 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ import (
77

88
"github.com/docker/machine/libmachine"
99
"github.com/docker/machine/libmachine/host"
10+
delete "k8s.io/minikube/cmd/minikube/cmd"
1011
"k8s.io/minikube/pkg/minikube/command"
1112
"k8s.io/minikube/pkg/minikube/config"
1213
"k8s.io/minikube/pkg/minikube/exit"
@@ -25,15 +26,15 @@ type Node interface {
2526
Add(cc *config.ClusterConfig, starter node.Starter) error
2627
}
2728

28-
type MinikubeNode struct {
29+
type MinikubeCluster struct {
2930
workerNodes int
3031
}
3132

32-
func NewMinikubeNode() *MinikubeNode {
33-
return &MinikubeNode{workerNodes: 0}
33+
func NewMinikubeCluster() *MinikubeCluster {
34+
return &MinikubeCluster{workerNodes: 0}
3435
}
3536

36-
func (m *MinikubeNode) Provision(cc *config.ClusterConfig, n *config.Node, apiServer bool, delOnFail bool) (command.Runner, bool, libmachine.API, *host.Host, error) {
37+
func (m *MinikubeCluster) Provision(cc *config.ClusterConfig, n *config.Node, apiServer bool, delOnFail bool) (command.Runner, bool, libmachine.API, *host.Host, error) {
3738
makeAllMinikubeDirectories()
3839
_, err := node.CacheKubectlBinary(cc.KubernetesConfig.KubernetesVersion, cc.BinaryMirror)
3940
if err != nil {
@@ -43,13 +44,13 @@ func (m *MinikubeNode) Provision(cc *config.ClusterConfig, n *config.Node, apiSe
4344
return node.Provision(cc, n, apiServer, delOnFail)
4445
}
4546

46-
func (m *MinikubeNode) Start(starter node.Starter, apiServer bool) (*kubeconfig.Settings, error) {
47+
func (m *MinikubeCluster) Start(starter node.Starter, apiServer bool) (*kubeconfig.Settings, error) {
4748

4849
return node.Start(starter, apiServer)
4950
}
5051

51-
//Add adds nodes to the clusters node pool
52-
func (m *MinikubeNode) Add(cc *config.ClusterConfig, starter node.Starter) error {
52+
// Add adds nodes to the clusters node pool
53+
func (m *MinikubeCluster) Add(cc *config.ClusterConfig, starter node.Starter) error {
5354
n := config.Node{
5455
Name: node.Name(m.workerNodes),
5556
Worker: true,
@@ -61,28 +62,33 @@ func (m *MinikubeNode) Add(cc *config.ClusterConfig, starter node.Starter) error
6162
return node.Add(cc, n, true)
6263
}
6364

64-
func (m *MinikubeNode) Delete(cc config.ClusterConfig, name string) (*config.Node, error) {
65-
node, err := node.Delete(cc, name)
66-
if err != nil {
67-
return node, err
65+
func (m *MinikubeCluster) Delete(cc config.ClusterConfig, name string) (*config.Node, error) {
66+
errs := delete.DeleteProfiles([]*config.Profile{
67+
{
68+
Name: name,
69+
Config: &cc,
70+
},
71+
})
72+
if len(errs) > 0 {
73+
return nil, errs[0]
6874
}
6975

7076
machineDir := filepath.Join(localpath.MiniPath(), "machines", name)
7177
profilesDir := filepath.Join(localpath.MiniPath(), "profiles", name)
72-
err = rmdir(machineDir)
78+
err := rmdir(machineDir)
7379
if err != nil {
74-
return node, err
80+
return nil, err
7581
}
7682

7783
err = rmdir(profilesDir)
7884
if err != nil {
79-
return node, err
85+
return nil, err
8086
}
8187

82-
return node, err
88+
return nil, err
8389
}
8490

85-
func (m *MinikubeNode) Get(name string) mustload.ClusterController {
91+
func (m *MinikubeCluster) Get(name string) mustload.ClusterController {
8692
return mustload.Running(name)
8793
}
8894

0 commit comments

Comments
 (0)