Refactor VSCode launch configuration by removing the 'showLog' parameter for cleaner setup. Update transaction error handling in tx_load_tool.go to prevent multiple error logs for the same failure, and add a warning for cases with no p2p events received.

Author: tosettil-polimi

.vscode/launch.json

@@ -8,17 +8,15 @@
       "mode": "auto",
       "program": "${workspaceFolder}/main.go",
       "args": ["--config", ".hack/devnet/generated-assertoor-config.yaml"],
-      "preLaunchTask": "devnet-setup",
-      "showLog": true
+      "preLaunchTask": "devnet-setup"
     },
     {
       "name": "Debug go assertoor",
       "type": "go",
       "request": "launch",
       "mode": "auto",
       "program": "${workspaceFolder}/main.go",
-      "args": ["--config", ".hack/devnet/generated-assertoor-config.yaml"],
-      "showLog": true
+      "args": ["--config", ".hack/devnet/generated-assertoor-config.yaml"]
     }
   ]
 }

pkg/coordinator/utils/tx_load_tool/tx_load_tool.go

@@ -130,9 +130,12 @@ func (l *Load) Execute() error {
 				l.Result.TxStartTime[i] = time.Now()
 				err = l.target.sendTransaction(tx)
 				if err != nil {
-					l.target.logger.WithField("client", l.target.client.GetName()).Errorf("Failed to send transaction: %v", err)
-					l.target.task_ctx.SetResult(types.TaskResultFailure)
-					l.Result.Failed = true
+					if !l.Result.Failed {
+						l.target.logger.WithField("client", l.target.client.GetName()).Errorf("Failed to send transaction: %v", err)
+						l.target.task_ctx.SetResult(types.TaskResultFailure)
+						l.Result.Failed = true
+					}
+
 					return
 				}
 
@@ -208,6 +211,11 @@ func (l *Load) MeasurePropagationLatencies() (*LoadResult, error) {
 			return l.Result, fmt.Errorf("measurement stopped: failed reading p2p events")
 		}
 
+		if txes == nil || len(*txes) == 0 {
+			l.target.logger.Warnf("No p2p events received")
+			continue
+		}
+
 		for i, tx := range *txes {
 			tx_data := tx.Data()
 			// read tx_data that is in the format "tx_index:<index>"