Re: [IPFIX] Review of draft-irtf-nmrg-location-ipfix-07.txt

"Eggert, Lars" <[email protected]> Wed, 29 Mar 2017 18:21:18 +0000
Newsgroups gmane.ietf.ipfix
Message-ID <[email protected]>
--===============9179009817480890900==
Content-Language: en-US
Content-Type: multipart/signed;
 boundary="Apple-Mail=_6D738C2D-8127-47C1-92E7-BA62D08ED4DD";
 protocol="application/pgp-signature"; micalg=pgp-sha512

--Apple-Mail=_6D738C2D-8127-47C1-92E7-BA62D08ED4DD
Content-Transfer-Encoding: quoted-printable
Content-Type: text/plain;
	charset=utf-8

Hi,

On 2017-3-29, at 12:57, Abdelkader Lahmadi <[email protected]> =
wrote:
> Dear Lars, and NMRG chairs,

Allison has taken over as chair, so this is a question for her. However:

> How we can proceed now with the draft?
>=20
> According to this: =
https://datatracker.ietf.org/doc/conflict-review-irtf-nmrg-location-ipfix/=
00/, It seems that the draft is not in conflict with IETF work.

The IESG has not yet concluded its review - that link at the moment =
captures the current status of individual ADs positions. They will send =
a formal response when their review has concluded.

> But, it seems that an IETF review and IESG approval are required for =
registering the location method tokens according to this:
> =
https://datatracker.ietf.org/doc/conflict-review-irtf-nmrg-location-ipfix/=


All IANA actions coming from the IETF require IESG approval. Typically, =
that approval is implicit in a review that results in a document being =
publishable.

> I suppose that we can now update the draft with the IANA =
recommendation and also the review provided IPFIX expert (Paul).

I'd wait until the IESG review has concluded.

Lars




> Thank you.
> Best regards.
>> On 03 Mar 2017, at 12:28, Eggert, Lars <[email protected]> wrote:
>>=20
>> On 2017-3-3, at 12:17, Abdelkader Lahmadi =
<[email protected]> wrote:
>>> So, we work on a new text to resolve ALL the raised issues and send =
you a version.
>>=20
>> Hang on. Your RG chair should have explained how the process works.
>>=20
>> This IRTF document is now being reviewed (per RFC5742) by the IESG. =
That review is *not* the same review that happens for IETF documents. =
Specifically, the IESG can really only say one of these five things:
>>=20
>>>   1. The IESG has concluded that there is no conflict between this
>>>      document and IETF work.
>>>=20
>>>   2. The IESG has concluded that this work is related to IETF work =
done
>>>      in WG <X>, but this relationship does not prevent publishing.
>>>=20
>>>   3. The IESG has concluded that publication could potentially =
disrupt
>>>      the IETF work done in WG <X> and recommends not publishing the
>>>      document at this time.
>>>=20
>>>   4. The IESG has concluded that this document violates IETF =
procedures
>>>      for <Y> and should therefore not be published without IETF =
review
>>>      and IESG approval.
>>>=20
>>>   5. The IESG has concluded that this document extends an IETF =
protocol
>>>      in a way that requires IETF review and should therefore not be
>>>      published without IETF review and IESG approval.
>>=20
>>=20
>> At the moment, you are getting individual comments from IPFIX experts =
as part of the IANA review process of the registry actions your document =
wants to make happen. Those are very valuable (thank you!), but before =
you make any changes to the document, please wait until an Area Director =
says that the issues raised are at a significance where they will ask =
for an IESG response other than #1 or #2 above.
>>=20
>> It sounds like this is likely going to be the case here, but please =
wait until the ADs have caught up on the discussion. And please wait =
with submitting a new revision until that happens as well.
>>=20
>> Lars
>>=20
>>=20
>>>=20
>>> Best,
>>>> On 03 Mar 2017, at 12:07, PJ Aitken <[email protected]> wrote:
>>>>=20
>>>> Abdelkader, it was me who did the IE-doctors review. That's only =
concerned with the IANA request; it's not an IPFIX review of the =
document.
>>>>=20
>>>> P.
>>>>=20
>>>>=20
>>>> On 03/03/17 11:01, Abdelkader Lahmadi wrote:
>>>>> Hello,
>>>>> We haven=E2=80=99t really get a "proper review" of the document by =
IPFIX experts. Recently, we had a discussion with IANA and they asked =
IE-doctors to make a review, since that we received some points to be =
fixed regarding the proposed IE. I can forward to you the other comments =
from IE-doctors that we have received by IANA.
>>>>>=20
>>>>> Thank you for your comments, Ok we will fix the raised issues in =
the document.
>>>>> Best regards.
>>>>>=20
>>>>>=20
>>>>>> On 03 Mar 2017, at 11:42, PJ Aitken <[email protected]> wrote:
>>>>>>=20
>>>>>> Authors, has this document been reviewed by any IPFIX experts?
>>>>>>=20
>>>>>> I see a request on November 23rd, but no reviews. So let me sign =
up for that.
>>>>>>=20
>>>>>>=20
>>>>>> First, I took a quick look at the Figures in Appendix B:
>>>>>>=20
>>>>>> Figure 1: the Field Count should be 5, not 2.
>>>>>>=20
>>>>>> Figure 2: the size of the optional Padding field is wrong: the =
figure shows 9 bits rather than 8.
>>>>>>=20
>>>>>> Figure 4: the Length of 32 should be 28. The =
"geospatialLocationPosLat" Information Element isn't defined.
>>>>>>=20
>>>>>> Figure 5: the Field Count of 2 should be 3.
>>>>>>=20
>>>>>> Figure 7: The "geospatialLocationPostLng" and =
"geospatialLocationtLng" Information Elements aren't defined.
>>>>>>=20
>>>>>> Figure 9: The sizes of the "CivicValue" data fields are not shown =
correctly. eg, "Inria Nancy-Grand Est" is depicted in 6 octets when it =
should contain 21. Therefore the Figure is misleading and difficult to =
understand; it is not a good example. Please redraw the figure =
correctly. Please mark the variable-lengths eg "vlen =3D 21".
>>>>>>=20
>>>>>> Figure 11:
>>>>>>  The Set IDs (311, 312, 313) do not correspond to the Template =
IDs in Figure 10 (306, 307, 308).
>>>>>>  Again, the "Inria Nancy-Grand Grand Est" field is depicted in 6 =
octets rather than the requisite 27. Without the repeated "Grand", the =
21 would be correct. Please write "vlen=3D21"
>>>>>>  The "Civic location Attr length" of 25 seems wrong.
>>>>>>=20
>>>>>>=20
>>>>>> This document is not ready for publication. Please post an =
updated version so I can check that all the IPFIX details are correct.
>>>>>>=20
>>>>>> Thanks,
>>>>>> P.
>>>>=20
>>>> _______________________________________________
>>>> IPFIX mailing list
>>>> [email protected]
>>>> https://www.ietf.org/mailman/listinfo/ipfix
>>>=20
>>=20
>=20


--Apple-Mail=_6D738C2D-8127-47C1-92E7-BA62D08ED4DD
Content-Transfer-Encoding: 7bit
Content-Disposition: attachment; filename="signature.asc"
Content-Type: application/pgp-signature; name="signature.asc"
Content-Description: Message signed with OpenPGP

-----BEGIN PGP SIGNATURE-----

iQIcBAEBCgAGBQJY2/saAAoJEFS1wwm/cMFX10cP/jiUj2AnkOradF2h9Q1KX/hb
5GrB6AXGgV4DvtmhL2bK0vJC/Eok0cxDHaEa1QpLNeQL3pUEL4XxvpZPeBkiOK5g
GSzBefTBeFkg6bos+6CY4bGA2uovDkzssZAshFCnNK6TTlRpqNx6pg1Y0I6eNdHW
+4GzjLha3w7YOzwa8K2MHcZrKbihSANvDU8+WEw9qBJOj8l3ol3XYUzIvZpU+D2i
rSBgiOYt/QUfQYSXYhBy+wX6+EUwj4Cwprxc8cWsSex8FR5ztDChziHPWS0JgBlU
pBy6Fn2oCBCCLjnmVA/QAnXdWC+cZjRf9kQVH4L9XupriXuW/bbDKTiu2CAdbVYT
1Rl+Fwh4xpQ3w1VzpF550BpYj3NQ2PCdmHGLpgQb12ymrOHoh+Wsvq+uSWUFO4JL
eW/lVXkqyNvkkRIelSb4Ujm6wtvdDfyOSV5p3xSdtUDitcGHziDXGyJDAsP2kPWl
gUJMxYVMs2LyrcROIGjp85CL4woBf79ey5VETKgH5pwk3AOAZmTyar+w6VvMhUhj
gT6/M8ppQ/sXyj7UiCI0lspkhVAD0k5SBHjABsfghb4qHUMw4fk5mB0Mt1+jJPhn
IAUKWTJibVsWGDAZfg4YotuG3/fuu9bDE/iMEiN17cvy4NQxDszQS37ssSEqtL5w
lX8OUzRGKLA1UE0Uq01h
=Ct6/
-----END PGP SIGNATURE-----

--Apple-Mail=_6D738C2D-8127-47C1-92E7-BA62D08ED4DD--


--===============9179009817480890900==
Content-Type: text/plain; charset="us-ascii"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Content-Disposition: inline

_______________________________________________
IPFIX mailing list
[email protected]
https://www.ietf.org/mailman/listinfo/ipfix

--===============9179009817480890900==--