Re: [openstack-dev] [neutron][dvr] DVR L2 agent is removing the br-int OVS flows

2015-08-21 Thread Anna Kamyshnikova
I pushed change for that case https://review.openstack.org/215596.

On Fri, Aug 21, 2015 at 2:45 PM, Anna Kamyshnikova <
akamyshnik...@mirantis.com> wrote:

> Hi, Artur!
>
> Thanks, for bringing this up! I missed that. I push change for that in a
> short time.
>
> On Fri, Aug 21, 2015 at 1:35 PM, Korzeniewski, Artur <
> artur.korzeniew...@intel.com> wrote:
>
>> Hi all,
>>
>> After merging the “Graceful ovs-agent restart”[1] (great work BTW!), I’m
>> seeing in DVR L2 agent code place where flows on br-int is removed in old
>> style:
>>
>>
>>
>> File
>> /neutron/plugins/ml2/drivers/openvswitch/agent/ovs_dvr_neutron_agent.py
>>
>> 200 def setup_dvr_flows_on_integ_br(self):
>>
>> 201 '''Setup up initial dvr flows into br-int'''
>>
>> 202 if not self.in_distributed_mode():
>>
>> 203 return
>>
>> 204
>>
>> 205 LOG.info(_LI("L2 Agent operating in DVR Mode with MAC %s"),
>>
>> 206  self.dvr_mac_address)
>>
>> 207 # Remove existing flows in integration bridge
>>
>> 208 self.int_br.delete_flows()
>>
>>
>>
>> This is kind of bummer when seeing the effort to preserve the flows in
>> [1].
>>
>> This should not affect VM network access, since the br-tun is configured
>> properly and br-int is in learning mode.
>>
>>
>>
>> Should this be fixed in Liberty cycle?
>>
>>
>>
>> This is something similar to submitted bug:
>> https://bugs.launchpad.net/neutron/+bug/1436156
>>
>>
>>
>> [1] https://bugs.launchpad.net/neutron/+bug/1383674
>>
>>
>>
>> Regards,
>>
>> Artur Korzeniewski
>>
>> 
>>
>> Intel Technology Poland sp. z o.o.
>>
>> KRS 101882
>>
>> ul. Slowackiego 173, 80-298 Gdansk
>>
>>
>>
>> __
>> OpenStack Development Mailing List (not for usage questions)
>> Unsubscribe:
>> openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
>> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>>
>>
>
>
> --
> Regards,
> Ann Kamyshnikova
> Mirantis, Inc
>



-- 
Regards,
Ann Kamyshnikova
Mirantis, Inc
__
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev


Re: [openstack-dev] [neutron][dvr] DVR L2 agent is removing the br-int OVS flows

2015-08-21 Thread Anna Kamyshnikova
Hi, Artur!

Thanks, for bringing this up! I missed that. I push change for that in a
short time.

On Fri, Aug 21, 2015 at 1:35 PM, Korzeniewski, Artur <
artur.korzeniew...@intel.com> wrote:

> Hi all,
>
> After merging the “Graceful ovs-agent restart”[1] (great work BTW!), I’m
> seeing in DVR L2 agent code place where flows on br-int is removed in old
> style:
>
>
>
> File
> /neutron/plugins/ml2/drivers/openvswitch/agent/ovs_dvr_neutron_agent.py
>
> 200 def setup_dvr_flows_on_integ_br(self):
>
> 201 '''Setup up initial dvr flows into br-int'''
>
> 202 if not self.in_distributed_mode():
>
> 203 return
>
> 204
>
> 205 LOG.info(_LI("L2 Agent operating in DVR Mode with MAC %s"),
>
> 206  self.dvr_mac_address)
>
> 207 # Remove existing flows in integration bridge
>
> 208 self.int_br.delete_flows()
>
>
>
> This is kind of bummer when seeing the effort to preserve the flows in [1].
>
> This should not affect VM network access, since the br-tun is configured
> properly and br-int is in learning mode.
>
>
>
> Should this be fixed in Liberty cycle?
>
>
>
> This is something similar to submitted bug:
> https://bugs.launchpad.net/neutron/+bug/1436156
>
>
>
> [1] https://bugs.launchpad.net/neutron/+bug/1383674
>
>
>
> Regards,
>
> Artur Korzeniewski
>
> 
>
> Intel Technology Poland sp. z o.o.
>
> KRS 101882
>
> ul. Slowackiego 173, 80-298 Gdansk
>
>
>
> __
> OpenStack Development Mailing List (not for usage questions)
> Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>
>


-- 
Regards,
Ann Kamyshnikova
Mirantis, Inc
__
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev


[openstack-dev] [neutron][dvr] DVR L2 agent is removing the br-int OVS flows

2015-08-21 Thread Korzeniewski, Artur
Hi all,
After merging the "Graceful ovs-agent restart"[1] (great work BTW!), I'm seeing 
in DVR L2 agent code place where flows on br-int is removed in old style:

File /neutron/plugins/ml2/drivers/openvswitch/agent/ovs_dvr_neutron_agent.py
200 def setup_dvr_flows_on_integ_br(self):
201 '''Setup up initial dvr flows into br-int'''
202 if not self.in_distributed_mode():
203 return
204
205 LOG.info(_LI("L2 Agent operating in DVR Mode with MAC %s"),
206  self.dvr_mac_address)
207 # Remove existing flows in integration bridge
208 self.int_br.delete_flows()

This is kind of bummer when seeing the effort to preserve the flows in [1].
This should not affect VM network access, since the br-tun is configured 
properly and br-int is in learning mode.

Should this be fixed in Liberty cycle?

This is something similar to submitted bug: 
https://bugs.launchpad.net/neutron/+bug/1436156

[1] https://bugs.launchpad.net/neutron/+bug/1383674

Regards,
Artur Korzeniewski

Intel Technology Poland sp. z o.o.
KRS 101882
ul. Slowackiego 173, 80-298 Gdansk

__
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev