I am the assigned Gen-ART reviewer for this draft. The General Area
Review Team (Gen-ART) reviews all IETF documents being processed
by the IESG for the IETF Chair. Please treat these comments just
like any other last call comments.
For more information, please see the FAQ at
<https://trac.ietf.org/trac/gen/wiki/GenArtfaq>.
Document: draft-ietf-anima-brski-prm-17
Reviewer: Paul Kyzivat
Review Date: 2025-01-27
IETF LC End Date: 2025-01-30
IESG Telechat date: ?
Summary:
This draft is basically ready for publication, but has nits that should
be fixed before publication.
This was this reviewer's first introduction to anima. As a result, this
review is limited to document form, not technical details.
It is a very well written document. It is however forbiddingly long and
intimidating to read. It has a very distinct repetitive structure. (Many
twisty little passages, all different, but very similar.) It seems like
it could perhaps be represented in a more concise way that would be
easier to read. But I don't have a specific suggestion.
ISSUES: 0
NITS: 3
1) NIT: Non-RFC2606-compliant FQDN
Section 6.1.2 (Discovery of the Pledge) uses an FQDN,
"_brski-pledge._tcp.local", that triggers an IdNits
non-RFC2606-compliant FQDN warning. This is perhaps mitigated somewhat
by being a local FQDN.
I don't have a specific recommendation whether or not to change this.
2) NIT: Language
In the first paragraph of Section 8 (Logging), I suggest minor syntax
tweaks in the first paragraph:
s/Recommend key events for logging comprise/Recommended key events for
logging comprise:/
3) NIT: Stale References
There are three outdated references to related anima documents. I trust
these will be dealt with in due course.
--
last-call mailing list -- last-call@xxxxxxxx
To unsubscribe send an email to last-call-leave@xxxxxxxx
