[nova] openstack-tox-pep8 job broken, hold your rechecks
Hey all, Sending this to the ML to try to help others who may not have heard about it yet. I didn't know about myself until I saw the job fail on a random nova patch and I looked around to figure out why. It looks like openstack-tox-pep8 is failing 100% in nova due to a newer version of the mypy library being pulled in since a recent upper-constraints bump: https://review.opendev.org/c/openstack/requirements/+/872065 And the job isn't going to pass until the following fix merges: https://review.opendev.org/c/openstack/nova/+/878693 Finally, to help prevent a breakage for this in the future, a cross-nova-pep8 job has been approved: https://review.opendev.org/c/openstack/requirements/+/878748 and will be on its way to the gate once the aforementioned fix merges. I'll post a reply to this email when it's OK to do rechecks again. Cheers, -melwitt
Heh, you missed my email ;-) https://lists.openstack.org/pipermail/openstack-discuss/2023-March/032979.ht... No worries tho :) Le mar. 28 mars 2023 à 23:11, melanie witt <melwittt@gmail.com> a écrit :
Hey all,
Sending this to the ML to try to help others who may not have heard about it yet. I didn't know about myself until I saw the job fail on a random nova patch and I looked around to figure out why.
It looks like openstack-tox-pep8 is failing 100% in nova due to a newer version of the mypy library being pulled in since a recent upper-constraints bump:
https://review.opendev.org/c/openstack/requirements/+/872065
And the job isn't going to pass until the following fix merges:
https://review.opendev.org/c/openstack/nova/+/878693
Finally, to help prevent a breakage for this in the future, a cross-nova-pep8 job has been approved:
https://review.opendev.org/c/openstack/requirements/+/878748
and will be on its way to the gate once the aforementioned fix merges.
I'll post a reply to this email when it's OK to do rechecks again.
Cheers, -melwitt
Fix is merged, everyone can recheck (with a written reason ;) ) Le mer. 29 mars 2023 à 11:49, Sylvain Bauza <sbauza@redhat.com> a écrit :
Heh, you missed my email ;-)
https://lists.openstack.org/pipermail/openstack-discuss/2023-March/032979.ht...
No worries tho :)
Le mar. 28 mars 2023 à 23:11, melanie witt <melwittt@gmail.com> a écrit :
Hey all,
Sending this to the ML to try to help others who may not have heard about it yet. I didn't know about myself until I saw the job fail on a random nova patch and I looked around to figure out why.
It looks like openstack-tox-pep8 is failing 100% in nova due to a newer version of the mypy library being pulled in since a recent upper-constraints bump:
https://review.opendev.org/c/openstack/requirements/+/872065
And the job isn't going to pass until the following fix merges:
https://review.opendev.org/c/openstack/nova/+/878693
Finally, to help prevent a breakage for this in the future, a cross-nova-pep8 job has been approved:
https://review.opendev.org/c/openstack/requirements/+/878748
and will be on its way to the gate once the aforementioned fix merges.
I'll post a reply to this email when it's OK to do rechecks again.
Cheers, -melwitt
On 03/29/23 02:49, Sylvain Bauza wrote:
Heh, you missed my email ;-)
https://lists.openstack.org/pipermail/openstack-discuss/2023-March/032979.ht... <https://lists.openstack.org/pipermail/openstack-discuss/2023-March/032979.html>
No worries tho :)
Ugh, sorry. I searched for "[nova]" and skimmed for the words "broken" or "gate" or "CI" and managed to miss it accordingly. Sorry about that.
Le mar. 28 mars 2023 à 23:11, melanie witt <melwittt@gmail.com <mailto:melwittt@gmail.com>> a écrit :
Hey all,
Sending this to the ML to try to help others who may not have heard about it yet. I didn't know about myself until I saw the job fail on a random nova patch and I looked around to figure out why.
It looks like openstack-tox-pep8 is failing 100% in nova due to a newer version of the mypy library being pulled in since a recent upper-constraints bump:
https://review.opendev.org/c/openstack/requirements/+/872065 <https://review.opendev.org/c/openstack/requirements/+/872065>
And the job isn't going to pass until the following fix merges:
https://review.opendev.org/c/openstack/nova/+/878693 <https://review.opendev.org/c/openstack/nova/+/878693>
Finally, to help prevent a breakage for this in the future, a cross-nova-pep8 job has been approved:
https://review.opendev.org/c/openstack/requirements/+/878748 <https://review.opendev.org/c/openstack/requirements/+/878748>
and will be on its way to the gate once the aforementioned fix merges.
I'll post a reply to this email when it's OK to do rechecks again.
Cheers, -melwitt
participants (2)
-
melanie witt
-
Sylvain Bauza