Re: [Softwires] WGLC for draft-ietf-softwire-yang-04 as Standard Track, closed by 27 June 2018

<> Wed, 27 June 2018 08:54 UTC

Return-Path: <>
Received: from localhost (localhost []) by (Postfix) with ESMTP id 38110130EE1; Wed, 27 Jun 2018 01:54:19 -0700 (PDT)
X-Virus-Scanned: amavisd-new at
X-Spam-Flag: NO
X-Spam-Score: -2.589
X-Spam-Status: No, score=-2.589 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_LOW=-0.7, SPF_PASS=-0.001, T_KAM_HTML_FONT_INVALID=0.01, UNPARSEABLE_RELAY=0.001] autolearn=ham autolearn_force=no
Received: from ([]) by localhost ( []) (amavisd-new, port 10024) with ESMTP id YAup3B87Z1RD; Wed, 27 Jun 2018 01:54:16 -0700 (PDT)
Received: from ( []) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by (Postfix) with ESMTPS id C9DF3130EA4; Wed, 27 Jun 2018 01:54:15 -0700 (PDT)
Received: from (unknown [xx.xx.xx.7]) by (ESMTP service) with ESMTP id 1DDFE60D8A; Wed, 27 Jun 2018 10:54:13 +0200 (CEST)
Received: from Exchangemail-eme2.itn.ftgroup (unknown [xx.xx.31.66]) by (ESMTP service) with ESMTP id E81A760074; Wed, 27 Jun 2018 10:54:12 +0200 (CEST)
Received: from OPEXCLILMA3.corporate.adroot.infra.ftgroup ([fe80::60a9:abc3:86e6:2541]) by OPEXCLILMA1.corporate.adroot.infra.ftgroup ([fe80::95e2:eb4b:3053:fabf%19]) with mapi id 14.03.0399.000; Wed, 27 Jun 2018 10:54:12 +0200
To: Sheng Jiang <>, Softwires WG <>
CC: "" <>
Thread-Topic: WGLC for draft-ietf-softwire-yang-04 as Standard Track, closed by 27 June 2018
Thread-Index: AdQC+hwArZYPu3auQDGWSI0pkruopgKz1XLAAAbmCKA=
Date: Wed, 27 Jun 2018 08:54:12 +0000
Message-ID: <787AE7BB302AE849A7480A190F8B93302DF4CA36@OPEXCLILMA3.corporate.adroot.infra.ftgroup>
References: <> <>
In-Reply-To: <>
Accept-Language: fr-FR, en-US
Content-Language: fr-FR
x-originating-ip: []
Content-Type: multipart/alternative; boundary="_000_787AE7BB302AE849A7480A190F8B93302DF4CA36OPEXCLILMA3corp_"
MIME-Version: 1.0
Archived-At: <>
Subject: Re: [Softwires] WGLC for draft-ietf-softwire-yang-04 as Standard Track, closed by 27 June 2018
X-Mailman-Version: 2.1.26
Precedence: list
List-Id: softwires wg discussion list <>
List-Unsubscribe: <>, <>
List-Archive: <>
List-Post: <>
List-Help: <>
List-Subscribe: <>, <>
X-List-Received-Date: Wed, 27 Jun 2018 08:54:20 -0000

Hi Sheng,

Many thanks for the careful review.

An updated version which integrates your comments is available online:

See more inline.


De : Softwires [] De la part de Sheng Jiang
Envoyé : mercredi 27 juin 2018 06:00
À : Softwires WG
Cc :
Objet : Re: [Softwires] WGLC for draft-ietf-softwire-yang-04 as Standard Track, closed by 27 June 2018

As the document shepherd, I have reviewed this document. Document editors and WG chairs should treat these comments just like any other last call comments. In general, I think this document is in a good shape. The YANG model is well defined and clearly described.
Here are some minor issues, mostly editorial, although there is 1 error report by the IETF Yang validation tool. It should be easy to be fixed, I believe

[Med] As you can see in, the module passes validation. The issue displayed by the tracker is related to iana-if-type@2018-06-22.yang<>.

There are some minor comments below, most of them are editorial.

Section 2.1
It may be better to add the statement names in the description of choice statement:
  a choice statement 'ce-type' is included for ...
  a choice statement 'data-plane' is included to ...

[Med] Fixed.

"For each module, a choice statement is included for either 'binding' or 'algorithmic'."
But in Table 1 it is 'algorithm'. Maybe 'algorithmic' should be changed to 'algorithm'.

[Med] Good catch. Fixed.

Section 2.2
The reference to Appendix A.3 should be Appendix A

[Med] Citing Appendix A.3 is on purpose. It is where NAT and RFC8349 examples are provided.

Section 3.1
"for all of the softwire mechanisms listed in Section 1"
It may be bette to avoid self citation and just list the mechanisms here.

[Med] OK.

"Figure 1 describes the tree structure of the CE softwire YANG module"
It's better to unify the terminology as "Softwire CE YANG Module"

[Med] OK.

Section 3.2
In the paragraph of "softwire-path-mru:":
It's confusing here whether the MRU is for IPv4 or IPv6.

[Med] The text indicates "to set the maximum IPv6 softwire packet size". Furthermore, the description "The path MRU for the softwire (payload + encapsulation
        overhead)" is also clear about the usage. I don't think a change is required.

There are two "br-ipv6-addr" defined. It may be better to add different prefixes or suffixes into the names, but I'm also OK with the current names..
[Med] We could add map or lw prefixes, but as you know adding prefixes is not helpful (and it even not recommended) as a leaf is identified by its parent. I suggest to leave those unmodified.

In the paragraph of "ce-binding-ipv6-addr-change:":
"binding-ipv6-address" is not defined in the whole document. It should be explained.

[Med] changed to "binding IPv6 address"

Section 4.2
"in Figure 1"
should be "in Section 3.2"

[Med] OK.

"for logging/data retention purposes" -> "for logging or data retention purposes"
[Med] OK.

"between 3-tuples, which contains the lwB4's IPv6 address..." -> "between 3-tuples: the lwB4's IPv6 address..."
[Med] Changed to "3-tuples {lwB4's IPv6 address/prefix, the allocated IPv4 address, restricted port-set}"

>From the description, I think it may be better to rename it to "softwire-num-max".
[Med] Makes sense.

In the paratraph of "invalid-entry, added-entry, modified-entry:":
"the client" -> "the NETCONF client"

[Med] OK.

Appendix A.1
"lwB4 IPv6 Address:          123"
What's the "lwB4 IPv6 Address" here?
[Med] Oops. This should be PSID.

Appendix A.2
"for the clients" -> "for the CEs"
[Med] Done.

Appendix A.3
The same "lwB4 IPv6 Address" issue
And the PSID and PSID offset should be provided in the example.
[Med] Idem as above. Should s/lwB4 IPv6 Address/PSID.



From: Softwires [] On Behalf Of Sheng Jiang
Sent: Wednesday, June 13, 2018 5:44 PM
To: Softwires WG <>
Subject: [Softwires] WGLC for draft-ietf-softwire-yang-04 as Standard Track, closed by 27 June 2018

This email announces a Softwire Working Group Last Call (WGLC) on:

Since both chairs of softwire WG are the co-authors of this document. I am now acting as the document shepherd for this draft.

YANG Modules for IPv4-in-IPv6 Address plus Port Softwires


This draft is intended to become a Standard Track RFC.

This WGLC will run through the end of the day on Wednesday, June 27, 2018.

Comments should be sent to the<> list, although purely

editorial comments may be sent directly to the author.

No IPR disclosures have been submitted directly on


Regards and thanks,

Sheng Jiang (document shepherd)