<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<title></title>
</head>
<body>
<div name="messageBodySection">
<div dir="auto">Thanks Matthew,
<div dir="auto"><br /></div>
<div dir="auto">For now we did this: <a href="https://review.opendev.org/#/c/685932/">https://review.opendev.org/#/c/685932/</a>. So we just added “kombu” explicitly into our dependencies that forces to load the right version of amqp before oslo.messaging. That works. If that looks OK for you we can skip the mentioned bump.</div>
</div>
</div>
<div name="messageSignatureSection"><br />
<div class="matchFont"><br />
<br />
Renat Akhmerov<br />
@Nokia</div>
</div>
<div name="messageReplySection">On 2 Oct 2019, 23:35 +0700, Matthew Thode <mthode@mthode.org>, wrote:<br />
<blockquote type="cite" class="spark_quote" style="margin: 5px 5px; padding-left: 10px; border-left: thin solid #1abc9c;">On 19-10-02 14:57:24, Renat Akhmerov wrote:<br />
<blockquote type="cite" class="spark_quote" style="margin: 5px 5px; padding-left: 10px; border-left: thin solid #e67e22;">Hi,<br />
<br />
We have a failing “docs” ([1]) CI job that fails because it implicitly brings amqp 2.5.2 but this lib is not allowed to be higher than 2.5.1 in the upper-constraings.txt in the requirements project ([2]). We see that there’s the patch [3] generated by the proposal bot that bumps the constraint to 2.5.2 for amqp (among others) but it was given -2.<br />
<br />
Please assist on how to address in the best way. Should we bump only amqp version in upper constraints for now?<br />
<br />
[1] https://zuul.opendev.org/t/openstack/build/6fe7c7d3e60b40458d2a98f3a293f412/log/job-output.txt#840<br />
[2] https://github.com/openstack/requirements/blob/master/upper-constraints.txt#L258<br />
[3] https://review.opendev.org/#/c/681382<br />
<br /></blockquote>
<br />
I'm going to be treating this as a FFE request to bump amqp from 2.5.1<br />
to 2.5.2.<br />
It looks like a bugfix only release so I'm fine with it. As long as we<br />
don't need to mask 2.5.1 in global-requirements (which would cause a<br />
re-release for openstack/oslo.messaging).<br />
<br />
https://github.com/celery/py-amqp/compare/2.5.1...2.5.2<br />
<br />
So, if you propose a constraints only bump of amqp-2.5.1 to 2.5.2 then I<br />
approve.<br />
<br />
--<br />
Matthew Thode<br /></blockquote>
</div>
</body>
</html>