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

Static route subscribe path #125

Merged
merged 4 commits into from
Nov 10, 2021
Merged

Conversation

zhaozhenhong
Copy link

No description provided.

@smahara
Copy link
Collaborator

smahara commented Oct 7, 2021

Everything under "/openconfig-network-instance:network-instances/network-instance[name=*]/protocols/protocol[identifier=STATIC][name=static]/static-routes|" supports on_change right? Add that in the description.
Also the title of the file at the top says "Interface Telemetry Subscribe". Instead you can say "Telemetry subscribe for various OC-yang/modules"

@zhaozhenhong
Copy link
Author

zhaozhenhong commented Nov 4, 2021

Everything under "/openconfig-network-instance:network-instances/network-instance[name=*]/protocols/protocol[identifier=STATIC][name=static]/static-routes|" supports on_change right? Add that in the description. Also the title of the file at the top says "Interface Telemetry Subscribe". Instead you can say "Telemetry subscribe for various OC-yang/modules"

@smahara The title you mentioned is only for interface model. Static route has it own title and more description was added to clarify the path hierarchy.

@bhavini-gada bhavini-gada merged commit 1ade910 into master Nov 10, 2021
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

Successfully merging this pull request may close these issues.

3 participants