[CCAMP] Comments on draft-ietf-ccamp-rsvp-te-srlg-collect-07

Lou Berger <lberger@labn.net> Wed, 22 October 2014 22:06 UTC

Return-Path: <lberger@labn.net>
X-Original-To: ccamp@ietfa.amsl.com
Delivered-To: ccamp@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id B4E861A7031 for <ccamp@ietfa.amsl.com>; Wed, 22 Oct 2014 15:06:40 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: 0.232
X-Spam-Level:
X-Spam-Status: No, score=0.232 tagged_above=-999 required=5 tests=[BAYES_40=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, IP_NOT_FRIENDLY=0.334, RCVD_IN_DNSWL_NONE=-0.0001, SPF_PASS=-0.001] autolearn=no
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id aW2bRxCK8oxP for <ccamp@ietfa.amsl.com>; Wed, 22 Oct 2014 15:06:39 -0700 (PDT)
Received: from gproxy6-pub.mail.unifiedlayer.com (gproxy6-pub.mail.unifiedlayer.com [67.222.39.168]) by ietfa.amsl.com (Postfix) with SMTP id B745E1AD0C3 for <ccamp@ietf.org>; Wed, 22 Oct 2014 15:06:39 -0700 (PDT)
Received: (qmail 19842 invoked by uid 0); 22 Oct 2014 22:06:37 -0000
Received: from unknown (HELO CMOut01) (10.0.90.82) by gproxy6.mail.unifiedlayer.com with SMTP; 22 Oct 2014 22:06:37 -0000
Received: from box313.bluehost.com ([69.89.31.113]) by CMOut01 with id 6N6S1p0052SSUrH01N6Vqf; Wed, 22 Oct 2014 16:06:36 -0600
X-Authority-Analysis: v=2.1 cv=Tr912lnh c=1 sm=1 tr=0 a=h1BC+oY+fLhyFmnTBx92Jg==:117 a=u9EReRu7m0cA:10 a=HFCU6gKsb0MA:10 a=MGRxyiorQo4A:10 a=8nJEP1OIZ-IA:10 a=wU2YTnxGAAAA:8 a=cNaOj0WVAAAA:8 a=-NfooI8aBGcA:10 a=uEJ9t1CZtbIA:10 a=_X-uKZgBjGTumMbn1toA:9 a=wPNLvfGTeEIA:10
DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=labn.net; s=default; h=Content-Transfer-Encoding:Content-Type:In-Reply-To:References:Subject:CC:To:MIME-Version:From:Date:Message-ID; bh=rrZGSdOwft6tlHiyc3xyfrIWpaeytc/+rG+DXTDoJ4Y=; b=IjWrjEbN/IYIWsQbIfqqvMJi9kSt9cTKvsYarNRdoYmG+aQP+KEffQLfjIfr/Gj9nntYQtQOubETgs9KgdviM2BtdFsz1BplvlQGkGjvVcLH2oZpJD42qvmpgVretr/l;
Received: from box313.bluehost.com ([69.89.31.113]:34906 helo=[127.0.0.1]) by box313.bluehost.com with esmtpa (Exim 4.82) (envelope-from <lberger@labn.net>) id 1Xh430-0005Lk-Vb; Wed, 22 Oct 2014 16:06:27 -0600
Message-ID: <54482AAA.2040602@labn.net>
Date: Wed, 22 Oct 2014 18:07:38 -0400
From: Lou Berger <lberger@labn.net>
User-Agent: Mozilla/5.0 (Windows NT 6.3; WOW64; rv:24.0) Gecko/20100101 Thunderbird/24.6.0
MIME-Version: 1.0
To: "draft-ietf-ccamp-rsvp-te-srlg-collect@tools.ietf.org" <draft-ietf-ccamp-rsvp-te-srlg-collect@tools.ietf.org>
References: <536AAB57.2090403@labn.net> <CFEED041.5B5DC%oscar.gonzalezdedios@telefonica.com>
In-Reply-To: <CFEED041.5B5DC%oscar.gonzalezdedios@telefonica.com>
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 8bit
X-Identified-User: {1038:box313.bluehost.com:labnmobi:labn.net} {sentby:smtp auth 69.89.31.113 authed with lberger@labn.net}
Archived-At: http://mailarchive.ietf.org/arch/msg/ccamp/do896Ffc6auLtc3b8WILd-9fhJ0
Cc: CCAMP <ccamp@ietf.org>
Subject: [CCAMP] Comments on draft-ietf-ccamp-rsvp-te-srlg-collect-07
X-BeenThere: ccamp@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: Discussion list for the CCAMP working group <ccamp.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ccamp>, <mailto:ccamp-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/ccamp/>
List-Post: <mailto:ccamp@ietf.org>
List-Help: <mailto:ccamp-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ccamp>, <mailto:ccamp-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 22 Oct 2014 22:06:40 -0000

Authors,
	I think we still have some unresolved comments from May:

>> - you use "should not" in lower case in a few spots in this section.
>>  While I think your usage *is* correct, my experience is that someone
>>  (probably in the IESG) will tell you that these need to be in upper
>>  case at some point.  Of course, they'll be wrong, and this will have
>>  to be explained.  I suggest avoiding 2119 language in lower case where
>>  easily avoided.  How about s/should not be/is not to be

The above comment also applies to "must".

Your current of 2119 language is a bit inconsistent. I think you should
review current uses of 2119 language and ensure that such usage is
limited to (protocol) mechanisms, behavior and interoperability.  If
doesn't the language is most likely informative in nature and should
avoid 2119 conformance language.

>> - The section is missing handling of RRO to big. Perhaps add it at ~line
>>  330.
> 

A few of new comments:

- The current text of section 4.1 could be read as only one ID may be
present in the SO.  you should explain under what conditions multiple
IDs are to be added.

-  Please provide a reference for "Confidential Path message. Segment (CPS)"

- (should have caught this one before, i.e., is in old text) In Section
5.1 it looks like you are applying policy on both Path and Resv
processing ("When a node receives a Resv message ... if local policy
determines ...). Given the "SRLG Recording Rejected" PathErr required
earlier in the section, is this really needed?  Are you assuming
separate upstream/downstream policies?  (Which would seem to be
overkill.) Covering a policy change race condition, something else? I
suspect the policy processing text on resv isn't needed.

Thanks,
Lou