[pve-devel] Process to submit patches

[Andrew Thrift]

FYI patch is:


*** Network.pm.orig     2013-04-02 10:19:42.000000000 +1300
--- Network.pm  2013-04-02 11:41:44.000000000 +1300
***************
*** 122,138 ****
       #check if we have an only one ethX or bondX interface in the bridge

       my $iface;
!     PVE::Tools::dir_glob_foreach($dir, '((eth|bond)\d+)', sub {
         my ($slave) = @_;

         die "more then one physical interfaces on bridge '$bridge'\n" 
if $iface;
!       $iface = $slave;

       });

       die "no physical interface on bridge '$bridge'\n" if !$iface;

!     my $ifacevlan = "${iface}.$tag";

       # create vlan on $iface is not already exist
       if (! -d "/sys/class/net/$ifacevlan") {
--- 122,138 ----
       #check if we have an only one ethX or bondX interface in the bridge

       my $iface;
!     PVE::Tools::dir_glob_foreach($dir, '((eth|bond)\d+\.?\d+)', sub {
         my ($slave) = @_;

         die "more then one physical interfaces on bridge '$bridge'\n" 
if $iface;
!       $iface = $bridge;

       });

       die "no physical interface on bridge '$bridge'\n" if !$iface;

!     my $ifacevlan = "${bridge}.$tag";

       # create vlan on $iface is not already exist
       if (! -d "/sys/class/net/$ifacevlan") {





On 4/2/2013 11:27 AM, Andrew Thrift wrote:
> Hi,
>
> We are wanting to submit a patch to Network.pm to be included upstream.
>
> Our patch changes the way Proxmox dynamically creates vlans allowing 
> for the current model, as well as for QinQ.
>
> Currently it is not possible to do QinQ on Proxmox as when you specify 
> a VLAN in the WebUI, Proxmox checks the parent bridge for a Physical 
> Interface, then creates the vlan sub-if on the Physical Interface and 
> then binds it to a new bridge.
>
> Our patches creates the vlan sub-if on the parent VM bridge, rather 
> than on the parent interface.   This works with both QinQ and non QinQ 
> configurations.
>
>
> What is the process to submit our patch ?
>
>
>
>
> Regards,
>
>
>
>
>
> Andrew Thrift
>