fix(das): Only debug log sampled header if it was actually successful (…

…#1295) Self explanatory Co-authored-by: Viacheslav <viacheslavgonkivskyi@gmail.com>
celestiaorg · Nov 1, 2022 · f5f5adb · f5f5adb
1 parent 9c996a7
commit f5f5adb
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/das/worker.go b/das/worker.go
@@ -70,8 +70,13 @@ func (w *worker) run(
 		}
 		w.setResult(curr, err)
 		metrics.observeSample(ctx, h, time.Since(startSample), err)
-		log.Debugw("sampled header", "height", h.Height, "hash", h.Hash(),
-			"square width", len(h.DAH.RowsRoots), "data root", h.DAH.Hash(), "finished (s)", time.Since(startSample))
+		if err != nil {
+			log.Debugw("failed to sampled header", "height", h.Height, "hash", h.Hash(),
+				"square width", len(h.DAH.RowsRoots), "data root", h.DAH.Hash(), "err", err)
+		} else {
+			log.Debugw("sampled header", "height", h.Height, "hash", h.Hash(),
+				"square width", len(h.DAH.RowsRoots), "data root", h.DAH.Hash(), "finished (s)", time.Since(startSample))
+		}
 	}
 
 	if w.state.Curr > w.state.From {