| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: Add release mappings for 19.0 and 19.1 bugfix https://review.opendev.org/c/openstack/ironic/+/828989 | 03:36 |
|---|---|---|
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: Update some releasenotes for 19.1 release https://review.opendev.org/c/openstack/ironic/+/828991 | 04:12 |
| arne_wiebalck | Good morning janders iurygregory (?) and Ironic! | 07:39 |
| rpittau | good morning ironic! o/ | 08:33 |
| arne_wiebalck | Hey rpittau o/ | 08:43 |
| rpittau | hey arne_wiebalck :) | 08:43 |
| dtantsur | morning folks, happy Monday | 10:14 |
| rpittau | hey dtantsur :) | 10:14 |
| dtantsur | what are everyone's plans for this week? I think I'll dive into bifrost | 10:14 |
| MahnoorAsghar | I'm working on my Sphinx extension for generating documentation from docstrings | 10:18 |
| MahnoorAsghar | Done with generating the method Parameter values bit, now onto Return values (including linking to exceptions etc.) | 10:19 |
| opendevreview | Arne Wiebalck proposed openstack/ironic-python-agent master: Move prepare_boot_partitions_for_softraid to raid_utils https://review.opendev.org/c/openstack/ironic-python-agent/+/826814 | 10:25 |
| arne_wiebalck | rpittau: dtantsur: finally rebased the RAID refactor ^^ | 10:28 |
| dtantsur | nice! | 10:44 |
| * dtantsur is debugging a transient issue in manage_uefi, SIGH | 10:44 | |
| dtantsur | arne_wiebalck: "partx: /dev/sda: error adding partitions 1-4" - rings any bells? | 10:47 |
| dtantsur | we really need to run it with -v flag | 10:49 |
| rpittau | that would probably help | 10:49 |
| iurygregory | good morning Ironic o/ | 10:49 |
| rpittau | hey iurygregory :) | 10:50 |
| iurygregory | we need reviews for https://review.opendev.org/c/openstack/ironic/+/828991 https://review.opendev.org/c/openstack/ironic/+/828989 so I can cut the release for Ironic o/ | 10:50 |
| arne_wiebalck | dtantsur: no ... is that from the partx -a call we do in IPA? | 10:51 |
| dtantsur | arne_wiebalck: yep. | 10:51 |
| dtantsur | and then it fails to mount the EFI partition | 10:51 |
| dtantsur | mount: /tmp/tmp0wftrg8k/boot/efi: special device /dev/sda2 does not exist. | 10:54 |
| arne_wiebalck | the IPA creates the EFI partition ... no complaints there? | 10:55 |
| arne_wiebalck | oh, wait, this is not s/w RAID, is it? | 10:56 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic-python-agent master: Run partx in verbose mode to simplify debugging https://review.opendev.org/c/openstack/ironic-python-agent/+/829011 | 11:02 |
| dtantsur | arne_wiebalck: no, it's not RAID, just a whole disk image | 11:02 |
| rpittau | it seems like partx is having trouble reading the partition table, verbose output will help | 11:05 |
| dtantsur | the bug is https://bugzilla.redhat.com/show_bug.cgi?id=2053752 btw, seems to be public | 11:06 |
| arne_wiebalck | rpittau: isn't partx successfully reading that there are 4 partitions, but then fails to tell the kernel? | 11:06 |
| rpittau | arne_wiebalck: yes, but it may be reading that wrong, like size, start-end | 11:07 |
| arne_wiebalck | rpittau: oh, could be yes | 11:08 |
| arne_wiebalck | dtantsur: there are no messages from the kernel, I guess? | 11:09 |
| dtantsur | Feb 11 18:06:04 master-0 kernel: GPT:Primary header thinks Alt. header is not at the end of the disk. | 11:09 |
| dtantsur | Feb 11 18:06:04 master-0 kernel: GPT:7884799 != 936640511 | 11:09 |
| dtantsur | Feb 11 18:06:04 master-0 kernel: GPT:Alternate GPT header not at the end of the disk. | 11:09 |
| dtantsur | Feb 11 18:06:04 master-0 kernel: GPT:7884799 != 936640511 | 11:09 |
| dtantsur | Feb 11 18:06:04 master-0 kernel: GPT: Use GNU Parted to correct GPT errors. | 11:09 |
| dtantsur | Feb 11 18:06:04 master-0 kernel: sda: sda1 sda2 sda3 sda4 | 11:09 |
| dtantsur | as far as I understand, nothing dangerous | 11:09 |
| rpittau | heh there it is | 11:09 |
| rpittau | well | 11:09 |
| rpittau | mmmmm | 11:10 |
| dtantsur | it's a normal situation for GPT whole disk images | 11:10 |
| arne_wiebalck | I have no idea, tbh. It seems the disk content is ok but the kernel refuses to acknowledge the reality. | 11:16 |
| dtantsur | oh, the same kpartx failure seems to be present on a successful run | 11:16 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: Add release mappings for 19.0 and 19.1 bugfix https://review.opendev.org/c/openstack/ironic/+/828989 | 11:17 |
| arne_wiebalck | are we still doing the wrong thing to inform the kernel? | 11:18 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: Update some releasenotes for 19.1 release https://review.opendev.org/c/openstack/ironic/+/828991 | 11:26 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic-python-agent master: Refactor efi_utils for easier maintaining and debugging https://review.opendev.org/c/openstack/ironic-python-agent/+/829015 | 11:48 |
| dtantsur | while I'm here ^^ | 11:48 |
| dtantsur | arne_wiebalck: maybe we need partprobe as well? dunno, shooting in darkness | 11:51 |
| dtantsur | or maybe we actually need `partx -u` (although that didn't work for you?). or both? | 11:55 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic-python-agent master: Add `mount` and `parted -l` to the collected commands https://review.opendev.org/c/openstack/ironic-python-agent/+/829016 | 12:02 |
| opendevreview | Ameya Raut proposed openstack/ironic-tempest-plugin master: Add iDRAC management cleaning steps tests https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/826646 | 12:41 |
| arne_wiebalck | dtantsur: from what I read when we changed from `-u` to `-a` was that there might be different situations where different calls are needed; if that is true we could add more calls to rescan_device ... not very scientific, though :) | 13:00 |
| arne_wiebalck | dtantsur: we probably should not remove calls as they have been working in the past, so removing `-u` was maybe a mistake | 13:01 |
| iurygregory | dtantsur, rpittau if you have some time to check if the releases for inspector and ipa are ok https://review.opendev.org/c/openstack/releases/+/828992 https://review.opendev.org/c/openstack/releases/+/828993 | 13:24 |
| rpittau | arne_wiebalck: not sure -u is useful in our case | 13:28 |
| rpittau | iurygregory: I'll have a look now | 13:29 |
| iurygregory | rpittau, ty | 13:30 |
| arne_wiebalck | rpittau: not sure either, but I guess someone put it there for a reason and rescan_device could become the one-stop shop for this kind of updates by calling everything (rather than we try to have tailored solutions in different situations and new problems every couple of months) ... not ideal either, I agree :) | 13:32 |
| * TheJulia yawns | 13:35 | |
| rpittau | arne_wiebalck: I mean that --update is very specific for partition changes, so in case of partitions already present in the partition table, while --add read the entire partition table anew, so in our case for the rescan_device function the --add option is more suited | 13:36 |
| rpittau | good morning TheJulia :) | 13:36 |
| iurygregory | good morning TheJulia =) | 13:42 |
| arne_wiebalck | rpittau: right, but when we do partition changes, we could use the same function to propagate the changes, just as we do with udev | 13:54 |
| arne_wiebalck | hey TheJulia o/ | 13:55 |
| TheJulia | good morning | 13:55 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: Update some releasenotes for 19.1 release https://review.opendev.org/c/openstack/ironic/+/828991 | 13:55 |
| dtantsur | TheJulia: morning | 14:12 |
| opendevreview | Nisha Agarwal proposed openstack/ironic master: Adds create_csr and add_https_certificate clean step https://review.opendev.org/c/openstack/ironic/+/804486 | 14:13 |
| dtantsur | okay, my handmade cirros partition image has deployed but failed to boot. that's a start? | 14:22 |
| dtantsur | okay, it does not have a kernel. fair enough. | 14:25 |
| arne_wiebalck | for our multi-step cleaning (cpu burn-in, mem burn-in, ... benchmark), it'd be nice to see at which stage a node is; we will start with the logging we send via fluentd from the IPA, but I guess notifications would be a sensible mechanism as well ... right? or is there something already I miss? | 15:05 |
| arne_wiebalck | (adding notifications to cleaning I mean) | 15:05 |
| TheJulia | Well... | 15:08 |
| TheJulia | hmm | 15:08 |
| dtantsur | yay, cirros booting from a partition image | 15:21 |
| dtantsur | now, does anyone know how to disable Nova metadata support in cirros? | 15:22 |
| iurygregory | no idea .-. | 15:23 |
| dtantsur | aha, https://serverfault.com/questions/646326/how-do-i-disable-the-metadata-lookup-at-cirros-boot | 15:23 |
| iurygregory | kernel args ++ | 15:29 |
| iurygregory | almost time for our weekly meeting | 15:57 |
| TheJulia | another week already?! | 15:57 |
| iurygregory | Feb is almost over =X | 15:57 |
| dtantsur | that's crazy | 15:58 |
| iurygregory | yeah, only 2 weeks to go | 15:59 |
| iurygregory | and for me the next two week will pass really fast =X | 15:59 |
| iurygregory | #startmeeting ironic | 16:00 |
| opendevmeet | Meeting started Mon Feb 14 16:00:00 2022 UTC and is due to finish in 60 minutes. The chair is iurygregory. Information about MeetBot at http://wiki.debian.org/MeetBot. | 16:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 16:00 |
| opendevmeet | The meeting name has been set to 'ironic' | 16:00 |
| iurygregory | Hello everyone, welcome to our weekly meeting \o/ | 16:00 |
| ajya | o/ | 16:00 |
| dtantsur | o/ | 16:00 |
| MahnoorAsghar | o/ | 16:00 |
| rpittau | o/ | 16:00 |
| TheJulia | o/ | 16:00 |
| rloo | o/ | 16:00 |
| arne_wiebalck | o/ | 16:01 |
| iurygregory | Our agenda is available in the wiki | 16:01 |
| iurygregory | #link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting | 16:01 |
| rpioso | o/ | 16:01 |
| ameya49 | o/ | 16:01 |
| iurygregory | #topic Announcements / Reminders | 16:01 |
| iurygregory | #info This week is the Non-client library freeze: February 17th, 2022 (R-6 week) | 16:02 |
| iurygregory | #info Next week is the deadline for Cycle Highlights, Client library freeze and Yoga-3 milestone February 24th, 2022 (R-5 week) | 16:03 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic-python-agent master: [WIP] Create mount points before mounting https://review.opendev.org/c/openstack/ironic-python-agent/+/829067 | 16:03 |
| * iurygregory will work on the cycle highlights this week | 16:03 | |
| iurygregory | #info Yoga final release: March 30th, 2022 | 16:03 |
| iurygregory | #info PTG: April 4-8, 2022 | 16:04 |
| iurygregory | #link https://openinfra-ptg.eventbrite.com/ | 16:05 |
| iurygregory | Seems like the PTG now have more restrictions regarding the amount of time we can use for discussion, Max 4hrs/day Max 16/hrs for the week =) | 16:06 |
| TheJulia | I wonder what drove that... | 16:06 |
| TheJulia | Then again some teams would historically just try to meet for endless hours | 16:07 |
| TheJulia | which impacted cross-polination | 16:07 |
| rloo | that should be fine for us/ironic. | 16:08 |
| iurygregory | yeah, I'm planning in have some sessions that would allow people from Australia/NZ to join =) | 16:08 |
| iurygregory | so it would cause problems with the 4hrs/day, I will try to figure out this during this week | 16:09 |
| iurygregory | Does anyone have anything to announce or remind us of this week? | 16:09 |
| iurygregory | moving on | 16:10 |
| iurygregory | #topic Review action items from previous meeting | 16:10 |
| iurygregory | I had an action item to fix releasenotes and push the release for the projects | 16:11 |
| iurygregory | almost everything is done | 16:11 |
| iurygregory | we need https://review.opendev.org/c/openstack/ironic/+/828991 https://review.opendev.org/c/openstack/ironic/+/828989 to cut the ironic release | 16:11 |
| iurygregory | IPA and Inspector already have the release patch ready, please take a look https://review.opendev.org/c/openstack/releases/+/828993 https://review.opendev.org/c/openstack/releases/+/828992 | 16:12 |
| iurygregory | For bifrost I will just request a normal release later today =) | 16:12 |
| iurygregory | since we don't need the bugfix branch anymore | 16:12 |
| dtantsur | right | 16:12 |
| iurygregory | #topic Deciding on priorities for the coming week | 16:13 |
| iurygregory | #link https://review.opendev.org/q/status:open+hashtag:ironic-week-prio | 16:13 |
| iurygregory | Any patches we should add to our list of priorities? | 16:13 |
| dtantsur | this could https://review.opendev.org/q/topic:unix-socket | 16:14 |
| rpittau | this probably https://review.opendev.org/c/openstack/bifrost/+/827293 but it depends on a change in DIB | 16:15 |
| iurygregory | the bifrost ones we can just have in the final release right? not in the one I'm planning to request today (just to double check) | 16:16 |
| dtantsur | right | 16:16 |
| rpittau | yep | 16:16 |
| iurygregory | feel free to add the tag =) | 16:17 |
| arne_wiebalck | I added the s/w RAID refactor earlier today | 16:18 |
| iurygregory | arne_wiebalck, ++ | 16:18 |
| iurygregory | shall we move on? | 16:20 |
| iurygregory | #topic Baremetal SIG | 16:22 |
| iurygregory | #link https://etherpad.opendev.org/p/bare-metal-sig | 16:22 |
| iurygregory | arne_wiebalck, the *mic* is yours =) | 16:22 |
| arne_wiebalck | video of last week's session is up | 16:22 |
| arne_wiebalck | session for March is already planned | 16:23 |
| arne_wiebalck | "Bare Metal for Health - Using OpenStack Ironic for HPC at Berlin Institute of Health" | 16:23 |
| arne_wiebalck | deployment show and tell :) | 16:23 |
| iurygregory | #link https://www.youtube.com/watch?v=yvdq8jPGPTw | 16:24 |
| iurygregory | oh wow, that sounds very interesting :D | 16:24 |
| iurygregory | tks for sharing the news arne_wiebalck o/ | 16:24 |
| arne_wiebalck | tahnks for the link iurygregory | 16:25 |
| arne_wiebalck | *thanks | 16:25 |
| iurygregory | #topic RFE review | 16:25 |
| iurygregory | we don't have any RFE, skipping | 16:25 |
| iurygregory | #topic Open discussion | 16:25 |
| iurygregory | Anything we should discuss today? \o/ | 16:25 |
| dtantsur | I'm trying to get somewhere with a partition cirros image | 16:27 |
| dtantsur | I think it actually works, but only with UEFI (using asset preservation) | 16:27 |
| dtantsur | are we fine to only test partition local boot under UEFI? so not actually testing grub-install? | 16:28 |
| dtantsur | (not that we're testing it now...) | 16:28 |
| iurygregory | I'm ok with this approach tbh =P (I remember we wanted to kill grub with fire in the past lol) | 16:28 |
| rloo | ideally we want to test both but if we're not testing grub-install, testing under UEFI is better than what we test now :) | 16:29 |
| dtantsur | right, that's my thinking | 16:29 |
| rloo | (unless the problem is an issue with our code that causes grub-install to fail..) | 16:30 |
| dtantsur | no, the problem is that cirros does not have grub-install :) | 16:31 |
| opendevreview | Merged openstack/ironic master: Update some releasenotes for 19.1 release https://review.opendev.org/c/openstack/ironic/+/828991 | 16:31 |
| rloo | heh, that would be a problem! | 16:32 |
| TheJulia | dtantsur: I think I'm okay with only testing local boot with asset preservation under uefi | 16:34 |
| TheJulia | dtantsur: But, I say this having actually looked at machines recently which don't support legacy boot | 16:34 |
| dtantsur | yay, just got a successful boot on bifrost | 16:36 |
| iurygregory | \o/ | 16:36 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic-python-agent-builder master: [WIP] Experimental script to build a cirros root image https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/828904 | 16:38 |
| dtantsur | this is the current version of the script ^^ | 16:38 |
| dtantsur | since it's very fast, I think I'll just move it to Ironic and use during devstack run | 16:38 |
| iurygregory | sounds like a plan =) | 16:39 |
| iurygregory | I'm wondering if we can't just have the cirros-partition-image available so we can just download... | 16:40 |
| dtantsur | there is such an image, but it's empty... | 16:40 |
| dtantsur | partition+local boot is not the exactly the cirros use case | 16:41 |
| iurygregory | we can't publish ours and have it in the opendev mirrors? | 16:41 |
| dtantsur | since it won't work on clouds out of box (without ironic) | 16:41 |
| dtantsur | we can, but the script takes a couple of seconds, I'm not sure it's worth it | 16:41 |
| iurygregory | any other topic for discussion? =) | 16:43 |
| iurygregory | #topic Who is going to run the next meeting? | 16:45 |
| iurygregory | Do we have any volunteers? | 16:45 |
| iurygregory | I will run the next meeting =) | 16:47 |
| iurygregory | Tks everyone! | 16:47 |
| iurygregory | #endmeeting | 16:47 |
| opendevmeet | Meeting ended Mon Feb 14 16:47:22 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:47 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2022/ironic.2022-02-14-16.00.html | 16:47 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2022/ironic.2022-02-14-16.00.txt | 16:47 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2022/ironic.2022-02-14-16.00.log.html | 16:47 |
| rpittau | thank you :) | 16:48 |
| iurygregory | Just FYI: this Friday RH folks will probably not be around =) / next week I will probably take 2 days off =) | 16:48 |
| iurygregory | dtantsur, congrats on being the new Outreachy OpenStack Coordinator \o/ | 16:50 |
| dtantsur | thank you :) | 16:51 |
| ameya49 | TheJulia : Thank You for review comments for : https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/826646. Have addressed the review comments. Thanks! | 16:54 |
| iurygregory | time to submit the patch in the election repo =) (almost forgot the deadline) | 16:54 |
| rpittau | bye everyone, see you tomorrow! o/ | 17:00 |
| iurygregory | OpenStack Zed... | 17:05 |
| TheJulia | Well, where did we put the Zed-PM to power the stargate up with?! | 17:06 |
| iurygregory | seems like a lot of people in our community enjoys playing LoL | 17:06 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: CI: use a custom cirros partition image instead of the default https://review.opendev.org/c/openstack/ironic/+/827882 | 17:08 |
| dtantsur | fingers crossed | 17:10 |
| dtantsur | on this positive note I'll go enjoy some Valentine's beer :) | 17:10 |
| dtantsur | see you tomorrow folks | 17:10 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-python-agent bugfix/8.4: Update .gitreview for bugfix/8.4 https://review.opendev.org/c/openstack/ironic-python-agent/+/829120 | 17:43 |
| opendevreview | Verification of a change to openstack/ironic master failed: Add release mappings for 19.0 and 19.1 bugfix https://review.opendev.org/c/openstack/ironic/+/828989 | 18:42 |
| opendevreview | Verification of a change to openstack/ironic master failed: Add release mappings for 19.0 and 19.1 bugfix https://review.opendev.org/c/openstack/ironic/+/828989 | 19:37 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-inspector bugfix/10.10: Update .gitreview for bugfix/10.10 https://review.opendev.org/c/openstack/ironic-inspector/+/829127 | 19:56 |
| TheJulia | stevebaker[m]: I think I fixed https://review.opendev.org/c/openstack/diskimage-builder/+/828617 for you. Now to let CI run and hopefully now pass on opensuse | 19:57 |
| stevebaker[m] | TheJulia: awesome, thanks! I think you could still +2 that with a change that small | 20:00 |
| TheJulia | Yeah, that was my plan | 20:00 |
| TheJulia | I went looking into the logs again and realized exactly what was going on | 20:00 |
| TheJulia | btw, creative use of word counting :) | 20:02 |
| stevebaker[m] | heh, there is probably a more bashy way | 20:05 |
| TheJulia | eh, reset the IFS maybe | 20:15 |
| TheJulia | but that can be brutal to fix | 20:15 |
| TheJulia | https://zuul.opendev.org/t/openstack/build/cf2e6663ccf04e8da84f974be7ad04d0 <-- this seems... odd | 20:15 |
| opendevreview | Verification of a change to openstack/ironic stable/xena failed: ImageCache: respect Cache-Control: no-store https://review.opendev.org/c/openstack/ironic/+/826370 | 20:41 |
| TheJulia | jeeze | 20:48 |
| TheJulia | .. | 21:00 |
| opendevreview | Merged openstack/ironic master: Clean up jobs with legacy names https://review.opendev.org/c/openstack/ironic/+/827752 | 22:08 |
| opendevreview | Merged openstack/ironic master: iRMC: Fix instructions for boot interface https://review.opendev.org/c/openstack/ironic/+/828448 | 22:08 |
| opendevreview | Merged openstack/ironic-python-agent-builder master: Use debian current stable for dib https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/827301 | 22:12 |
| opendevreview | Merged openstack/ironic-python-agent-builder master: Update documentation on supported CentOS version https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/827605 | 22:12 |
| opendevreview | Merged openstack/ironic-python-agent-builder master: Build tinyipa on tinycore 13.x https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/827137 | 22:12 |
| opendevreview | Merged openstack/ironic-python-agent-builder master: Remove ussuri job https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/828495 | 22:12 |
| opendevreview | Merged openstack/ironic master: Add release mappings for 19.0 and 19.1 bugfix https://review.opendev.org/c/openstack/ironic/+/828989 | 22:22 |
| TheJulia | ajya: quick question with regards to https://review.opendev.org/c/openstack/ironic/+/823312 is redfish raid tested in dell's ci? | 22:30 |
| TheJulia | ajya: also, fwiw, I forgot to actually click post on the comments, on that one. :( | 22:31 |
| TheJulia | forgot is probably wrong, I likely got distracted | 22:32 |
| erbarr | hi TheJulia, no dell's ci currently is only doing basic ops with nova, the tempest work will lead to other tests getting coverage | 22:33 |
| TheJulia | erbarr: much appreciated, thanks!\ | 22:33 |
| opendevreview | Merged openstack/ironic stable/xena: ImageCache: respect Cache-Control: no-store https://review.opendev.org/c/openstack/ironic/+/826370 | 22:50 |
| *** fdegir16 is now known as fdegir | 23:25 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!