fix

Author: nexustar

pkg/telemetry/meta.go

@@ -37,8 +37,6 @@ const (
 	DisableStatus Status = "disable"
 )
 
-const defaultStatus = ""
-
 // Meta data of telemetry.
 type Meta struct {
 	UUID   string `yaml:"uuid,omitempty"`
@@ -88,11 +86,11 @@ func LoadFrom(fname string) (meta *Meta, err error) {
 
 	// populate UUID and secret if not set
 	var updated bool
-	if meta.UUID == "" {
+	if meta.UUID == "" && meta.Status == EnableStatus {
 		meta.UUID = NewUUID()
 		updated = true
 	}
-	if meta.Secret == "" {
+	if meta.Secret == "" && meta.Status == EnableStatus {
 		meta.Secret = NewSecret()
 		updated = true
 	}

pkg/telemetry/meta_test.go

@@ -23,7 +23,7 @@ func (s *teleSuite) TestTelemetry(c *check.C) {
 	// Should no error and get a default meta.
 	meta, err := LoadFrom(fname)
 	c.Assert(err, check.IsNil)
-	c.Assert(meta.Status, check.Equals, defaultStatus)
+	c.Assert(meta.Status, check.Equals, Status(""))
 	c.Assert(len(meta.UUID), check.Equals, 0)
 
 	// Save and load back