Open Stack

Hey,

with patching global-requirements [0] our change now pass [1].

[0] https://review.opendev.org/#/c/682588/1
[1] https://review.opendev.org/#/c/681562/4

Maciej

On Tue, Sep 17, 2019 at 11:07 AM Maciej Jozefczyk <mjozefcz at redhat.com>
wrote:

> Hello,
>
> I'm trying to bump-up ovsdbapp requirement in networking-ovn [0] from
> 0.8.0 [1] to 0.10.4 [2]. Those two are in the same stable/queens release
> and we need that change to merge some serious performance improvements to
> stable/queens.
>
> Unfortunately the requirements-check jobs fails on this change [3] with:
>
> Requirement for package ovsdbapp : Requirement(package=u'ovsdbapp', location='', specifiers='>=0.10.4', markers=u'', comment=u'# Apache-2.0', extras=frozenset([])) does not match openstack/requirements value : set([Requirement(package='ovsdbapp', location='', specifiers='>=0.8.0', markers='', comment='# Apache-2.0', extras=frozenset([]))])
>
> The only place where >=0.8.0 is set is global-requirements [4]. Do we need
> to bump up it also there, even the upper-requirements bot proposal [5] has
> been merged? It is string match?
>
> I proposed a change to bump it in global-requirements [6].
>
> Thanks,
> Maciej
>
> [0] https://review.opendev.org/#/c/681562/
> [1] https://github.com/openstack/ovsdbapp/releases/tag/0.8.0
> [2] https://github.com/openstack/ovsdbapp/releases/tag/0.10.4
> [3]
> https://bb8048f0749367929365-38c02a6f4c2535c3f3f9bfdb5440d261.ssl.cf1.rackcdn.com/681562/3/check/requirements-check/84e1e97/job-output.txt
> [4]
> https://github.com/openstack/requirements/blob/stable/queens/global-requirements.txt#L402
> [5] https://review.opendev.org/#/c/682323
> [6] https://review.opendev.org/#/c/682588
>
> --
> Best regards,
> Maciej Józefczyk
>


-- 
Best regards,
Maciej Józefczyk
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-discuss/attachments/20190917/74c97b24/attachment.html>