Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Source VRF or leafref to ietf-network-instance #5

Closed
ericvoit opened this issue Oct 16, 2017 · 1 comment
Closed

Source VRF or leafref to ietf-network-instance #5

ericvoit opened this issue Oct 16, 2017 · 1 comment

Comments

@ericvoit
Copy link
Contributor

We have a choice or source-vrf as

(a) a string or (b) a leafref to ietf-network-instance model.

Nothing is wrong with ietf-network-instance model, but it adds the following:

  • timeframe for WGLC
  • draft has a dependency on schema-mount
  • no insights into industry adoption

For the short term at least, (b) adds extra complexity, as the subscribed-notifications model will have to include custom augmentations for vendors to support different VRFs.

@ericvoit
Copy link
Contributor Author

ericvoit commented Dec 7, 2017

Based on the discussions completing, I believe there to be rough consensus on the configuration of a VRF for a configured subscription being via a leafref to network-instances with vrf-support being an if-feature. I.e.:
+--ro source-vrf? -> /ni:network-instances/network-instance/name {supports-vrf}?

Based on the planned time frame for network-instances.yang going to the IESG, I don't expect there to be any timing issues. But if it does turn out that there is a delay in this draft, this issue many need to be revisited.

@ericvoit ericvoit closed this as completed Dec 7, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant