Skip to main content

Telechat Review of draft-ietf-ippm-stamp-on-lag-05
review-ietf-ippm-stamp-on-lag-05-intdir-telechat-fressancourt-2023-11-21-00

Request Review of draft-ietf-ippm-stamp-on-lag
Requested revision No specific revision (document currently at 06)
Type Telechat Review
Team Internet Area Directorate (intdir)
Deadline 2023-11-23
Requested 2023-11-14
Requested by Éric Vyncke
Authors Zhenqiang Li , Tianran Zhou , Guo Jun , Greg Mirsky , Rakesh Gandhi
I-D last updated 2023-11-21
Completed reviews Intdir Telechat review of -05 by Haoyu Song (diff)
Intdir Telechat review of -05 by Antoine Fressancourt (diff)
Secdir Last Call review of -05 by Nancy Cam-Winget (diff)
Assignment Reviewer Antoine Fressancourt
State Completed
Request Telechat review on draft-ietf-ippm-stamp-on-lag by Internet Area Directorate Assigned
Posted at https://mailarchive.ietf.org/arch/msg/int-dir/zxVq3OIi6ZhIL28EHDvw6iiUajo
Reviewed revision 05 (document currently at 06)
Result Ready w/issues
Completed 2023-11-21
review-ietf-ippm-stamp-on-lag-05-intdir-telechat-fressancourt-2023-11-21-00
I am an assigned INT directorate reviewer for
draft-ietf-ippm-stamp-on-lag-05.txt. These comments were written primarily for
the benefit of the Internet Area Directors. Document editors and shepherd(s)
should treat these comments just like they would treat comments from any other
IETF contributors and resolve them along with any other Last Call comments that
have been received. For more details on the INT Directorate, see
https://datatracker.ietf.org/group/intdir/about/
<https://datatracker.ietf.org/group/intdir/about/>.

Based on my review, if I was on the IESG I would ballot this document as
DISCUSS.

* I have the following DISCUSS/ABSTAIN level issues:

** In Section 3.2, the behavior of the Session-Sender and of the
Session-Reflector regarding the value of the Reflector Micro-session ID field
is problematic. Indeed, the 3rd paragraph of Section 3.2 states that the
Session-Sender MUST set the Reflector Micro-session ID field if he knows it, or
set it to ZERO otherwise. Yet, the conditions in which this field is supposed
to be known are unclear, and the last sentence of the paragraph states that how
the Reflector is supposed to know this ID is outside the document's scope. As a
potential implementer of this protocol, I find this description puzzling, and
let me wonder when the Session-Reflector is supposed to know this ID. The 5th
paragraph of Section 3.2 mentions that the Session-Reflector MUST check the
value of the Reflector Micro-session ID if it is not set to ZERO, but is rather
unclear about what is the benefits one can take out of this verification. This
use and management of the Reflector Micro-session ID is even more confusing
when reading the last sentence of Section 3.2 which mentions that any procedure
with regards to the Micro-session ID is stateless.

The document should either mention the benefit that can be taken from having
the Session-Sender set the proper value for the Reflector Micro-session ID and
give some more details about how and when the Session-Sender are supposed to
learn about this ID, or be more relaxed with regards to the value of this
field. Besides, given that, to the best of my knowledge, RFC 8972 does not give
any constraint about the length of STAMP optionnal TLVs (even if the examples
given in RFC 8972 are all aligned to 4 bytes...), I wonder what is the benefit
from keeping the Reflector Micro-session ID in the TLV, so the overall
Micro-session ID TLV could be 6 bytes long.

* The following are other issues I found with this document that SHOULD be
corrected before publication:

** In Section 2, in the 4th paragraph, it is stated that "each micro STAMP
session MUST be assigned with a unique SSID", yet, if I read correctly, in RFC
8972 this MUST is a MAY (3rd paragraph in Section 3 of RFC 8972: "A STAMP
Session-Sender MAY generate a locally unique STAMP session Identifier
(SSID)."). This should be either harmonized or, if there is a reason for
requiring a MUST here, it should be clearly stated.

* The following are minor issues (typos, misspelling, minor text improvements)
with the document:

** In Section 1, the document should include a clear reference to OWAMP and
TWAMP to help the reader refer to the document describing those protocols.