Merge pull request #79 from openconfig/origin-fix2

Fix setting origin for root path
openconfig · Dec 21, 2022 · 897697d · 897697d
2 parents 66e9876 + 43c0249
commit 897697d
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/internal/testutil/schema.go b/internal/testutil/schema.go
@@ -244,7 +244,7 @@ func GNMIPath(t testing.TB, s string) *gpb.Path {
 		t.Fatal(err)
 	}
 	// TODO: remove when fixed https://github.com/openconfig/ygot/issues/615
-	if len(p.Elem) > 0 && p.Elem[0].Name != "meta" {
+	if p.Origin == "" && (len(p.Elem) == 0 || p.Elem[0].Name != "meta") {
 		p.Origin = "openconfig"
 	}
 	return p

diff --git a/ygnmi/gnmi.go b/ygnmi/gnmi.go
@@ -446,7 +446,7 @@ func resolvePath(q PathStruct) (*gpb.Path, error) {
 	}
 
 	// TODO: remove when fixed https://github.com/openconfig/ygot/issues/615
-	if len(path.Elem) > 0 && path.Elem[0].Name != "meta" && path.Origin == "" {
+	if path.Origin == "" && (len(path.Elem) == 0 || path.Elem[0].Name != "meta") {
 		path.Origin = "openconfig"
 	}