<div dir="ltr">Thanks again. <div><br><div>This kombu >=3.0.7 requirement is added in commit 5b9fb6980220dbfa18bac4c3231d57efb493ebf0, which is from a Bot with no reason.</div><div><br></div><div> As i see, we are directly requiring amqp >=1.4.0 in requirement.txt from commit 0c954cffa2f3710acafa79f01b958a8955823640 on.</div></div><div>So maybe there is no need to require kombu >= 3.0.7 ?</div><div><br></div><div>Another reason I am asking about this, in the latest centos 7 epel repo, the versions of these two packages are :</div><div>python-kombu : 2.5.16-1.el7</div><div>python-amqp : 1.4.5-1.el7</div><div><br></div><div>I have not test whether there are problems about heartbeat with this version pair though.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Oct 27, 2015 at 11:02 PM, Mehdi Abaakouk <span dir="ltr"><<a href="mailto:sileht@sileht.net" target="_blank">sileht@sileht.net</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">
[1] seems just socket timeout issue, and admins can adjust those kernel<br>
params themselves.<br>
</blockquote>
<br></span>
Yes, but if you trick kernel settings, like putting very low tcp keepalive values, you don't need to have/enable heartbeat.<span class=""><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
[2] and [3] truly a problem about the heartbeat implementation, but it says<br>
the fix is a part of py-amqp 1.4.0, and the dependency with kombu was not<br>
specified.<br>
[4] is a bug of kombu's autoretry method which is said to be fixed in kombu<br>
3.0.0, it is not directly related to heartbeat.<br>
</blockquote>
<br></span>
As far as I can remember, this is because oslo.messaging doesn't really require py-amqp but only kombu, so to ensure kombu depends of py-amqp 1.4.0 we have to depends on kombu 3.0.7 (that have amqp>=1.4.0 in its requirements I guess).<div class="HOEnZb"><div class="h5"><br>
<br>
---<br>
Mehdi Abaakouk<br>
mail: <a href="mailto:sileht@sileht.net" target="_blank">sileht@sileht.net</a><br>
irc: sileht<br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr">Regards,<div>apporc</div></div></div>
</div>