Not following upper constraints

Bug #1965512 reported by Joel Capitao
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
oslo.cache
Fix Released
High
Takashi Kajinami

Bug Description

At RDO, when trying to build oslo.cache-2.6.3 for Victoria [1], we're hitting issue [2] because of [3].
The conditional should be "dogpile.__version__ < '1.1.4'". We have same issue in [4] and [5].

This was not caught up in CI because the requirements downloaded are not
tight to Victoria UC. As you can see in [6] (CI result of [5]), dogpile.cache 1.1.5 is downloaded (see after "py38 installed:"), whereas in UC we have 1.0.2 [7].

I think the requirements.txt file should be added as deps, alongside test-requirements.txt and uc file, in testenv section of tox.ini file [8]. But maybe there is a reason to not follow UC.

[1] https://github.com/openstack/requirements/commit/ec5a9b3157409082651ba1acb848256a81b70000
[2] https://logserver.rdoproject.org/75/40575/5/check/rdoinfo-DLRN-check-centos8/0e7a47c/buildset/centos-victoria-rdo/repos/component/common/84/b3/84b3519499b55332bfd150d537ad008b6d1c41b5_dev/rpmbuild.log
[3] https://review.opendev.org/c/openstack/oslo.cache/+/828725
[4] https://review.opendev.org/c/openstack/oslo.cache/+/829493
[5] https://review.opendev.org/c/openstack/oslo.cache/+/829494

[6] https://384c64727f31e7a7ec3c-70aee045aa856a76767e0cd0433cf359.ssl.cf2.rackcdn.com/829494/1/check/openstack-tox-py38/71d97ed/job-output.txt
[7] https://github.com/openstack/requirements/blob/ec5a9b3157409082651ba1acb848256a81b70000/upper-constraints.txt#L402
[8] https://github.com/openstack/oslo.cache/blob/d75020b33541ade07c8c9415e29992042896ef19/tox.ini#L10#L12

Revision history for this message
Joel Capitao (jcapitao) wrote :
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to oslo.cache (master)

Fix proposed to branch: master
Review: https://review.opendev.org/c/openstack/oslo.cache/+/908729

Changed in oslo.cache:
status: New → In Progress
Changed in oslo.cache:
importance: Undecided → High
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to oslo.cache (master)

Reviewed: https://review.opendev.org/c/openstack/oslo.cache/+/908729
Committed: https://opendev.org/openstack/oslo.cache/commit/5475c35117f4e18fa7561eb084fab9e14ae5dc09
Submitter: "Zuul (22348)"
Branch: master

commit 5475c35117f4e18fa7561eb084fab9e14ae5dc09
Author: Takashi Kajinami <email address hidden>
Date: Sun Feb 11 14:17:27 2024 +0900

    Ensure requirements are capped by upper constraints

    Closes-Bug: #1965512
    Change-Id: Iff6643f97346862573500d3dce706c3120bea64a

Changed in oslo.cache:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to oslo.cache (stable/2023.2)

Fix proposed to branch: stable/2023.2
Review: https://review.opendev.org/c/openstack/oslo.cache/+/908975

Changed in oslo.cache:
assignee: nobody → Takashi Kajinami (kajinamit)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to oslo.cache (stable/2023.1)

Fix proposed to branch: stable/2023.1
Review: https://review.opendev.org/c/openstack/oslo.cache/+/908976

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to oslo.cache (stable/2023.2)

Reviewed: https://review.opendev.org/c/openstack/oslo.cache/+/908975
Committed: https://opendev.org/openstack/oslo.cache/commit/b2e51643b19e1ef1aa5db4711b493f556ba6e61e
Submitter: "Zuul (22348)"
Branch: stable/2023.2

commit b2e51643b19e1ef1aa5db4711b493f556ba6e61e
Author: Takashi Kajinami <email address hidden>
Date: Sun Feb 11 14:17:27 2024 +0900

    Ensure requirements are capped by upper constraints

    Conflicts:
            tox.ini

    Closes-Bug: #1965512
    Change-Id: Iff6643f97346862573500d3dce706c3120bea64a
    (cherry picked from commit 5475c35117f4e18fa7561eb084fab9e14ae5dc09)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to oslo.cache (stable/2023.1)

Reviewed: https://review.opendev.org/c/openstack/oslo.cache/+/908976
Committed: https://opendev.org/openstack/oslo.cache/commit/d313c169d6daeca84f6c095ac5f760ba35b7259e
Submitter: "Zuul (22348)"
Branch: stable/2023.1

commit d313c169d6daeca84f6c095ac5f760ba35b7259e
Author: Takashi Kajinami <email address hidden>
Date: Sun Feb 11 14:17:27 2024 +0900

    Ensure requirements are capped by upper constraints

    Conflicts:
            tox.ini

    Closes-Bug: #1965512
    Change-Id: Iff6643f97346862573500d3dce706c3120bea64a
    (cherry picked from commit 5475c35117f4e18fa7561eb084fab9e14ae5dc09)
    (cherry picked from commit b2e51643b19e1ef1aa5db4711b493f556ba6e61e)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to oslo.cache (stable/zed)

Fix proposed to branch: stable/zed
Review: https://review.opendev.org/c/openstack/oslo.cache/+/909071

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to oslo.cache (stable/zed)

Reviewed: https://review.opendev.org/c/openstack/oslo.cache/+/909071
Committed: https://opendev.org/openstack/oslo.cache/commit/2a9c4f6ddf47a6973cd5ecb11594632063593510
Submitter: "Zuul (22348)"
Branch: stable/zed

commit 2a9c4f6ddf47a6973cd5ecb11594632063593510
Author: Takashi Kajinami <email address hidden>
Date: Sun Feb 11 14:17:27 2024 +0900

    Ensure requirements are capped by upper constraints

    Conflicts:
            tox.ini

    Closes-Bug: #1965512
    Change-Id: Iff6643f97346862573500d3dce706c3120bea64a
    (cherry picked from commit 5475c35117f4e18fa7561eb084fab9e14ae5dc09)
    (cherry picked from commit b2e51643b19e1ef1aa5db4711b493f556ba6e61e)
    (cherry picked from commit d313c169d6daeca84f6c095ac5f760ba35b7259e)

tags: added: in-stable-zed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/oslo.cache 3.7.0

This issue was fixed in the openstack/oslo.cache 3.7.0 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/oslo.cache zed-eom

This issue was fixed in the openstack/oslo.cache zed-eom release.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.