fix(analytics): respect environment variable to disable reporting eve…

…nt analytics Signed-off-by: Ethan Mosbaugh <[email protected]>
openebs · Nov 7, 2024 · dbc556a · dbc556a
1 parent f0f4acc
commit dbc556a
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 9 deletions.
diff --git a/changelogs/unreleased/210-disable-event-analytics b/changelogs/unreleased/210-disable-event-analytics
@@ -0,0 +1 @@
+Fix respect OPENEBS_IO_ENABLE_ANALYTICS environment variable to disable reporting event analytics.
diff --git a/cmd/provisioner-localpv/app/provisioner.go b/cmd/provisioner-localpv/app/provisioner.go
@@ -23,6 +23,7 @@ import (
 	analytics "github.com/openebs/google-analytics-4/usage"
 	"github.com/openebs/maya/pkg/alertlog"
 	mconfig "github.com/openebs/maya/pkg/apis/openebs.io/v1alpha1"
+	menv "github.com/openebs/maya/pkg/env/v1alpha1"
 	"github.com/pkg/errors"
 	v1 "k8s.io/api/core/v1"
 	"k8s.io/apimachinery/pkg/api/resource"
@@ -192,15 +193,17 @@ func (p *Provisioner) Delete(ctx context.Context, pv *v1.PersistentVolume) (err
 
 // sendEventOrIgnore sends anonymous local-pv provision/delete events
 func sendEventOrIgnore(pvcName, pvName, capacity, stgType, method string) {
-	stgType = "local-" + stgType
-
-	analytics.New().CommonBuild(stgType).ApplicationBuilder().
-		SetVolumeName(pvName).
-		SetVolumeClaimName(pvcName).
-		SetReplicaCount(DefaultUnknownReplicaCount).
-		SetCategory(method).
-		SetVolumeCapacity(capacity).
-		Send()
+	if menv.Truthy(menv.OpenEBSEnableAnalytics) {
+		stgType = "local-" + stgType
+
+		analytics.New().CommonBuild(stgType).ApplicationBuilder().
+			SetVolumeName(pvName).
+			SetVolumeClaimName(pvcName).
+			SetReplicaCount(DefaultUnknownReplicaCount).
+			SetCategory(method).
+			SetVolumeCapacity(capacity).
+			Send()
+	}
 }
 
 // validateVolumeSource validates datasource field of the pvc.