Hello,
I implemented the “draft-ietf-opsawg-l3sm-l3nm-03” (Revision 2020-04-03) on
Cisco NSO and Cisco Crosswork Network Controller. I am happy to fill any forms
(sorry but new to the WG).
The good news is that I did not find any syntax problem and was able to compile
and implement the full model.
I do have two on this new version for the WG:
1) Major: Why importing the "ietf-l3vpn-svc" module?
I think this is a major problem as it violates the architecture in Section 4
and the note in section 5 that states: "Note that the use of L3NM within a
Service Provider does assume nor preclude exposing the VPN service via L3SM.".
If there is no dependencies with L3SM, we should not force the network
controller to support it, neither its libraries. I noted that the SVC data
model is mainly important to have a common library of types, identities and
features. However, the correct implementation would be a separate YANG module
for these libraries that both L3SM and L3NM import from.
All in all, importing the SVC module breaks the separation between the two
layers. If we would like to re-use types, identities, features definitions, a
separate module for them should be defined.
2) Minor: RD Semantic:
The document proposes:
leaf rd {
type union {
type rt-types:route-distinguisher;
type empty;
}
}
Leafs have an implicit empty state (unless you use the default
statement) but the authors are trying to give a specific meaning to the empty
state by using a union statement. I believe a much natural representation would
be:
leaf rd {
type rt-types:route-distinguisher;
}
leaf auto-assign-rd {
when "not(../rd)";
type boolean;
}
Regards,
Roque
_______________________________________________
OPSAWG mailing list
[email protected]
https://www.ietf.org/mailman/listinfo/opsawg
