I'm happy to keep the current subcode. In today's IOS-XR implementation, we simply hexdump and trailing octets after the subcode in "show bgp neighbor". The new code could be to additionally print it in UTF8, after validating it (no terminal escape sequences or other rubbish) and truncating it at 128 octets. And syslog it if a valid string exists.
Thanks, Jakob. > -----Original Message----- > From: Idr [mailto:[email protected]] On Behalf Of Jeffrey Haas > Sent: Wednesday, November 16, 2016 5:47 PM > To: Job Snijders <[email protected]> > Cc: [email protected]; [email protected] > Subject: Re: [Idr] [GROW] draft-snijders-idr-shutdown-00: Drop a line in the > peer's syslog at shutdown > > On Thu, Nov 17, 2016 at 10:42:15AM +0900, Job Snijders wrote: > > On Wed, Nov 16, 2016 at 08:36:40PM -0500, Jeffrey Haas wrote: > > > On Wed, Nov 16, 2016 at 03:15:56PM +0900, Job Snijders wrote: > > > > Some might wonder, why "Cease"? > > > > > > > > The beauty of using a new Cease subcode, is that the NOTIFICATION > > > > message type already allows extra data to be attached, so for most > > > > implementations it will be very simple to bolt what is specified in > > > > draft-snijders-idr-shutdown-00 on top of their existing code. In some > > > > cases we are looking at just a handful of lines. > > > > > > As I commented elsewhere, changing subcodes ends up being painful from a > > > conformance standpoint. I would honestly not recommend a new subcode. > > > > I don't follow, it seems you recommend to not change the behaviour of an > > existing subcode (conformance), but you also recommend against getting a > > new subcode? Can you clarify? > > No new subcodes. > Add the text to the NOTIFICATION Data section of existing subcodes, when it > makes sense and that Data section isn't otherwise specified. > > -- Jeff > > _______________________________________________ > Idr mailing list > [email protected] > https://www.ietf.org/mailman/listinfo/idr _______________________________________________ GROW mailing list [email protected] https://www.ietf.org/mailman/listinfo/grow
