| arne_wiebalck | Good morning, Ironic! | 06:48 |
|---|---|---|
| iurygregory | good morning arne_wiebalck and Ironic o/ | 07:03 |
| arne_wiebalck | Hey iurygregoryo/ | 07:28 |
| cenne | Good morning Ironic! | 07:29 |
| cenne | Hey arne_wiebalck, iurygregory o/ | 07:29 |
| iurygregory | hey cenne o/ | 07:29 |
| arne_wiebalck | hey cenne o/ | 07:32 |
| *** rpittau|afk is now known as rpittau | 07:54 | |
| rpittau | good morning ironic! o/ | 07:54 |
| frigo | morning! | 07:54 |
| iurygregory | morning rpittau o/ | 07:54 |
| cenne | Good morning rpittau ! o/ | 07:54 |
| rpittau | hey iurygregory :) | 07:54 |
| iurygregory | welcome back after the long weekend =) | 07:54 |
| cenne | Morning frigo. | 07:54 |
| rpittau | thanks! | 07:54 |
| cenne | Has someone looked into the failing builds? Has it been fixed? | 07:56 |
| cenne | Is there something I can do to help? | 07:56 |
| cenne | Kinda have some time. | 07:57 |
| rpittau | cenne: which failing builds? | 08:05 |
| cenne | Zuul suddenly became unhappy with all the patches I was working on towards the end of last week. I saw someone else talking of the same too. | 08:06 |
| cenne | Lemme get the some links. | 08:06 |
| cenne | Ex: https://review.opendev.org/c/openstack/openstacksdk/+/802101 | 08:07 |
| cenne | Also : https://review.opendev.org/c/openstack/python-ironicclient/+/803970 | 08:09 |
| iurygregory | ERROR: Cannot uninstall 'libvirt-python'. It is a distutils installed project and thus we cannot accurately determine which files belong to it which would lead to only a partial uninstall. | 08:13 |
| iurygregory | to me this is like a problem in devstack, I asked yesterday but they only pointed to a launchpad https://bugs.launchpad.net/devstack/+bug/1896327 | 08:13 |
| iurygregory | the openstacksdk failures I have no idea... .-. | 08:14 |
| rpittau | I'll ahve a look after (ice) coffee | 08:20 |
| cenne | Just to make sure, am i allwed to send 'recheck's? | 08:22 |
| iurygregory | cenne, on openstacksdk it should be fine I would say | 08:26 |
| cenne | thanks iurygregory | 08:27 |
| cenne | thanks rpittau | 08:27 |
| rpittau | cenne, iurygregory, I could reproduce the issue locally on focal | 08:55 |
| rpittau | I think the problem comes from a conflict between the version of libvirt-python (or any other python lib) and what's in upper-constraints, as well as the version of pip | 08:55 |
| rpittau | a way to bypass that would be ignoring installed python libs (but it's not ideal as we don't respect the constrained version), or use a virtualenv | 08:55 |
| rpittau | in short that happens because we (devstack) uses pip as root, that behavior is just a safeguard | 08:55 |
| *** pmannidi is now known as pmannidi|AFK | 08:56 | |
| iurygregory | rpittau, yeah I'm aware of that hehe, not sure where we should put the fix :D | 08:58 |
| iurygregory | I'm wondering if this is not affecting other projects (no email or more people talking about it?) | 08:59 |
| rpittau | I don't see any fix possible on our side to be honest | 08:59 |
| iurygregory | yeah =) | 09:00 |
| rpittau | one way could be using --ignore-installed and completely bypass system installed libraries, kind of risky | 09:05 |
| iurygregory | maybe we should move the conversation to #openstack-qa? | 09:14 |
| rpittau | yeah | 09:15 |
| fmuyassarov | Hello. We would like DM multipathing to be enabled on the IPA image so that when it is booted and searches for the root device (specified via root device hints), it treats multiple disks (part of the same multipath group/device) as one device. Is it even possible? And does installing device-mapper-multipath on the base CentOS for IPA is enough for enabling it? | 09:49 |
| rpittau | fmuyassarov: I'm not sure that would be enough, it would definitely need to be tested | 09:58 |
| fmuyassarov | rpittau, OK, I'm gonna try it. Thanks | 10:09 |
| fmuyassarov | @rpittau, one more question. Is my assumption correct that, If a node has sda, sdb, and sdzz disks and sdzz is specified to be a root device, while searching for root device IPA will be iterating over sda, sdb, sdc...sdz, sdaa...sdaz and etc, in linear order even though some of the disks don't exist in that range? | 10:15 |
| arne_wiebalck | fmuyassarov: I don't think so. The IPA will get a list of available devices and make its pick. | 10:16 |
| arne_wiebalck | fmuyassarov: Either via hints or some heuristics. | 10:17 |
| arne_wiebalck | fmuyassarov: https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/hardware.py#L1199 | 10:19 |
| rpittau | fmuyassarov: what arne_wiebalck says :) | 10:19 |
| fmuyassarov | arne_wiebalck, rpittau thank you. Probably I've done my tests wrongly. Because I kind of saw from the IPA logs that it was checking disks that don't even exist. But I will redo my tests, because that code also tells it. | 10:22 |
| opendevreview | Aija Jauntēva proposed openstack/ironic master: Add clear foreign config to idrac-redfish https://review.opendev.org/c/openstack/ironic/+/804848 | 11:55 |
| rpittau | iurygregory, cenne I proposed https://review.opendev.org/c/openstack/requirements/+/804874 as a workaround for the libvirt-python issue, not sure how well-received will that be :) | 13:24 |
| iurygregory | rpittau, you have my support I will give +1 :D | 13:24 |
| rpittau | eheh thanks | 13:24 |
| cenne | Thanks rpittau ^^ | 13:28 |
| opendevreview | Aija Jauntēva proposed x/sushy-oem-idrac master: Fix IncludeInExport allowed value validation https://review.opendev.org/c/x/sushy-oem-idrac/+/800496 | 13:57 |
| opendevreview | Aija Jauntēva proposed x/sushy-oem-idrac master: Add RAID clear foreign config method https://review.opendev.org/c/x/sushy-oem-idrac/+/800071 | 14:31 |
| opendevreview | Merged openstack/ironic-prometheus-exporter master: Add lower-constraints job to current development branch https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/803893 | 15:35 |
| arne_wiebalck | Bye everyone o/ | 15:45 |
| rpittau | bye everyone! o/ | 16:31 |
| *** rpittau is now known as rpittau|afk | 16:31 | |
| opendevreview | Tadeas Kot proposed openstack/ironic master: [WIP] support GET /v1/drivers/?fields=... https://review.opendev.org/c/openstack/ironic/+/804416 | 17:14 |
| opendevreview | Merged openstack/metalsmith master: Use importlib instead of imp https://review.opendev.org/c/openstack/metalsmith/+/804803 | 22:25 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!