Re: [ippm] Erik Kline's No Objection on draft-ietf-ippm-ioam-conf-state-07: (with COMMENT)

[[e-mail]]

Hi Erik,






Thank you for the review and thoughtful comments.


Please check inline the proposed changes that will be incorporated into the next revision.





Best Regards,


Xiao Min







Original



From: ErikKlineviaDatatracker <[e-mail]>
To: The IESG <[e-mail]>;
Cc: [e-mail] <[e-mail]>;[e-mail] <[e-mail]>;[e-mail] <[e-mail]>;[e-mail] <[e-mail]>;[e-mail] <[e-mail]>;
Date: 2022年10月27日 16:00
Subject: Erik Kline's No Objection on draft-ietf-ippm-ioam-conf-state-07: (with COMMENT)


Erik Kline has entered the following ballot position for
draft-ietf-ippm-ioam-conf-state-07: No Objection
 
When responding, please keep the subject line intact and reply to all
email addresses included in the To and CC lines. (Feel free to cut this
introductory paragraph, however.)
 
 
Please refer to https://www.ietf.org/about/groups/iesg/statements/handling-ballot-positions/  
for more information about how to handle DISCUSS and COMMENT positions.
 
 
The document, along with other ballot positions, can be found here:
https://datatracker.ietf.org/doc/draft-ietf-ippm-ioam-conf-state/
 
 
 
----------------------------------------------------------------------
COMMENT:
----------------------------------------------------------------------
 
# Internet AD comments for draft-ietf-ippm-ioam-conf-state-07
CC @ekline
 
I support John's request for clarification.
 
## Comments
 
### S3.1
 
* "MUST send an echo reply without IOAM capabilities or no echo reply" 
 
  I think I understand the intent, and overall I think this text is likely
  to not be an issue in practice.  But I'll note that anything "MUST" where
  ICMP is concerned may not exactly be enforceable.

[XM]>>> Understood. Will remain the text you quoted as is. At the same time, I propose a change in S3.2 as below because I found there is exception out of MUST.

OLD

A list of IOAM capabilities objects (one
 or more objects) which contains the enabled IOAM capabilities MUST be
 included in this container of echo reply.NEW

A list of IOAM capabilities objects (one
 or more objects) which contains the enabled IOAM capabilities MUST be
 included in this container of echo reply except the sender encounters 
 an error (e.g., no matched Namespace-ID).


## Nits
 
### S2.2
 
* Maybe consider adding DEX to this list (it's also fully expanded on first
  use in S1, but it might be nice for ease of reference).

[XM]>>> OK. Will add DEX to this list.


### S3.1, S3.2
 
* /depends on the specific environment it is applied at/ -> 
  /depends on the specific deployment environment/ maybe?
 [XM]>>> OK. Will change the text as you suggested.