Hi WG,
Support both the documents, and are basically ready for publications.
I re-read the documents again, here are some nits/comments which can
be handled along the process.
--------
For draft-ietf-pce-stateful-pce-app: (listed as a contributor)
- In sec 3. Overview of the Stateful PCE Protocol Extensions; the word
tunnel is used in below paragraph, for consistency with rest of the
document can it be changed to LSP.
[I-D.ietf-pce-stateful-pce] specifies a set of extensions to PCEP to
enable stateful control of tunnels within and across PCEP sessions in
compliance with [RFC4657]. It includes mechanisms to effect tunnel
state synchronization between PCCs and PCEs, delegation of control
over tunnels to PCEs, and PCE control of timing and sequence of path
computations within and across PCEP sessions.
Since we are not using RFC2119 keywords in this document, we should
not use 'SHOULD'.
LSP state synchronization (C-E): after the session between a PCC and
a stateful PCE is initialized, the PCE can perform path
computation and update attributes in a PCC. However, if the goal
of the PCE is to provide accurate path information based on the
most up-to-date state of the network, the PCE SHOULD wait until it
learns the state of the PCC's LSP states before doing so.
- In sec 4.3. PCE Survivability; a reference to
draft-ietf-pce-stateful-sync-optimizations can be added for
readability.
- Expand on first use PCReq, LER, NMS, RRO, RSVP-TE
- Suggest if you can add IANA consideration section and explicitly say
that there are no request for IANA in this document.
- Some unused references like MXMN-TE, MPLS-PC, NET-REC (perhaps
because of older version text from draft-ietf-pce-stateful-pce)
--------
For draft-ietf-pce-stateful-pce:
- RFC2119 keywords requirement language should be added as subsection
in introduction.
1.1 Requirements Language
The key words "MUST", "MUST NOT", "REQUIRED", "SHALL", "SHALL NOT",
"SHOULD", "SHOULD NOT", "RECOMMENDED", "MAY", and "OPTIONAL" in this
document are to be interpreted as described in RFC 2119 [RFC2119].
- In sec 3.2. Objectives
OLD:
Allow a PCC to delegate control of its LSPs to an active stateful
PCE such that a single LSP is under the control a single PCE at
any given time.
NEW:
Allow a PCC to delegate control of its LSPs to an active stateful
PCE such that a LSP is under the control a single PCE at
any given time.
The current text gives an impression that only one LSP (a single LSP)
is under control of a PCE.
- In sec 5.4. State Synchronization, for figure 2 and 3 'PCErr=?' is
used, I suggest either to put the error type/code or remove '=?'.
- Update reference for minei-pce-stateful-sync-optimizations to
ietf-pce-stateful-sync-optimizations
- I am not sure what is the correct way to put comments inside RBNF,
or better yet they should simply be avoided. In section 6.3, 6.4 and
6.5 "<<<<" and "<---" are used.
- The alignment for the byte markers is incorrect in figure 10
- In sec 7.3.2. Symbolic Path Name TLV, can the following text be added?
The Symbolic Path Name is padded to 4-bytes alignment; padding
itself is not included in the Length field.
- In sec 7.3.3. LSP Error Code TLV, we should say - "The following LSP
Error Codes are *currently* defined:"; so that future documents can
easily extend it.
- In 9.6. Impact on Network Operation s/PcE/PCE
- ietf-pce-gmpls-pcep-extensions can be removed as a normative reference.
- there are bunch of idnits that you may wish to handle (see attached)
--------
Dhruv
On Sun, Sep 14, 2014 at 3:42 PM, JP Vasseur (jvasseur)
<[email protected]> wrote:
> Dear WG,
>
> Both draft-ietf-pce-stateful-pce-app-02 and draft-ietf-pce-stateful-pce-09
> are now stable ready for WG Last Call.
> This starts a 3-week WG Last call, which will end on Oct 6 at noon ET.
>
> Please send your comments on the mailing list.
>
> ps: this WG LC will be followed by another WG LC on
> draft-ietf-pce-stateful-sync-optimizations-01
> and draft-ietf-pce-pce-initiated-lsp-01.
>
>
> Thanks.
>
> JP and Julien.
>
> _______________________________________________
> Pce mailing list
> [email protected]
> https://www.ietf.org/mailman/listinfo/pce
>
<!DOCTYPE HTML PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"
"http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd";>
<html xmlns="http://www.w3.org/1999/xhtml";><!-- InstanceBegin
template="/Templates/GeneralPage.dwt" codeOutsideHTMLIsLocked="true" -->
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8" />
<!-- InstanceBeginEditable name="title" -->
<title>IETF IDNITS Tool</title>
<!-- InstanceEndEditable -->
<script type="text/javascript" language="JavaScript"
src="/css/ietf.js"></script>
<script type="text/javascript">
document.write('<link id="sheet4" href="/css/ietf4.css" rel="stylesheet"
type="text/css" />');
document.write('<link id="sheet3" href="/css/ietf3.css" rel="stylesheet"
type="text/css" />');
document.write('<link id="sheet2" href="/css/ietf2.css" rel="stylesheet"
type="text/css" />');
</script>
<link id="sheet1" href="/css/ietf.css" rel="stylesheet" type="text/css" />
</head>
<body onload="setStyle(0);">
<div id="container">
<table border="0" cellpadding="0" cellspacing="0">
<tbody>
<tr>
<td id="sidebar1"><div id="sidebar2"><a href="/"><img
src="/images/ietflogotrans.gif" alt="Logo" height="75" width="140"
border="0"/></a>
<div style="margin:0;padding:0;white-space: nowrap;">
<form method="get" action="http://www.google.com/u/ietf";
style="margin:0;padding:0;white-space: nowrap;">
<p style="margin:0;padding:0;white-space: nowrap;">
<input name="q" size="16" maxlength="255" value=""
type="text" style="margin:1px;padding:0;width:100px;border:1px solid #89d;" />
<input name="Search" value="Search" type="submit"
style="margin:0;padding:0;font-size:85%;" />
</p>
</form>
</div>
<ul>
<li class="subhdr00"><a class="subhdr00"
href="/"><strong>Home</strong></a></li>
<li class="subhdr00"><a class="subhdr00"
href="/about/"><strong>About the IETF</strong></a></li>
<li class="subhdr02"><a class="subhdr02"
href="/about/mission.html">Mission</a></li>
<li class="subhdr02"><a class="subhdr02"
href="/about/standards-process.html">Standards Process</a></li>
<li class="subhdr02"><a class="subhdr02"
href="/about/note-well.html">Note Well</a></li>
<li class="subhdr02"><a class="subhdr02"
href="/nomcom/">NomCom</a></li>
<li class="subhdr02"><a class="subhdr02"
href="/newcomers.html">Info for Newcomers</a></li>
<li class="subhdr00"><a class="subhdr00"
href="/id-info/"><strong>Internet-Drafts</strong></a></li>
<li class="subhdr02"><a class="subhdr02"
href="https://datatracker.ietf.org/";>Datatracker</a></li>
<li class="subhdr02"><a class="subhdr02"
href="https://datatracker.ietf.org/drafts/";>Search</a></li>
<li class="subhdr02"><a class="subhdr02"
href="https://datatracker.ietf.org/idst/upload.cgi";>Submit</a></li>
<li class="subhdr00"><a class="subhdr00"
href="/rfc.html"><strong>RFC Pages</strong></a></li>
<li class="subhdr02"><a class="subhdr02"
href="http://rfc-editor.org/rfcsearch.html";>Search RFC Ed Index</a></li>
<li class="subhdr02"><a class="subhdr02"
href="http://www.rfc-editor.org/queue2.html";>RFC Editor Queue</a></li>
<li class="subhdr00"><a class="subhdr00"
href="/iana.html"><strong>IANA Pages</strong></a></li>
<li class="subhdr02"><a class="subhdr02"
href="/assignments/">Protocol Parameters</a></li>
<li class="subhdr00"><a class="subhdr00"
href="/wg/"><strong>Working Groups</strong></a></li>
<li class="subhdr02"><a class="subhdr02"
href="/dyn/wg/charter.html">WG Charters</a></li>
<li class="subhdr02"><a class="subhdr02"
href="http://datatracker.ietf.org/list/wg/";>Email Lists</a></li>
<li class="subhdr02"><a class="subhdr02"
href="/wg/chairs-page.html">WG Chairs' Page</a></li>
<li class="subhdr00"><a class="subhdr00"
href="/tools/"><strong>Resources</strong></a></li>
<li class="subhdr02"><a class="subhdr02"
href="/tools/tools.html">Community Tools</a></li>
<li class="subhdr02"><a class="subhdr02"
href="http://tools.ietf.org/";>Tools Team Pages</a></li>
<li class="subhdr02"><a class="subhdr02"
href="http://wiki.tools.ietf.org/group/";>Wikis</a></li>
<li class="subhdr00"><a class="subhdr00"
href="/meeting/"><strong>Meetings</strong></a></li>
<li class="subhdr02"><a class="subhdr02"
href="/meeting/upcoming.html">Upcoming Meetings</a></li>
<li class="subhdr02"><a class="subhdr02"
href="/meeting/past.html">Past Meetings</a></li>
<li class="subhdr02"><a class="subhdr02"
href="/meeting/cutoff-dates.html">Important Dates</a></li>
<li class="subhdr02"><a class="subhdr02"
href="/meeting/proceedings.html">Proceedings</a></li>
<li class="subhdr00"><a class="subhdr00"
href="/list/"><strong>Mailing Lists</strong></a></li>
<li class="subhdr02"><a class="subhdr02"
href="/list/announcement.html">Announcement Lists</a></li>
<li class="subhdr02"><a class="subhdr02"
href="/list/discussion.html">Discussion Lists</a></li>
<li class="subhdr02"><a class="subhdr02"
href="/list/nonwg.html">Non-WG Lists</a></li>
<li class="subhdr00"><a class="subhdr00"
href="/iesg/"><strong>IESG </strong></a></li>
<li class="subhdr02"><a class="subhdr02"
href="https://datatracker.ietf.org/iesg/ann/new/";>Announcements</a></li>
<li class="subhdr02"><a class="subhdr02"
href="/iesg/statement.html">Statements</a></li>
<li class="subhdr02"><a class="subhdr02"
href="/iesg/members.html">Members</a></li>
<li class="subhdr02"><a class="subhdr02"
href="/iesg/minutes.html">Minutes</a></li>
<li class="subhdr00"><a class="subhdr00"
href="/ipr/"><strong>IPR</strong></a></li>
<li class="subhdr02"><a class="subhdr02"
href="/ipr/policy.html">IPR Policy</a></li>
<li class="subhdr02"><a class="subhdr02"
href="/ipr/file-disclosure">File a Disclosure</a></li>
<li class="subhdr02"><a class="subhdr02"
href="https://datatracker.ietf.org/ipr/search/";>Disclosure Search</a></li>
<li class="subhdr00"><a class="subhdr00"
href="/liaison/"><strong>Liaisons</strong></a></li>
<li class="subhdr02"><a class="subhdr02"
href="https://datatracker.ietf.org/liaison/";>Liaison Statements</a></li>
<li class="subhdr02"><a class="subhdr02"
href="/liaison/managers.html">Liaison Managers</a></li>
<li class="subhdr00"><a class="subhdr00"
href="/contact-the-ietf.html"><strong>Contact</strong></a></li>
<li class="subhdr02"><a class="subhdr02"
href="/secretariat.html">IETF Secretariat</a><sup>®</sup></li>
<li class="subhdr02"><a class="subhdr02"
href="http://iaoc.ietf.org/subpoena.html";>Legal Matters</a></li>
<li class="subhdr02"><a class="subhdr02"
href="mailto:[email protected]";>Report Web Site Errors</a></li>
<li class="subhdr00"><strong>Customize View</strong><br />
</li>
<li class="subhdr01"><a class="subhdr01"
href="javascript:setStyle('2');" title="Brief">Brief</a></li>
<li class="subhdr01"><a class="subhdr01"
href="javascript:setStyle('1');" title="Normal">Normal</a></li>
<li class="subhdr01"><a class="subhdr01"
href="javascript:setStyle('3');" title="Extended">Extended</a></li>
<li class="subhdr01"><a class="subhdr01"
href="javascript:setStyle('4');" title="Advanced">Advanced</a></li>
<li class="subhdr01"><small>(Requires Javascript)</small></li>
</ul>
</div></td>
<td id="content1"><div id="content2"> <!-- InstanceBeginEditable
name="body" -->
<p class="ptitle">IDNITS</p>
<p>
<form action="/tools/idnits" method="post"
enctype="multipart/form-data">
Check a draft for nits:<br />
<input type="checkbox" name="verbose" /> verbose
output<br />
<input type="checkbox" name="vverbose" /> very
verbose output<br />
<input type="checkbox" name="submitcheck" /> only
do submission checks<br />
<input type="text" size=4 name="year" /> copyright
year (if different from current)<br />
<!-- <input type="file" name="file1" size="40" /> -->
<input type="file" name="filename" size="40" />
<input type="submit" name="submit" value="Check file" />
</form>
</p>
<!-- InstanceEndEditable --> </div></td>
</tr>
<tr>
<td id="footer1" colspan="2"><div id="footer2">
<p><a href="http://www.isoc.org/";><img src="/images/isoc_logo.gif"
alt="Internet Society" align="left" border="0" hspace="5" /></a><a
href="http://www.amsl.com/";><img src="/images/ams_logo.png" alt="AMS"
align="right" border="0" hspace="5" /></a><a href="/">Home</a> - <a
href="http://tools.ietf.org/";>Tools</a> - <a
href="https://datatracker.ietf.org/";>Datatracker</a> - <a
href="/usagedata/">Web Site Usage Statistics</a> - <a
href="http://iaoc.ietf.org/";>IASA</a> - <a href="http://www.iab.org/";>IAB</a> -
<a href="http://www.rfc-editor.org/";>RFC Editor</a> - <a
href="http://www.iana.org/";>IANA</a> - <a href="http://www.irtf.org/";>IRTF</a>
- <a href="http://trustee.ietf.org/";>IETF Trust</a> - <a
href="http://www.isoc.org/";>ISOC</a> - <a
href="/secretariat.html">Contact Us</a><br />
Secretariat services provided by <a
href="http://www.amsl.com/";>Association Management Solutions, LLC (AMS)</a>.<br
/>
Please send problem reports to: <a
href="mailto:[email protected]";>[email protected]</a>.<br />
</p>
</div></td>
</tr>
</tbody>
</table>
</div>
</body>
<!-- InstanceEnd --></html>
_______________________________________________
Pce mailing list
[email protected]
https://www.ietf.org/mailman/listinfo/pce
