| arddennis | Hello, | 07:37 |
|---|---|---|
| arddennis | I have a trivial patch which I sent to review https://review.opendev.org/c/openstack/diskimage-builder/+/841465 | 07:37 |
| arddennis | And it does not pass all required tests. I am not sure if I have to fix erros like: | 07:37 |
| arddennis | ``` | 07:37 |
| arddennis | error: invalid command 'bdist_wheel' | 07:37 |
| arddennis | ``` | 07:37 |
| arddennis | missing package. Or incorrect requirements in ironic-python-agent-check-image-dib-centos8 | 07:37 |
| arddennis | ``` | 07:37 |
| arddennis | ERROR: Package 'ironic-python-agent' requires a different Python: 3.6.8 not in '>=3.8' | 07:37 |
| arddennis | ``` | 07:37 |
| arddennis | I'd opt for recheck, but don't know the chances to succeed. | 07:37 |
| arddennis | What is normal procedure for such cases and which steps I have to perform as I don't believe the errors are related to my change at all? | 07:37 |
| frickler | arddennis: mentioning the issue here is a good first step. I agree that this failure seems unrelated to your change. | 08:02 |
| frickler | arddennis: you can now either simply just wait for someone else to fix the issue or, if you want to take a closer look for yourself, look at the build history for that job and try to find out what might have changed https://zuul.opendev.org/t/openstack/builds?job_name=ironic-python-agent-check-image-dib-centos8&project=openstack/diskimage-builder | 08:03 |
| frickler | since this seems to be the first failure after a long row of successes, doing a recheck to see whether it is reproducible would also seem appropriate | 08:05 |
| frickler | dtantsur: TheJulia: seems fallout from https://opendev.org/openstack/ironic-python-agent/commit/c2bbcd7dc39d9a8ea8b5791192cb1c3fc8de04b4 , see also https://zuul.opendev.org/t/openstack/build/fc947c79ce5a43aa8cc25f034449feb0 | 08:13 |
| frickler | not sure why that job only runs in post and not in gate it seems | 08:13 |
| dtantsur | frickler: this job seems green on IPA itself, weird | 09:38 |
| dtantsur | ah, probably because of the source install | 09:40 |
| dtantsur | I'll propose a revert | 09:40 |
| dtantsur | OR we should switch DIB to a centos9 job | 09:40 |
| dtantsur | iurygregory, rpittau: opinions? IPA can no longer be installed on Python 3.6, are we dropping it for good or reverting? | 09:41 |
| dtantsur | in any case, I think it's safe to switch DIB to the newer job | 09:42 |
| opendevreview | Dmitry Tantsur proposed openstack/diskimage-builder master: Switch to the CentOS 9 IPA job https://review.opendev.org/c/openstack/diskimage-builder/+/841558 | 09:43 |
| dtantsur | frickler, arddennis ^^ | 09:43 |
| rpittau | let's drop it | 09:48 |
| rpittau | dtantsur: I think we're planning the same also on our side, no? | 09:48 |
| dtantsur | rpittau: okay. I'm looking into IPA-builder now unless you already have a patch? | 09:49 |
| rpittau | dtantsur: I don't, wanted to discuss first, and I guess we did :) | 09:49 |
| frickler | dtantsur: thx, the only concern that I have is that the job looks to be n-v for ipa, so I wouldn't want to have it voting for dib. do you intend to change that for the ironic projects? | 10:02 |
| dtantsur | frickler: it's voting on IPA-builder, on IPA itself it only serves as a canary | 10:02 |
| dtantsur | counter-intuitive, but it's a building job, the source code of IPA can barely break it (even in this case when the IPA change was actually breaking, it stayed green) | 10:03 |
| frickler | dtantsur: ah, ok then, thx | 10:04 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/diskimage-builder master: Adopted dkms element to work on Ubuntu Jammy and nvidia drivers https://review.opendev.org/c/openstack/diskimage-builder/+/841465 | 10:20 |
| opendevreview | Merged openstack/diskimage-builder master: Switch to the CentOS 9 IPA job https://review.opendev.org/c/openstack/diskimage-builder/+/841558 | 13:05 |
| *** prometheanfire is now known as Guest0 | 22:26 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!