[Gen-art] Gen-ART Last Call review of draft-ietf-pcp-port-set-10

Meral Shirazipour <meral.shirazipour@ericsson.com> Mon, 12 October 2015 05:49 UTC

From: Meral Shirazipour <meral.shirazipour@ericsson.com>
To: "draft-ietf-pcp-port-set.all@tools.ietf.org" <draft-ietf-pcp-port-set.all@tools.ietf.org>, "gen-art@ietf.org" <gen-art@ietf.org>
Thread-Topic: Gen-ART Last Call review of draft-ietf-pcp-port-set-10
Thread-Index: AdEEsYXry8YyyGNTSQm7Xt/5JSS3SQ==
Date: Mon, 12 Oct 2015 05:49:06 +0000
Message-ID: <ABCAA4EF18F17B4FB619EA93DEF7939A45432563@eusaamb107.ericsson.se>
Accept-Language: en-US
Content-Language: en-US
Content-Type: multipart/alternative; boundary="_000_ABCAA4EF18F17B4FB619EA93DEF7939A45432563eusaamb107erics_"
MIME-Version: 1.0
Archived-At: <http://mailarchive.ietf.org/arch/msg/gen-art/SeiOAfhgvM9wOH8kT-hT9jaCBDo>
Subject: [Gen-art] Gen-ART Last Call review of draft-ietf-pcp-port-set-10
Precedence: list

I am the assigned Gen-ART reviewer for this draft. For background on Gen-ART, please see the FAQ at http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq.

Please resolve these comments along with any other Last Call comments you may receive.

Document: draft-ietf-pcp-port-set-10
Reviewer: Meral Shirazipour
Review Date: 2015-10-10
IETF LC End Date: 2015-10-14
IESG Telechat date: NA

Summary:
This draft is ready to be published as Standards Track RFC but I have some comments.

Major issues:
N/A

Minor issues:
-[Page 7], Section 4.1, "If the PCP Client does not know the exact number of ports its requires, it MAY then set the Port Set Size to 0xffff, indicating that it is willing to accept as many ports as the PCP server can offer."
Question/clarification to add: Mention if there a mechanism where the server will know which of the mapped ports are going to be used by the client? and which mappings can be discarded/reused in a subsequent request.

Nits/editorial comments:
-[Page 6], "In particular, the PREFER_FAILURE option MUST NOT be present in a request that contains a PORT_SET option.".
Suggestion: Please add a sentence after this one suggesting why PREFER_FAILURE option MUST NOT be used. It was not clear to me until I read the rest of the draft...although I am still not sure why this behavior is to be a MUST NOT.

-[Page 8], Section 4.3, "There is intentionally no port set capability discovery mechanism.".
What is the intention? I could not find anything on the list discussion. It would be good to clarify this to make this section puroposeful.

-[Page 16] , Ref. [RFC7596] should be revised-it still refers to the draft.

Best Regards,
Meral
---
Meral Shirazipour
Ericsson
Research
www.ericsson.com

[Gen-art] Gen-ART Last Call review of draft-ietf-… Meral Shirazipour
Re: [Gen-art] Gen-ART Last Call review of draft-i… mohamed.boucadair
Re: [Gen-art] Gen-ART Last Call review of draft-i… Simon Perreault
Re: [Gen-art] Gen-ART Last Call review of draft-i… Meral Shirazipour
Re: [Gen-art] Gen-ART Last Call review of draft-i… Meral Shirazipour
Re: [Gen-art] Gen-ART Last Call review of draft-i… mohamed.boucadair
Re: [Gen-art] Gen-ART Last Call review of draft-i… Meral Shirazipour
Re: [Gen-art] Gen-ART Last Call review of draft-i… mohamed.boucadair
Re: [Gen-art] Gen-ART Last Call review of draft-i… Meral Shirazipour