SONiC icon indicating copy to clipboard operation
SONiC copied to clipboard

NETCONF Server HLD

Open ntt-omw opened this issue 2 years ago • 9 comments

NETCONF Server HLD

ntt-omw avatar Jul 14 '23 11:07 ntt-omw

CLA Signed

The committers listed above are authorized under a signed CLA.

  • :white_check_mark: login: nakano-omw / name: Kanji Nakano (3f6984140effb92c5bfd0e45b9b76d40e8c96c11)
  • :white_check_mark: login: ebiken-ntt / name: Kentaro Ebisawa (32e394b6e48a4e91bb78617d69bb61e83010e60e)

Will the NETCONF server be disabled by default ? since we already have REST, and Telemetry running.

mbalachandar avatar Aug 04 '23 23:08 mbalachandar

NETCONF server will be running in separate docker container?

mbalachandar avatar Aug 05 '23 00:08 mbalachandar

@mbalachandar

Will the NETCONF server be disabled by default ? since we already have REST, and Telemetry running.

Yes. NETCONF will be disabled by default. We are considering making it configurable (e.g., via config_db), once we decide if it should be a separate container or not.

NETCONF server will be running in separate docker container?

Currently, no. It's running as a process in mgmt-framework container. But we can consider separating it to a separate docker container if that's your recommendation and fits better with the SONiC architecture.

ebiken-ntt avatar Aug 08 '23 14:08 ebiken-ntt

How do ensure that the NetConf running database and Redis database are synchronized? for example CLI ? Is it possible for NetConf to fetch directly from the Redis database and deprecate its own running database

zyboy2000 avatar Aug 10 '23 01:08 zyboy2000

In the context of NETCONF capabilities, does this HLD will cover extended capabilities as well?

ridahanif96 avatar Sep 18 '23 04:09 ridahanif96

Noting as future reference. (Comment from Kwan Kim)

In today’s (14th Sep) meeting, one of questions from NTT is about NETCONF Get-config operation.

NETCONF Get-config can be translated to REST Get with the “content” query parameter, i.e. content=config. https://datatracker.ietf.org/doc/html/rfc8040#section-4.8.1

We have opened PRs for query parameter support. https://github.com/sonic-net/sonic-mgmt-common/pull/102 https://github.com/sonic-net/sonic-mgmt-framework/pull/119

ebiken-ntt avatar Sep 21 '23 00:09 ebiken-ntt

not target 202311 per Kanji update. Keep a record.

zhangyanzhao avatar Sep 21 '23 01:09 zhangyanzhao

@ntt-omw @nakano-omw @ebiken-ntt Hi , I have read the document you submitted and I fully agree with your design for integrating netconf into Sonic. I wanted to inquire if you have already completed the development work to support netconf in Sonic. Recently, I have been considering enabling netconf functionality in my Sonic switch. Although your documentation is very detailed, I am not sure where to start. Could you provide me with some advice or suggest some source code for me to study?

burnCalories avatar Apr 02 '24 06:04 burnCalories