Re: [netmod] augment and if-feature

[[e-mail] (Dale R. Worley)]

JOEY BOYD <[e-mail]> writes:
> module base-module {
>   prefix bmod;
>
>   feature do-things;
>
>   container things {
>     if-feature do-things;
>     ...
>   }
> }
>
> module augment-module {
>   prefix amod;
>
>   augment "/bmod:do-things" {
>     container other-things {
>     }
>   }
> }

First question:  I'm not expert in Yang, but as far as I can figure out,
the augment statement is augmenting "container things", right?  So the
augment statement should be 'augment "/bmod:things"' not 'augment
"/bmod:do-things"'.

But on the important question, I don't see it as at all unreasonable
that the augment needs to be qualified by the same if-feature.  The
reason is that if you're reading the text of module augment-module, it's
helpful to have documented, right there, that the augmentation depends
on the presence of a particular feature in the augmented module.  And
it's helpful to know that the designer did, at least for one moment,
think about the fact that the augmentation is conditional.

Dale