Last Call Review of draft-ietf-nfsv4-rfc5661sesqui-msns-03
review-ietf-nfsv4-rfc5661sesqui-msns-03-secdir-lc-turner-2019-12-09-00
| Request | Review of | draft-ietf-nfsv4-rfc5661sesqui-msns |
|---|---|---|
| Requested revision | No specific revision (document currently at 04) | |
| Type | Last Call Review | |
| Team | Security Area Directorate (secdir) | |
| Deadline | 2019-11-25 | |
| Requested | 2019-11-04 | |
| Authors | David Noveck , Chuck Lever | |
| Draft last updated | 2019-12-09 | |
| Completed reviews |
Secdir Last Call review of -03
by
Sean Turner
(diff)
Genart Last Call review of -03 by Vijay K. Gurbani (diff) |
|
| Assignment | Reviewer | Sean Turner |
| State | Completed | |
| Review |
review-ietf-nfsv4-rfc5661sesqui-msns-03-secdir-lc-turner-2019-12-09
|
|
| Posted at | https://mailarchive.ietf.org/arch/msg/secdir/vqPpJS7g8OT5LQcs3MIdRzuLmf0 | |
| Reviewed revision | 03 (document currently at 04) | |
| Result | Ready | |
| Completed | 2019-12-09 |
review-ietf-nfsv4-rfc5661sesqui-msns-03-secdir-lc-turner-2019-12-09-00
I'm no NFS expert, but I did review draft-ietf-nfsv4-mv1-msns-update [0], which dealt with fixes in RFC5661 for multi-server namespace. This draft replaces that draft and obsoletes RFC5661 to incorporate changes documented in draft-ietf-nfsv4-mv1-msns-update. Almost all of the changes are to section 11 and 21. I briefly examined the changes in this draft and those noted in draft-ietf-nfsv4-mv1-msns-update, and it appears the changes are incorporated (I trust the authors faithfully made the changes). As far as the changes I suggested during my previous review [1], it appears as if the authors have: - major: 0-2 accepted and incorporated. The spirit of 3 was incorporated; makes it clear that AUTH_SYS provides "no provision to avoid modification of data in flight". - nits: incorporated 0-4, 6-8, and 10. I see the point to not making the changes suggested in 5 and 9. Cheers, spt [0] https://datatracker.ietf.org/doc/review-ietf-nfsv4-mv1-msns-update-04-secdir-lc-turner-2019-02-27/ [1] https://mailarchive.ietf.org/arch/msg/secdir/zjgQKz0KnQbQHrJmQGwWioNwEpg