From 6fbba33f8123b69f398ac3cd7166ebac9311fde1 Mon Sep 17 00:00:00 2001 From: urvisavla Date: Tue, 24 Sep 2024 01:25:00 -0700 Subject: [PATCH] fix CI failures due to race condition in recoverysigner unit tests (#5472) * fix CI failures due to race condition in recoverysigner unit tests * Update support/db/dbtest/db.go Co-authored-by: tamirms --- support/db/dbtest/db.go | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/support/db/dbtest/db.go b/support/db/dbtest/db.go index 18de073224..42f252b6f8 100644 --- a/support/db/dbtest/db.go +++ b/support/db/dbtest/db.go @@ -128,9 +128,11 @@ func checkReadOnly(t testing.TB, DSN string) { if !rows.Next() { _, err = tx.Exec("CREATE ROLE user_ro WITH LOGIN PASSWORD 'user_ro';") if err != nil { - // Handle race condition by ignoring the error if it's a duplicate key violation - if pqErr, ok := err.(*pq.Error); ok && pqErr.Code == "23505" { + // Handle race condition by ignoring the error if it's a duplicate key violation or duplicate object error + if pqErr, ok := err.(*pq.Error); ok && (pqErr.Code == "23505" || pqErr.Code == "42710") { return + } else if ok { + t.Logf("pq error code: %s", pqErr.Code) } } require.NoError(t, err)