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

Ben Pfaff blp at nicira.com
Fri Oct 26 14:44:20 UTC 2012


On Fri, Oct 26, 2012 at 09:22:35PM +0800, Cong Wang wrote:
> 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.
> 
> 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.

ovs-vswitchd doesn't have a fd for the console; it daemonized itself and
closed that fd long ago.  (That's especially important if one started
ovs-vswitchd over an SSH connection; otherwise ovs-vswitchd holds the
SSH connection open.)

I think the key is probably to learn to look at the log if something
goes wrong.



More information about the dev mailing list