[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Working Group Last Call complete: draft-ietf-ccamp-pc-and-sc-reqs-02
Hi,
I have done a chair review on this draft and found some small editorial
updates that should be made before the I-D goes forward.
If the editors could resolve these and submit an update, we will be ready
for Deborah to take the I-D to the IESG.
Thanks,
Adrian
idnits
== It seems as if not all pages are separated by form feeds - found
1 form feeds but 12 pages
== Unused Reference: 'RFC 3473' is defined on line 406, but no explicit
reference was found in the text
===
General points
- I think you should decide whether management plane, control plane, and
data plane are capitalized or not, and apply this to the whole
document.
- Conventionally, references (such as [RFC1234]) do not include spaces
(i.e., not [RFC 1234])
===
1. Introduction
s/endpoints/end-points/
OLD
At least some control plane initiated aspects of a connection must be
capable of being queried by the management plane. These aspects
should be independent of how the connection was established.
NEW
At least some aspects of a control plane initiated connection must be
capable of being queried by the management plane. These aspects
should be independent of how the connection was established.
===
2. Motivation
s/or SC interoperation is achieved/or before SC interoperation is achieved/
s/is proposed as/is stated as/
s/is seen as a nice to have, or desirable, feature/is a desirable feature/
s/should be scoped as/is scoped as/
===
3. Label Switched Path Terminology
s/(such as Path and Resv state)/(such as RSVP-TE [RFC3473] Path and Resv
state)/
I think you should note that control plane LSPs may be visible and
controlable through the management plane. That is, the control plane
LSP may have been requested through a management plane operation, and
can be reported and inspected by/at the transit LSR. Further, it is
generally the case that management plane operations can be carried
out on the resources of active control plane LSPs even at transit
LSRs. You can then say that this is not the function that you are
talking about.
===
4. LSP within GMPLS Control Plane
Include a refernce to RFC 3945 as you are talking about GMPLS architecture.
===
4.1. Resource Ownership
At the end of the first paragraph, you have...
Note that the management plane assigns
resources to the control plane.
I think this is confusing in the context of the rest of the text in the
paragrpah that defines 'ownership' of resources in relation to the setup
of an LSP. Doesn't the sentence actually belong in the next paragraph?
I.e.,
Note that the management plane assigns
resources as available for ownership by the control plane.
Or even just delete this sentence.
===
4.2. Setting Up a GMPLS Controlled Network
You have...
When converting from an SC to a PC, the
management plane must change the owner of each hop. Somehow, then the
instance in the control plane must be removed without affecting the
data plane. This may best be done via a make before break operation.
This last sentence is odd! There is no "make-before-break operation" to
be performed as there is no change in the data plane. I suggest that
you simply remove this sentence which, in any case, is pre-judging the
solution (which this document should avoid doing).
===
5.1. PC to SC/SPC Conversion
s/Next step in such conversion/The next step in this conversion/
You have...
In this case a network upgrade by a Control Plane coverage extension
may be required.
Huh?
The phrase "Control Plane coverage extension" didn't mean anything to me
(although I can make some guesses). Can you clarify in the text, please.
===
6. Requirements
Suggest you add...
Notation from [RFC2119] is used to clarify the level of each
requirement.
===
6.2. No Disruption of User Traffic
SC to PC conversion and vice-versa shall occur without generating
s/shall/SHALL/ ?
===
6.5. Synchronization of state among nodes during conversion
Please capitalize the section heading
It MUST be assured that the state of the LSP is synchronized among
all nodes traversed by it before proceeding to the conversion.
Does "before proceeding to the conversion" mean "before the conversion
is considered complete"?