<div dir="ltr">Sorry, that I don't see this earlier. Yes, cookies have integer values, so we won't be able to set string there. May be we can have a reserved integer cookie value for a project like all "1".<div><br></div><div>I won't support idea of modifying cleanup logic not to drop 0x0 cookies. During implementation of graceful restart it was not dropped at first, but I get rid of it as having a lot of flows not related to anything was not desirable, so we should try to avoid it here, too.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Dec 16, 2015 at 7:46 AM, Soichi Shigeta <span dir="ltr"><<a href="mailto:shigeta.soichi@jp.fujitsu.com" target="_blank">shigeta.soichi@jp.fujitsu.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
o) An idea to fix:<br>
<br>
1. Set "taas" stamp(*) to taas flows.<br>
2. Modify the cleanup logic in ovs-agent not to delete entries<br>
stamped as "taas".<br>
<br>
* Maybe a static string.<br>
If we need to use a string which generated dynamically<br>
(e.g. uuid), API to interact with ovs-agent is required.<br>
<br>
</blockquote>
<br></span>
Last week I proposed to set a static string (e.g. "taas") as cookie<br>
of flows created by taas agent.<br>
<br>
But I found that the value of a cookie should not be a string,<br>
but an integer.<br>
<br>
At line 187 in "neutron/neutron/plugins/ml2/drivers/openvswitch/agent/ovs_neutron_agent.py":<br>
self.agent_uuid_stamp = uuid.uuid4().int & UINT64_BITMASK<br>
<br>
In case of we set an integer value to cookie, coordination<br>
(reservation of range) is required to avoid conflict of cookies with<br>
other neutron sub-projects.<br>
<br>
As an alternative (*** short term ***) solution, my idea is:<br>
Modify the clean up logic in ovs agent not to delete flows whose<br>
"cookie = 0x0".<br>
Because old flows created by ovs agent have an old stamp, "cookie =<br>
0x0" means it was created by other than ovs agent.<br>
<br>
# But, this idea has a disadvantage:<br>
If there are flows which have been created by older version of ovs<br>
agent, they can not be cleaned.<div class="HOEnZb"><div class="h5"><br>
<br>
---<br>
Soichi Shigeta<br>
<br>
<br>
<br>
<br>
__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr">Regards,<div>Ann Kamyshnikova</div><div>Mirantis, Inc</div></div></div>
</div>