diff --git a/linkerd/app/inbound/src/policy/api.rs b/linkerd/app/inbound/src/policy/api.rs index ab646fe5b3..87e4638997 100644 --- a/linkerd/app/inbound/src/policy/api.rs +++ b/linkerd/app/inbound/src/policy/api.rs @@ -109,7 +109,11 @@ impl Recover for GrpcRecover { return Err(status); } - tracing::trace!(%status, "Recovering"); + tracing::warn!( + grpc.status = %status.code(), + grpc.message = status.message(), + "Unexpected policy controller response; retrying with a backoff", + ); Ok(self.0.stream()) } } diff --git a/linkerd/app/outbound/src/policy/api.rs b/linkerd/app/outbound/src/policy/api.rs index 57c55eff34..d4020677c8 100644 --- a/linkerd/app/outbound/src/policy/api.rs +++ b/linkerd/app/outbound/src/policy/api.rs @@ -113,8 +113,12 @@ impl Recover for GrpcRecover { tonic::Code::InvalidArgument | tonic::Code::FailedPrecondition => Err(status), // Indicates no policy for this target tonic::Code::NotFound | tonic::Code::Unimplemented => Err(status), - _ => { - tracing::debug!(%status, "Recovering"); + code => { + tracing::warn!( + grpc.status = %code, + grpc.message = status.message(), + "Unexpected policy controller response; retrying with a backoff", + ); Ok(self.0.stream()) } } diff --git a/linkerd/app/src/dst.rs b/linkerd/app/src/dst.rs index 3c6c66f0ac..f0ce8622c4 100644 --- a/linkerd/app/src/dst.rs +++ b/linkerd/app/src/dst.rs @@ -94,7 +94,11 @@ impl Recover for BackoffUnlessInvalidArgument { return Err(status); } - tracing::trace!(%status, "Recovering"); + tracing::warn!( + grpc.status = %status.code(), + grpc.message = status.message(), + "Unexpected destination controller response; retrying with a backoff", + ); Ok(self.0.stream()) } }