[ovs-dev] [PATCH] ovs-vsctl: check if the device name is valid

Cong Wang xiyou.wangcong at gmail.com
Fri Oct 26 13:22:35 UTC 2012


On 10/25/2012 11:20 PM, Ben Pfaff wrote:
> Furthermore, ovs-vsctl is meant to be generic in the sense that you can
> use an ovs-vsctl built in one place (on one OS) to control a switch and
> database running elsewhere (on another OS).  You don't want to apply the
> local host's restrictions to changes to a remote db.

Makes sense.

The problem without this patch is that when we add some port whose name 
is longer than IFNAMSIZ, ovs-vsctl doesn't give any error, the error is 
actually "hidden" in the log, which is actually written by ovs-vswitchd. 
So, it is not easy for inexperienced people to find where the error is.

How about raising the console log level? So that such errors will be 
seen on the terminal.

Thanks!




More information about the dev mailing list