Thanks for fixing the bug!

I think we should not log a warning about this.  It is pretty much
unavoidable that sometimes the controller will be disconnected from the
switch, especially at controller startup.  WARN level log messages
should be reserved for situations that are likely to indicate a problem.
It might make sense to log something at INFO level, but the code for
maintaining OpenFlow connections already logs plenty of INFO messages
about connecting and disconnecting, so I don't think we really need
another one here.
I remove WARN.
Like Mark, I am not sure that the test is necessary.
I optimized the test case.
Will you submit a v2?
https://mail.openvswitch.org/pipermail/ovs-dev/2018-February/344491.html

Thanks,

Ben.

_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to