-
Notifications
You must be signed in to change notification settings - Fork 105
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(analytics): adding google analytics for ZFSPV #49
Conversation
f39c106
to
e297fcc
Compare
Codecov Report
@@ Coverage Diff @@
## master #49 +/- ##
===========================================
- Coverage 83.33% 23.57% -59.76%
===========================================
Files 1 14 +13
Lines 72 475 +403
===========================================
+ Hits 60 112 +52
- Misses 12 362 +350
- Partials 0 1 +1
Continue to review full report at Codecov.
|
Hi @pawanpraka1 : Can you update the PR description with the following details:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've got to suggest some more code deletions.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
minor small change suggestios, PTAL
@@ -647,6 +650,10 @@ spec: | |||
value: unix:///var/lib/csi/sockets/pluginproxy/csi.sock | |||
- name: OPENEBS_NAMESPACE | |||
value: openebs | |||
- name: OPENEBS_IO_INSTALLER_TYPE |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pending addition to helm charts cc @ChandanSagar
// sendEventOrIgnore sends anonymous local-pv provision/delete events | ||
func sendEventOrIgnore(pvName, capacity, stgType, method string) { | ||
if zfs.GoogleAnalyticsEnabled == "true" { | ||
analytics.New().Build().ApplicationBuilder(). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
DoubleCheck: Confirm if AppName, AppID, AppVersion, etc are being set, etc.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Whenever a volume is provisioned and de-provisioned we will send a google event with mainly following details : 1. pvName (will shown as app title in google analytics) 2. size of the volume 3. event type : volume-provision, volume-deprovision 4. storage type zfs-localpv 5. replicacount as 1 6. ClientId as default namespace uuid Apart from this, we send the event once in 24 hr, which will have some info like number of nodes, node type, kubernetes version etc. This metric is cotrolled by OPENEBS_IO_ENABLE_ANALYTICS env. We can set it to false if we don't want to send the metrics. Signed-off-by: Pawan <pawan@mayadata.io>
Signed-off-by: Pawan <pawan@mayadata.io>
Whenever a volume is provisioned and de-provisioned we will send a google event with following details :
Apart from this, we send the event once in 24 hr, which will have some info like number of nodes, node type, kubernetes version etc.
This metric is cotrolled by OPENEBS_IO_ENABLE_ANALYTICS env. We can set it to false if we don't want to send the metrics.
Signed-off-by: Pawan pawan@mayadata.io