Fix CI failures in deploy telemetry

Two fixes:

1. Remove redundant empty BundleDeployEvent logging in root.go Execute.
   The defer in process.go already logs detailed deploy telemetry via
   LogDeployTelemetry on all exit paths. The root.go code caused duplicate
   events and Upload retries (mock returns numProtoSuccess=1).

2. Add HasConfigUsed guard in telemetry.Upload before calling ConfigUsed.
   When bundle config fails early (e.g., restricted script execution
   blocking preinit), ConfigUsed is never set on the context. The
   LogDeployTelemetry defer still logs an event, so Upload would panic
   trying to create an API client without auth config.

Co-authored-by: Isaac

Author: shreyas-goenka

cmd/root/root.go

@@ -179,15 +179,6 @@ Stack Trace:
 	commandStr := commandString(cmd)
 	ctx = cmd.Context()
 
-	// Log bundle deploy failures. Only log if we have successfully configured
-	// an authenticated Databricks client. We cannot log unauthenticated telemetry
-	// from the CLI yet.
-	if cmdctx.HasConfigUsed(ctx) && commandStr == "bundle_deploy" && exitCode != 0 {
-		telemetry.Log(ctx, protos.DatabricksCliLog{
-			BundleDeployEvent: &protos.BundleDeployEvent{},
-		})
-	}
-
 	telemetryErr := telemetry.Upload(cmd.Context(), protos.ExecutionContext{
 		CmdExecID:       cmdctx.ExecId(ctx),
 		Version:         build.GetInfo().Version,

libs/telemetry/logger.go

@@ -76,6 +76,11 @@ func Upload(ctx context.Context, ec protos.ExecutionContext) error {
 		protoLogs[i] = string(b)
 	}
 
+	if !cmdctx.HasConfigUsed(ctx) {
+		log.Debugf(ctx, "no auth config available; skipping telemetry upload")
+		return nil
+	}
+
 	apiClient, err := client.New(cmdctx.ConfigUsed(ctx))
 	if err != nil {
 		return fmt.Errorf("failed to create API client: %w", err)