Re: [sidr] I-D Action: draft-ietf-sidr-pfx-validate-04.txt

"George, Wes" <wesley.george@twcable.com> Wed, 14 March 2012 15:57 UTC

Return-Path: <wesley.george@twcable.com>
X-Original-To: sidr@ietfa.amsl.com
Delivered-To: sidr@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 15B7C21F85D6; Wed, 14 Mar 2012 08:57:44 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.041
X-Spam-Level:
X-Spam-Status: No, score=-1.041 tagged_above=-999 required=5 tests=[AWL=0.422, BAYES_00=-2.599, HELO_EQ_MODEMCABLE=0.768, HOST_EQ_MODEMCABLE=1.368, RCVD_IN_DNSWL_LOW=-1]
Received: from mail.ietf.org ([12.22.58.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id LUBbXJguTPan; Wed, 14 Mar 2012 08:57:43 -0700 (PDT)
Received: from cdpipgw01.twcable.com (cdpipgw01.twcable.com [165.237.59.22]) by ietfa.amsl.com (Postfix) with ESMTP id 14AEA21F85CC; Wed, 14 Mar 2012 08:57:42 -0700 (PDT)
X-SENDER-IP: 10.136.163.12
X-SENDER-REPUTATION: None
X-IronPort-AV: E=Sophos;i="4.73,584,1325480400"; d="scan'208";a="353624045"
Received: from unknown (HELO PRVPEXHUB03.corp.twcable.com) ([10.136.163.12]) by cdpipgw01.twcable.com with ESMTP/TLS/RC4-MD5; 14 Mar 2012 11:56:37 -0400
Received: from PRVPEXVS03.corp.twcable.com ([10.136.163.26]) by PRVPEXHUB03.corp.twcable.com ([10.136.163.12]) with mapi; Wed, 14 Mar 2012 11:57:09 -0400
From: "George, Wes" <wesley.george@twcable.com>
To: Pradosh Mohapatra <pmohapat@cisco.com>
Date: Wed, 14 Mar 2012 11:57:06 -0400
Thread-Topic: [sidr] I-D Action: draft-ietf-sidr-pfx-validate-04.txt
Thread-Index: Ac0BVNUs+c6Dnf/3TK2U3kLseh4cmAAlVvvA
Message-ID: <DCC302FAA9FE5F4BBA4DCAD465693779173BA12B5C@PRVPEXVS03.corp.twcable.com>
References: <20120312232702.17194.34028.idtracker@ietfa.amsl.com> <DCC302FAA9FE5F4BBA4DCAD465693779173B937614@PRVPEXVS03.corp.twcable.com> <C030F426-6133-415A-9AB5-847081C62904@cisco.com>
In-Reply-To: <C030F426-6133-415A-9AB5-847081C62904@cisco.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
acceptlanguage: en-US
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
Cc: "sidr@ietf.org" <sidr@ietf.org>, "internet-drafts@ietf.org" <internet-drafts@ietf.org>, "i-d-announce@ietf.org" <i-d-announce@ietf.org>
Subject: Re: [sidr] I-D Action: draft-ietf-sidr-pfx-validate-04.txt
X-BeenThere: sidr@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: Secure Interdomain Routing <sidr.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/sidr>, <mailto:sidr-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/sidr>
List-Post: <mailto:sidr@ietf.org>
List-Help: <mailto:sidr-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/sidr>, <mailto:sidr-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 14 Mar 2012 15:57:44 -0000

> -----Original Message-----
> From: Pradosh Mohapatra [mailto:pmohapat@cisco.com]
> Sent: Tuesday, March 13, 2012 4:07 PM
> To: George, Wes
> Cc: internet-drafts@ietf.org; i-d-announce@ietf.org; sidr@ietf.org
> Subject: Re: [sidr] I-D Action: draft-ietf-sidr-pfx-validate-04.txt
>
> > In section 2:
> > "No ROA can match an origin
> >      AS number of "NONE".  No Route can match a ROA whose origin AS
> >      number is zero."
> >
> > I'm wondering if there should be a 2119 normative or two in there?
> > This sounds like a validation instruction. (eg MUST/SHOULD declare
> > prefixes covered by an origin AS number of none/zero invalid)
>
>
> Could you suggest text with 2119 language?

[WEG] Originally I stopped short of fully suggesting text because I didn't think that I had a complete grasp of what the authors are suggesting should happen here based on the combination of the text above.
In rereading the surrounding text to make another attempt at it, I don't think that this sentence belongs in the definition for Route Origin ASN at all, because it's not really part of the definition. This is instructional about a special case of match/cover, and should probably be moved down a few sentences to where you talk about valid/invalid/unknown. The same is also true for the following from the definition of Matched.
        "keeping in mind that a ROA ASN of zero can never be matched, nor can a route origin AS
      number of "NONE"."

So I would strike the references to ASN 0 and origin AS NONE from the definitions altogether, and then reword the next section as follows:
CURRENT TEXT

" Given these definitions, any given BGP Route will be found to have
   one of the following "validation states":

   o  NotFound: No ROA Covers the Route Prefix.

   o  Valid: At least one ROA Matches the Route Prefix.

   o  Invalid: At least one ROA Covers the Route Prefix, but no ROA
      Matches it."

NEW TEXT

"Given these definitions, any given BGP route MUST [SHOULD?] be found to have one of the following "validation states":
   o  NotFound: No ROA Covers the Route Prefix.

   o  Valid: At least one ROA Matches the Route Prefix.

   o  Invalid: At least one ROA Covers the Route Prefix, but no ROA
      Matches it.
It should be noted that a ROA ASN of zero or a route origin AS number of "NONE" MUST NOT ever be considered matches. This means that routes with a covering ROA ASN of zero MUST be declared Invalid, and routes with a route origin AS number of "NONE" and one or more covering ROAs MUST be declared Invalid."

Is that a reasonably accurate interpretation of the intent?

>
> > Lastly:
> > "We observe that a Route can be Matched or Covered by more than one
> >   ROA.  This procedure does not mandate an order in which ROAs must be
> >   visited; however, the "validation state" output is fully
> > determined."
> > Is there guidance on this in one of the other documents? If so,
> > please reference it here. Does longest-match still apply? This seems
> > a fairly big question to simply leave open to implementation.
> > Please apply cluebrick liberally if I'm being thick.
>
>
> I looked around in sidr-usecases and origin-ops, but couldn't find an
> example. May be we should add one. But is there anything that you are
> specifically worried about? All that the text says is that ordering is
> not relevant. It's a classic OR operation for the match.

[WEG] I didn't get "ordering not relevant" from the current text, but now that you say it, I see how it could be interpreted that way. See my suggested change as a reply to Randy's explanation.

Thanks
Wes George

This E-mail and any of its attachments may contain Time Warner Cable proprietary information, which is privileged, confidential, or subject to copyright belonging to Time Warner Cable. This E-mail is intended solely for the use of the individual or entity to which it is addressed. If you are not the intended recipient of this E-mail, you are hereby notified that any dissemination, distribution, copying, or action taken in relation to the contents of and attachments to this E-mail is strictly prohibited and may be unlawful. If you have received this E-mail in error, please notify the sender immediately and permanently delete the original and any copy of this E-mail and any printout.