On 2/28/11 3:17 PM, Ethan Jackson wrote:
>> +
>> + # Change bridge disable_in_band option if XAPI and OVS states
>> differ.
>> + xapi_dib = network['other_config'].get('vswitch-disable-in-band')
>> + if not xapi_dib:
> I think we should enumerate the valid possibilities here e.g.:
> if xapi_dib not in ['true', 'false']:
Ok, the behavior now is that if the value is 'true' or 'false', it gets
set in OVSDB. If it's empty ('' or None), it gets cleared in OVSDB.
Otherwise, an error is logged. This is true of all three scripts, though
in the XAPI plugin the error is returned to the caller rather than being
logged.
> Otherwise looks good. Go ahead and merge when you are ready.
Thanks, I've pushed it to master.
-Andrew
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev_openvswitch.org
