Re: [ovs-dev] [PATCH v4 2/2] ovs-vsctl: Improve error reporting

2014-03-28 Thread Ben Pfaff
On Fri, Mar 28, 2014 at 10:49:29AM +0100, Thomas Graf wrote: > Obviously we don't want to change behaviour of ovs-vsctl show in > terms of exit code. So how about a new command 'ovs-vsctl status' > which acts as a dashboard. It would be based on 'show' but without > port options to keep it simple.

Re: [ovs-dev] [PATCH v4 2/2] ovs-vsctl: Improve error reporting

2014-03-28 Thread Ben Pfaff
Great. Thanks Andy and Thomas, I applied these two patches to master. I added a NEWS item: - ovs-vsctl now reports when ovs-vswitchd fails to create a new port or bridge. On Fri, Mar 28, 2014 at 12:13:45AM -0700, Andy Zhou wrote: > Not sure if I suppose to ack this patch since I am one

Re: [ovs-dev] [PATCH v4 2/2] ovs-vsctl: Improve error reporting

2014-03-28 Thread Thomas Graf
On 03/28/2014 05:02 AM, Ben Pfaff wrote: From: Andy Zhou ovs-vsctl is a command-line interface to the Open vSwitch database, and as such it just modifies the desired Open vSwitch configuration in the database. ovs-vswitchd, on the other hand, monitors the database and implements the actual con

Re: [ovs-dev] [PATCH v4 2/2] ovs-vsctl: Improve error reporting

2014-03-28 Thread Andy Zhou
Not sure if I suppose to ack this patch since I am one of authors. I did review v4 in detail and tested it . Just in case it is useful, here is my ack. Acked-by: Andy Zhou On Thu, Mar 27, 2014 at 9:02 PM, Ben Pfaff wrote: > From: Andy Zhou > > ovs-vsctl is a command-line interface to the Op