| *** pmannidi|AFK is now known as pmannidi | 00:01 | |
| *** pmannidi is now known as pmannidi|brb | 00:38 | |
| *** pmannidi|brb is now known as pmannidi | 01:58 | |
| *** pmannidi is now known as pmannidi|Lunch | 03:14 | |
| *** pmannidi|Lunch is now known as pmannidi | 04:24 | |
| arne_wiebalck | Goof morning janders and Ironic! | 06:08 |
|---|---|---|
| arne_wiebalck | even Good morning | 06:08 |
| arne_wiebalck | even Good morning | 06:08 |
| * arne_wiebalck thinks the morning starts well | 06:09 | |
| iurygregory | good morning arne_wiebalck and Ironic o/ | 06:25 |
| * iurygregory is still trying to wake up | 06:26 | |
| arne_wiebalck | hey iurygregory o/ | 06:30 |
| iurygregory | Today we have Bare Metal SIG \o/ | 06:32 |
| iurygregory | arne_wiebalck, if you have some time for a quick review https://review.opendev.org/c/openstack/ironic-inspector/+/807394 this is just to save some CI resources in the ironic gate when we only update unit tests (so we don't need to run inspector jobs) | 07:11 |
| arne_wiebalck | iurygregory: done! | 07:20 |
| iurygregory | arne_wiebalck, tks! :D | 07:20 |
| dtantsur | morning ironic | 07:52 |
| janders | good morning arne_wiebalck iurygregory dtantsur and Ironic o/ | 07:57 |
| iurygregory | morning dtantsur janders o/ | 07:57 |
| arne_wiebalck | hey janders o/ | 08:27 |
| * dtantsur -> jogging, brb | 08:51 | |
| opendevreview | Verification of a change to openstack/ironic-inspector master failed: Include ironic/tests in irrelevant-files https://review.opendev.org/c/openstack/ironic-inspector/+/807394 | 08:59 |
| opendevreview | Tadeas Kot proposed openstack/ironic-inspector master: Add support for state selector in the list introspection https://review.opendev.org/c/openstack/ironic-inspector/+/807578 | 09:04 |
| janders | see you in a bit Ironic o/ | 11:33 |
| iurygregory | bye janders, enjoy the PTO | 11:33 |
| janders | starting my PTO tomorrow, pfizer-2 after that, should be back not later than the 20th of September | 11:33 |
| janders | (possibly earlier) | 11:33 |
| janders | thank you iurygregory | 11:33 |
| janders | will watch those pesky crocs | 11:34 |
| dtantsur | :D | 11:36 |
| dtantsur | have fun! | 11:36 |
| janders | thank you dtantsur! :) | 11:38 |
| arne_wiebalck | dtantsur: patches to https://github.com/metal3-io/ironic-hardware-inventory-recorder-image are done via github PRs? | 11:48 |
| iurygregory | arne_wiebalck, correct | 11:48 |
| dtantsur | arne_wiebalck: yep. I'm also not sure this image is used by anyone. | 11:48 |
| arne_wiebalck | well :-D | 11:48 |
| arne_wiebalck | not in its current state | 11:49 |
| dtantsur | yeah. so if you want to use it, you may end up on the hook to maintain it ;) | 11:49 |
| arne_wiebalck | heh | 11:54 |
| arne_wiebalck | will I also be called the expert ? | 11:55 |
| iurygregory | arne_wiebalck, yes! | 11:56 |
| * arne_wiebalck got a fright from dtantsur joining the meeting early | 12:03 | |
| dtantsur | yeah, I'm too dump to remember UTC vs my time apparently :( | 12:03 |
| dtantsur | oh, and too dumb to write dumb correctly | 12:04 |
| * arne_wiebalck gets scared every time "has joined your meeting" pops up :-D | 12:05 | |
| dtantsur | sorry | 12:05 |
| arne_wiebalck | no worries :) | 12:08 |
| arne_wiebalck | sent a PR to the above repo now | 12:08 |
| opendevreview | Merged openstack/ironic-inspector master: Include ironic/tests in irrelevant-files https://review.opendev.org/c/openstack/ironic-inspector/+/807394 | 12:20 |
| iurygregory | arne_wiebalck++ | 12:21 |
| opendevreview | Merged x/sushy-oem-idrac master: Exclude BOSS controllers in clear_foreign_config https://review.opendev.org/c/x/sushy-oem-idrac/+/806888 | 12:39 |
| TheJulia | good morning | 12:41 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic-python-agent master: Trivial: minor fixes in error messages https://review.opendev.org/c/openstack/ironic-python-agent/+/807713 | 12:42 |
| iurygregory | good morning TheJulia =) | 12:46 |
| bfournie | good morning! Do we know when the next sushy-tools release will be? There are some fixes from July I'd like to get out there | 12:47 |
| iurygregory | bfournie, you can push a release and we will be happy to approve =D | 12:47 |
| dtantsur | morning TheJulia | 12:48 |
| bfournie | iurygregory: ok cool | 12:48 |
| dtantsur | bfournie, iurygregory, I'd prefer to hold until at least some of my patches merge... | 12:48 |
| bfournie | dtantsur: ok no problem | 12:48 |
| iurygregory | dtantsur, good point | 12:48 |
| dtantsur | https://review.opendev.org/c/openstack/sushy-tools/+/805255 is very good to have | 12:49 |
| dtantsur | https://review.opendev.org/c/openstack/sushy-tools/+/805621 too | 12:49 |
| iurygregory | I did a review in 3 patches, I will check the other 3 patches today | 12:49 |
| * TheJulia is trying to wake up | 12:49 | |
| dtantsur | https://review.opendev.org/c/openstack/sushy-tools/+/803596 has merged, so it's good | 12:49 |
| dtantsur | all other patches can wait | 12:49 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Trivial: shorten the deploy/clean step failure message https://review.opendev.org/c/openstack/ironic/+/807718 | 12:57 |
| dtantsur | also looking for a 2nd +2 on https://review.opendev.org/c/openstack/ironic-lib/+/806392 | 12:58 |
| dtantsur | it fixes error message when the instance image is not actually an image | 12:58 |
| TheJulia | heh, I just went to go review that and it and I already had | 13:11 |
| dtantsur | :) | 13:12 |
| TheJulia | iurygregory: is the tl;dr on https://review.opendev.org/c/openstack/ironic/+/807278 that because a fresh engine is not being used, tests collide with the shared connection? | 13:14 |
| arne_wiebalck | dtantsur: the m3 container use case is for me is to update inspection data via manual inspection | 13:14 |
| TheJulia | (if so, might be good to update the commit message) | 13:15 |
| arne_wiebalck | dtantsur: I plan to use this for nodes where we had this lshw parsing issue some months ago and the inventory data in S3 is now missing the correct RAM information | 13:16 |
| iurygregory | TheJulia, only in the PostgreSQL tests (mysql was working fine without a nested connection to test failures | 13:16 |
| arne_wiebalck | dtantsur: and the nodes are now in production ofc | 13:16 |
| dtantsur | arne_wiebalck: ah, interesting | 13:16 |
| iurygregory | we need this to unblock oslo.db 11.0.0 =) | 13:16 |
| arne_wiebalck | dtantsur: I never used it for this, but let' see :) | 13:16 |
| * TheJulia glances at her laptop screen, still playing ksp, and the spacecraft's speed is 1.20x "c" | 13:17 | |
| TheJulia | arne_wiebalck: you likely noticed, but I added a successes question to the round table today | 13:18 |
| TheJulia | arne_wiebalck: unfortunately, I have a board meeting conflicting :( | 13:19 |
| arne_wiebalck | TheJulia: yep, I noticed and add sth there :) | 13:19 |
| arne_wiebalck | TheJulia: oh, ok | 13:19 |
| TheJulia | That being said, it sounds like it may be a full hous | 13:19 |
| TheJulia | house | 13:19 |
| arne_wiebalck | I tried to summon some people, so there might be some just looking into Ironic atm. | 13:20 |
| TheJulia | I had a few folks message me back stating they would be joining | 13:20 |
| arne_wiebalck | great! | 13:21 |
| timeu | meeting is at 4pm CET right ? ;-) | 13:21 |
| arne_wiebalck | timeu: it is in 40 mins | 13:21 |
| timeu | ah great thanks! | 13:22 |
| timeu | just need to find a free seminar room/office until then ;-) | 13:23 |
| * TheJulia needs to refill brain with coffee | 13:27 | |
| iurygregory | TheJulia, I can update the commit message to explain why we need the change =) | 13:47 |
| arne_wiebalck | Bare Metal SIG meeting starting in about 1 min! | 13:58 |
| * dtantsur has conflicts :( | 13:59 | |
| TheJulia | oh noes :( | 14:12 |
| TheJulia | iurygregory: up to you | 14:12 |
| opendevreview | Merged openstack/ironic-lib master: Avoid using blkid in favour of lsblk and parted https://review.opendev.org/c/openstack/ironic-lib/+/806392 | 14:47 |
| * arne_wiebalck is flashed by all the feedback and questions today | 15:08 | |
| iurygregory | it was amazing :D | 15:08 |
| TheJulia | "flashed" ? | 15:08 |
| iurygregory | great job arne_wiebalck =) | 15:08 |
| arne_wiebalck | TheJulia: overwhelmed | 15:08 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic-lib stable/wallaby: Avoid using blkid in favour of lsblk and parted https://review.opendev.org/c/openstack/ironic-lib/+/807752 | 15:09 |
| arne_wiebalck | TheJulia: wasn't expecting such a lively discussion | 15:09 |
| TheJulia | ahh! | 15:11 |
| arne_wiebalck | :) | 15:14 |
| dtantsur | oh, I'm intrigued! | 15:15 |
| * dtantsur is still in meetings | 15:15 | |
| * arne_wiebalck does not envy dtantsur | 15:20 | |
| * TheJulia is also still in the board meeting | 15:28 | |
| * arne_wiebalck should come to the SIG meeting next time: MUCH shorter :-P | 15:28 | |
| * arne_wiebalck dtantsur and TheJulia should come to the SIG meeting next time: MUCH shorter :-P | 15:29 | |
| dtantsur | yeah.. | 15:29 |
| opendevreview | Aija Jauntēva proposed openstack/ironic master: Fix RAID steps for non-RAID in idrac-redfish https://review.opendev.org/c/openstack/ironic/+/804112 | 15:29 |
| TheJulia | Timing is... hard :( | 15:29 |
| TheJulia | I'm surprised to hear feedback about the chassis api | 15:51 |
| TheJulia | combined with nobodycam's feedback, sounds like something that might need more operational weight | 15:51 |
| arne_wiebalck | dtantsur: TheJulia: manual introspection from a container to update/correct missing introspection data in S3 works :) | 15:54 |
| arne_wiebalck | this way, things can be corrected when there is an instance already | 15:55 |
| TheJulia | \o/ | 15:58 |
| NobodyCam | \o/ | 15:59 |
| NobodyCam | Good morning ironic’ers | 15:59 |
| arne_wiebalck | Good morning NobodyCam o/ | 15:59 |
| iurygregory | good morning NobodyCam o/ | 15:59 |
| NobodyCam | Morning arne_wiebalck | 16:00 |
| opendevreview | Aija Jauntēva proposed openstack/ironic master: Fix clear foreign config in idrac-redfish RAID https://review.opendev.org/c/openstack/ironic/+/804848 | 16:04 |
| arne_wiebalck | bye everyone, see you next week o/ | 16:05 |
| TheJulia | o/ | 16:05 |
| TheJulia | Are we review jamming on anything today? | 16:05 |
| NobodyCam | Morning TheJulia | 16:06 |
| dtantsur | arne_wiebalck: sweet! | 16:07 |
| rpioso | Good morning, ironic :) | 18:21 |
| rpioso | TheJulia: We have seen the message "ACPI: Preparing to enter system sleep state S5" on the console of a system which fails to soft power off. After emitting that, it seems to hang ... Sound familiar? | 18:22 |
| TheJulia | rpioso: That sounds like a potential ACPI bug/issue | 18:49 |
| rpioso | TheJulia: We need to determine if the ones which ultimately are successfully turned off also failed to soft power off. The messages scroll by too quickly to see on the console. | 18:53 |
| TheJulia | rpioso: so it should be just trying to issue poweroff | 18:53 |
| TheJulia | rpioso: *but* if that fails, it falls back to using sysrq directly | 18:53 |
| TheJulia | both should end up at the same place, fwiw | 18:53 |
| TheJulia | https://github.com/openstack/ironic-python-agent/blob/71f54b7f9864b0cb51e59b3a157df986302b6073/ironic_python_agent/extensions/standby.py#L791 | 18:54 |
| rpioso | TheJulia: That ACPI message seems to have been emitted after the command from the conductor scrolled off the screen. | 18:58 |
| rpioso | TheJulia: Thank you for the link :) | 18:59 |
| rpioso | ajya: ^^^ | 18:59 |
| opendevreview | Merged openstack/ironic-ui master: Remove horizon from doc requirements https://review.opendev.org/c/openstack/ironic-ui/+/806865 | 19:31 |
| opendevreview | Merged openstack/sushy-tools master: Handle HTTP errors when downloading virtual media https://review.opendev.org/c/openstack/sushy-tools/+/805255 | 20:43 |
| opendevreview | Merged openstack/sushy-tools master: Support credentials for virtual media https://review.opendev.org/c/openstack/sushy-tools/+/802493 | 20:43 |
| stevebaker | TheJulia: related to our grub chat, what is the status of syslinux as a supported bootloader? Is it fair to say it is supported in theory, no longer used in practice, and de-facto but not explicitly deprecated? | 21:53 |
| opendevreview | Steve Baker proposed openstack/ironic master: Deprecate [pxe]pxe_config_subdir, write to root dir https://review.opendev.org/c/openstack/ironic/+/807804 | 23:10 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!