Merge pull request #1 from kubesimplify/new-version

updated to v1.1.1 of ksctl core
ksctl · Oct 24, 2023 · 91219d5 · 91219d5
2 parents 6167f61 + 1792daf
commit 91219d5
Show file tree

Hide file tree

Showing 14 changed files with 163 additions and 58 deletions.
diff --git a/.github/workflows/test-mock.yml b/.github/workflows/test-mock.yml
@@ -0,0 +1,44 @@
+name: "Test Functionality of the Mock cli"
+on:
+  pull_request:
+    paths:
+      - 'cli/**'
+      - 'scripts/**'
+      - 'go.mod'
+      - 'go.sum'
+    types: [ labeled ]
+    branches: [ main ]
+  push:
+    paths:
+      - 'scripts/**'
+      - 'cli/**'
+      - 'go.mod'
+      - 'go.sum'
+    branches: [ main ]
+jobs:
+  civo:
+#    if: ${{ github.event.label.name == 'pr/lgtm' }}
+    runs-on: ubuntu-latest
+    env:
+      KSCTL_FAKE_FLAG_ENABLED: "1"
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v3
+
+      - name: Set up Go
+        uses: actions/setup-go@v4
+        with:
+          go-version: 1.21.0
+
+      - name: build ksctl
+        run: |
+          make install_linux
+          ksctl -h
+
+      - name: civo tests
+        run: |
+          ksctl create civo -n demo -r LON1 -v --approve
+          ksctl switch -p civo -n demo -r LON1
+          ksctl create ha-civo -n demo -r FRA1 -v --approve
+          ksctl switch -p ha-civo -n demo -r FRA1
+          ksctl get -p civo
diff --git a/.idea/workspace.xml b/.idea/workspace.xml
diff --git a/cli/cmd/addNodesHAAzure.go b/cli/cmd/addNodesHAAzure.go
@@ -24,8 +24,8 @@ ksctl create-cluster ha-azure add-nodes <arguments to civo cloud provider>
 			panic(err)
 		}
 		SetRequiredFeatureFlags(cmd)
-		cli.Client.Metadata.Provider = consts.CLOUD_AZURE
-		SetDefaults(consts.CLOUD_AZURE, consts.CLUSTER_TYPE_HA)
+		cli.Client.Metadata.Provider = consts.CloudAzure
+		SetDefaults(consts.CloudAzure, consts.ClusterTypeHa)
 		cli.Client.Metadata.NoWP = noWP
 		cli.Client.Metadata.WorkerPlaneNodeType = nodeSizeWP
 		cli.Client.Metadata.ClusterName = clusterName

diff --git a/cli/cmd/addNodesHACivo.go b/cli/cmd/addNodesHACivo.go
@@ -24,8 +24,8 @@ ksctl create-cluster ha-civo add-nodes <arguments to civo cloud provider>
 			panic(err)
 		}
 		SetRequiredFeatureFlags(cmd)
-		cli.Client.Metadata.Provider = consts.CLOUD_CIVO
-		SetDefaults(consts.CLOUD_CIVO, consts.CLUSTER_TYPE_HA)
+		cli.Client.Metadata.Provider = consts.CloudCivo
+		SetDefaults(consts.CloudCivo, consts.ClusterTypeHa)
 		cli.Client.Metadata.NoWP = noWP
 		cli.Client.Metadata.WorkerPlaneNodeType = nodeSizeWP
 		cli.Client.Metadata.ClusterName = clusterName

diff --git a/cli/cmd/createCluster.go b/cli/cmd/createCluster.go
@@ -43,8 +43,8 @@ var createClusterAzure = &cobra.Command{
 			panic(err)
 		}
 		SetRequiredFeatureFlags(cmd)
-		cli.Client.Metadata.Provider = consts.CLOUD_AZURE
-		SetDefaults(consts.CLOUD_AZURE, consts.CLUSTER_TYPE_MANG)
+		cli.Client.Metadata.Provider = consts.CloudAzure
+		SetDefaults(consts.CloudAzure, consts.ClusterTypeMang)
 		createManaged(cmd.Flags().Lookup("approve").Changed)
 	},
 }
@@ -63,8 +63,8 @@ ksctl create-cluster civo <arguments to civo cloud provider>
 		}
 
 		SetRequiredFeatureFlags(cmd)
-		cli.Client.Metadata.Provider = consts.CLOUD_CIVO
-		SetDefaults(consts.CLOUD_CIVO, consts.CLUSTER_TYPE_MANG)
+		cli.Client.Metadata.Provider = consts.CloudCivo
+		SetDefaults(consts.CloudCivo, consts.ClusterTypeMang)
 		createManaged(cmd.Flags().Lookup("approve").Changed)
 	},
 }
@@ -83,8 +83,8 @@ ksctl create-cluster local <arguments to civo cloud provider>
 		}
 
 		SetRequiredFeatureFlags(cmd)
-		cli.Client.Metadata.Provider = consts.CLOUD_LOCAL
-		SetDefaults(consts.CLOUD_LOCAL, consts.CLUSTER_TYPE_MANG)
+		cli.Client.Metadata.Provider = consts.CloudLocal
+		SetDefaults(consts.CloudLocal, consts.ClusterTypeMang)
 		createManaged(cmd.Flags().Lookup("approve").Changed)
 	},
 }
@@ -103,8 +103,8 @@ ksctl create-cluster ha-civo <arguments to civo cloud provider>
 		}
 		SetRequiredFeatureFlags(cmd)
 
-		cli.Client.Metadata.Provider = consts.CLOUD_CIVO
-		SetDefaults(consts.CLOUD_CIVO, consts.CLUSTER_TYPE_HA)
+		cli.Client.Metadata.Provider = consts.CloudCivo
+		SetDefaults(consts.CloudCivo, consts.ClusterTypeHa)
 		createHA(cmd.Flags().Lookup("approve").Changed)
 	},
 }
@@ -122,8 +122,8 @@ var createClusterHAAzure = &cobra.Command{
 			panic(err)
 		}
 		SetRequiredFeatureFlags(cmd)
-		cli.Client.Metadata.Provider = consts.CLOUD_AZURE
-		SetDefaults(consts.CLOUD_AZURE, consts.CLUSTER_TYPE_HA)
+		cli.Client.Metadata.Provider = consts.CloudAzure
+		SetDefaults(consts.CloudAzure, consts.ClusterTypeHa)
 		createHA(cmd.Flags().Lookup("approve").Changed)
 	},
 }

diff --git a/cli/cmd/deleteCluster.go b/cli/cmd/deleteCluster.go
@@ -33,8 +33,8 @@ ksctl create-cluster azure <arguments to civo cloud provider>
 		}
 
 		SetRequiredFeatureFlags(cmd)
-		cli.Client.Metadata.Provider = consts.CLOUD_AZURE
-		SetDefaults(consts.CLOUD_AZURE, consts.CLUSTER_TYPE_MANG)
+		cli.Client.Metadata.Provider = consts.CloudAzure
+		SetDefaults(consts.CloudAzure, consts.ClusterTypeMang)
 
 		deleteManaged(cmd.Flags().Lookup("approve").Changed)
 	},
@@ -54,8 +54,8 @@ ksctl delete-cluster civo
 		}
 
 		SetRequiredFeatureFlags(cmd)
-		cli.Client.Metadata.Provider = consts.CLOUD_CIVO
-		SetDefaults(consts.CLOUD_CIVO, consts.CLUSTER_TYPE_MANG)
+		cli.Client.Metadata.Provider = consts.CloudCivo
+		SetDefaults(consts.CloudCivo, consts.ClusterTypeMang)
 
 		deleteManaged(cmd.Flags().Lookup("approve").Changed)
 
@@ -76,8 +76,8 @@ var deleteClusterHAAzure = &cobra.Command{
 		}
 
 		SetRequiredFeatureFlags(cmd)
-		cli.Client.Metadata.Provider = consts.CLOUD_AZURE
-		SetDefaults(consts.CLOUD_AZURE, consts.CLUSTER_TYPE_HA)
+		cli.Client.Metadata.Provider = consts.CloudAzure
+		SetDefaults(consts.CloudAzure, consts.ClusterTypeHa)
 
 		deleteHA(cmd.Flags().Lookup("approve").Changed)
 	},
@@ -97,8 +97,8 @@ ksctl delete-cluster ha-civo <arguments to civo cloud provider>
 		}
 
 		SetRequiredFeatureFlags(cmd)
-		cli.Client.Metadata.Provider = consts.CLOUD_CIVO
-		SetDefaults(consts.CLOUD_CIVO, consts.CLUSTER_TYPE_HA)
+		cli.Client.Metadata.Provider = consts.CloudCivo
+		SetDefaults(consts.CloudCivo, consts.ClusterTypeHa)
 
 		deleteHA(cmd.Flags().Lookup("approve").Changed)
 	},
@@ -118,8 +118,8 @@ ksctl delete-cluster local <arguments to local/Docker provider>
 		}
 
 		SetRequiredFeatureFlags(cmd)
-		cli.Client.Metadata.Provider = consts.CLOUD_LOCAL
-		SetDefaults(consts.CLOUD_LOCAL, consts.CLUSTER_TYPE_MANG)
+		cli.Client.Metadata.Provider = consts.CloudLocal
+		SetDefaults(consts.CloudLocal, consts.ClusterTypeMang)
 
 		deleteManaged(cmd.Flags().Lookup("approve").Changed)
 	},

diff --git a/cli/cmd/deleteNodesHAAzure.go b/cli/cmd/deleteNodesHAAzure.go
@@ -23,10 +23,10 @@ ksctl delete-cluster ha-azure delete-nodes <arguments to civo cloud provider>
 			panic(err)
 		}
 		SetRequiredFeatureFlags(cmd)
-		cli.Client.Metadata.Provider = consts.CLOUD_AZURE
+		cli.Client.Metadata.Provider = consts.CloudAzure
 		cli.Client.Metadata.IsHA = true
 
-		SetDefaults(consts.CLOUD_AZURE, consts.CLUSTER_TYPE_HA)
+		SetDefaults(consts.CloudAzure, consts.ClusterTypeHa)
 		cli.Client.Metadata.NoWP = noWP
 		cli.Client.Metadata.ClusterName = clusterName
 		cli.Client.Metadata.Region = region

diff --git a/cli/cmd/deleteNodesHACivo.go b/cli/cmd/deleteNodesHACivo.go
@@ -23,10 +23,10 @@ ksctl delete-cluster ha-civo delete-nodes <arguments to civo cloud provider>
 			panic(err)
 		}
 		SetRequiredFeatureFlags(cmd)
-		cli.Client.Metadata.Provider = consts.CLOUD_CIVO
+		cli.Client.Metadata.Provider = consts.CloudCivo
 		cli.Client.Metadata.IsHA = true
 
-		SetDefaults(consts.CLOUD_CIVO, consts.CLUSTER_TYPE_HA)
+		SetDefaults(consts.CloudCivo, consts.ClusterTypeHa)
 		cli.Client.Metadata.NoWP = noWP
 		cli.Client.Metadata.ClusterName = clusterName
 		cli.Client.Metadata.Region = region

diff --git a/cli/cmd/featureFlags.go b/cli/cmd/featureFlags.go
@@ -31,7 +31,7 @@ func SetRequiredFeatureFlags(cmd *cobra.Command) {
 
 		switch consts.KsctlSpecialFlags(feature) {
 		case ksctl_feature_auto_scale:
-			if err := os.Setenv(string(consts.KSCTL_FEATURE_FLAG_HA_AUTOSCALE), "true"); err != nil {
+			if err := os.Setenv(string(consts.KsctlFeatureFlagHaAutoscale), "true"); err != nil {
 				if cli.Client.Storage != nil {
 					cli.Client.Storage.Logger().Err("Unable to set the ha autoscale feature")
 				} else {
@@ -40,7 +40,7 @@ func SetRequiredFeatureFlags(cmd *cobra.Command) {
 			}
 
 		case ksctl_feature_applications:
-			if err := os.Setenv(string(consts.KSCTL_FEATURE_FLAG_APPLICATIONS), "true"); err != nil {
+			if err := os.Setenv(string(consts.KsctlFeatureFlagApplications), "true"); err != nil {
 				if cli.Client.Storage != nil {
 					cli.Client.Storage.Logger().Err("Unable to set applications feature")
 				} else {

diff --git a/cli/cmd/root.go b/cli/cmd/root.go
@@ -65,12 +65,12 @@ func Execute() {
 	controller = controlPkg.GenKsctlController()
 
 	cloud = map[int]string{
-		1: string(consts.CLOUD_AWS),
-		2: string(consts.CLOUD_AZURE),
-		3: string(consts.CLOUD_CIVO),
-		4: string(consts.CLOUD_LOCAL),
+		1: string(consts.CloudAws),
+		2: string(consts.CloudAzure),
+		3: string(consts.CloudCivo),
+		4: string(consts.CloudLocal),
 	}
-	cli.Client.Metadata.StateLocation = consts.STORE_LOCAL
+	cli.Client.Metadata.StateLocation = consts.StoreLocal
 
 	timer := time.Now()
 	err := rootCmd.Execute()

diff --git a/cli/cmd/switchCluster.go b/cli/cmd/switchCluster.go
@@ -26,22 +26,22 @@ ksctl switch-context -p <civo,local,civo-ha,azure-ha,azure>  -n <clustername> -r
 		cli.Client.Metadata.Region = region
 
 		switch provider {
-		case string(consts.CLOUD_LOCAL):
-			cli.Client.Metadata.Provider = consts.CLOUD_LOCAL
+		case string(consts.CloudLocal):
+			cli.Client.Metadata.Provider = consts.CloudLocal
 
-		case string(consts.CLUSTER_TYPE_HA) + "-" + string(consts.CLOUD_CIVO):
-			cli.Client.Metadata.Provider = consts.CLOUD_CIVO
+		case string(consts.ClusterTypeHa) + "-" + string(consts.CloudCivo):
+			cli.Client.Metadata.Provider = consts.CloudCivo
 			cli.Client.Metadata.IsHA = true
 
-		case string(consts.CLOUD_CIVO):
-			cli.Client.Metadata.Provider = consts.CLOUD_CIVO
+		case string(consts.CloudCivo):
+			cli.Client.Metadata.Provider = consts.CloudCivo
 
-		case string(consts.CLUSTER_TYPE_HA) + "-" + string(consts.CLOUD_AZURE):
-			cli.Client.Metadata.Provider = consts.CLOUD_AZURE
+		case string(consts.ClusterTypeHa) + "-" + string(consts.CloudAzure):
+			cli.Client.Metadata.Provider = consts.CloudAzure
 			cli.Client.Metadata.IsHA = true
 
-		case string(consts.CLOUD_AZURE):
-			cli.Client.Metadata.Provider = consts.CLOUD_AZURE
+		case string(consts.CloudAzure):
+			cli.Client.Metadata.Provider = consts.CloudAzure
 		}
 
 		stat, err := controller.SwitchCluster(&cli.Client)

diff --git a/cli/cmd/utils.go b/cli/cmd/utils.go
@@ -169,15 +169,15 @@ func createApproval(showMsg bool) error {
 
 func SetDefaults(provider consts.KsctlCloud, clusterType consts.KsctlClusterType) {
 	switch string(provider) + string(clusterType) {
-	case string(consts.CLOUD_LOCAL) + string(consts.CLUSTER_TYPE_MANG):
+	case string(consts.CloudLocal) + string(consts.ClusterTypeMang):
 		if noMP == -1 {
 			noMP = 2
 		}
 		if len(k8sVer) == 0 {
 			k8sVer = "1.27.1"
 		}
 
-	case string(consts.CLOUD_AZURE) + string(consts.CLUSTER_TYPE_MANG):
+	case string(consts.CloudAzure) + string(consts.ClusterTypeMang):
 		if len(nodeSizeMP) == 0 {
 			nodeSizeMP = "Standard_DS2_v2"
 		}
@@ -191,8 +191,7 @@ func SetDefaults(provider consts.KsctlCloud, clusterType consts.KsctlClusterType
 			k8sVer = "1.27"
 		}
 
-
-	case string(consts.CLOUD_CIVO) + string(consts.CLUSTER_TYPE_MANG):
+	case string(consts.CloudCivo) + string(consts.ClusterTypeMang):
 		if len(nodeSizeMP) == 0 {
 			nodeSizeMP = "g4s.kube.small"
 		}
@@ -206,7 +205,7 @@ func SetDefaults(provider consts.KsctlCloud, clusterType consts.KsctlClusterType
 			k8sVer = "1.27.1"
 		}
 
-	case string(consts.CLOUD_AZURE) + string(consts.CLUSTER_TYPE_HA):
+	case string(consts.CloudAzure) + string(consts.ClusterTypeHa):
 		if len(nodeSizeCP) == 0 {
 			nodeSizeCP = "Standard_F2s"
 		}
@@ -235,10 +234,10 @@ func SetDefaults(provider consts.KsctlCloud, clusterType consts.KsctlClusterType
 			k8sVer = "1.27.1"
 		}
 		if len(distro) == 0 {
-			distro = string(consts.K8S_K3S)
+			distro = string(consts.K8sK3s)
 		}
 
-	case string(consts.CLOUD_CIVO) + string(consts.CLUSTER_TYPE_HA):
+	case string(consts.CloudCivo) + string(consts.ClusterTypeHa):
 		if len(nodeSizeCP) == 0 {
 			nodeSizeCP = "g3.small"
 		}
@@ -267,7 +266,7 @@ func SetDefaults(provider consts.KsctlCloud, clusterType consts.KsctlClusterType
 			k8sVer = "1.27.1"
 		}
 		if len(distro) == 0 {
-			distro = string(consts.K8S_K3S)
+			distro = string(consts.K8sK3s)
 		}
 	}
 }