opendevreview | Vanou Ishii proposed openstack/ironic master: Fix failure of create_vfat_image function https://review.opendev.org/c/openstack/ironic/+/826576 | 01:15 |
---|---|---|
vanou | hi o/ | 01:21 |
arne_wiebalck | Good morning, Ironic! | 07:24 |
opendevreview | Jacob Anders proposed openstack/sushy master: Re-trying InsertMedia call with TransferProtocolType if required https://review.opendev.org/c/openstack/sushy/+/830322 | 07:31 |
janders | hey arne_wiebalck o/ | 07:32 |
rpittau | good morning ironic! o/ | 07:32 |
janders | hey rpittau o/ | 07:32 |
rpittau | hey janders :) | 07:33 |
arne_wiebalck | hey janders and rpittau o/ | 07:35 |
rpittau | hey arne_wiebalck :) | 07:35 |
dtantsur | morning folks | 07:43 |
rpittau | hey dtantsur :) | 07:44 |
arne_wiebalck | morning dtantsur o/ | 07:44 |
janders | good morning dtantsur o/ | 07:52 |
opendevreview | Merged openstack/ironic master: [Trivial] Fix formatting in troubleshooting docs https://review.opendev.org/c/openstack/ironic/+/831348 | 08:33 |
opendevreview | Merged openstack/ironic master: Troubleshooting guide: node locked error https://review.opendev.org/c/openstack/ironic/+/831351 | 08:34 |
opendevreview | Radosław Piliszek proposed openstack/tenks master: Add Debian support https://review.opendev.org/c/openstack/tenks/+/830675 | 09:28 |
opendevreview | Roman Popelka proposed openstack/ironic-tempest-plugin master: Ironic scenario manager cleanup https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/831160 | 09:30 |
opendevreview | Merged openstack/sushy master: Follow up to Attempt using Redfish settings resource to get URI https://review.opendev.org/c/openstack/sushy/+/831355 | 09:30 |
iurygregory | good morning Ironic | 09:54 |
* iurygregory goes and grabs more coffee to stay awake =) | 09:56 | |
opendevreview | Roman Popelka proposed openstack/ironic-tempest-plugin master: Ironic scenario manager cleanup https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/831159 | 09:57 |
opendevreview | Roman Popelka proposed openstack/ironic-tempest-plugin master: Ironic scenario manager cleanup https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/831160 | 10:03 |
opendevreview | Roman Popelka proposed openstack/ironic-tempest-plugin master: Ironic scenario manager cleanup https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/831158 | 10:18 |
opendevreview | Roman Popelka proposed openstack/ironic-tempest-plugin master: Ironic scenario manager cleanup https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/831159 | 10:20 |
opendevreview | Roman Popelka proposed openstack/ironic-tempest-plugin master: Ironic scenario manager cleanup https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/831160 | 10:20 |
janders | hey iurygregory o/ | 10:32 |
rpittau | whooo almost burned the lasagna, not good | 11:04 |
arne_wiebalck | please don't! | 11:05 |
rpittau | I blame ansible :) | 11:06 |
iurygregory | ansible-playbook -vvvv -i inventory/food.py lasagna.yaml | 11:11 |
opendevreview | Merged openstack/tenks master: [CI] Stop overriding default python https://review.opendev.org/c/openstack/tenks/+/831434 | 11:14 |
zigo | Hi there! I'm having troubles building Ironic backport for Bullseye (building in Debian Unstable works without any issue). Here's the build log: | 11:45 |
zigo | https://paste.opendev.org/show/bgirNHUGSZtfaXwkxDca/ | 11:45 |
zigo | What component needs an update?!? | 11:45 |
zigo | There's 449 error like this one about sushy ... | 11:46 |
zigo | (pip freeze output in the paste.o.o) | 11:46 |
dtantsur | zigo: sushy version mismatch? | 11:46 |
zigo | dtantsur: It has 4.1.0. | 11:46 |
dtantsur | can you try importing sushy isolated, just in a python interpreter? | 11:47 |
dtantsur | maybe a broken transient dependency? | 11:47 |
zigo | For in Unstable, it was sushy 3.12.0, so probably my 4.1.0 version is broken indeed, checking if I can import it. | 11:47 |
zigo | Oh ... | 11:48 |
zigo | https://paste.opendev.org/show/bmuy8bwnvoBmonz4wDab/ | 11:48 |
opendevreview | Merged openstack/tenks master: Add Debian support https://review.opendev.org/c/openstack/tenks/+/830675 | 11:48 |
zigo | :( | 11:48 |
zigo | I wonder how this happened. | 11:48 |
zigo | Oh, I know !!! | 11:49 |
zigo | sushy/resources/system/network/ has no __init__.py !!! | 11:50 |
zigo | So it's IMO an upstream issue that got to be fixed. | 11:50 |
zigo | dtantsur: ^' | 11:50 |
iurygregory | zigo, can you push a patch? | 11:50 |
zigo | Sure, will do. | 11:50 |
iurygregory | we will be happy to merge and push a new release | 11:50 |
dtantsur | oops | 11:50 |
dtantsur | I suspect it works in newer pythons | 11:50 |
iurygregory | python *magic* | 11:51 |
zigo | The issue is that setuptools will *not* package the folder at all in Debian is the __init__.py is missing. | 11:51 |
zigo | It may work with pip though... | 11:52 |
dtantsur | interesting | 11:52 |
opendevreview | Thomas Goirand proposed openstack/sushy master: Add missing __init__.py https://review.opendev.org/c/openstack/sushy/+/831515 | 11:52 |
zigo | That's unless MANIFEST.in has a recursive include thingy... | 11:53 |
dtantsur | we have that generated by pbr, I think | 11:53 |
zigo | pbr is stupid regarding this, and it's been since it exists that it's annoying me about it. | 11:53 |
zigo | Most Debian packages for services have a: | 11:54 |
zigo | recursive-include ironic * | 11:54 |
zigo | patch ... | 11:54 |
zigo | Otherwise, the alembic migration alembic.ini is missing from the package for example. | 11:54 |
opendevreview | Jacob Anders proposed openstack/sushy master: Re-trying InsertMedia call with TransferProtocolType if required https://review.opendev.org/c/openstack/sushy/+/830322 | 12:19 |
zigo | Now I'm getting: AttributeError: None does not have the attribute 'IloClient' | 12:51 |
zigo | What lib is that?!? | 12:51 |
zigo | Oh, proliantutils, no ? | 12:52 |
zigo | I'm using 2.12.0... | 12:55 |
zigo | https://paste.opendev.org/show/bryz3BRTDjGa3uWfgNLK/ | 12:56 |
zigo | Ironic pretends >= 2.11.0 is ok, is there something wrong? | 12:56 |
zigo | iurygregory: dtantsur: ^ | 12:56 |
iurygregory | huh O.o | 12:58 |
zigo | iurygregory: Do you know what version of proliantutils the gate is using? | 13:01 |
zigo | 2.13.0 maybe ? | 13:01 |
zigo | I can upgrade to it if needed ... | 13:02 |
rpittau | zigo: that looks like a problem with mock, proliantutils version seems correct | 13:02 |
zigo | ok | 13:02 |
iurygregory | I can check in a few, a lot of meetings atm =( | 13:02 |
rpittau | zigo: you said debian bullseye ? | 13:02 |
zigo | rpittau: Yeah, my usual backport repository... | 13:02 |
zigo | It builds fine in unstable. | 13:02 |
zigo | So I'm double-guessing some libs may need an update, but I can't figure out which one. | 13:03 |
rpittau | ok, give me a minute to bring up a vm and I'll have a look | 13:05 |
dtantsur | podman run -it debian:bullseye ;) | 13:06 |
zigo | Then thing that is breaking should be: | 13:06 |
zigo | if not mock._is_instance_mock(ilo_client): | 13:06 |
zigo | mock.patch.object(ilo_client, 'IloClient', autospec=True).start() | 13:06 |
rpittau | or yeah, a container :) | 13:07 |
zigo | rpittau: If you want the same setup as me, you can use this repo: | 13:08 |
zigo | deb http://localhost/debian bullseye-yoga-backports main | 13:08 |
zigo | deb-src http://localhost/debian bullseye-yoga-backports main | 13:08 |
zigo | deb http://localhost/debian bullseye-yoga-backports-nochange main | 13:08 |
zigo | deb-src http://localhost/debian bullseye-yoga-backports-nochange main | 13:08 |
zigo | the repo key is: http://bullseye-yoga.debian.net/debian/dists/pubkey.gpg | 13:08 |
zigo | then apt-get build-dep ironic | 13:08 |
zigo | (without any venv...) | 13:08 |
rpittau | ack | 13:08 |
zigo | Woops, not localhost, but bullseye-yoga.debian.net sorry. | 13:08 |
zigo | Thanks a lot. | 13:09 |
rpittau | zigo: running "apt-get build-dep ironic" does not throw any error for me | 13:24 |
zigo | rpittau: Yeah, though you need to run the unit tests of Ironic using what you downloaded... | 13:58 |
rpittau | zigo: I'm running debuild now | 13:58 |
zigo | rpittau: Using the latest git? | 13:59 |
zigo | https://salsa.debian.org/openstack-team/services/ironic | 13:59 |
saop | @TheJulia Hi | 14:00 |
TheJulia | Good morning | 14:03 |
saop | TheJulia: Hi, HPE Ironic CI is planning to shift testing to a containerised version of ironic using openStack Kolla project, is that okay with upstream? | 14:03 |
TheJulia | I don't see any issue with that | 14:04 |
TheJulia | iurygregory: FYI ^^ | 14:04 |
saop | TheJulia: Thanks. | 14:04 |
iurygregory | ack | 14:06 |
iurygregory | I also don't see a problem with it =) | 14:06 |
saop | iurygregory: Thanks for the info | 14:07 |
TheJulia | I've become too predictable | 14:09 |
iurygregory | why? .-. | 14:10 |
TheJulia | Morning greeting right at 6 AM :) | 14:11 |
iurygregory | wow | 14:22 |
dtantsur | heh, morning TheJulia | 14:58 |
opendevreview | Riccardo Pittau proposed openstack/ironic master: Update minimum requirements for ilo drivers https://review.opendev.org/c/openstack/ironic/+/831545 | 15:04 |
rpittau | zigo ^ | 15:04 |
rpittau | if anyone has a minute please check https://review.opendev.org/c/openstack/ironic-python-agent/+/830264 thanks! | 15:06 |
zigo | +1 | 15:08 |
iurygregory | rpittau, looking now | 15:11 |
rpittau | thanks iurygregory | 15:12 |
iurygregory | maybe we need some unit tests? .-. | 15:12 |
TheJulia | Eh, I'm thinking that might just be okay | 15:21 |
TheJulia | Then again, I know of at least one vendor that was shipping ascii only csv files for a while | 15:22 |
TheJulia | but le makes sence from a multiarch standpoint | 15:22 |
TheJulia | and... yeah | 15:22 |
TheJulia | I'm going to workflow it in 5 minutes if there are no objections | 15:23 |
iurygregory | yeah | 15:26 |
iurygregory | I don't have objections, and it looks pretty straight forward | 15:26 |
TheJulia | iurygregory: thanks | 15:33 |
rpittau | thanks iurygregory and TheJulia :) | 15:44 |
opendevreview | Merged openstack/sushy master: Add missing __init__.py https://review.opendev.org/c/openstack/sushy/+/831515 | 16:03 |
opendevreview | Merged openstack/sushy master: Re-trying InsertMedia call with TransferProtocolType if required https://review.opendev.org/c/openstack/sushy/+/830322 | 16:46 |
opendevreview | Merged openstack/ironic-python-agent master: Use utf-16-le if BOM not present https://review.opendev.org/c/openstack/ironic-python-agent/+/830264 | 16:55 |
rpittau | good night! o/ | 17:15 |
TheJulia | iurygregory: do we have a ptg etherpad yet? | 17:20 |
iurygregory | TheJulia, I was about to create it and send an email to collect topics =) | 17:22 |
TheJulia | excellent | 17:22 |
arne_wiebalck | bye everyone o/ | 18:01 |
dtantsur | o/ | 18:43 |
admiyo | OCI runtime create failed: container_linux.go:380: starting container process caused: process_linux.go:545: container init caused: rootfs_linux.go:76: mounting "/run/openvswitch" to rootfs at "/run/openvswitch" caused: mount through procfd: no space left on device: unknown | 18:56 |
admiyo | the joys of alternate platforms | 18:56 |
TheJulia | admiyo: oh my | 19:06 |
admiyo | TheJulia, its the toolbox container and I have no idea why it is unhappy, but I suspect hardward | 19:09 |
admiyo | hardware. Like, df -h is taking an real long time to return, but then there is no real shortage of space anywhere | 19:10 |
TheJulia | admiyo: any errors in dmesg output? | 19:10 |
admiyo | nope | 19:11 |
admiyo | its an 80 Core system, in two chips, and I wonder if something is just getting overwhelmed | 19:11 |
TheJulia | thermal throttling on core 0? | 19:12 |
admiyo | it feels along those lines, yea | 19:12 |
admiyo | Um...I think I just found a smoking gun | 19:12 |
TheJulia | are interrupts being spaced across cores? | 19:12 |
admiyo | I just ran mount and it has about 1000+ lines of tmpfs on /run/openvswitch type tmpfs (rw,nosuid,nodev,noexec,relatime,size=26301852k,mode=755) | 19:13 |
* TheJulia is intruiged | 19:13 | |
admiyo | Just trying to run kolla all in one here | 19:13 |
* TheJulia wonders why kolla sets that up | 19:13 | |
admiyo | $ mount | wc -l | 19:14 |
admiyo | 32813 | 19:14 |
TheJulia | oh... wow | 19:14 |
admiyo | something got into a loop....feels like I left the dog unattended with food on the table. | 19:15 |
TheJulia | That feels like https://www.youtube.com/watch?v=t9wmWZbr_wQ | 19:15 |
admiyo | ok, umount cleaned that up | 19:16 |
admiyo | running destroy and then deploy and lets see what breaks next. | 19:17 |
admiyo | Yep, same thing...time to go bug someone in the kolla channnel | 19:18 |
stevebaker[m] | good morning! | 19:36 |
TheJulia | good morning stevebaker[m] | 19:47 |
TheJulia | erbarr: is the log publishing url link broken? | 21:11 |
erbarr | yea, I'm setting up the log server | 21:13 |
erbarr | only have an internal one right now | 21:14 |
erbarr | on https://review.opendev.org/c/openstack/ironic/+/830275 I'm seeing a tempest issue, at first devstack was failing to to install_tempest because it does a "git checkout ..." but that was failing due to tempest checkout having local changes that idk where they come from, zuul I guess. So I changed jobs to delete the tempest folder injected by zuul before starting the job but that leads to plugin being installed and | 21:24 |
erbarr | uninstalled during stacking so regex to run test is not matched | 21:24 |
TheJulia | erbarr: good to know, sounds weird about the clone is blowing up :\ | 21:43 |
opendevreview | Merged openstack/ironic master: Fix failure of create_vfat_image function https://review.opendev.org/c/openstack/ironic/+/826576 | 22:59 |
opendevreview | Merged openstack/ironic master: Update minimum requirements for ilo drivers https://review.opendev.org/c/openstack/ironic/+/831545 | 23:04 |
opendevreview | Merged openstack/ironic master: Add idrac-wsman clean steps to not require ramdisk https://review.opendev.org/c/openstack/ironic/+/825489 | 23:04 |
opendevreview | Merged openstack/ironic master: Fix prepare ramdisk for 'wait' states https://review.opendev.org/c/openstack/ironic/+/823311 | 23:04 |
opendevreview | Merged openstack/ironic master: Fix Redfish RAID for non-immediate controllers https://review.opendev.org/c/openstack/ironic/+/823312 | 23:24 |
opendevreview | Ruby Loo proposed openstack/ironic master: Minor updates to rbac doc https://review.opendev.org/c/openstack/ironic/+/831615 | 23:40 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!