Job,

> From: Job Snijders [mailto:j...@ntt.net]
 > Sent: Tuesday, October 10, 2017 12:06 PM
> 
 > On Tue, Oct 10, 2017 at 09:56:45AM +0000, bruno.decra...@orange.com wrote:
 > > > Minor issues:
 > > >
 > > > In Section 4. "EBGP graceful shutdown procedure", it states that 0
 > > > can used in all cases except where the AS already has a special
 > > > meaning for 0. It seems to me more ought to be said, but I admit I'm
 > > > not well-versed on (I) BGP and might be seeing dragons where only
 > > > windmills are present.
 > >
 > > [Bruno]
 > > I'm not seeing much more to say. It's intended as a warning that an AS
 > > may already use LOCAL_PREF zero to mean something specific. In which
 > > case, the AS knows the specifics,  authors/IETF/draft/RFC do not.
 > > I'm changing to the following reformulation. Please feel free to
 > > suggest text if you prefer.
 > >
 > > OLD: The LOCAL_PREF value must be lower than the one of the alternate
 > > path. 0 being the lowest value, it can be used in all cases, except if
 > > it already has a special meaning within the AS.
 > > NEW: The LOCAL_PREF value SHOULD be lower than the one of the
 > > alternate path. Zero being the lowest value, it MAY be used whichever
 > > LOCAL_PREF values are used by the AS. If LOCAL_PREF zero already has a
 > > special meaning within the AS, and there is a need to distinguish both
 > > usages, another low value MAY be used.
 > 
 > Any attribute (origin, as_path, aggregator) anywhere can be overloaded
 > to mean something only significant to the local network. I think the
 > document is simpler without this and see no point in mentioning this. I
 > propose:
 > 
 > OLD:
 >     The LOCAL_PREF value must be lower than the one of the alternate
 >     path. 0 being the lowest value, it can be used in all cases, except
 >     if it already has a special meaning within the AS.
 > NEW:
 >     The LOCAL_PREF value SHOULD be lower than any of the alternative
 >     paths. It is RECOMMEND to use 0, the lowest LOCAL_PREF value.

What is really needed is "The LOCAL_PREF value SHOULD be lower than the one of 
the alternative path."
Looks reasonable to extend it to your proposition " The LOCAL_PREF value SHOULD 
be lower than any of the alternative paths." So I'm changing for this.

Now the value is truly local to an AS, and I'm not sure to see the technical 
reason to RECOMMEND (SHOULD) a specific value. MAY seems more appropriate to 
me. Hence I'm proposing to keep
"Zero being the lowest value, it MAY be used whichever LOCAL_PREF values are 
used by the AS."

I'm open to remove "If LOCAL_PREF zero already has a special meaning within the 
AS, and there is a need to distinguish both usages, another low value MAY be 
used." If you believe that this sentence add complexity. I'd agree that it is 
somewhat redundant, although it does provides a specific point to consider.

 
 > Kind regards,
 > 
 > Job
 > 
 > ps. I-D.ietf-idr-shutdown can reference RFC 8203 now.

Thanks. Fixed.

Kind regards
--Bruno

_________________________________________________________________________________________________________________________

Ce message et ses pieces jointes peuvent contenir des informations 
confidentielles ou privilegiees et ne doivent donc
pas etre diffuses, exploites ou copies sans autorisation. Si vous avez recu ce 
message par erreur, veuillez le signaler
a l'expediteur et le detruire ainsi que les pieces jointes. Les messages 
electroniques etant susceptibles d'alteration,
Orange decline toute responsabilite si ce message a ete altere, deforme ou 
falsifie. Merci.

This message and its attachments may contain confidential or privileged 
information that may be protected by law;
they should not be distributed, used or copied without authorisation.
If you have received this email in error, please notify the sender and delete 
this message and its attachments.
As emails may be altered, Orange is not liable for messages that have been 
modified, changed or falsified.
Thank you.

_______________________________________________
GROW mailing list
GROW@ietf.org
https://www.ietf.org/mailman/listinfo/grow

Reply via email to