openconfig / ygot / 14265122775

89%

master: 89%

Build Type

Pull #1028

github

Committed by tengyiG

Commit Message

Fix a bug in protoFromPathsInternal function in proto.go causing encapsulation header details to be lost when mapping gNMI paths to protobuf fiedls.

Root cause: the issue stems from incorrect path construction when recursing into nested messages. Something like this will happen: "DEBUG: recursing into nested message gribi_aft.Afts.NextHop.EncapHeader.mpls, path: elem:{name:"afts"} elem:{name:"next-hops"} elem:{name:"next-hop"} elem:{name:"encap-headers"} elem:{name:"encap-header" key:{key:"index" value:"11"}} elem:{name:"afts"} elem:{name:"next-hops"} elem:{name:"next-hop"} elem:{name:"encap-headers"} elem:{name:"encap-header"} elem:{name:"mpls"}.

When findChilren uses these malformed paths to search for nested fields, it returns 0 children because no paths in the input map match this duplicated structure.

Pull Request Pull Request #1028: Fix a bug in protoFromPathsInternal function in proto.go causing enca…

Run Details

26 of 26 new or added lines in 2 files covered. (100.0%)

48 existing lines in 2 files now uncovered.

13940 of 15698 relevant lines covered (88.8%)

481.8 hits per line