Re: [Isis-wg] WG Last Call for draft-ietf-isis-reverse-metric-07

["Les Ginsberg (ginsberg)" <[e-mail]>]

Acee –

The IS-IS draft is more flexible in this regard than the OSPF equivalent (https://tools.ietf.org/html/draft-ietf-ospf-link-overload-10).

In the IS-IS draft the text states:

“The Metric Offset field contains a 24-bit unsigned integer of an IS-
   IS metric that a neighbor SHOULD add to the existing, configured
   "default metric" contained within its IS Neighbors TLV…”

This allows that the operator could choose to set the neighbor metric to something other than “max-metric-1”.

Contrast this with the OSPF Draft which states:

“The node that has the link to be taken out of service MUST set metric
   of the link to MaxLinkMetric (0xffff)…”

The text in the IS-IS draft needs to remain as it is.

   Les


From: Isis-wg [mailto:[e-mail]] On Behalf Of Acee Lindem (acee)
Sent: Monday, November 27, 2017 3:33 PM
To: Naiming Shen (naiming) <[e-mail]>
Cc: Christian Hopps <[e-mail]>; [e-mail]; [e-mail]
Subject: Re: [Isis-wg] WG Last Call for draft-ietf-isis-reverse-metric-07

Hi Naiming,

From: "Naiming Shen (naiming)" <[e-mail]<mailto:[e-mail]>>
Date: Monday, November 27, 2017 at 4:38 PM
To: Acee Lindem <[e-mail]<mailto:[e-mail]>>
Cc: Christian Hopps <[e-mail]<mailto:[e-mail]>>, "[e-mail]<mailto:[e-mail]>" <[e-mail]<mailto:[e-mail]>>, "[e-mail]<mailto:[e-mail]>" <[e-mail]<mailto:[e-mail]>>
Subject: Re: [Isis-wg] WG Last Call for draft-ietf-isis-reverse-metric-07


Hi Acee,

thanks for the comments. replies inline with <NS>…</NS>

On Nov 24, 2017, at 3:05 PM, Acee Lindem (acee) <[e-mail]<mailto:[e-mail]>> wrote:

Hi,

I support publication of the subject document. I do have comments.

  1. Why do you call the reverse-metricn field, “Metric Offset”? This
sounds like a remnant of some bad 20th century CLI….

<NS>
the draft used to have the field as just ‘Metric’, since version 06, we responded
to the comments:


Section 2

From the description what is being advertised in the new TLV is not a metric but a metric offset i.e. you want the receiving IS to add the advertised value to its existing configured metric. Identifying the metric field as "metric offset" would make this point more clearly.


which is a good point I think. do you think there is an alternative name we
can use in replacing the ‘offset’ here?
</NS>

How about just calling it the “Reverse Metric”?

Thanks,
Acee




  2. Please split the single sentence description immediately following
figure 1 into multiple sentences. Maybe just refer to “Elements of
Procedure” sections rather than one incomprehensible sentence.
  3. Pervasive editorial comment, the plural of acronyms is does NOT have
an apostrophe. For example, it is TLVs, not TLV’s.

<NS>
will change those.

thanks.
- Naiming
</NS>



Thanks,
Acee


On 11/15/17, 5:43 PM, "Isis-wg on behalf of Christian Hopps"
<[e-mail]<mailto:[e-mail]> on behalf of [e-mail]<mailto:[e-mail]>> wrote:



The authors have asked for and we are starting a WG Last Call on

https://datatracker.ietf.org/doc/draft-ietf-isis-reverse-metric/

which will last an extended 3 weeks to allow for IETF100.

Thanks,
Chris.

_______________________________________________
Isis-wg mailing list
[e-mail]<mailto:[e-mail]>
https://www.ietf.org/mailman/listinfo/isis-wg

_______________________________________________
Isis-wg mailing list
[e-mail]<mailto:[e-mail]>
https://www.ietf.org/mailman/listinfo/isis-wg