-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Sonic Netconf Server Proposal #1466
base: master
Are you sure you want to change the base?
Conversation
I believe there is another proposal going on for NetCONF server support in SONiC: |
Yes. We have discussed our (NTT's) NETCONF HLD #1428 in today's SONiC UMF working group meeting. We would also like @aseaudi to share this #1466 proposal to the UMF WG community. (Ritu or someone from the WG should contact you to join the WG meeting) |
BRCM registered as reviewer. |
@zhangyanzhao please add @prvattem, @rathnasabapathyv, @mhampasagar, @joyas-joseph as reviewers from Dell. |
Adding @anand-kumar-subramanian @a-barboza @mbalachandar @sachinholla @faraazbrcm to review and share comments |
added except mhampassagar who needs to accept the invitation to become a reviewer. |
invitation is sent out, will add them after accepting the invitation |
Can you please add the NETCONF server capabilities supported by the proposal? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need more information on the Netconf server similar to the sections 3.2.2.5 gNMI server and 4.4 gNMI flow.
doc/mgmt/Management Framework.md
Outdated
@@ -174,30 +175,37 @@ Management framework is a SONiC application which is responsible for providing v | |||
1. Standard [YANG](https://tools.ietf.org/html/rfc7950) models (e.g. OpenConfig, IETF, IEEE) | |||
2. Custom YANG models ([SONiC YANG](https://github.com/sonic-net/SONiC/blob/master/doc/mgmt/SONiC_YANG_Model_Guidelines.md)) | |||
3. Industry-standard CLI / Cisco like CLI | |||
|
|||
* Must provide support for [OpenAPI spec](http://spec.openapis.org/oas/v3.0.3) to generate REST server side code | |||
* Must provide support for [OpenAPI spec](http://spec.openapis.org/oas/v3.0.3) to generate REST server side code and NETCONF server side code |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is OpenAPI spec going to be used to generate NETCONF server side code?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it should be REST only, we removed NETCONF part
Done |
resolves #1498 |
code PR is not ready, move to backlog for future release |
This PR is for an update in the Sonic Management Framework to add an RFC Compliant Netconf Server, which reuses the Translib and the yang models that are already part of the management framework.
The updated management framework HLD that includes the Netconf Server is under /doc/mgmt/Management Framework.md