Re: [ovs-dev] [PATCH ovn] northd: Allow DHCP request from the lport if enabled DHCPv4

2024-04-19 Thread Numan Siddique
On Thu, Apr 18, 2024 at 11:20 PM shy liu wrote: > > > > > > > > On Tue, Apr 16, 2024 at 1:22 AM wrote: > >> > >> From: shylou > >> > >> DHCP for VM fails when removing default security group rules > >> using a CMS like Neutron ML2/OVN [1]. This is because DHCP > >> requests from VMs may be

Re: [ovs-dev] [PATCH ovn] northd: Allow DHCP request from the lport if enabled DHCPv4

2024-04-18 Thread shy liu
> > > > On Tue, Apr 16, 2024 at 1:22 AM wrote: >> >> From: shylou >> >> DHCP for VM fails when removing default security group rules >> using a CMS like Neutron ML2/OVN [1]. This is because DHCP >> requests from VMs may be dropped by ACLs. To fix this issue, >> we add a lflow with a priority of

Re: [ovs-dev] [PATCH ovn] northd: Allow DHCP request from the lport if enabled DHCPv4

2024-04-18 Thread Numan Siddique
On Tue, Apr 16, 2024 at 1:22 AM wrote: > From: shylou > > DHCP for VM fails when removing default security group rules > using a CMS like Neutron ML2/OVN [1]. This is because DHCP > requests from VMs may be dropped by ACLs. To fix this issue, > we add a lflow with a priority of 34000 to allow

Re: [ovs-dev] [PATCH ovn] northd: Allow DHCP request from the lport if enabled DHCPv4

2024-04-15 Thread 0-day Robot
Bleep bloop. Greetings 刘勰, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. checkpatch: WARNING: The subject summary should end with a dot. Subject: northd: Allow DHCP request from the lport if

[ovs-dev] [PATCH ovn] northd: Allow DHCP request from the lport if enabled DHCPv4

2024-04-15 Thread liuxie_11
From: shylou DHCP for VM fails when removing default security group rules using a CMS like Neutron ML2/OVN [1]. This is because DHCP requests from VMs may be dropped by ACLs. To fix this issue, we add a lflow with a priority of 34000 to allow DHCP requests from the logical port if the CMS has