Re: [bess] ID-Nits for draft-ietf-bess-evpn-lsp-ping

2022-02-02 Thread Parag Jain (paragj)
Hi Matthew

Will do.

Thanks
Parag

From: "Bocci, Matthew (Nokia - GB)" 
Date: Wednesday, February 2, 2022 at 11:10 AM
To: "draft-ietf-bess-evpn-lsp-p...@ietf.org" 

Cc: "bess-cha...@ietf.org" , "bess@ietf.org" 

Subject: Re: ID-Nits for draft-ietf-bess-evpn-lsp-ping
Resent-From: 
Resent-To: , , , 
, 
Resent-Date: Wednesday, February 2, 2022 at 11:09 AM

Hi Authors

While going through this I have a couple of other comments:

Please can you clean up the use of RFC2119 language, in particular capitalizing 
SHOULD, MUST etc where appropriate. For example, I think the following in 
Section 4.3 should be corrected:


Ethernet Tag field value in EVPN Ethernet AD Sub-TLV, should be set

   according to the context:



   o  For per-ES context, the Ethernet Tag field in the sub-TLV must be

  set to the reserved MAX-ET value 
[RFC7432]



   o  For per-EVI context, the Ethernet Tag field in the sub-TLV must be

  set to the non-reserved value


Thanks
Matthew

From: Bocci, Matthew (Nokia - GB) 
Date: Wednesday, 2 February 2022 at 16:00
To: draft-ietf-bess-evpn-lsp-p...@ietf.org 

Cc: bess-cha...@ietf.org 
Subject: ID-Nits for draft-ietf-bess-evpn-lsp-ping
Hi Authors

I am finally doing the document shepherds write up for this draft. ID Nits 
throws up a few warnings, which I would appreciate if you could fix now before 
I request publication:

https://www.ietf.org/tools/idnits?url=https://www.ietf.org/archive/id/draft-ietf-bess-evpn-lsp-ping-06.txt


Thanks

Matthew
___
BESS mailing list
BESS@ietf.org
https://www.ietf.org/mailman/listinfo/bess


Re: [bess] ID-Nits for draft-ietf-bess-evpn-lsp-ping

2022-02-02 Thread Bocci, Matthew (Nokia - GB)
Hi Authors

While going through this I have a couple of other comments:

Please can you clean up the use of RFC2119 language, in particular capitalizing 
SHOULD, MUST etc where appropriate. For example, I think the following in 
Section 4.3 should be corrected:


Ethernet Tag field value in EVPN Ethernet AD Sub-TLV, should be set

   according to the context:



   o  For per-ES context, the Ethernet Tag field in the sub-TLV must be

  set to the reserved MAX-ET value 
[RFC7432]



   o  For per-EVI context, the Ethernet Tag field in the sub-TLV must be

  set to the non-reserved value


Thanks
Matthew

From: Bocci, Matthew (Nokia - GB) 
Date: Wednesday, 2 February 2022 at 16:00
To: draft-ietf-bess-evpn-lsp-p...@ietf.org 

Cc: bess-cha...@ietf.org 
Subject: ID-Nits for draft-ietf-bess-evpn-lsp-ping
Hi Authors

I am finally doing the document shepherds write up for this draft. ID Nits 
throws up a few warnings, which I would appreciate if you could fix now before 
I request publication:

https://www.ietf.org/tools/idnits?url=https://www.ietf.org/archive/id/draft-ietf-bess-evpn-lsp-ping-06.txt


Thanks

Matthew
___
BESS mailing list
BESS@ietf.org
https://www.ietf.org/mailman/listinfo/bess