| janders | good morning Ironic o/ | 00:18 |
|---|---|---|
| *** akahat is now known as akahat|rover | 07:00 | |
| iurygregory | good morning Ironic o/ | 07:11 |
| arne_wiebalck | Good morning janders iurygregory and Ironic! | 07:17 |
| iurygregory | arne_wiebalck, o/ | 07:18 |
| opendevreview | Arne Wiebalck proposed openstack/ironic-python-agent master: Re-read the partition table with partx -a https://review.opendev.org/c/openstack/ironic-python-agent/+/816941 | 07:22 |
| arne_wiebalck | dtantsur: ^^ it seems 'partx -a' fixes the issue I see the same way 'blockdev --rereadpt' does | 07:24 |
| arne_wiebalck | dtantsur: since the udev settle call comes after I guess there is no issue with /dev/disk/by-partuuid not being populated | 07:25 |
| arne_wiebalck | dtantsur: I have no partition images to test with, but will try without | 07:25 |
| *** pmannidi is now known as pmannidi|AFK | 07:27 | |
| *** pmannidi|AFK is now known as pmannidi | 07:49 | |
| *** sshnaidm|off is now known as sshnaidm | 08:04 | |
| rpittau | good morning ironic! o/ | 08:20 |
| iurygregory | good morning rpittau o/ | 08:20 |
| rpittau | hey iurygregory :) | 08:21 |
| rpittau | arne_wiebalck: should we change also https://opendev.org/openstack/ironic-python-agent/src/branch/master/ironic_python_agent/hardware.py#L2169 ? | 08:51 |
| arne_wiebalck | rpittau: I think so, yes. Once the above change is merge, the lines you mention should just call _rescan_device(). | 08:52 |
| arne_wiebalck | *merged | 08:52 |
| rpittau | arne_wiebalck: ok, I started the process to move that to utils, I'l lwait until that is merged to finalize the change https://review.opendev.org/c/openstack/ironic-python-agent/+/816916 | 08:53 |
| arne_wiebalck | rpittau: great, thanks! | 08:54 |
| arne_wiebalck | rpittau: I am not sure how I can test the move to 'partx -a' properly. Last time it only created issues when zuul tried to merge it. | 08:54 |
| arne_wiebalck | rpittau: It fixes the original issue from what I see, but my previous patch did this as well. | 08:55 |
| rpittau | I'm a bit confused as well also because we really didn't see so frequent issues until recently | 09:01 |
| rpittau | looking again at the code of partx, the -a option seems more correct as it will read all the partitions, while -u just update what's already present | 09:06 |
| rpittau | I guess it was working before just because of a matter of timing in the command execution :/ | 09:06 |
| dtantsur | morning folks | 09:07 |
| dtantsur | arne_wiebalck: neat! | 09:07 |
| arne_wiebalck | rpittau: probably, yes | 09:11 |
| dtantsur | let's see what the metalsmith job says | 09:30 |
| janders | hey iurygregory arne_wiebalck rpittau dtantsur and Ironic o/ | 09:32 |
| rpittau | hey janders :) | 09:32 |
| iurygregory | hey janders o/ | 09:32 |
| janders | I finally fixed my gerrit account problem... had to generate an ecdsa keypair, rsa was no longer good. All other services were fine. Have you guys ever seen anything like that? | 09:33 |
| janders | (I am guessing recent fresh install of FC34 had something to do with it as it worked before) | 09:33 |
| dtantsur | yep | 09:34 |
| dtantsur | this is because of Fedora security policies | 09:34 |
| rpittau | janders: time to upgrade to FC35 :) | 09:35 |
| dtantsur | IIUC, an RSA key exchange involved weak server keys, which Fedora rejected | 09:35 |
| janders | rpittau that's the plan for next week (can't afford an outage this week in an unlikely case something breaks, too much going on) | 09:35 |
| dtantsur | Time for ED25519 :) (ECDSA works as well, but is a half-measure I guess) | 09:35 |
| janders | there is no point in half doing something, as they say :) | 09:35 |
| janders | thanks guys! very useful hints | 09:39 |
| *** pmannidi is now known as pmannidi|AFK | 09:40 | |
| iurygregory | I had some fun after upgrading to F35 yesterday | 09:43 |
| dtantsur | fun \o/ | 09:46 |
| dtantsur | it was quite boring for me, module wireplumber migration | 09:46 |
| rpittau | was really boring and loooong | 09:46 |
| iurygregory | I was having some problems with x11 and wayland in F34, in F35 they are worse :D | 09:49 |
| iurygregory | I need to run startx from the console now :D | 09:50 |
| janders | iurygregory wow that's like going back to late 1990s :D | 09:50 |
| rpittau | WUT | 09:50 |
| iurygregory | janders, yup! | 09:50 |
| rpittau | iurygregory: upgrade or reinstall ? | 09:50 |
| iurygregory | rpittau, upgrade | 09:51 |
| dtantsur | I must admit, I haven't tried wayland yet: MATE is not compatible with it | 09:51 |
| iurygregory | but I will probably reinstall from scratch when I'm on PTO | 09:51 |
| iurygregory | in my personal laptop the upgrade went well, 0 problems :D | 09:51 |
| dtantsur | do you have a separate GPU? | 09:52 |
| iurygregory | in my personal laptop yes | 09:52 |
| dtantsur | hmm, I would expect it to cause issues | 09:53 |
| iurygregory | https://paste.opendev.org/show/810828/ https://paste.opendev.org/show/810829/ | 09:53 |
| iurygregory | some of the funny things me and my friend found yesterday when trying to fix the problems :D | 09:53 |
| dtantsur | wow | 09:54 |
| iurygregory | yeah | 09:54 |
| dtantsur | make sure to file it | 09:54 |
| iurygregory | I'll try to check a few more things to file a bug, I think the FW upgrade caused some issues | 09:54 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Enable iPXE boot interface by default https://review.opendev.org/c/openstack/ironic/+/816824 | 11:24 |
| janders | see you tomorrow Ironic o/ | 12:20 |
| iurygregory | bye janders o/ | 12:58 |
| dtantsur | this is not too great: https://zuul.openstack.org/builds?job_name=ironic-standalone | 13:11 |
| iurygregory | mdadm: cannot open /dev/vdb1: No such file or directory | 13:20 |
| TheJulia | Good morning everyone | 13:20 |
| * TheJulia feels like lots of coffee is needed this morning | 13:20 | |
| iurygregory | good morning TheJulia | 13:20 |
| iurygregory | coffee is always good | 13:21 |
| dtantsur | iurygregory: oh, so maybe arne_wiebalck's patch will help? | 13:24 |
| iurygregory | dtantsur, yeah I have this feeling | 13:24 |
| rpittau | that should help, yeah, that's what we were discussing saying last week on thursday and friday | 13:33 |
| rpittau | mmm fantastic, I have 3 meetings at the same time :/ | 13:33 |
| dtantsur | \o/ | 13:42 |
| dtantsur | are you becoming a manager? :D | 13:42 |
| rpittau | lol | 13:42 |
| * dtantsur brb | 13:43 | |
| rpittau | gotta love time zone change shenanigan | 13:43 |
| TheJulia | UTC. UTC for all meetings. | 13:44 |
| rpittau | heh would appreciate that :/ | 13:44 |
| rpittau | mmmmmmm is it normal that I can't see the zuul status for https://review.opendev.org/c/openstack/ironic-python-agent/+/816941 ? | 13:49 |
| TheJulia | hmm, is it on the status page at all? | 14:03 |
| rpittau | nope | 14:03 |
| iurygregory | yeah it doesn't show to me also D: | 14:04 |
| iurygregory | recheck ftw | 14:04 |
| iurygregory | zuul didn't receive the cookies probably :D | 14:04 |
| rpittau | I have a pack of Oreo here I can share | 14:04 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/xena: Fix UEFI record regex https://review.opendev.org/c/openstack/ironic-python-agent/+/817008 | 14:05 |
| TheJulia | It seems to prefer Oreo, I think | 14:06 |
| rpittau | :) | 14:06 |
| TheJulia | rpittau: that ^^^ is on top of https://review.opendev.org/c/openstack/ironic-python-agent/+/816489 | 14:07 |
| TheJulia | I'd honestly just prefer to cherry pick the two down instead of trying to squash | 14:07 |
| rpittau | oh yeah, saw that right after approving | 14:07 |
| TheJulia | We ran into this on lenovo sr650s where they would basically reset their boot orders | 14:07 |
| rpittau | I remember the discussion | 14:08 |
| TheJulia | okay, i see you +2+A'ed | 14:09 |
| TheJulia | now onward to train :\ | 14:09 |
| rpittau | yeah | 14:09 |
| rpittau | oh wow, up to that? | 14:09 |
| TheJulia | Yeah, it was against osp16.2 | 14:10 |
| rpittau | ah I see | 14:10 |
| TheJulia | so I have to go *way* back | 14:10 |
| TheJulia | hmm | 14:15 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/wallaby: Output verbose info from efibootmgr https://review.opendev.org/c/openstack/ironic-python-agent/+/817012 | 14:19 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/wallaby: Delete EFI boot entry duplicate labels first https://review.opendev.org/c/openstack/ironic-python-agent/+/817013 | 14:19 |
| TheJulia | wheeeeeeeeee | 14:19 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/wallaby: Fix UEFI record regex https://review.opendev.org/c/openstack/ironic-python-agent/+/817014 | 14:19 |
| TheJulia | so many uefi fixes over the past couple of years | 14:20 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/victoria: Output verbose info from efibootmgr https://review.opendev.org/c/openstack/ironic-python-agent/+/817016 | 14:20 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/victoria: Delete EFI boot entry duplicate labels first https://review.opendev.org/c/openstack/ironic-python-agent/+/817017 | 14:20 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/victoria: Fix UEFI record regex https://review.opendev.org/c/openstack/ironic-python-agent/+/817018 | 14:20 |
| *** pmannidi|AFK is now known as pmannidi | 14:21 | |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/ussuri: Output verbose info from efibootmgr https://review.opendev.org/c/openstack/ironic-python-agent/+/817019 | 14:22 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/ussuri: Delete EFI boot entry duplicate labels first https://review.opendev.org/c/openstack/ironic-python-agent/+/817020 | 14:22 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/ussuri: Fix UEFI record regex https://review.opendev.org/c/openstack/ironic-python-agent/+/817021 | 14:22 |
| iurygregory | wheeeeeeeeeee \o/ | 14:22 |
| iurygregory | spam time :D | 14:22 |
| TheJulia | sorry | 14:23 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/train: Output verbose info from efibootmgr https://review.opendev.org/c/openstack/ironic-python-agent/+/817024 | 14:24 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/train: Delete EFI boot entry duplicate labels first https://review.opendev.org/c/openstack/ironic-python-agent/+/817025 | 14:24 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/train: Fix UEFI record regex https://review.opendev.org/c/openstack/ironic-python-agent/+/817026 | 14:24 |
| iurygregory | no need to be sorry =) | 14:24 |
| TheJulia | With the pain of merge conflicts we've had on the older branches, I've really become a fan of just never squashing anything | 14:24 |
| TheJulia | makes it easier to trace down what is going on | 14:24 |
| TheJulia | *when* there is a conflict | 14:24 |
| iurygregory | ++ | 14:24 |
| * TheJulia curses waking up at 1 am | 14:32 | |
| opendevreview | Aija Jauntēva proposed x/sushy-oem-idrac master: Migrate constants to enums https://review.opendev.org/c/x/sushy-oem-idrac/+/817028 | 14:32 |
| TheJulia | I think I'm going to lay down for a little bit. I think I just got up way too early this morning | 14:38 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic-specs master: Yoga Themes https://review.opendev.org/c/openstack/ironic-specs/+/815308 | 14:41 |
| iurygregory | TheJulia, take care =) | 14:41 |
| opendevreview | Merged openstack/virtualbmc master: Update pep8 test requirements https://review.opendev.org/c/openstack/virtualbmc/+/814540 | 14:42 |
| dtantsur | TheJulia: once you get better, we probably need to backport it to the latest bugfix branch (18.1?) | 14:55 |
| iurygregory | ++ ^ | 14:58 |
| iurygregory | #startmeeting ironic | 15:00 |
| opendevmeet | Meeting started Mon Nov 8 15:00:00 2021 UTC and is due to finish in 60 minutes. The chair is iurygregory. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
| opendevmeet | The meeting name has been set to 'ironic' | 15:00 |
| dtantsur | o/ | 15:00 |
| iurygregory | Hello everyone, welcome to our weekly meeting =) | 15:00 |
| iurygregory | o/ | 15:00 |
| rpittau | o/ | 15:00 |
| rpioso | o/ | 15:00 |
| stendulker | o/ | 15:00 |
| rloo | o/ | 15:00 |
| iurygregory | Our agenda can be found on the wiki. | 15:01 |
| iurygregory | #link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting | 15:01 |
| iurygregory | #topic Announcements / Reminders | 15:01 |
| iurygregory | #info ajya is now part of the sushy-core group | 15:02 |
| iurygregory | congratulations ajya =) | 15:02 |
| rpittau | congrats ajya :) | 15:02 |
| rpioso | Congratulations, ajya \o/ | 15:02 |
| * iurygregory wondering if we should announce the sig meeting in announcements or not (and skip the topic) | 15:03 | |
| * iurygregory thinks it makes sense to use Announcements so he will do =) | 15:04 | |
| iurygregory | #info Tomorrow we will have our monthly SIG Meeting: Emmanouil Bagakis on "Hardware Burn-in with Ironic" - 2 PM UTC | 15:04 |
| iurygregory | does anyone have anything to announce/remind us of? | 15:04 |
| stendulker | Congras ajya :) | 15:04 |
| opendevreview | Merged openstack/bifrost master: Bump ansible lint to latest version https://review.opendev.org/c/openstack/bifrost/+/815293 | 15:06 |
| iurygregory | we don't have any action items from previous meeting or subteam status report so I will skip this topics | 15:06 |
| iurygregory | but please take a look at the Yoga Themes https://review.opendev.org/c/openstack/ironic-specs/+/815308 if there is something you would like to work on or something we should change | 15:06 |
| ajya | o/ | 15:07 |
| ajya | thank you everyone :) | 15:07 |
| iurygregory | #topic Deciding on priorities for the coming week | 15:08 |
| iurygregory | #link https://review.opendev.org/q/status:open+hashtag:ironic-week-prio | 15:08 |
| iurygregory | I see Julia pushed a few backports, I'm adding them to the list | 15:09 |
| iurygregory | anyone has patches that we should review? =) | 15:09 |
| rpittau | iurygregory: I guess we should add arne_wiebalck ipa fix | 15:09 |
| dtantsur | yeah | 15:09 |
| iurygregory | rpittau, ++ | 15:09 |
| dtantsur | first, https://review.opendev.org/c/openstack/ironic/+/816721 and https://review.opendev.org/c/openstack/ironic/+/816824 make our defaults better | 15:09 |
| dtantsur | then https://review.opendev.org/c/openstack/ironic/+/816726 is the first part of unblocking the sushy enum work | 15:09 |
| dtantsur | because, as I've discovered, switching sushy to enums breaks ironic :( | 15:10 |
| iurygregory | ouch D: | 15:10 |
| dtantsur | https://review.opendev.org/c/openstack/ironic-python-agent/+/815651 is something I really need downstream, but it conflicts with arne_wiebalck's fix which has a higher priority | 15:10 |
| iurygregory | ack, I'm ok with adding them to our list | 15:10 |
| dtantsur | finally, https://review.opendev.org/c/openstack/sushy/+/814191 is a nice sushy addition | 15:10 |
| iurygregory | added to our list | 15:12 |
| iurygregory | I think we can move on =) | 15:14 |
| iurygregory | #topic Discussion | 15:14 |
| iurygregory | Do we have anything we would like to discuss? | 15:14 |
| rloo | i took a quick look at the yoga themes PR. | 15:15 |
| rloo | there is a lot there, not many primary contacts. is the goal to try to do all of them? | 15:15 |
| iurygregory | not really, we only a few people provided feedback on the first draft | 15:16 |
| rloo | i'm a bit concerned that it might take several weeks to approve that pr: https://review.opendev.org/c/openstack/ironic-specs/+/815308. which might be ok? | 15:16 |
| iurygregory | I know a few topics that Julia will work on (forgot to add that /facepalm) | 15:16 |
| rloo | goal details are missing. I am happy to review but seems like it isn't really ready to merge. | 15:17 |
| iurygregory | yeah, I'll also update with more details about each Theme so people can have a better idea of what they mean | 15:17 |
| rloo | thanks! | 15:17 |
| iurygregory | tks for the feedback rloo =) | 15:18 |
| rloo | fwiw, i feel like if there are no primary contacts by some date (this week or next Mon?) we remove. | 15:18 |
| iurygregory | makes sense to me | 15:18 |
| iurygregory | moving on | 15:20 |
| iurygregory | I already added the sig information and we don't have any rfe to review - skipping | 15:21 |
| iurygregory | #topic Who is going to run the next meeting? | 15:21 |
| iurygregory | Do we have a volunteer? | 15:21 |
| iurygregory | I will run the meeting next week =) | 15:23 |
| iurygregory | Thanks everyone! | 15:23 |
| iurygregory | #endmeeting | 15:23 |
| opendevmeet | Meeting ended Mon Nov 8 15:23:46 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:23 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2021/ironic.2021-11-08-15.00.html | 15:23 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2021/ironic.2021-11-08-15.00.txt | 15:23 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2021/ironic.2021-11-08-15.00.log.html | 15:23 |
| rloo | ajya: You could approve this? https://review.opendev.org/c/openstack/sushy/+/814191 :D | 15:49 |
| ajya | rloo: ok :) | 15:51 |
| rloo | ajya: your first of many ;) | 15:51 |
| dtantsur | always a nice feeling :) | 15:52 |
| opendevreview | Merged openstack/sushy-tools master: Update pep8 test requirements https://review.opendev.org/c/openstack/sushy-tools/+/814541 | 15:53 |
| opendevreview | Verification of a change to openstack/ironic bugfix/18.1 failed: Fix various issues in the anaconda deploy interface https://review.opendev.org/c/openstack/ironic/+/815871 | 15:56 |
| opendevreview | Merged openstack/ironic-python-agent stable/xena: Delete EFI boot entry duplicate labels first https://review.opendev.org/c/openstack/ironic-python-agent/+/816489 | 15:56 |
| * TheJulia feels much more awake now | 15:59 | |
| opendevreview | Aija Jauntēva proposed openstack/ironic stable/xena: Fix idrac-wsman deploy with existing non-BIOS jobs https://review.opendev.org/c/openstack/ironic/+/816963 | 16:20 |
| opendevreview | Aija Jauntēva proposed openstack/ironic stable/wallaby: Fix idrac-wsman deploy with existing non-BIOS jobs https://review.opendev.org/c/openstack/ironic/+/816964 | 16:21 |
| opendevreview | Aija Jauntēva proposed openstack/ironic stable/victoria: Fix idrac-wsman deploy with existing non-BIOS jobs https://review.opendev.org/c/openstack/ironic/+/816965 | 16:21 |
| opendevreview | Aija Jauntēva proposed openstack/ironic stable/ussuri: Fix idrac-wsman deploy with existing non-BIOS jobs https://review.opendev.org/c/openstack/ironic/+/816966 | 16:22 |
| opendevreview | Aija Jauntēva proposed openstack/ironic stable/train: Fix idrac-wsman deploy with existing non-BIOS jobs https://review.opendev.org/c/openstack/ironic/+/816967 | 16:22 |
| opendevreview | Merged openstack/sushy master: Add some VirtualMedia fields from 1.3.0 and 1.4.0 https://review.opendev.org/c/openstack/sushy/+/814191 | 16:40 |
| arne_wiebalck | rpittau and arne_wiebalck align on backports :-D | 16:43 |
| rpittau | :D | 16:44 |
| rpittau | at this point I'm wondering if udevadm settle is even needed | 16:47 |
| rpittau | heh, it cna't hurt | 16:47 |
| arne_wiebalck | hmm, good point | 16:49 |
| arne_wiebalck | I think it is though | 16:50 |
| arne_wiebalck | since the entries in /dev/disk/by-* are populated by udev | 16:50 |
| rpittau | mmm ok, makes sense | 16:50 |
| arne_wiebalck | bye everyone, see you tomorrow o/ | 17:21 |
| rpittau | bye arne_wiebalck :) | 17:21 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent master: Move rescan device function to general utils https://review.opendev.org/c/openstack/ironic-python-agent/+/816916 | 17:28 |
| rpittau | good night! o/ | 17:29 |
| opendevreview | Merged openstack/ironic-python-agent stable/xena: Fix UEFI record regex https://review.opendev.org/c/openstack/ironic-python-agent/+/817008 | 17:31 |
| dtantsur | o/ | 17:53 |
| NobodyCam | Good Morning and Happy Monday Ironic folks! | 17:57 |
| NobodyCam | woo +2 +1 on https://review.opendev.org/c/openstack/nova/+/813263 | 18:03 |
| tzumainn | hi! I'm running into an issue using the instance_info hardware interface override (https://docs.openstack.org/ironic/latest/user/deploy.html#overriding-a-hardware-interface), and I think I've traced it back to an issue I'm not sure how to solve | 18:41 |
| tzumainn | the basic issue is: the task driver appears to be created properly with values from instance_info overriding *_interface | 18:41 |
| tzumainn | however there are a few places in the ironic code such as https://opendev.org/openstack/ironic/src/branch/master/ironic/drivers/modules/deploy_utils.py#L631 where there's logic that directly compares against a node's *_interface value | 18:42 |
| tzumainn | I was wondering if there was an ideal fix for this that people could think of? something like a node method to retrieve an interface that checks instance_info first, or something like that? | 18:43 |
| rloo | tzumainn: that seems like a bug to me and yes, having a new node method to get the interface should work. | 19:13 |
| tzumainn | rloo, okay, thanks! I'll try to implement that then | 19:18 |
| rloo | tzumainn: thank YOU! | 19:19 |
| opendevreview | Verification of a change to openstack/ironic bugfix/18.1 failed: Fix various issues in the anaconda deploy interface https://review.opendev.org/c/openstack/ironic/+/815871 | 20:03 |
| opendevreview | Verification of a change to openstack/ironic bugfix/18.1 failed: Fix various issues in the anaconda deploy interface https://review.opendev.org/c/openstack/ironic/+/815871 | 20:44 |
| opendevreview | Merged openstack/ironic-python-agent master: Re-read the partition table with partx -a https://review.opendev.org/c/openstack/ironic-python-agent/+/816941 | 20:58 |
| *** dviroel is now known as dviroel|out | 21:11 | |
| opendevreview | Tzu-Mainn Chen proposed openstack/ironic master: Create node get_interface method https://review.opendev.org/c/openstack/ironic/+/817086 | 21:27 |
| janders | good morning Ironic o/ | 22:33 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!