Re: [netconf] [yang-doctors] Yangdoctors last call review of draft-ietf-netconf-subscribed-notifications-21

[Martin Bjorklund <[e-mail]>]

"Eric Voit (evoit)" <[e-mail]> wrote:
> > "Eric Voit (evoit)" <[e-mail]> wrote:
> > > > "Eric Voit (evoit)" <[e-mail]> wrote:
> > > > > > From: Martin Bjorklund, January 24, 2019 9:40 AM
> > > > > >
> > > > > > "Eric Voit (evoit)" <[e-mail]> wrote:
> > > > > > > > From: Martin Bjorklund, January 24, 2019 8:17 AM

[...]

> > > > > > > > Thinking some more, what is supposed to happen if the client on
> > > > > > > > the same session sends first an establish-subscription with dscp
> > > > > > > > 42, and then another establish-subscription with dscp 10?
> > > > > > >
> > > > > > > This would be allowed.
> > > > > >
> > > > > > On linux at least this is a sockopt, i.e., the option applies to the
> > > > > > socket, which means all packets on the session.  So how is this
> > > > > > supposed to be implemented if different messages on the session
> > > > > > should have different dscp values?
> > > > > > Or is the
> > > > > > idea that you send the msg, flush all data from ssh/tls to tcp, then
> > > > > > flush the tcp buffers (not that easy...)?
> > > > > >
> > > > > > Even if there's just one establish-subscription with a dscp value,
> > > > > > since it applies to the session it means that all normal rpcs on
> > > > > > this session will get the same dscp value.  It is not clear that
> > > > > > this is the intention?
> > 
> > Did you miss these questions?
> 
> With NETCONF, one DSCP will apply to single TCP session.  So there is no issue.

The problem is if a client sends two establish-subscriptions
with different values for dscp on the same session.  Or even if the
client sends one establish-subscription with som dscp value, then the
dscp will be applied to all packets from other rpcs as well.

... and even worse, with SSH channels you can have multiple NETCONF
session on the same TCP session, which means that the dscp value
applies to all packets in all NETCONF sessions sharing the same TCP
session.

I don't know what the right thing to do is.  Probably first agree that
this is in fact a problem, then maybe simply document this somehow.


> For RESTCONF, there is no requirement that there is a single TCP
> session between publisher and subscriber.

I agree that this is not a problem with RESTCONF, since there is no
multiplexing going on.  When the server accepts the SSE GET, it can
safely set the dscp value.

> Where there are different
> DSCPs requested, a different logical connection can be established
> should the publisher want to concurrently support several DSCPs.

I don't think is an issue, see above.


/martin