Skip to main content

BGP Extended Communities Attribute
draft-ietf-idr-bgp-ext-communities-09

Discuss


Yes

(Bill Fenner)

No Objection

(Alex Zinin)
(Allison Mankin)
(Brian Carpenter)
(Jon Peterson)
(Margaret Cullen)
(Mark Townsley)
(Sam Hartman)
(Ted Hardie)

Note: This ballot was opened for revision 09 and is now closed.

Harald Alvestrand Former IESG member
Discuss
Discuss [Treat as non-blocking comment] (2005-02-17) Unknown
Got a review from Mark Allman just before telechat, indicating that stuff needs to be cleared up. This is a placeholder.
Thomas Narten Former IESG member
Discuss
Discuss [Treat as non-blocking comment] (2005-03-09) Unknown
High-level: this document could be a lot clearer in terms of what it
does. After re-reading multiple times, I conclude that what this
document could/should do is:

Defines a 1 octet type field (not 2-octet), but that  some of the
attributes have a 1-octet "code" field (ala ICMP). The codes are
specific to the type message, and some type codes don't even bother
defining any codes.

Thus, what IANA needs to do is create a type registry, where types are
1-octet values, and that each assigned value may (or may not) have
code field, as defined by the specifically defined type.

So there is no "extended type" per se. It's just that some types have a
sub-type or "code". 

>             I - IANA authority bit
> 
>                Value 0: IANA assignable type using the "First Come First
>                Serve" policy
> 
>                Value 1: IANA assignable type using the IETF Consensus
>                policy and experimental

I don't understand the value of this or what the technical point is.

Is there a _technical_ distinction between types with the high-order
bit set? I think no. But in that case, a better way to specify
the above is to just  say:

  Types 0-127 are assigned through "FCFS", types 128-255 are assigned
  via "IETF Consensus".

Finally, the above policies make me a little uncomfortable. We have a
number of exmaples where FCFS means someone goes to IANA and gets an
assignment. No Internet draft. No IETF review (even cursory). Is this
_really_ what the WG wants? I.e., couldn't you at least require an
Internet-Draft with (say) approval of the WG? While this isn't an
RFC2434 policy, the WG can certainly invent one, tailered to its
needs. For example, something like:

  Types 0-127 are assigned through "Expert Review" per RFC 2434. The
  intention is that the Expert Review consist of review of an Internet
  Draft (for which there is an expectation that an RFC will eventually
  be publishd), with approval of the IDR WG of the allocation.  In the
  event that the IDR WG no longer exists, Expert review would include
  consultation with the IDR mailing list or appropriate follow-on list
  where the BGP community can be openly consulted.

Finally, "experimental" use has a very specific meaning (see RFC
3692) and IANA doesn't assign them. IF you want to reserve some for
experimentation, this document should just do so by saying something
like:

Types 250-255 are reserved for Experinemtnal use as defined in RFC
3692.

>             T - Transitive bit
> 
>                Value 0: The community is transitive across ASes
> 
>                Value 1: The community is non-transitive across ASes

Here is a bit that does have technical significance. Too bad its not
the most significant bit, so that you coudl say something like:

types 0-127 are transitive across ASes, types 128-255 are
non-transitive.

But with the bit where it is, you really want to split the registry
into sub-registries, one for transitive and one for non-transitive.

Can this bit be made the most significant or is it too late for that?
I.e., I think what you need to say now is something like:

There are two kinds of types, transitive and
non-transitive. Transitive types include the ranges 0-63, 128-192,
Non-transitive types are in the range 64-127 and 193-255. Future
requests for assignment of a Type value must specify the specific type
of assignment that is requested.

Finally, I assume the1 T-bit really does  need to be part of the type value
itself? I.e, can the specific type (as part of its definition) state
whether an attribute is transitive or not, or is the intent that an
implementation be able to specify how to handle unrecognized types? (I
suspect so)
Bill Fenner Former IESG member
Yes
Yes () Unknown

                            
Alex Zinin Former IESG member
(was Discuss) No Objection
No Objection () Unknown

                            
Allison Mankin Former IESG member
No Objection
No Objection () Unknown

                            
Bert Wijnen Former IESG member
No Objection
No Objection (2005-07-06) Unknown
No further objections
Brian Carpenter Former IESG member
(was Discuss) No Objection
No Objection () Unknown

                            
David Kessens Former IESG member
(was Discuss) No Objection
No Objection (2005-02-17) Unknown
This document lacks an index and contains non US ASCII characters.

---
Value Field:

The encoding of the Value Field is dependent on the "type" of
the community as specified by the Type Field.
                  
Two extended communities are declared equal only when all 8 octets of
their encoding are equal.
---

the word 'encoding' is used in slightly different ways. I suggest to
avoid the word in the second sentence.
Jon Peterson Former IESG member
No Objection
No Objection () Unknown

                            
Margaret Cullen Former IESG member
No Objection
No Objection () Unknown

                            
Mark Townsley Former IESG member
(was Discuss) No Objection
No Objection () Unknown

                            
Russ Housley Former IESG member
(was Discuss) No Objection
No Objection (2005-02-16) Unknown
  The Abstract is insufficient.  It does not even mention the
  Extended Community Attribute.

  The document authors need to turn off hyphenation.
Sam Hartman Former IESG member
No Objection
No Objection () Unknown

                            
Scott Hollenbeck Former IESG member
No Objection
No Objection (2005-02-15) Unknown
The text in section 2 could be clearer about what actually identifies a regular type and an extended type.  After reading the text I half expected to see something like a bit flag being used to ID one or the other, but there's no bit flag and little additional text included to explain what distinguishes one type from the other.  It might be a good idea to supplement "The value of the high-order octet of the Type Field determines if an extended community is a Regular type or an Extended type" with some additional words to note that the value registered with IANA determines if a type is extended or regular.
Ted Hardie Former IESG member
No Objection
No Objection () Unknown