rodecker / draft-yang-bgp-communities Goto Github PK
View Code? Open in Web Editor NEWYANG model for BGP communities
YANG model for BGP communities
00 supports Two-Octet and Four-Octet AS-Specific Extended Communities.
Should we extend the model to include IPv6-Specific Extended Communities and possibly others?
Reported by: Luuk Hendriks
https://datatracker.ietf.org/doc/html/rfc7950#section-7.21.4
Add in (00) section 5.6, 5.7, 5.8, 5.6.4.2.3.
Reported by: Tom Petch
https://mailarchive.ietf.org/arch/msg/grow/F2Nn5KiB2asa15lUldZMqZKxlbQ/
The abstract is not specific enough about what we are trying to achieve ("data model defining the structures to be used to specify a BGP community").
Reported by: Tom Petch
https://mailarchive.ietf.org/arch/msg/grow/F2Nn5KiB2asa15lUldZMqZKxlbQ/
When a parser finds a community with a Private ASN in the globaladmin field, it cannot determine which ASN has set the community. We need some language/guidelines about this.
Should there be enumerated fields in the model denoting this?
Reported by: Dale W. Carder
https://mailarchive.ietf.org/arch/msg/grow/ieXVv5Z_YZQ0153AsLY34o3Xd2s/
Switch section order.
Reported by: Tom Petch
https://mailarchive.ietf.org/arch/msg/grow/F2Nn5KiB2asa15lUldZMqZKxlbQ/
Instead of "ordered from most to least specific" language, make it explicit that the first match wins (as written in https://github.com/rodecker/draft-yang-bgp-communities/blob/main/draft/draft-yang-bgp-communities.xml#L503).
Reported by: Luuk Hendriks
Would it be helpful to template common scenario's like do-not-send-to-xxx and prepend-x-times?
Reported by: Dale W. Carder
https://mailarchive.ietf.org/arch/msg/grow/ieXVv5Z_YZQ0153AsLY34o3Xd2s/
Datatype "string" has very few constraints. Use custom datatypes where possible (e.g. in contacturl and emailaddress leafs). In other cases (e.g. description leafs) at least add some limitation on length and permitted characters.
Reported by: Tom Petch
https://mailarchive.ietf.org/arch/msg/grow/F2Nn5KiB2asa15lUldZMqZKxlbQ/
Miscellaneous warnings:
----------------------------------------------------------------------------
== Line 178 has weird spacing: '...aladmin uin...'
== Using lowercase 'not' together with uppercase 'MUST', 'SHALL', 'SHOULD',
or 'RECOMMENDED' is not an accepted usage according to RFC 2119. Please
use uppercase 'NOT' together with RFC 2119 keywords (if that is what you
mean).
Found 'MUST not' in this paragraph:
A list of objects that together form the Local Administrator part
of the community. The combined length values of all fields MUST not
exceed the maximum length of the Local Administrator part of the
community.
== Using lowercase 'not' together with uppercase 'MUST', 'SHALL', 'SHOULD',
or 'RECOMMENDED' is not an accepted usage according to RFC 2119. Please
use uppercase 'NOT' together with RFC 2119 keywords (if that is what you
mean).
Found 'MUST not' in this paragraph:
A list of objects that together form the Local Administrator part
of the community. The combined length values of all fields MUST not
exceed the maximum length of the Local Administrator part of the
community.
== Using lowercase 'not' together with uppercase 'MUST', 'SHALL', 'SHOULD',
or 'RECOMMENDED' is not an accepted usage according to RFC 2119. Please
use uppercase 'NOT' together with RFC 2119 keywords (if that is what you
mean).
Found 'MUST not' in this paragraph:
A list of objects that together form the Local Data Part 1 section
of the community. The combined length values of all fields MUST not
exceed the maximum length of the "Local Data Part 1" section of the
community.
== Using lowercase 'not' together with uppercase 'MUST', 'SHALL', 'SHOULD',
or 'RECOMMENDED' is not an accepted usage according to RFC 2119. Please
use uppercase 'NOT' together with RFC 2119 keywords (if that is what you
mean).
Found 'MUST not' in this paragraph:
A list of objects that together form the Local Data Part 2 section
of the community. The combined length values of all fields MUST not
exceed the maximum length of the "Local Data Part 2" section of the
community.
== Couldn't figure out when the document was first submitted -- there may
comments or warnings related to the use of a disclaimer for pre-RFC5378
work that could not be issued because of this. Please check the Legal
Provisions document at https://trustee.ietf.org/license-info to determine
if you need the pre-RFC5378 disclaimer.
Checking references for intended status: Informational
----------------------------------------------------------------------------
== Outdated reference: A later version (-24) exists of
draft-ietf-core-sid-20
A declarative, efficient, and flexible JavaScript library for building user interfaces.
๐ Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
An Open Source Machine Learning Framework for Everyone
The Web framework for perfectionists with deadlines.
A PHP framework for web artisans
Bring data to life with SVG, Canvas and HTML. ๐๐๐
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
Some thing interesting about web. New door for the world.
A server is a program made to process requests and deliver data to clients.
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
Some thing interesting about visualization, use data art
Some thing interesting about game, make everyone happy.
We are working to build community through open source technology. NB: members must have two-factor auth.
Open source projects and samples from Microsoft.
Google โค๏ธ Open Source for everyone.
Alibaba Open Source for everyone
Data-Driven Documents codes.
China tencent open source team.