> On Apr 7, 2015, at 8:03 AM, Russell Bryant <rbry...@redhat.com> wrote:
> 
> On 04/06/2015 06:53 PM, Justin Pettit wrote:
> 
>> It might be nice to print the parent and tag in "lport-list".
> 
> Any thoughts on formatting?  I don't care much.  I was trying to keep it
> simple so it can be parsed without too much pain in scripts.

I agree simple is a good approach.  Looking back at it, maybe it's not worth 
printing those values in the Northbound database, since those values are closer 
to physical space than logical.  It would be nice at some point to print all 
the fields in the record, but we can look to add that when we extend the 
utility later.

By the way, would you be okay with our changing ovn-nbctl commands to more 
closely mimic ovs-vsctl?  The main differences I see are changing the names of 
the following commands:

        lswitch-add -> add-lswitch
        lswitch-del -> del-lswitch
        lswitch-list -> list-lswitch
        lport-add -> add-lport
        lport-del -> del-lport
        lport-list -> list-lport

And, finally, change the order of the "lport-add" arguments so it would be 
"add-lport lswitch lport".

It's pretty minor (and I realize your naming convention is more consistent), 
but I think it will cause less confusion when switching between the utilities.  
I'm happy to write up a patch if you want, but I imagine it will require 
changing the OpenStack portion, too, so I wanted to check with you.

Thanks!

--Justin


_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to