[SAM] Review of draft-samrg-sam-baseline-protocol-00

[Marc Petit-Huguenin <[e-mail]>]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Sorry for the delay in reviewing this.  I waited to base the review on an
implementation, but unfortunately I never got the time to do so.  So this is
only a partial review.


1. Title

I do not know if the rules are different for the IRTF, but shouldn't this draft
been called draft-irtf-samrg-sam-baseline-protocol instead?


2. Section 7.2.1 "...peer id closest to and less than the GroupId"

This idea of a Node-ID been close to another ID is specific to certain types of
overlays (like the CHORD-RELOAD default), but will probably do not make sense
for other types.  The text either need to be modified to work with any type of
overlay, or to explicitly say that this draft is only for RELOAD overlays in
which this make sense.


3. Section 7.2.1 structure definition

The Dictionary type is not defined in this spec.

Also I would suggest to follow the standard convention of using lowercase for
the character of a field.


4. Section 7.2.[2-5]

RELOAD message are client/server based.  A request is sent and retransmitted
until a matching response is received.  I am sure how the
Join/JoinAccept/JoinConfirm/JoinDecline mechanism is working inside this framework.


5. Section 10.1. Data Model section

The Data Model is probably SINGLE here.


6. Section 10.1. Access Control section

The Access Control policy NODE-MATCH assumes that the Resource Name is the
Node-ID of signer of the ALMtree entry.  If you want to use the SessionKey as
Resource Name, you probably need to define a new Access Control policy.


Nits
- ----

- - Section 5

s/Register Kind-ID points/Register Kind-ID codepoints/

- - Section 7.3.3

s/diffent/different/

- - Section 7.2.6

s/idenfied/identified/

- - Section 8.2

s/rendevous/rendezvous/

- - Section 8.5

s/conirm/confirm/

- -- 
Marc Petit-Huguenin
Personal email: [e-mail]
Professional email: [e-mail]
Blog: http://blog.marc.petit-huguenin.org
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (GNU/Linux)

iEYEARECAAYFAk62ze0ACgkQ9RoMZyVa61cC4ACdGKlVx8bm1sQr017foMaDhyze
Eg4AnR557TEUHu6CTi1qcyrRGnACiuHO
=cqrN
-----END PGP SIGNATURE-----