From 35c07ba3b91ed2162b27748e51c564a77ea017d8 Mon Sep 17 00:00:00 2001 From: Daniel Grau Date: Wed, 26 Apr 2023 22:01:36 +0000 Subject: [PATCH] Make error message easier --- ygnmi/gnmi.go | 2 +- ygnmi/ygnmi.go | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/ygnmi/gnmi.go b/ygnmi/gnmi.go index 6fb7780..aebe8e7 100644 --- a/ygnmi/gnmi.go +++ b/ygnmi/gnmi.go @@ -123,7 +123,7 @@ func (gs *getSubscriber) Send(req *gpb.SubscribeRequest) error { if st, ok := status.FromError(err); ok && st.Code() == codes.NotFound { // Make this behave like Subscribe, where non-existent paths don't return values. return nil } else if err != nil { - return err + return fmt.Errorf("gNMI.Get request failed: %w", err) } gs.notifs = resp.GetNotification() return nil diff --git a/ygnmi/ygnmi.go b/ygnmi/ygnmi.go index 3893d78..9614d8d 100644 --- a/ygnmi/ygnmi.go +++ b/ygnmi/ygnmi.go @@ -414,10 +414,10 @@ func LookupAll[T any](ctx context.Context, c *Client, q WildcardQuery[T], opts . return nil, fmt.Errorf("failed to unmarshal data: %w", err) } if v.ComplianceErrors != nil { + log.V(0).Infof("noncompliant data encountered while unmarshalling: %v", v.ComplianceErrors) if q.isLeaf() { continue } - log.V(0).Infof("noncompliant data encountered while unmarshalling: %v", v.ComplianceErrors) } vals = append(vals, v) }