openconfig/public

aft route summary model refactor required

dplore opened this issue · 1 comments

dplore commented
          @dplore Looks like this introduced an OpenConfig style error that unfortunately the script didn't catch. I have a rough draft modification that will catch this error, but I want to start the conversation on how to fix it:
public/release/models/network-instance/openconfig-network-instance.yang:1394 (at public/release/models/aft/openconfig-aft.yang:192): error: Parent node (ipv4-unicast) has multiple children where one is a list, but a list is not allowed to have siblings: list: ipv4-entry, container: summary
public/release/models/network-instance/openconfig-network-instance.yang:1394 (at public/release/models/aft/openconfig-aft.yang:206): error: Parent node (ipv6-unicast) has multiple children where one is a list, but a list is not allowed to have siblings: list: ipv6-entry, container: summary

Originally posted by @wenovus in #783 (comment)

dplore commented

The fix for this is probably to move the summary container to instantiated at /afts/ipv4-unicast/summary and /afts/ipv6-unicast/summary