[pve-devel] venet firewall broken?

[Alexandre DERUMIER]

Ok thanks !


>>Please can you review them? If you think we can go that way, please add 
>>add 'Signed-off-by' line and cleanup the commit messages (remove 'based on 
>>patch from Alexandre' note) 

This is my first review ;) I'll try to do it cleanly

----- Mail original ----- 

De: "Dietmar Maurer" <dietmar at proxmox.com> 
À: "Alexandre DERUMIER" <aderumier at odiso.com> 
Cc: pve-devel at pve.proxmox.com 
Envoyé: Lundi 12 Mai 2014 10:21:51 
Objet: RE: venet firewall broken? 

> >>Which is obviously wrong. So why do you want to keep that patch? 
> 
> Yes,I think you are right, we can revert that patch. 

I sent a rework to the list. Those patches apply on top of: 

commit 81a1a25884420d50fc3cc0cd68e01befeb547e7e 
Author: Dietmar Maurer <dietmar at proxmox.com> 
Date: Tue May 6 11:18:25 2014 +0200 

set RELEASE to 3.2 

Please can you review them? If you think we can go that way, please add 
add 'Signed-off-by' line and cleanup the commit messages (remove 'based on 
patch from Alexandre' note)