Remove enableAnalytics flag

Signed-off-by: Tamal Saha <[email protected]>
voyagermesh · Jan 2, 2022 · 3583265 · 3583265
1 parent 21e6643
commit 3583265
Show file tree

Hide file tree

Showing 5 changed files with 1 addition and 8 deletions.
diff --git a/apis/installer/v1alpha1/voyager_types.go b/apis/installer/v1alpha1/voyager_types.go
@@ -84,9 +84,7 @@ type VoyagerSpec struct {
 	// +optional
 	IngressClass *string     `json:"ingressClass"`
 	Apiserver    WebHookSpec `json:"apiserver"`
-	//+optional
-	EnableAnalytics bool      `json:"enableAnalytics"`
-	Templates       Templates `json:"templates"`
+	Templates    Templates   `json:"templates"`
 	// +optional
 	License string `json:"license"`
 }

diff --git a/charts/voyager/README.md b/charts/voyager/README.md
@@ -91,7 +91,6 @@ The following table lists the configurable parameters of the `voyager` chart and
 | apiserver.servingCerts.caCrt          | CA certficate used by serving certificate of webhook server.                                                                                                                                                                                                                                                                                        | `""`                                                                  |
 | apiserver.servingCerts.serverCrt      | Serving certficate used by webhook server.                                                                                                                                                                                                                                                                                                          | `""`                                                                  |
 | apiserver.servingCerts.serverKey      | Private key for the serving certificate used by webhook server.                                                                                                                                                                                                                                                                                     | `""`                                                                  |
-| enableAnalytics                       | If true, sends usage analytics                                                                                                                                                                                                                                                                                                                      | `true`                                                                |
 | templates.cfgmap                      | Name of configmap with custom templates                                                                                                                                                                                                                                                                                                             | ``                                                                    |
 
 

diff --git a/charts/voyager/templates/deployment-operator.yaml b/charts/voyager/templates/deployment-operator.yaml
@@ -50,7 +50,6 @@ spec:
         - --coordinator-image={{ list (include "operator.registry" .) .Values.operator.repository | compact | join "/" }}:{{ .Values.operator.tag }}
         - --bypass-validating-webhook-xray={{ .Values.apiserver.bypassValidatingWebhookXray }}
         - --use-kubeapiserver-fqdn-for-aks={{ .Values.apiserver.useKubeapiserverFqdnForAks }}
-        - --enable-analytics={{ .Values.enableAnalytics }}
       {{- if .Values.templates.cfgmap }}
         - --custom-templates=/srv/voyager/custom/*.cfg
       {{- end }}

diff --git a/charts/voyager/templates/deployment-webhook-server.yaml b/charts/voyager/templates/deployment-webhook-server.yaml
@@ -51,7 +51,6 @@ spec:
         - --tls-private-key-file=/var/serving-cert/tls.key
         - --bypass-validating-webhook-xray={{ .Values.apiserver.bypassValidatingWebhookXray }}
         - --use-kubeapiserver-fqdn-for-aks={{ .Values.apiserver.useKubeapiserverFqdnForAks }}
-        - --enable-analytics={{ .Values.enableAnalytics }}
       {{- if include "appscode.license" . }}
         - --license-file=/var/run/secrets/appscode/license/key.txt
       {{- end }}

diff --git a/charts/voyager/values.yaml b/charts/voyager/values.yaml
@@ -126,8 +126,6 @@ apiserver:
     serverCrt: ""
     # Private key for the serving certificate used by webhook server.
     serverKey: ""
-# If true, sends usage analytics
-enableAnalytics: true
 # Customization of templates
 templates:
   # Name of configmap with custom templates