Re: [Softwires] intdir Telechat Review requested: draft-ietf-softwire-map-radius

"Bernie Volz (volz)" <volz@cisco.com> Tue, 14 May 2019 01:21 UTC

Return-Path: <volz@cisco.com>
X-Original-To: softwires@ietfa.amsl.com
Delivered-To: softwires@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id E39B11200FF; Mon, 13 May 2019 18:21:03 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -14.5
X-Spam-Level:
X-Spam-Status: No, score=-14.5 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, RCVD_IN_DNSWL_HI=-5, SPF_PASS=-0.001, URIBL_BLOCKED=0.001, USER_IN_DEF_DKIM_WL=-7.5] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=cisco.com header.b=GZjxjJQ/; dkim=pass (1024-bit key) header.d=cisco.onmicrosoft.com header.b=XVezjp8X
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 0LQLIN-Oj7Uc; Mon, 13 May 2019 18:21:01 -0700 (PDT)
Received: from alln-iport-8.cisco.com (alln-iport-8.cisco.com [173.37.142.95]) (using TLSv1.2 with cipher DHE-RSA-SEED-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 1C55F12008C; Mon, 13 May 2019 18:21:01 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=5236; q=dns/txt; s=iport; t=1557796861; x=1559006461; h=from:to:cc:subject:date:message-id:references: in-reply-to:content-transfer-encoding:mime-version; bh=f+Wg35CzPjI4sPZe7fMLbzngU0/0qEfUmX3LqEOWahs=; b=GZjxjJQ/3nBNagoJ5O5i1gEaqHpS3Y0WkRjVl/UPwqKK473/x8fqrNX3 GWv/cbqQsIH1Yc8Y47Xk5KiHk0CGLU0mUI0KaAujQxugW5JTmb4yLv6Mw zS4l2+HpswGP7ZjGHZZaZ75DzK6tlU26msIQjcc5CJLW4crwmqAgAQyPL w=;
IronPort-PHdr: =?us-ascii?q?9a23=3AERMPShzh2xYqHmTXCy+N+z0EezQntrPoPwUc9p?= =?us-ascii?q?sgjfdUf7+++4j5YRGN/u1j2VnOW4iTq+lJjebbqejBYSQB+t7A+GsHbIQKUh?= =?us-ascii?q?YEjcsMmAl1HsmBG2XwLeXhaGoxG8ERHFI=3D?=
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-Anti-Spam-Result: =?us-ascii?q?A0BIAAAHF9pc/5NdJa1kHAEBAQQBAQc?= =?us-ascii?q?EAQGBUQcBAQsBgT0pJwNpVSAECygKhAeDRwOEUoosgleXJYEugSQDVAkBAQE?= =?us-ascii?q?MAQElCAIBAYRAAheBfCM0CQ4BAwEBBAEBAgEEbRwMhUoBAQEBAxIREQwBATc?= =?us-ascii?q?BCwQCAQgRBAEBAwImAgICMBUICAIEAQ0FCBqDAYFqAx0BAgyhZgKBNYhfcYE?= =?us-ascii?q?vgnkBAQWBMgETQYMDGIIPAwaBCygBi04XgX+BV4JMPoEEAYFcAQEDAYFfgwg?= =?us-ascii?q?ygiaLEYJNmRZlCQKCCYYgiAdjg22VbIwyhlaOMAIEAgQFAg4BAQWBTziBV3A?= =?us-ascii?q?VGoJZAQEBMYEXeIEkAQmCQYUUhT9yCYEgjV4BgSABAQ?=
X-IronPort-AV: E=Sophos;i="5.60,466,1549929600"; d="scan'208";a="272331730"
Received: from rcdn-core-11.cisco.com ([173.37.93.147]) by alln-iport-8.cisco.com with ESMTP/TLS/DHE-RSA-SEED-SHA; 14 May 2019 01:21:00 +0000
Received: from XCH-ALN-018.cisco.com (xch-aln-018.cisco.com [173.36.7.28]) by rcdn-core-11.cisco.com (8.15.2/8.15.2) with ESMTPS id x4E1L0NT021300 (version=TLSv1.2 cipher=AES256-SHA bits=256 verify=FAIL); Tue, 14 May 2019 01:21:00 GMT
Received: from xhs-aln-002.cisco.com (173.37.135.119) by XCH-ALN-018.cisco.com (173.36.7.28) with Microsoft SMTP Server (TLS) id 15.0.1473.3; Mon, 13 May 2019 20:20:59 -0500
Received: from xhs-aln-001.cisco.com (173.37.135.118) by xhs-aln-002.cisco.com (173.37.135.119) with Microsoft SMTP Server (TLS) id 15.0.1473.3; Mon, 13 May 2019 20:20:58 -0500
Received: from NAM02-SN1-obe.outbound.protection.outlook.com (173.37.151.57) by xhs-aln-001.cisco.com (173.37.135.118) with Microsoft SMTP Server (TLS) id 15.0.1473.3 via Frontend Transport; Mon, 13 May 2019 20:20:58 -0500
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cisco.onmicrosoft.com; s=selector1-cisco-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=f+Wg35CzPjI4sPZe7fMLbzngU0/0qEfUmX3LqEOWahs=; b=XVezjp8XDeHXOMNqQt48YJmJ22WJIeCf5S+1MSpWct6U4nPxJ7ywRGV0O7optEh5RSJRIq2upjRKOQDycE212JmzzfwmKznytLDQXbhBaAmCMzT0kwrcdIHE5x5uq02/laBPJgYwDgKm1Uguv87jbr53Nmul66+G9HyH6/dqaUk=
Received: from BN8PR11MB3601.namprd11.prod.outlook.com (20.178.219.23) by BN8PR11MB3620.namprd11.prod.outlook.com (20.178.219.87) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.1878.25; Tue, 14 May 2019 01:20:57 +0000
Received: from BN8PR11MB3601.namprd11.prod.outlook.com ([fe80::5dc0:cbba:a126:65bf]) by BN8PR11MB3601.namprd11.prod.outlook.com ([fe80::5dc0:cbba:a126:65bf%6]) with mapi id 15.20.1878.024; Tue, 14 May 2019 01:20:57 +0000
From: "Bernie Volz (volz)" <volz@cisco.com>
To: "int-ads@ietf.org" <int-ads@ietf.org>, "int-dir@ietf.org" <int-dir@ietf.org>, "draft-ietf-softwire-map-radius@ietf.org" <draft-ietf-softwire-map-radius@ietf.org>
CC: "softwires@ietf.org" <softwires@ietf.org>
Thread-Topic: intdir Telechat Review requested: draft-ietf-softwire-map-radius
Thread-Index: AQHU+fjZtWwHlo4VX06MHb4N50Y5/qZp6kAA
Date: Tue, 14 May 2019 01:20:57 +0000
Message-ID: <BN8PR11MB3601B9BC50D9FC2A2E88065ECF080@BN8PR11MB3601.namprd11.prod.outlook.com>
References: <155604000110.32429.11279582703707003716.idtracker@ietfa.amsl.com>
In-Reply-To: <155604000110.32429.11279582703707003716.idtracker@ietfa.amsl.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
authentication-results: spf=none (sender IP is ) smtp.mailfrom=volz@cisco.com;
x-originating-ip: [173.38.117.73]
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: 02f71cf3-ed55-4789-1390-08d6d80a6ae7
x-microsoft-antispam: BCL:0; PCL:0; RULEID:(2390118)(7020095)(4652040)(8989299)(4534185)(4627221)(201703031133081)(201702281549075)(8990200)(5600141)(711020)(4605104)(2017052603328)(7193020); SRVR:BN8PR11MB3620;
x-ms-traffictypediagnostic: BN8PR11MB3620:
x-ms-exchange-purlcount: 2
x-microsoft-antispam-prvs: <BN8PR11MB36202B346D6EE097FF5881F6CF080@BN8PR11MB3620.namprd11.prod.outlook.com>
x-ms-oob-tlc-oobclassifiers: OLM:5236;
x-forefront-prvs: 0037FD6480
x-forefront-antispam-report: SFV:NSPM; SFS:(10009020)(376002)(136003)(39860400002)(396003)(346002)(366004)(199004)(189003)(13464003)(4326008)(26005)(229853002)(6246003)(450100002)(86362001)(66066001)(316002)(2201001)(6116002)(5660300002)(478600001)(52536014)(305945005)(53936002)(73956011)(2906002)(66946007)(446003)(7696005)(66574012)(6506007)(110136005)(99286004)(76116006)(66446008)(66556008)(64756008)(186003)(66476007)(11346002)(6436002)(486006)(2501003)(966005)(81156014)(14454004)(6306002)(71200400001)(71190400001)(9686003)(476003)(102836004)(25786009)(68736007)(3846002)(7736002)(74316002)(55016002)(33656002)(14444005)(256004)(8936002)(8676002)(53546011)(81166006)(76176011); DIR:OUT; SFP:1101; SCL:1; SRVR:BN8PR11MB3620; H:BN8PR11MB3601.namprd11.prod.outlook.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; A:1; MX:1;
received-spf: None (protection.outlook.com: cisco.com does not designate permitted sender hosts)
x-ms-exchange-senderadcheck: 1
x-microsoft-antispam-message-info: RcQRLLri5R2JMPDCmo7/F8F1iw+s9r59mAFQWo2H24LXpEN7916m9xO+6L8XchwKPnaqV6mabBylWUTAzXzS9ySuVta1ji9YoLAzIHWq0WPhAz+2nmNw0SpyxMCh9jUz/kes632GaS4789J8zGyJZSF3DQ9ewyiN3zZIzKoeitq8BASLuiSTvecYcazX39uxk9DbTYE4e/ZcFRr+iBogO8ldyPXqYpdAzxKsUo8yAEn4ER51Yufu61msVv0tEknEogrBNkAL9061VLgwkJ1Ci9BGfMTBB39eaZvsZJbTAfpaY57Fit6mSA4bdcjzYwBU6fz23t37sOHUdRhTR1XxFSOJmo7ApeSpmloNWBLbGx/wxwmuh6aTamP9PTt+6iy707/XqJDgO3q2a6pZAcHqlS2rQT2yWxM2xKgTdWi+JL8=
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: base64
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: 02f71cf3-ed55-4789-1390-08d6d80a6ae7
X-MS-Exchange-CrossTenant-originalarrivaltime: 14 May 2019 01:20:57.3675 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 5ae1af62-9505-4097-a69a-c1553ef7840e
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN8PR11MB3620
X-OriginatorOrg: cisco.com
X-Outbound-SMTP-Client: 173.36.7.28, xch-aln-018.cisco.com
X-Outbound-Node: rcdn-core-11.cisco.com
Archived-At: <https://mailarchive.ietf.org/arch/msg/softwires/UBEJLevus4l1K5Jsp47C4D91AoQ>
Subject: Re: [Softwires] intdir Telechat Review requested: draft-ietf-softwire-map-radius
X-BeenThere: softwires@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: softwires wg discussion list <softwires.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/softwires>, <mailto:softwires-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/softwires/>
List-Post: <mailto:softwires@ietf.org>
List-Help: <mailto:softwires-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/softwires>, <mailto:softwires-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 14 May 2019 01:21:04 -0000

I am an assigned INT directorate reviewer for draft-ietf-softwire-map-radius-22. These comments were written primarily for the benefit of the Internet Area Directors. Document editors and shepherd(s) should treat these comments just like they would treat comments from any other IETF contributors and resolve them along with any other Last Call comments that have been received. For more details on the INT Directorate, see https://datatracker.ietf.org/group/intdir/about/.

This draft looks pretty good but there are a few quickly fixed issues and a bunch of minor nits. But, otherwise the draft looks ready to move forward.

Issues:

Section 3.1.3.1

I think the following text is in error:
   Defining multiple TLV-types achieves the same design goals as the
   "Softwire46 Rule Flags" defined in Section 4.1 of [RFC7598].  Using
   TLV-type set to 4 is equivalent to setting the F-flag in the
   OPTION_S46_RULE S46 Rule Flags field.
It should say (s/ 4 / 5 /):
   Defining multiple TLV-types achieves the same design goals as the
   "Softwire46 Rule Flags" defined in Section 4.1 of [RFC7598].  Using
   TLV-type set to 5 is equivalent to setting the F-flag in the
   OPTION_S46_RULE S46 Rule Flags field.
(I assume that "setting the F-flag" means setting it to 1.)

I'm also not sure what the following means:
	     5 Forwarding Permitted Mapping Rule (may be used for
	        forwarding. Can also be a Basic Mapping Rule)
Shouldn't this just be:
	     5 Forwarding Permitted Mapping Rule

FYI - The text in RFC7598 is:
   o  F-flag: 1-bit field that specifies whether the rule is to be used
      for forwarding (FMR).  If set, this rule is used as an FMR; if not
      set, this rule is a BMR only and MUST NOT be used for forwarding.
      Note: A BMR can also be used as an FMR for forwarding if the
      F-flag is set.  The BMR is determined by a longest-prefix match of
      the Rule IPv6 prefix against the End-user IPv6 prefix(es).

Section 5:
The "CoA-Request" message is not mentioned in this table, but was mentioned in 3.1:
      The Softwire46-Configuration Attribute MAY appear in a CoA-Request
      packet.
It may also be appropriate to include a table number/title?


Minor Nits:

Section 3.1:
	s/ efer / refer /

Section 3.1.2:
	Remove the 0+ definition under Table 2 as it is not used and therefore not needed.

Section 3.2:
	s/ orderd / ordered /
	s/ attribute include one or / attributes includes one or /		(use includes)

Section 3.3: Suggestion
	It may be more consistent and shorter to combine "MAY appear", "MAY contain" rules? For example:

      The Softwire46-Multicast Attribute MAY appear in an Access-Request,
      Access-Accept, CoA-Request, and Accounting-Request packet.

      The Softwire46-Multicast Attribute MAY contain ASM-Prefix64 (see
      Section 3.3.1), SSM-Prefix64 (see Section 3.3.2), and U-Prefix64 (see
      Section 3.3.3) attributes.

Section 4:
	In 4, s/Theses are/These are/
	In 5, s/CE send a/CE sends a/

Appendix A.7:
	The "TLV Field" column is a bit odd since these are really subfields from RFC8044.
	So, rename "TLV Subfield"? And, the fields are "Prefix-Length" and "Prefix" from
	the TLV attribute.

- Bernie

-----Original Message-----
From: Éric Vyncke via Datatracker <noreply@ietf.org> 
Sent: Tuesday, April 23, 2019 1:20 PM
To: Bernie Volz (volz) <volz@cisco.com>om>; Carlos Bernardos <cjbc@it.uc3m.es>
Subject: intdir Telechat Review requested: draft-ietf-softwire-map-radius


Telechat review of: draft-ietf-softwire-map-radius (no specific version)
Deadline: 2019-05-15
Requested by: Éric Vyncke

https://datatracker.ietf.org/doc/draft-ietf-softwire-map-radius/reviewrequest/11924/login/

intdir Telechat Review requested: draft-ietf-softwire-map-radius