Éric Vyncke has entered the following ballot position for draft-ietf-anima-grasp-api-08: 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/iesg/statement/discuss-criteria.html for more information about IESG DISCUSS and COMMENT positions. The document, along with other ballot positions, can be found here: https://datatracker.ietf.org/doc/draft-ietf-anima-grasp-api/ ---------------------------------------------------------------------- COMMENT: ---------------------------------------------------------------------- Thank you for the work put into this document. Please find below one some non-blocking COMMENT points, and one nits. I have also request IoT directorate and INT directorate reviews, so, you may expect more reviews. I hope that this helps to improve the document, Regards, -éric == COMMENTS == -- Section 1 -- In figure 1, is the "GRASP API Library" identical to the "basic GRASP library" mentioned later in the text? -- Section 2.1 -- May I assume that the bulleted list is not exhaustive? Probably worth stating "For example, ..." if this is the case. -- Section 2.2.1 -- This whole section looks more like a tutorial than something useful in an IETF document ;-) but no problem to leave it. Same applies for section 2.2.2 and even to 2.2.3. -- Section 2.3.2.2 -- Should it be specified that the timeout is an *unsigned* integer? Same applies for "loop_count" in section 2.3.2.3 -- Section 2.3 -- Several occurrences of "returned parameters"... should it better be "returned values" ? -- Section 2.3.3 -- "All ASAs must use this call." should it be followed by "before issuing any other API calls" ? "automatically if an ASA crashes" but what about "graceful termination" ? == NITS == -- Section 1 -- Suggestion move figure 1 earlier in the text to improve readability. _______________________________________________ Anima mailing list [email protected] https://www.ietf.org/mailman/listinfo/anima
