[pm-dir] Fwd: Request for RFC 6390 review of https://datatracker.ietf.org/doc/draft-ietf-ippm-alt-mark/

Al mentioned in a private message that this message below did not make
it across. So sending it again with the authors in cc for review.

-- Vinayak

---------- Forwarded message ----------
From: Vinayak Hegde <vinayakh@gmail.com>
Date: Mon, Sep 25, 2017 at 3:02 AM
Subject: Re: [pm-dir] Request for RFC 6390 review of
https://datatracker.ietf.org/doc/draft-ietf-ippm-alt-mark/
To: "MORTON, ALFRED C (AL)" <acmorton@att.com>
Cc: Spencer Dawkins at IETF <spencerdawkins.ietf@gmail.com>, Benoit
Claise <bclaise@cisco.com>, "pm-dir@ietf.org" <pm-dir@ietf.org>

I have done the review of the draft below as per RFC 6390

The RFC mostly conforms to the RFC 6390 recommended format (separately
noted in a section "8.  Compliance with RFC6390 guidelines"). There
are some small changes that might make the document more clear and
readable.

1. Define steady state explicitly - there is an attempt made though
noting that when the alternate color is being marked in the flow but
calling it out would make it clearer.

2. It might be useful to also explicitly state what the timeout should
be for marking the packet as marked. Currently it seems that it is
left as an exercise to the implementer of the draft. One way could be
to not consider the packets when the same colour is marked again on
the flow. The old packets of the same colour "expire" or are lost. For
example the draft recomends 5 min fr switching the colour. A similar
recommendation might be made for marking packets as lost after a
certain timeout (either as a percentage of the measurement period or a
flat time).

3. There are two sections - Normative and Informative. The draft
addresses the normative and informative parts except the verification
section of the informative part. If it is not applicable (may not be
possible to reproduce results for verification), then it should be
explicitly noted in the draft.

Regards
Vinayak

On Sun, Sep 24, 2017 at 2:02 PM, MORTON, ALFRED C (AL) <acmorton@att.com> wrote:
> Hi Vin,
>
> reminder for this review,
>
> Al
>
>
>
> From: Spencer Dawkins at IETF [mailto:spencerdawkins.ietf@gmail.com]
> Sent: Wednesday, September 13, 2017 10:59 PM
> To: MORTON, ALFRED C (AL)
> Cc: Vinayak Hegde; Benoit Claise; pm-dir@ietf.org
>
>
> Subject: Re: [pm-dir] Request for RFC 6390 review of
> https://datatracker.ietf.org/doc/draft-ietf-ippm-alt-mark/
>
>
>
> Hi, Al,
>
>
>
> On Wed, Sep 13, 2017 at 5:14 PM, MORTON, ALFRED C (AL) <acmorton@att.com>
> wrote:
>
> Thanks Vin! you have about two weeks till the
>
> deadline in Spencer’s mail below.
>
>
>
> Thanks to the both of you!
>
>
>
> Spencer
>
>
>
>
>
> @Spencer: Reviewer Assigned.
>
> Al
>
>
>
> From: Vinayak Hegde [mailto:vinayakh@gmail.com]
> Sent: Wednesday, September 13, 2017 5:21 PM
> To: MORTON, ALFRED C (AL)
> Cc: Spencer Dawkins at IETF; Benoit Claise; pm-dir@ietf.org
> Subject: Re: [pm-dir] Request for RFC 6390 review of
> https://datatracker.ietf.org/doc/draft-ietf-ippm-alt-mark/
>
>
>
> I am okay with doing it over the weekend.
>
>
>
> Regards
>
> Vinayak
>
>
>
> On 13 Sep 2017 23:19, "MORTON, ALFRED C (AL)" <acmorton@att.com> wrote:
>
> Thank you for your request, Spencer.
>
>
>
> Are there any volunteers to review this draft?
>
>
>
> Al
>
> pm-dir admin
>
>
>
> From: pm-dir [mailto:pm-dir-bounces@ietf.org] On Behalf Of Spencer Dawkins
> at IETF
> Sent: Wednesday, September 13, 2017 1:31 PM
> To: pm-dir@ietf.org
> Cc: Benoit Claise
> Subject: [pm-dir] Request for RFC 6390 review of
> https://datatracker.ietf.org/doc/draft-ietf-ippm-alt-mark/
>
>
>
> Dear PM-Dir,
>
>
>
> As the responsible AD for this draft, I'd like to request an RFC 6390 review
> for https://datatracker.ietf.org/doc/draft-ietf-ippm-alt-mark/.
>
>
>
> The draft is in Last Call Requested state, with Last Call ending around
> 9-27-2017.
>
>
>
> Please let me know if you need anything else, and thanks,
>
>
>
> Spencer
>
>
> _______________________________________________
> pm-dir mailing list
> pm-dir@ietf.org
> https://www.ietf.org/mailman/listinfo/pm-dir
>
>