From dcefb1fe1e1ff2c2a063a7a5d65231457915d58e Mon Sep 17 00:00:00 2001
From: Tim Vaillancourt <tim@timvaillancourt.com>
Date: Mon, 10 Feb 2025 03:52:39 +0100
Subject: [PATCH] improve logging

Signed-off-by: Tim Vaillancourt <tim@timvaillancourt.com>
---
 go/vt/vtorc/logic/cell_discovery.go      | 5 +++--
 go/vt/vtorc/logic/cell_discovery_test.go | 2 +-
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/go/vt/vtorc/logic/cell_discovery.go b/go/vt/vtorc/logic/cell_discovery.go
index 647ccc8e557..1e21178f3a1 100644
--- a/go/vt/vtorc/logic/cell_discovery.go
+++ b/go/vt/vtorc/logic/cell_discovery.go
@@ -49,7 +49,7 @@ func refreshCells(cells []string) (err error) {
 	for _, cell := range cells {
 		err = inst.SaveCell(cell)
 		if err != nil {
-			log.Error(err)
+			log.Errorf("Failed to save cell %q: %+v", cell, err)
 			return err
 		}
 		updated[cell] = true
@@ -59,6 +59,7 @@ func refreshCells(cells []string) (err error) {
 	// be changing because we are holding a lock.
 	cells, err = inst.ReadCells()
 	if err != nil {
+		log.Errorf("Failed to read cells: %+v", err)
 		return err
 	}
 
@@ -69,7 +70,7 @@ func refreshCells(cells []string) (err error) {
 		}
 		log.Infof("Forgetting stale cell %q", cell)
 		if err = inst.DeleteCell(cell); err != nil {
-			return err
+			log.Errorf("Failed to delete cell %q: %+v", cell, err)
 		}
 	}
 	return nil
diff --git a/go/vt/vtorc/logic/cell_discovery_test.go b/go/vt/vtorc/logic/cell_discovery_test.go
index 0ed4fa54b8a..9dcabaebfe1 100644
--- a/go/vt/vtorc/logic/cell_discovery_test.go
+++ b/go/vt/vtorc/logic/cell_discovery_test.go
@@ -27,7 +27,7 @@ import (
 	"vitess.io/vitess/go/vt/vtorc/inst"
 )
 
-func TestRefreshAllCells(t *testing.T) {
+func TestRefreshCells(t *testing.T) {
 	// Store the old flags and restore on test completion
 	oldTs := ts
 	defer func() {