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@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.rackcd... [4] https://github.com/openstack/requirements/blob/stable/queens/global-requirem... [5] https://review.opendev.org/#/c/682323 [6] https://review.opendev.org/#/c/682588
-- Best regards, Maciej Józefczyk