Add error to metrics

.github/workflows/release.yml

@@ -69,5 +69,5 @@ jobs:
         env:
           # https://docs.github.com/en/actions/security-guides/automatic-token-authentication#about-the-github_token-secret
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
-          TELEMETRY_TOKEN: ${{ secrets.POSTHOG_PROD_TOKEN }}
+          AVALANCHE_CLI_METRICS_TOKEN: ${{ secrets.POSTHOG_PROD_TOKEN }}
           DOCKERFILE: ./Dockerfile.release

.goreleaser.yml

@@ -9,7 +9,7 @@ builds:
     # contains linux and darwin
     ldflags:
       - -X 'github.com/ava-labs/avalanche-cli/cmd.Version={{.Version}}'
-      - -X 'github.com/ava-labs/avalanche-cli/pkg/metrics.telemetryToken={{ .Env.TELEMETRY_TOKEN }}'
+      - -X 'github.com/ava-labs/avalanche-cli/pkg/metrics.telemetryToken={{ .Env.AVALANCHE_CLI_METRICS_TOKEN }}'
     goos:
       - linux
       - darwin

VERSION

@@ -1 +1 @@
-1.8.8
+1.8.9

cmd/blockchaincmd/create.go

@@ -455,7 +455,7 @@ func createBlockchainConfig(cmd *cobra.Command, args []string) error {
 	}
 
 	if vmType == models.SubnetEvm {
-		err = sendMetrics(cmd, vmType.RepoName(), blockchainName)
+		err = sendMetrics(vmType.RepoName(), blockchainName)
 		if err != nil {
 			return err
 		}
@@ -486,7 +486,7 @@ func addSubnetEVMGenesisPrefundedAddress(genesisBytes []byte, address string, ba
 	return json.MarshalIndent(genesisMap, "", "  ")
 }
 
-func sendMetrics(cmd *cobra.Command, repoName, blockchainName string) error {
+func sendMetrics(repoName, blockchainName string) error {
 	flags := make(map[string]string)
 	flags[constants.SubnetType] = repoName
 	genesis, err := app.LoadEvmGenesis(blockchainName)
@@ -513,7 +513,7 @@ func sendMetrics(cmd *cobra.Command, repoName, blockchainName string) error {
 	precompilesJoined := strings.Join(precompiles, ",")
 	flags[constants.PrecompileType] = precompilesJoined
 	flags[constants.NumberOfAirdrops] = strconv.Itoa(numAirdropAddresses)
-	metrics.HandleTracking(cmd, constants.MetricsSubnetCreateCommand, app, flags)
+	metrics.HandleTracking(app, flags, nil)
 	return nil
 }
 

cmd/blockchaincmd/deploy.go

@@ -950,7 +950,7 @@ func deployBlockchain(cmd *cobra.Command, args []string) error {
 
 	flags := make(map[string]string)
 	flags[constants.MetricsNetwork] = network.Name()
-	metrics.HandleTracking(cmd, constants.MetricsSubnetDeployCommand, app, flags)
+	metrics.HandleTracking(app, flags, nil)
 
 	if network.Kind == models.Local && !simulatedPublicNetwork() {
 		ux.Logger.PrintToUser("")

cmd/blockchaincmd/export_test.go

@@ -33,7 +33,7 @@ func TestExportImportSubnet(t *testing.T) {
 	mockAppDownloader := mocks.Downloader{}
 	mockAppDownloader.On("Download", mock.Anything).Return(testSubnetEVMCompat, nil)
 
-	app.Setup(testDir, logging.NoLog{}, nil, "", prompts.NewPrompter(), &mockAppDownloader)
+	app.Setup(testDir, logging.NoLog{}, nil, "", prompts.NewPrompter(), &mockAppDownloader, nil)
 	ux.NewUserLog(logging.NoLog{}, io.Discard)
 	genBytes, err := os.ReadFile("../../" + utils.SubnetEvmGenesisPath)
 	require.NoError(err)

cmd/blockchaincmd/publish_test.go

@@ -415,7 +415,7 @@ func setupTestEnv(t *testing.T) (*require.Assertions, *mocks.Prompter) {
 	ux.NewUserLog(logging.NoLog{}, io.Discard)
 	app = &application.Avalanche{}
 	mockPrompt := mocks.NewPrompter(t)
-	app.Setup(testDir, logging.NoLog{}, config.New(), "", mockPrompt, application.NewDownloader())
+	app.Setup(testDir, logging.NoLog{}, config.New(), "", mockPrompt, application.NewDownloader(), nil)
 
 	return require, mockPrompt
 }

cmd/blockchaincmd/upgradecmd/vm_test.go

@@ -306,7 +306,7 @@ func TestUpdateToCustomBin(t *testing.T) {
 	ux.NewUserLog(log, os.Stdout)
 
 	app = &application.Avalanche{}
-	app.Setup(testDir, log, config.New(), "", prompts.NewPrompter(), application.NewDownloader())
+	app.Setup(testDir, log, config.New(), "", prompts.NewPrompter(), application.NewDownloader(), nil)
 
 	err = os.MkdirAll(app.GetSubnetDir(), constants.DefaultPerms755)
 	assert.NoError(err)

cmd/nodecmd/create.go

@@ -791,7 +791,7 @@ func createNodes(cmd *cobra.Command, args []string) error {
 		printResults(cloudConfigMap, publicIPMap, monitoringPublicIP)
 		ux.Logger.PrintToUser(logging.Green.Wrap("AvalancheGo and Avalanche-CLI installed and node(s) are bootstrapping!"))
 	}
-	sendNodeCreateMetrics(cmd, cloudService, network.Name(), numNodesMetricsMap)
+	sendNodeCreateMetrics(cloudService, network.Name(), numNodesMetricsMap)
 	return nil
 }
 
@@ -1403,7 +1403,7 @@ func defaultAvalancheCLIPrefix(region string) (string, error) {
 	return usr.Username + "-" + region + constants.AvalancheCLISuffix, nil
 }
 
-func sendNodeCreateMetrics(cmd *cobra.Command, cloudService, network string, nodes map[string]NumNodes) {
+func sendNodeCreateMetrics(cloudService, network string, nodes map[string]NumNodes) {
 	flags := make(map[string]string)
 	totalValidatorNodes := 0
 	totalAPINodes := 0
@@ -1428,7 +1428,7 @@ func sendNodeCreateMetrics(cmd *cobra.Command, cloudService, network string, nod
 		populateSubnetVMMetrics(flags, wizSubnet)
 		flags[constants.MetricsCalledFromWiz] = strconv.FormatBool(true)
 	}
-	metrics.HandleTracking(cmd, constants.MetricsNodeCreateCommand, app, flags)
+	metrics.HandleTracking(app, flags, nil)
 }
 
 func getPrometheusTargets(clusterName string) ([]string, []string, []string, error) {

cmd/nodecmd/wiz.go

@@ -457,7 +457,7 @@ func wiz(cmd *cobra.Command, args []string) error {
 	if err := deployClusterYAMLFile(clusterName, subnetName); err != nil {
 		return err
 	}
-	sendNodeWizMetrics(cmd)
+	sendNodeWizMetrics()
 	return nil
 }
 
@@ -889,10 +889,10 @@ func setICMRelayerSecurityGroupRule(clusterName string, awmRelayerHost *models.H
 	return nil
 }
 
-func sendNodeWizMetrics(cmd *cobra.Command) {
+func sendNodeWizMetrics() {
 	flags := make(map[string]string)
 	populateSubnetVMMetrics(flags, wizSubnet)
-	metrics.HandleTracking(cmd, constants.MetricsNodeDevnetWizCommand, app, flags)
+	metrics.HandleTracking(app, flags, nil)
 }
 
 func populateSubnetVMMetrics(flags map[string]string, subnetName string) {

cmd/root.go

@@ -63,7 +63,6 @@ To get started, look at the documentation for the subcommands or jump right
 in with avalanche blockchain create myNewBlockchain.`,
 		PersistentPreRunE: createApp,
 		Version:           Version,
-		PersistentPostRun: handleTracking,
 		SilenceErrors:     true,
 		SilenceUsage:      true,
 	}
@@ -138,7 +137,7 @@ func createApp(cmd *cobra.Command, _ []string) error {
 	log.Info("-----------")
 	log.Info(fmt.Sprintf("cmd: %s", strings.Join(os.Args[1:], " ")))
 	cf := config.New()
-	app.Setup(baseDir, log, cf, Version, prompts.NewPrompter(), application.NewDownloader())
+	app.Setup(baseDir, log, cf, Version, prompts.NewPrompter(), application.NewDownloader(), cmd)
 
 	if err := initConfig(); err != nil {
 		return err
@@ -240,10 +239,6 @@ func checkForUpdates(cmd *cobra.Command, app *application.Avalanche) error {
 	return nil
 }
 
-func handleTracking(cmd *cobra.Command, _ []string) {
-	metrics.HandleTracking(cmd, cmd.CommandPath(), app, nil)
-}
-
 func setupEnv() (string, error) {
 	// Set base dir
 	usr, err := user.Current()
@@ -383,6 +378,7 @@ func Execute() {
 	app = application.New()
 	rootCmd := NewRootCmd()
 	err := rootCmd.Execute()
+	metrics.HandleTracking(app, nil, err)
 	cobrautils.HandleErrors(err)
 }
 

go.mod

@@ -33,7 +33,7 @@ require (
 	github.com/otiai10/copy v1.14.1
 	github.com/pborman/ansi v1.0.0
 	github.com/pingcap/errors v0.11.4
-	github.com/posthog/posthog-go v1.2.24
+	github.com/posthog/posthog-go v1.4.7
 	github.com/prometheus/client_golang v1.21.1
 	github.com/schollz/progressbar/v3 v3.17.1
 	github.com/shirou/gopsutil v3.21.11+incompatible

go.sum

@@ -748,8 +748,8 @@ github.com/pkg/term v0.0.0-20180730021639-bffc007b7fd5/go.mod h1:eCbImbZ95eXtAUI
 github.com/pmezard/go-difflib v1.0.0/go.mod h1:iKH77koFhYxTK1pcRnkKkqfTogsbg7gZNVY4sRDYZ/4=
 github.com/pmezard/go-difflib v1.0.1-0.20181226105442-5d4384ee4fb2 h1:Jamvg5psRIccs7FGNTlIRMkT8wgtp5eCXdBlqhYGL6U=
 github.com/pmezard/go-difflib v1.0.1-0.20181226105442-5d4384ee4fb2/go.mod h1:iKH77koFhYxTK1pcRnkKkqfTogsbg7gZNVY4sRDYZ/4=
-github.com/posthog/posthog-go v1.2.24 h1:A+iG4saBJemo++VDlcWovbYf8KFFNUfrCoJtsc40RPA=
-github.com/posthog/posthog-go v1.2.24/go.mod h1:uYC2l1Yktc8E+9FAHJ9QZG4vQf/NHJPD800Hsm7DzoM=
+github.com/posthog/posthog-go v1.4.7 h1:2DOcy1pLeLbfEG+WgK9S2WOoXk+N2DXYAN5/S9pwOh8=
+github.com/posthog/posthog-go v1.4.7/go.mod h1:uYC2l1Yktc8E+9FAHJ9QZG4vQf/NHJPD800Hsm7DzoM=
 github.com/prometheus/client_golang v0.9.1/go.mod h1:7SWBe2y4D6OKWSNQJUaRYU/AaXPKyh/dDVn+NZz0KFw=
 github.com/prometheus/client_golang v1.0.0/go.mod h1:db9x61etRT2tGnBNRi70OPL5FsnadC4Ky3P0J6CfImo=
 github.com/prometheus/client_golang v1.21.1 h1:DOvXXTqVzvkIewV/CDPFdejpMCGeMcbGCQ8YOmu+Ibk=

internal/migrations/migrations_test.go

@@ -23,7 +23,7 @@ func TestRunMigrations(t *testing.T) {
 	testDir := t.TempDir()
 
 	app := &application.Avalanche{}
-	app.Setup(testDir, logging.NoLog{}, config.New(), "", prompts.NewPrompter(), application.NewDownloader())
+	app.Setup(testDir, logging.NoLog{}, config.New(), "", prompts.NewPrompter(), application.NewDownloader(), nil)
 
 	type migTest struct {
 		migs           map[int]migrationFunc

internal/migrations/subnet_evm_rename_test.go

@@ -62,7 +62,7 @@ func TestSubnetEVMRenameMigration(t *testing.T) {
 			testDir := t.TempDir()
 
 			app := &application.Avalanche{}
-			app.Setup(testDir, logging.NoLog{}, config.New(), "", prompts.NewPrompter(), application.NewDownloader())
+			app.Setup(testDir, logging.NoLog{}, config.New(), "", prompts.NewPrompter(), application.NewDownloader(), nil)
 
 			err := app.CreateSidecar(tt.sc)
 			require.NoError(err)
@@ -91,7 +91,7 @@ func TestSubnetEVMRenameMigration_EmptyDir(t *testing.T) {
 	testDir := t.TempDir()
 
 	app := &application.Avalanche{}
-	app.Setup(testDir, logging.NoLog{}, config.New(), "", prompts.NewPrompter(), application.NewDownloader())
+	app.Setup(testDir, logging.NoLog{}, config.New(), "", prompts.NewPrompter(), application.NewDownloader(), nil)
 
 	emptySubnetName := "emptySubnet"
 

internal/migrations/toplevel_files_test.go

@@ -25,7 +25,7 @@ func TestTopLevelFilesMigration(t *testing.T) {
 	testDir := t.TempDir()
 
 	app := &application.Avalanche{}
-	app.Setup(testDir, logging.NoLog{}, config.New(), "", prompts.NewPrompter(), application.NewDownloader())
+	app.Setup(testDir, logging.NoLog{}, config.New(), "", prompts.NewPrompter(), application.NewDownloader(), nil)
 
 	testSC1 := &models.Sidecar{
 		Name: "test1",

internal/testutils/setup.go

@@ -24,7 +24,7 @@ func SetupTestInTempDir(t *testing.T) *application.Avalanche {
 	testDir := t.TempDir()
 
 	app := application.New()
-	app.Setup(testDir, logging.NoLog{}, &config.Config{}, "", nil, nil)
+	app.Setup(testDir, logging.NoLog{}, &config.Config{}, "", nil, nil, nil)
 	ux.NewUserLog(logging.NoLog{}, io.Discard)
 	return app
 }

pkg/apmintegration/file_test.go

@@ -63,7 +63,7 @@ const (
 func newTestApp(t *testing.T, testDir string) *application.Avalanche {
 	tempDir := t.TempDir()
 	app := application.New()
-	app.Setup(tempDir, logging.NoLog{}, nil, "", prompts.NewPrompter(), application.NewDownloader())
+	app.Setup(tempDir, logging.NoLog{}, nil, "", prompts.NewPrompter(), application.NewDownloader(), nil)
 	app.ApmDir = testDir
 	return app
 }

pkg/application/app.go

@@ -23,6 +23,7 @@ import (
 	"github.com/ava-labs/avalanchego/utils/set"
 	"github.com/ava-labs/subnet-evm/core"
 
+	"github.com/spf13/cobra"
 	"golang.org/x/exp/maps"
 )
 
@@ -35,6 +36,7 @@ type Avalanche struct {
 	Apm        *apm.APM
 	ApmDir     string
 	Downloader Downloader
+	Cmd        *cobra.Command
 }
 
 func New() *Avalanche {
@@ -48,13 +50,15 @@ func (app *Avalanche) Setup(
 	version string,
 	prompt prompts.Prompter,
 	downloader Downloader,
+	cmd *cobra.Command,
 ) {
 	app.baseDir = baseDir
 	app.Log = log
 	app.Conf = conf
 	app.Version = version
 	app.Prompt = prompt
 	app.Downloader = downloader
+	app.Cmd = cmd
 }
 
 func (app *Avalanche) GetRunFile(prefix string) string {

pkg/binutils/release_test.go

@@ -37,7 +37,7 @@ func setupInstallDir(require *require.Assertions) *application.Avalanche {
 	defer os.RemoveAll(rootDir)
 
 	app := application.New()
-	app.Setup(rootDir, logging.NoLog{}, &config.Config{}, "", prompts.NewPrompter(), application.NewDownloader())
+	app.Setup(rootDir, logging.NoLog{}, &config.Config{}, "", prompts.NewPrompter(), application.NewDownloader(), nil)
 	return app
 }