Hi George,

You are certainly right about this – this is quite bad.
I saw your pull request on github and will try it out.

Thanks for reporting this and submitting a fix as well!

Cheers,
Rob

On 18 Dec 2012, at 14:24, George Shuklin <george.shuk...@gmail.com> wrote:

> I found some kind of horrible bug in XCP 1.6.
> 
> After looking to src on github (master branch) I found it still there.
> 
> Way to reproduce:
> 
> create vif with device=15 (or any other >0).
> set up locking-mode=locked
> set up some ipv4-allowed.
> 
> xe vif-plug ....
> 
> expected result: rules for OVS applied to xenbr, corresponding to vif 
> network's bridge.
> 
> actual result: device number (15) is is used as bridge number (xenbr15!).
> 
> I done some source code review:
> Dec 18 18:17:54 rvc2-xh43 python: 
> /opt/xensource/libexec/setup-vif-rules[8505] - Called with vif_type=vif, 
> domid=1, devid=15, network_mode=openvswitch, action=filter
> 
> devid=15 <- WRONG
> 
> It use vif id from xenstore instead of proper 'bridge' field from 
> network object.
> 
> 
> 
> 
> _______________________________________________
> Xen-api mailing list
> Xen-api@lists.xen.org
> http://lists.xen.org/cgi-bin/mailman/listinfo/xen-api


_______________________________________________
Xen-api mailing list
Xen-api@lists.xen.org
http://lists.xen.org/cgi-bin/mailman/listinfo/xen-api

Reply via email to