Hi, "Clyde Wildes (cwildes)" <[email protected]> wrote: > Any > > My comments inline as [clyde2]… > > From: Andy Bierman <[email protected]> > Date: Saturday, December 31, 2016 at 8:24 AM > To: "Clyde Wildes (cwildes)" <[email protected]> > Cc: Alex Campbell <[email protected]>, "[email protected]" > <[email protected]> > Subject: Re: [netmod] WG Last Call for > draft-ietf-netmod-syslog-model-11 > > > > On Fri, Dec 30, 2016 at 10:16 AM, Clyde Wildes (cwildes) > <[email protected]<mailto:[email protected]>> wrote: > Hi Andy, > > Thanks for taking the time to review the model. > > My comments are inline as [clyde]… > > From: netmod <[email protected]<mailto:[email protected]>> > on behalf of Andy Bierman > <[email protected]<mailto:[email protected]>> > Date: Tuesday, December 27, 2016 at 3:04 PM > To: Alex Campbell <[email protected]> > Cc: "[email protected]<mailto:[email protected]>" > <[email protected]<mailto:[email protected]>> > Subject: Re: [netmod] WG Last Call for > draft-ietf-netmod-syslog-model-11 > > Hi, > > I am also considering an implementation. > I share the same concerns that Alex has brought up. > > Some detailed comments: > > 1) /syslog/actions: seems like everything is in this container. > Why is it needed? Seems like it could be removed as it serves no > purpose > > [clyde] Although this model is currently designated as config only, we > could add operational data and rpc leaves in the future. The actions > container is to future-proof the model. > > 2) 8 features: the granularity seems wrong. The main container for > each section > should have its own if-feature > /console > /buffer > /file > /remote > > [clyde] We have gone back and forth on this…some have complained that > there are too many features. I will be happy to add a feature for each > action. Note that we studied the implementation of each action by six > vendors including Linux and opted to not add features for actions > implemented by at least 3 vendors. Vendors not implementing an action > could create a deviation.
This is not a good usage of deviations. Deviations should be used as a last resort by vendors that cannot comply with the standard. Designing the usage of deviations into the overall solution is not a good idea. These should really be features, even if the number of features becomes higher. > I prefer 1 mandatory-to-implement and a minimal number of additional > options. > > /console > /file > /remote > > These are all mandatory-to-implement.. > IMO only /file should be mandatory-to-implement. But not all systems have a local file system to write logs to. If there must be one mandatory-to-implement, shouldn't it be 'remote'? > [clyde2] I will remove the buffer and session actions in the next > draft and will make the remaining three features. Good! /martin _______________________________________________ netmod mailing list [email protected] https://www.ietf.org/mailman/listinfo/netmod
