From 0c7f2c9c0c7b2bda57d99ffc70e4c3535eb48a1c Mon Sep 17 00:00:00 2001 From: Miguel Martinez Trivino Date: Sat, 21 Mar 2026 12:44:26 +0100 Subject: [PATCH] fix(cli): demote retry log messages to debug level Transient retry errors during attestation operations were logged at error level, causing noisy output for users. These are expected during normal operation and only useful for debugging. Signed-off-by: Miguel Martinez Trivino --- app/cli/cmd/errors.go | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/app/cli/cmd/errors.go b/app/cli/cmd/errors.go index f2449c3f9..8273cd0a1 100644 --- a/app/cli/cmd/errors.go +++ b/app/cli/cmd/errors.go @@ -1,5 +1,5 @@ // -// Copyright 2023 The Chainloop Authors. +// Copyright 2023-2026 The Chainloop Authors. // // Licensed under the Apache License, Version 2.0 (the "License"); // you may not use this file except in compliance with the License. @@ -87,6 +87,11 @@ func runWithBackoffRetry(fn func() error) error { }, backoff.NewExponentialBackOff(backoff.WithMaxElapsedTime(3*time.Minute)), func(err error, delay time.Duration) { - logger.Err(err).Msgf("retrying in %s", delay) + l := logger.Debug().Err(err) + if v1.IsAttestationStateErrorConflict(err) { + l.Msgf("concurrent state update detected, retrying in %s", delay) + } else { + l.Msgf("retrying in %s", delay) + } }) }