Merge branch 'telemetry-shift' of github.com:kubefirst/kubefirst-api …

…into telemetry-shift
konstructio · Oct 22, 2023 · 66cef8e · 66cef8e
2 parents 6ec6cd2 + 1794dc2
commit 66cef8e
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 4 deletions.
diff --git a/internal/controller/argocd.go b/internal/controller/argocd.go
@@ -179,7 +179,6 @@ func (clctrl *ClusterController) DeployRegistryApplication() error {
 		return err
 	}
 
-	// telemetry todo
 
 	if !cl.ArgoCDCreateRegistryCheck {
 		var kcfg *k8s.KubernetesClient

diff --git a/internal/controller/state.go b/internal/controller/state.go
@@ -127,7 +127,7 @@ func (clctrl *ClusterController) StateStoreCredentials() error {
 			if err != nil {
 				msg := fmt.Sprintf("error creating google bucket %s: %s", clctrl.KubefirstStateStoreBucketName, err)
 				log.Error()
-				telemetry.SendCountMetric(clctrl.Telemetry, metrics.StateStoreCreateFailed, msg)
+				telemetry.SendCountMetric(clctrl.Telemetry, metrics.StateStoreCredentialsCreateFailed, msg)
 				return fmt.Errorf(msg)
 			}
 
@@ -142,7 +142,7 @@ func (clctrl *ClusterController) StateStoreCredentials() error {
 
 			objst, err := vultrConf.CreateObjectStorage(clctrl.KubefirstStateStoreBucketName)
 			if err != nil {
-				telemetry.SendCountMetric(clctrl.Telemetry, metrics.StateStoreCreateFailed, err.Error())
+				telemetry.SendCountMetric(clctrl.Telemetry, metrics.StateStoreCredentialsCreateFailed, err.Error())
 				log.Error(err.Error())
 				return err
 			}

diff --git a/providers/vultr/delete.go b/providers/vultr/delete.go
@@ -43,7 +43,7 @@ func DeleteVultrCluster(cl *pkgtypes.Cluster, segmentClient *telemetry.SegmentCl
 	log.SetReportCaller(false)
 	log.SetOutput(os.Stdout)
 
-	telemetry.SendCountMetric(segmentClient, metrics.ClusterInstallStarted, "")
+	telemetry.SendCountMetric(segmentClient, metrics.ClusterDeleteStarted, "")
 
 	// Instantiate vultr config
 	config := providerConfigs.GetConfig(cl.ClusterName, cl.DomainName, cl.GitProvider, cl.GitAuth.Owner, cl.GitProtocol, cl.CloudflareAuth.Token, "")