From 0c3cd8a99529849ee2e3f9661ebfa937f3f9be66 Mon Sep 17 00:00:00 2001 From: Derek McGowan Date: Thu, 29 May 2025 15:34:06 -0700 Subject: [PATCH] Add debug log when transfer returns not implemented Currently the error details are not included in the output error and there is no log. One of the reasons a the transferer was skipped could be do to a specific component which is not implemented (such as trying to use erofs differ) or unsupported image (pulling schema1). This information is useful to find a bad configuration. Signed-off-by: Derek McGowan --- plugins/services/transfer/service.go | 1 + 1 file changed, 1 insertion(+) diff --git a/plugins/services/transfer/service.go b/plugins/services/transfer/service.go index 9b4113f204..fa862a4c33 100644 --- a/plugins/services/transfer/service.go +++ b/plugins/services/transfer/service.go @@ -138,6 +138,7 @@ func (s *service) Transfer(ctx context.Context, req *transferapi.TransferRequest } else if !errdefs.IsNotImplemented(err) { return nil, errgrpc.ToGRPC(err) } + log.G(ctx).WithError(err).Debugf("transfer not implemented for %T to %T", src, dst) } return nil, status.Errorf(codes.Unimplemented, "method Transfer not implemented for %s to %s", req.Source.GetTypeUrl(), req.Destination.GetTypeUrl()) }