+1Thank yo for feature request however here are few points against implementing such specification:
- General argument used to support such new BGP features:
Since BGP is such a stable and widely deployed thing,
let's add all kind of random stuff to it.
Once you start adding stuff, BGP implementation ceases
to be simple, stable, and gets whole lot of new
failure modes. Quoting rfc:
"While it is certainly possible to address this problem using other
mechanisms, the authors believe that this solution offers the
substantial advantage of being an incremental addition to already
deployed mechanisms."
But you still need to configure it on each BGP speaker.
Why not, under the hood, make it a separate protocol so
that bugs in this thing don't pull whole BGP under?
- BGP NLRI is meant to carry address prefixes, which, in the
case of VPNv6, are 8 bytes RD + 16 bytes IPv6.
Data structures that handle such short ( <= 24 bytes)
prefixes well are generally not suited for storing
arbitrary length (into kilobytes) NLRI this RFC proposes to use.
The SANE way would have been to use NLRI as a short ID,
and keep flow matchers in the attributes. No reason in RFC is
given for their choice.
- If BGP speaker that redistributes flowspec gets DOSed,
flowspecs are removed from BGP peers. RFC does not
explain how this case is handled.
- This thing needs a way for administrator to monitor/limit
the amount of distributed info. You cannot use existing
tools for this thing.
Regardless of your like or dislike for Flowspec, that's what the community is moving forward with, so you need to get on board or you get replaced.Thank yo for feature request however here are few points against implementing such specification:
- General argument used to support such new BGP features:
Since BGP is such a stable and widely deployed thing,
let's add all kind of random stuff to it.
Once you start adding stuff, BGP implementation ceases
to be simple, stable, and gets whole lot of new
failure modes. Quoting rfc:
"While it is certainly possible to address this problem using other
mechanisms, the authors believe that this solution offers the
substantial advantage of being an incremental addition to already
deployed mechanisms."
But you still need to configure it on each BGP speaker.
Why not, under the hood, make it a separate protocol so
that bugs in this thing don't pull whole BGP under?
- BGP NLRI is meant to carry address prefixes, which, in the
case of VPNv6, are 8 bytes RD + 16 bytes IPv6.
Data structures that handle such short ( <= 24 bytes)
prefixes well are generally not suited for storing
arbitrary length (into kilobytes) NLRI this RFC proposes to use.
The SANE way would have been to use NLRI as a short ID,
and keep flow matchers in the attributes. No reason in RFC is
given for their choice.
- If BGP speaker that redistributes flowspec gets DOSed,
flowspecs are removed from BGP peers. RFC does not
explain how this case is handled.
- This thing needs a way for administrator to monitor/limit
the amount of distributed info. You cannot use existing
tools for this thing.
v7 is a mythIt probably would require the new routing engine in v7, so whenever that happens is the earliest we can expect FlowSpec.
I still want it ASAP, though.