Re: [mpls] Mirja Kühlewind's No Objection on draft-ietf-mpls-ldp-mrt-06: (with COMMENT)

That was just a comment. Just do the right thing…

> Am 13.11.2017 um 17:59 schrieb Chris Bowers <chrisbowers.ietf@gmail.com>:
> 
> Mirja,
> 
> With respect to the second comment, since the behavior is defined in RFC5661, in this case I think it is better to not be more explicit in this document because it might confuse matters and be interpreted as specifying behavior that is in some way different from RFC5661.  
> 
> If it is OK with you, I will close out these two items without making changes to the current text.
> 
> Thanks,
> Chris
> 
> On Thu, Nov 9, 2017 at 3:28 PM, Mirja Kuehlewind (IETF) <ietf@kuehlewind.net> wrote:
> Hi Chris,
> 
> see below.
> 
> > Am 07.11.2017 um 01:14 schrieb Chris Bowers <cbowers@juniper.net>:
> >
> > Mirja,
> >
> > Thanks for the feedback.  Responses are inline with [CB].
> >
> > Thanks,
> > Chris
> >
> > -----Original Message-----
> > From: mpls [mailto:mpls-bounces@ietf.org] On Behalf Of Mirja Kühlewind
> > Sent: Monday, September 11, 2017 6:37 AM
> > To: The IESG <iesg@ietf.org>
> > Cc: mpls@ietf.org; draft-ietf-mpls-ldp-mrt@ietf.org; mpls-chairs@ietf.org
> > Subject: [mpls] Mirja Kühlewind's No Objection on draft-ietf-mpls-ldp-mrt-06: (with COMMENT)
> >
> > Mirja Kühlewind has entered the following ballot position for
> > draft-ietf-mpls-ldp-mrt-06: No Objection
> >
> > When responding, please keep the subject line intact and reply to all email addresses included in the To and CC lines. (Feel free to cut this introductory paragraph, however.)
> >
> >
> > Please refer to https://www.ietf.org/iesg/statement/discuss-criteria.html
> > for more information about IESG DISCUSS and COMMENT positions.
> >
> >
> > The document, along with other ballot positions, can be found here:
> > https://datatracker.ietf.org/doc/draft-ietf-mpls-ldp-mrt/
> >
> >
> >
> > ----------------------------------------------------------------------
> > COMMENT:
> > ----------------------------------------------------------------------
> >
> > 1) I was just wondering if it was considered to just use one flag of the reserved field of the multi-topology extension in RFC7307 to advertise MRT support, given that MT must always advertised as well...?
> >
> > [CB]  I don't really understand the suggestion.  The document is requesting three values from the MPLS Multi-Topology Identifiers Registry (defined in RFC 7307). I think that we need all three, MRT-Red, MRT-Blue, and Rainbow MRT, for the protocol to work.  The registry has 64k possible values (with 4k being most useful), of which only 6 are currently used.
> 
> I was thinking just about the MRT Capability Advertisement (use a flag instead of a new TLV Type, namely TBD-MRT-LDP-1). However this is not important...
> 
> >
> > Or are you referring to the request for a value from the Label Distribution Protocol (LDP) Parameters registry "TLV Type Name Space"?
> >
> > 2) Why would a node withdraw the MRT capability/ when would it send a MRT advertisement with S=0?
> >
> > [CB]  As an example, a configuration change on the node to no longer participate in label distribution for MRT would generally result in a withdrawal of the MRT capability, which is done by setting S=0.  The document currently has a  reference to RFC 5661 "LDP Capabilities" for more information on this.  Do you think more text is needed in the document to clarify this?
> 
> Not necessarily. But it always helps to be rather explicit than implicit. It might be good to note that this is send based on things like config changes and not based on state changes that are happening during usual operation. However, maybe this is more obvious to people who are more expert than I am.
> 
> Mirja
> 
> >
> > 3) If you update the document, please also see the gen-art review: there are many small nits which will probably also be caught by the RFC editor but if you can fix them now, that's even better!
> >
> > [CB]  Thanks.  I did that in a separate response.
> >
> >
> > _______________________________________________
> > mpls mailing list
> > mpls@ietf.org
> > https://www.ietf.org/mailman/listinfo/mpls
> 
> _______________________________________________
> mpls mailing list
> mpls@ietf.org
> https://www.ietf.org/mailman/listinfo/mpls
> 