Uploaded image for project: 'yangtools'
  1. yangtools
  2. YANGTOOLS-859

Augmenting a container with if-feature

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Done
    • Icon: Medium Medium
    • 4.0.14, 6.0.1, 5.0.8
    • 1.2.1, 2.0.9
    • parser
    • None

      I would like ODL to reconsider the point of YANGTOOLS-811.

      I've been researching this a little, and it seems people from the RFC itself acknowledge that the RFC is unclear on this point:

      https://www.ietf.org/mail-archive/web/netmod/current/msg17835.html

      However, there are standardized YANG modules that are relying on the assumption that no if-feature is needed in the augment, see the following standardized models from BBF:

      https://github.com/BroadbandForum/yang/blob/master/standard/interface/bbf-sub-interfaces.yang

      https://github.com/BroadbandForum/yang/blob/master/standard/interface/bbf-sub-interface-tagging.yang

      The first model contains at the end containers ingress-rewrite and egress-rewrite that are conditional under the feature tag-rewrites.   The second module contains an augment on these containers, without an if-feature statement.

      It has also been brought to my attention that ConfD from Tail-F had provided a fix for this problem.    So it seems there is some kind of consensus out there that the if-feature statement is not needed in the augment.

      That is why I request that ODL reconsiders this.

            rovarga Robert Varga
            verthezpw Peter Verthez
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: