On 03/02/2016 14:39, "Ben Pfaff" <b...@ovn.org> wrote:

>On Wed, Feb 03, 2016 at 02:38:00PM -0800, Ben Pfaff wrote:
>> On Tue, Feb 02, 2016 at 05:56:34PM -0800, Daniele Di Proietto wrote:
>> > This effectively stops vswitchd from creating bridges with '/' in the
>> > name. OVS used to print a warning but the bridge was created anyway.
>> > 
>> > This restriction is implemented because the bridge name is part of a
>> > filesystem path.
>> > 
>> > This check is no substitute for Mandatory Access Control, but it
>> > certainly helps to catch the error early.
>> > 
>> > Signed-off-by: Daniele Di Proietto <diproiet...@vmware.com>
>> 
>> Please add a test.  You can take this one:
>
>Oh, with a test:
>Acked-by: Ben Pfaff <b...@ovn.org>

I didn't think it was possible to write a unit test for this. I added it
and push this up to branch-2.3.

Thanks!

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

Reply via email to