Make error message easier (#91)

openconfig · Apr 26, 2023 · 87c93e3 · 87c93e3
1 parent 87b71a8
commit 87c93e3
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git 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
@@ -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)
 	}