Hi Les,
On Wed, Dec 5, 2018 at 4:51 PM Les Ginsberg (ginsberg)
wrote:
> Yoshi -
>
> Thanx for taking the time to review.
>
> I can appreciate that this may the first time you have looked at RFC7810 -
> let alone the bis draft. As a result you have commented on content which is
> common to the bis draft and the RFC it is modifying (RFC 7810).
>
> While your questions in isolation may be interesting, I believe they are
> out of scope for the review of the bis draft. What the bis draft is doing
> is addressing two modest errata - details of which can be found in
> https://tools.ietf.org/html/draft-ietf-lsr-isis-rfc7810bis-03#appendix-A
> Comments on content not related to those changes is out of scope.
>
Just to be clear, my comments on this draft were clarification questions
which won't request changes in the spec of the draft because I thought
there are a certain ambiguities in the draft.
But, if what you really mean is there is no ambiguities or confusions among
expected readers on the points I made, it makes a sense to me and I can
think I had unnecessary concerns.
However, I'm not fully sure about it yet.
For example, the draft mentions about delay variation and it seems to me
this term is a bit ambiguous. If expected readers can understand it means
(for example) mean deviation, I am fine with it.
But, in this case,I might want to see supportive comments on this view from
the community.
Or, if you can clarify that even if there are some discrepancies between
implementations it won't affect overall performance of the applications, I
am also fine with it.
Anyway, I just tried to do my best to read and review the draft. I hope
not, but if the points I made don't make sense please let me know.
I would like to leave further decisions to ADs.
If you have an interest in this topic and want to comment on the substance
> of RFC 7810 and its companion document for OSPF RFC 7471, I encourage you
> to do so. Note that all of your comments (save the one on Security) are
> also applicable to RFC 7471 - so any agreed upon modification would need to
> be made to both documents. But I do not want to even start discussing such
> changes in the context of reviewing the bis draft changes. I hope you can
> understand why.
>
> As regards your Security comment, I am not sure I understand what you are
> suggesting. As IGP info is flooded hop-by-hop, man-in-the-middle attacks
> have to be able to insert themselves on an IGP enabled link. Use of
> cryptographic authentication prevents untrusted sources from being accepted
> - which is the point being made.
>
As Spencer already made my point clear (Thanks Spencer!), I was wondering
if we need a normative language here as the draft mentions these info can
be sensitive.
I was also wondering why the draft didn't mention encryption while it
conveys sensitive info.
Thanks,
--
Yoshi
> > -Original Message-
> > From: Yoshifumi Nishida
> > Sent: Wednesday, December 05, 2018 11:12 AM
> > To: tsv-...@ietf.org
> > Cc: lsr@ietf.org; i...@ietf.org;
> draft-ietf-lsr-isis-rfc7810bis....@ietf.org
> > Subject: Tsvart last call review of draft-ietf-lsr-isis-rfc7810bis-03
> >
> > Reviewer: Yoshifumi Nishida
> > Review result: Almost Ready
> >
> > This document has been reviewed as part of the transport area review
> > team's
> > ongoing effort to review key IETF documents. These comments were written
> > primarily for the transport area directors, but are copied to the
> document's
> > authors and WG to allow them to address any issues raised and also to the
> > IETF
> > discussion list for information.
> >
> > When done at the time of IETF Last Call, the authors should consider this
> > review as part of the last-call comments they receive. Please always CC
> > tsv-...@ietf.org if you reply to or forward this review.
> >
> > Summary: This document is almost ready for publication, but several
> points
> > need
> > to be clarified.
> >
> > 1: In Section 1:
> > "While this document does not specify how the performance
> information
> > should be obtained, the
> > measurement of delay SHOULD NOT vary significantly based upon the
> > offered
> > traffic load."
> >
> >It is not clear to me that why the measurement of delay should not
> vary
> >here. Also, queuing delay might be useful info to infer path status.
> Could
> >you elaborate the delays that the draft tries to capture?
> >
> > 2: In Section 1:
> > "Thus, queuing delays SHOULD NOT be included in the delay
> > measurement. "
> >
> >Is it clear for expected readers how to exclude queuing delays in
> their
> >
