Skip to content

Commit

Permalink
Revert "Telemetry: Send preset value to telemetry"
Browse files Browse the repository at this point in the history
This reverts commit 275b2ad.
Previous commit take care of sending the preset value for each event and
now we don't require it only for start action.
  • Loading branch information
praveenkumar committed Jul 3, 2023
1 parent 0a0a111 commit b09f30a
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 6 deletions.
1 change: 0 additions & 1 deletion pkg/crc/machine/start.go
Original file line number Diff line number Diff line change
Expand Up @@ -269,7 +269,6 @@ func (client *client) Start(ctx context.Context, startConfig types.StartConfig)
telemetry.SetCPUs(ctx, startConfig.CPUs)
telemetry.SetMemory(ctx, uint64(startConfig.Memory)*1024*1024)
telemetry.SetDiskSize(ctx, uint64(startConfig.DiskSize)*1024*1024*1024)
telemetry.SetPreset(ctx, startConfig.Preset)

if err := client.validateStartConfig(startConfig); err != nil {
return nil, err
Expand Down
5 changes: 0 additions & 5 deletions pkg/crc/telemetry/telemetry.go
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@ import (
"sync"

"github.com/crc-org/crc/pkg/crc/constants"
crcpreset "github.com/crc-org/crc/pkg/crc/preset"
)

type contextKey struct{}
Expand Down Expand Up @@ -78,10 +77,6 @@ func SetConfigurationKey(ctx context.Context, value string) {
setContextProperty(ctx, "key", value)
}

func SetPreset(ctx context.Context, value crcpreset.Preset) {
setContextProperty(ctx, "preset", value)
}

func SetStartType(ctx context.Context, value StartType) {
setContextProperty(ctx, "start-type", value)
}
Expand Down

0 comments on commit b09f30a

Please sign in to comment.