Telechat Review of draft-ietf-pwe3-static-pw-status-
review-ietf-pwe3-static-pw-status-genart-telechat-campbell-2012-01-12-00
Request | Review of | draft-ietf-pwe3-static-pw-status |
---|---|---|
Requested revision | No specific revision (document currently at 10) | |
Type | Telechat Review | |
Team | General Area Review Team (Gen-ART) (genart) | |
Deadline | 2011-11-01 | |
Requested | 2012-01-12 | |
Authors | Luca Martini , George Swallow , Giles Heron , Matthew Bocci | |
I-D last updated | 2012-01-12 | |
Completed reviews |
Genart Telechat review of -??
by Ben Campbell
Genart Telechat review of -?? by Ben Campbell |
|
Assignment | Reviewer | Ben Campbell |
State | Completed | |
Request | Telechat review on draft-ietf-pwe3-static-pw-status by General Area Review Team (Gen-ART) Assigned | |
Completed | 2012-01-12 |
review-ietf-pwe3-static-pw-status-genart-telechat-campbell-2012-01-12-00
I am the assigned Gen-ART reviewer for this draft. For background on Gen-ART, please see the FAQ at <http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>. Please resolve these comments along with any other Last Call comments you may receive. Document: draft-ietf-pwe3-static-pw-status-08 Reviewer: Ben Campbell Review Date: 2011-10-04 IETF LC End Date: 2011-10-05 Summary: The draft is almost ready for publication as a proposed standard, but there are a few minor issues and editorial issues that need addressing first. Major issues: None Minor issues: -- 5.3: Has the work group considered how the retransmit scheme and 30 second refresh default will scale to very large deployments? Seems like this could result in a lot of retransmissions. -- 5.3, last paragraph: " This will cause the PE sending the all clear message to stop sending, and to continue normal operation." Where is that specified? Can you offer a reference? -- 5.3.1, 1st paragraph: "The receiving PE will then set its timeout timer according to the timer value that is in the packet received, regardless of what timer value it sent." It's probably worth making a normative statement here, since it seems like this could easily result in an argument if the PEs disagree on the timer value. -- 5.3.1, 2nd paragraph: Please elaborate on "match exactly". What uniquely matches a message to an ack? How do you compare them? "… the sender PE MUST use the new timer value received." Doesn't this contradict the previous paragraph that lets the sender disagree? Is the subsequent treated different than the first attempt? (If so, is there a state machine that could be elaborated on?) -- 5.3.2: I don't understand the guidance here. Are you saying a PE should send status even when it can't? -- 5.5, 1st paragraph: "… MUST be supported by both T-PEs to be enabled." How does one T-PE know another supports this? -- 5.5, last paragraph: This could use some elaboration. What is the purpose of having to send both ways? Nits/editorial comments: -- general: IDNits returns some comments. I think something about the header format is confusing it. -- abstract: Please expand BFD on first mention -- Section 2: Please expand LDP and PE on first mention. (even though they are in the terminology section, since they are mentioned here before that section.) -- section 2: "… without control plane." Missing article ("a" or "the") -- section 4: Please expand PSN, MPLS-TP, and BFD on first mention. -- section 5.1, 2nd to last paragraph: "...PW OAM Message code" Missing "the" -- section 5.1, last paragraph: This seems redundant with the IANA considerations section. -- 5.2, 1st paragraph: "PW Status messages are indicated…" Indicated by what? (passive voice obscures responsible actor). -- same paragraph: "PW Status TLV defined in [RFC4447]. The PW Status TLV format is as follows:" I'm confused is defined in 4447 and just repeated here, or defined here? If the first, please be very clear about that, so there's no question about where the normative definition lives. For example: "PW Status TLV defined in [RFC4447], and is repeated here for the reader's convenience:" -- 5.2, last paragraph: "PW OAM Status Messages MUST NOT be used as a connectivity verification method." That sounds like it belongs in the "applicability" section. -- 5.3, 2nd paragraph: "...will be transmitted immediately." Transmitted by what? (passive voice obscures responsible actor). -- 5.3.1, 1st paragraph: "The timer value set in the reply packet SHOULD then be used as the new transmit interval." By what? (passive voice obscures responsible actor). -- 5.3.1, last paragraph: "standard procedures" Reference? -- 5.5.1, last paragraph: "If Bit "B" is set , then the T-PE can receive S-PE bypass status messages in the G-ACH. If Bit "B" is not set the T-PE MUST NOT send S-PE bypass status messages in the G-ACH." I think the sender and receiver are mixed up here. The text seems to say if the bit is set you may receive but if the bit is not set you must not send? -- 8 : IANA Considerations It would help to include the formal definition tables here, or reference them here. Also, can you include the URLs for each registry? 0x0022 appears to be already assigned to MPLS-TP CC message. "Pseudowire Switching Point PE TLV Type"" I don't find that registry. Did you mean sub-type? 0x16 appears to be already assigned to "Stack capability"