Re: [Idr] I-D Action: draft-lin-idr-bgp-nof-nlri-01.txt

[Robert Raszuk <[e-mail]>]

Dear authors of  draft-lin-idr-bgp-nof-nlri/
<https://datatracker.ietf.org/doc/draft-lin-idr-bgp-nof-nlri/>,

I have read your draft and have few comments.

First if we consider BGP to be a database distribution protocol the
described idea seems pretty interesting (in a positive sense). I am not
sure why you would like to use BGP and RRs to discover storage devices,
especially considering that all you need is a p2p information channel.
Moreover PUB-SUB model seems to be also requirement here (for example to
limit received information to a given zone). IMO use of DROID (
https://www.ietf.org/id/draft-li-lsr-droid-00.html) or any other pub-sub
package may be a much better fit.

But let's assume you do want to use the BGP protocol. With that I have two
technical suggestions:

**A** Use Transport Instance BGP with new port number as described in
https://datatracker.ietf.org/doc/html/draft-raszuk-ti-bgp-01

**B** Redefine BGP encoding to define one short NLRI to be an information
key and put all TLVs into the BGP Attribute. Currently "Device Discovery
NLRI" contains in itself all discovery information which is in parallel
accompanied by "Device Zone NLRI". In BGP NLRIs are independent entities
and your encoding model is trying to glue them. Redefining it should be a
pretty simple task from all of the above.

Kind regards,
Robert

On Wed, May 11, 2022 at 4:32 AM <[e-mail]> wrote:

>
> A New Internet-Draft is available from the on-line Internet-Drafts
> directories.
>
>
>         Title           : Distribution of Device Discovery Information in
> NVMe Over RoCEv2 Storage Network Using BGP
>         Authors         : Changwang Lin
>                           Mengxiao Chen
>                           Hao Li
>                           Ruixue Wang
>                           Fengwei Qin
>                           Qi Zhang
>         Filename        : draft-lin-idr-bgp-nof-nlri-01.txt
>         Pages           : 16
>         Date            : 2022-05-10
>
> Abstract:
>    This document proposes a method of distributing device discovery
>    information in NVMe over RoCEv2 storage network using the BGP
>    routing protocol. A new BGP Network Layer Reachability Information
>    (NLRI) encoding format, named NoF NLRI, is defined.
>
>
> The IETF datatracker status page for this draft is:
> https://datatracker.ietf.org/doc/draft-lin-idr-bgp-nof-nlri/
>
> There is also an htmlized version available at:
> https://datatracker.ietf.org/doc/html/draft-lin-idr-bgp-nof-nlri-01
>
> A diff from the previous version is available at:
> https://www.ietf.org/rfcdiff?url2=draft-lin-idr-bgp-nof-nlri-01
>
>
> Internet-Drafts are also available by rsync at rsync.ietf.org:
> :internet-drafts
>
>
> _______________________________________________
> I-D-Announce mailing list
> [e-mail]
> https://www.ietf.org/mailman/listinfo/i-d-announce
> Internet-Draft directories: http://www.ietf.org/shadow.html
> or ftp://ftp.ietf.org/ietf/1shadow-sites.txt
>