<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>