| opendevreview | Steve Baker proposed openstack/ironic master: Use test_with_retry to get the tap device name https://review.opendev.org/c/openstack/ironic/+/818703 | 00:51 |
|---|---|---|
| opendevreview | Steve Baker proposed openstack/bifrost master: WIP support grub network boot https://review.opendev.org/c/openstack/bifrost/+/807220 | 04:28 |
| opendevreview | Steve Baker proposed openstack/bifrost master: WIP support grub network boot https://review.opendev.org/c/openstack/bifrost/+/807220 | 04:33 |
| opendevreview | Steve Baker proposed openstack/bifrost master: Use distro conf loader_file_paths to copy ipxe/grub files https://review.opendev.org/c/openstack/bifrost/+/818709 | 04:33 |
| opendevreview | Steve Baker proposed openstack/bifrost master: Use distro conf loader_file_paths to copy ipxe/grub files https://review.opendev.org/c/openstack/bifrost/+/818709 | 04:36 |
| opendevreview | Steve Baker proposed openstack/bifrost master: WIP support grub network boot https://review.opendev.org/c/openstack/bifrost/+/807220 | 04:36 |
| opendevreview | Jacob Anders proposed openstack/ironic-python-agent master: Improve efficiency of storage cleaning in mixed media envs https://review.opendev.org/c/openstack/ironic-python-agent/+/818712 | 06:03 |
| janders | arne_wiebalck when you have time I'd appreciate your early feedback on https://review.opendev.org/c/openstack/ironic-python-agent/+/818712 | 06:16 |
| hgy | Good afternoon, Ironic! | 06:37 |
| muellerbe | good morning ironic o/ | 07:34 |
| iurygregory | good morning janders stevebaker[m] hgy__ muellerbe and Ironic o/ | 07:37 |
| muellerbe | hello iurygregory | 07:38 |
| rpittau | good morning ironic! o/ | 07:38 |
| stevebaker[m] | Good evening! | 07:39 |
| hgy__ | good morning iurygregory muellerbe rpittau! | 07:39 |
| iurygregory | morning rpittau o/ | 07:39 |
| hgy__ | Good evening <stevebaker[m]>! | 07:39 |
| rpittau | hey iurygregory :) | 07:43 |
| muellerbe | hgy__ o/ | 07:43 |
| opendevreview | Riccardo Pittau proposed openstack/python-ironic-inspector-client master: Improve testing https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/818523 | 07:58 |
| arne_wiebalck | Good morning janders hgy muellerbe rpittau iurygregory stevebaker[m] and Ironic! | 08:07 |
| rpittau | hey arne_wiebalck :) | 08:07 |
| muellerbe | hey ho arne_wiebalck | 08:07 |
| rpittau | dtantsur: about bullseye revert https://review.opendev.org/c/openstack/bifrost/+/818600 that error you're seeing is quite odd, the packages names should be python3-pip and python3-virtualenv, not sure why you're getting the python2 versions | 08:09 |
| rpittau | they were updated here too https://opendev.org/openstack/bifrost/src/branch/master/playbooks/roles/bifrost-ironic-install/defaults/required_defaults_Debian_family.yml | 08:10 |
| iurygregory | arne_wiebalck, o/ | 08:24 |
| janders | hey hgy_ muellerbe rpittau iurygregory arne_wiebalck and Ironic o/ | 09:06 |
| iurygregory | janders, o/ | 09:07 |
| rpittau | hey janders :) | 09:07 |
| hgy_ | hey janders | 09:10 |
| arne_wiebalck | janders: thanks for the patch! I had a first look: from what I see the suggestion is to erase the whole nvme device inside erase_meta_data? | 09:10 |
| janders | arne_wiebalck yeah what I thought may be the simplest approach is to check if a device is an NVMe in erase_device_metadata. If it is, use native NVMe erasure. If it is not, just do metadata. This way we can have the hybrid functionality with relatively minimal changes. | 09:16 |
| janders | I remember there were suggestions for taking this approach (as opposed to creating a whole new erasure "mode") during the previous PTG | 09:16 |
| janders | but - I am open to different approaches, too | 09:17 |
| janders | lets pick what we think is best | 09:17 |
| arne_wiebalck | my concern is that we basically do more than advertised (erase the whole device rather than only metadata) | 09:17 |
| janders | fair point - I'm open to considering a whole new cleaning "mode" instead | 09:18 |
| arne_wiebalck | either this or configuration for erase_block_devices, e.g.: do_hybrid=true/false | 09:19 |
| arne_wiebalck | or shred=true/false :-D | 09:20 |
| arne_wiebalck | skip_loooong_shred | 09:20 |
| arne_wiebalck | I think it would be nice if operators could configure this easily | 09:22 |
| iurygregory | skip_loooong_shred LOL | 09:23 |
| arne_wiebalck | non-automatic steps can be pulled in with the cleaning prio, no? | 09:24 |
| iurygregory | by non-automatic steps you mean the ones that are not enabled by default right? | 09:29 |
| *** TheJulia_ is now known as TheJulia | 09:32 | |
| *** buhman_ is now known as buhman | 09:32 | |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Use UEFI by default and deprecate legacy boot https://review.opendev.org/c/openstack/bifrost/+/818549 | 09:34 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Fix the DHCP port in iptables (68 vs 67 and TCP vs UDP) https://review.opendev.org/c/openstack/bifrost/+/818592 | 09:34 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Use UEFI by default and deprecate legacy boot https://review.opendev.org/c/openstack/bifrost/+/818549 | 09:35 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Revert "Explicitly trap on ERR" and fix exit code hanlding https://review.opendev.org/c/openstack/bifrost/+/817978 | 09:35 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Revert the default DIB distribution to Buster https://review.opendev.org/c/openstack/bifrost/+/818600 | 09:38 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Use boot.ipxe generated by Ironic https://review.opendev.org/c/openstack/bifrost/+/818000 | 09:41 |
| opendevreview | Merged openstack/ironic-python-agent-builder master: add element-deps for element burn-in delete entries https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/817408 | 11:01 |
| dtantsur | morning ironic | 12:09 |
| dtantsur | rpittau: I suspect the unversioned dependencies may be coming from DIB itself | 12:09 |
| dtantsur | since it happens inside the ramdisk, not on bifrost | 12:09 |
| dtantsur | https://review.opendev.org/c/openstack/bifrost/+/818592 is a very embarrassing issue that could use some reviews | 12:17 |
| dtantsur | also https://review.opendev.org/c/openstack/bifrost/+/806207 has been waiting for a 2nd +2 for a month: https://review.opendev.org/c/openstack/bifrost/+/806207 | 12:26 |
| rpittau | dtantsur: that's weird though, I see a bullseye image built this morning with ipa-builder and it went through just fine | 13:31 |
| dtantsur | hmmm | 13:39 |
| dtantsur | I wonder if there is anything unusual about the procedure in bifrost | 13:44 |
| iurygregory | dtantsur, you mentioned something about newer ansible version not supporting py36 last week I think, right? | 13:50 |
| dtantsur | yeah, I've seen a warning somewhere | 13:52 |
| TheJulia | good morning | 13:57 |
| iurygregory | dtantsur, well Yoga will be 3.8 and 3.9 =D | 13:57 |
| iurygregory | good morning TheJulia =) | 13:57 |
| dtantsur | right, but we keep 3.6 because RHEL.. | 13:57 |
| dtantsur | hey TheJulia | 13:58 |
| rpittau | iurygregory, dtantsur, seems like the discussion about dropping py3.6 in ansible-core is going on since 1 year https://github.com/ansible/ansible/issues/72668 | 13:58 |
| * TheJulia attempts caffination | 13:58 | |
| rpittau | good morning TheJulia :) | 13:58 |
| iurygregory | rpittau, oh wow :O | 13:58 |
| dtantsur | do we need to cap ansible-core? or does capping ansible do it for us? | 13:59 |
| TheJulia | rpittau: just from the sound of it... it sounds like python 2.6 all over again | 13:59 |
| rpittau | TheJulia: looks like it, yeah :) | 13:59 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/train: CI: Work around tinyipa build issues https://review.opendev.org/c/openstack/ironic/+/818735 | 14:02 |
| rpittau | dtantsur: so ansible 5 will need ansible-core 2.12 that requires python3.8 in the control node https://github.com/ansible/ansible/blob/stable-2.12/changelogs/CHANGELOG-v2.12.rst | 14:03 |
| rpittau | I guess we could cap ansible | 14:03 |
| dtantsur | okay, so we need to decide what we do | 14:03 |
| dtantsur | I think we're fine with "less supported" 3.9 (?) on RHEL 8 | 14:03 |
| dtantsur | but I guess it leaves out openSUSE and Buster? | 14:03 |
| dtantsur | and Bionic? | 14:03 |
| 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:05 |
| rpittau | mmm if yoga really supports python>=3.8 all of the distros you mentioned won't be 100% supported | 14:05 |
| 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:05 |
| 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:05 |
| dtantsur | rpittau: was a decision made to drop 3.6? if yes, we just follow the pack. if not, we need to decide for ourselves | 14:05 |
| rpittau | iurygregory: where did you see the yoga approved python versions ? | 14:06 |
| iurygregory | rpittau, fyi https://governance.openstack.org/tc/reference/runtimes/yoga.html | 14:06 |
| rpittau | thanks! | 14:06 |
| iurygregory | I'm going to provide the info in the meeting also :D | 14:06 |
| dtantsur | "stable distribution" "CentOS Stream" LOL | 14:06 |
| rpittau | :D | 14:06 |
| rpittau | centos stream 9 !!! | 14:07 |
| dtantsur | yeah, I would take 8 as semi-stable | 14:07 |
| dtantsur | but 9 is beta, who in their right mind would install that in production? | 14:07 |
| rpittau | it does not seem py3.6 is dropped, it's just the min tested runtimes | 14:08 |
| dtantsur | yeah, it's ambiguous | 14:08 |
| rpittau | so we could probably cap ansible if we want to keep support for another round | 14:08 |
| dtantsur | we do seem to still run 3.6 unit tests? | 14:09 |
| rpittau | yeah | 14:09 |
| dtantsur | ah, we'll stop https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/818609/2/zuul.d/project-templates.yaml | 14:10 |
| rpittau | ah there you go | 14:10 |
| rpittau | well that means if we want to keep support we'll have to run our unit py36 unit tests | 14:11 |
| iurygregory | yeah | 14:11 |
| dtantsur | honestly, I don't care enough if we can make Stream 8 work. | 14:11 |
| rpittau | lol | 14:11 |
| dtantsur | it's a bit sad to drop openSUSE without deprecation, but what can I do? | 14:11 |
| * TheJulia wonders when we can put dtantsur on a mountain next | 14:12 | |
| dtantsur | TheJulia: assuming the world will not go down in flames, I hope to be on a mountain after x-mas :) | 14:12 |
| TheJulia | dtantsur: ++ | 14:12 |
| dtantsur | rpittau: which means, we need to actually fix bullseye :) | 14:13 |
| rpittau | oh yeah | 14:13 |
| dtantsur | okay, I'll look into some deprecations once I receive my groceries | 14:14 |
| TheJulia | iurygregory: I've left some comments on https://review.opendev.org/c/openstack/ironic-specs/+/815308 with suggested text and one "do we really want to do this" question | 14:14 |
| rpittau | we still ahve the meeting in 45 minutes ? | 14:15 |
| dtantsur | for suse we could use https://build.opensuse.org/project/show/devel%3alanguages%3apython%3aFactory but I'd rather not | 14:15 |
| iurygregory | TheJulia, looking now | 14:15 |
| dtantsur | heh, buster has 3.7. still not enough | 14:16 |
| iurygregory | TheJulia, re privsep memory consumption maybe we can discuss in the meeting to see what people think about it? | 14:17 |
| TheJulia | ++ | 14:21 |
| TheJulia | I'm just not aware of the value/loss proposition there | 14:21 |
| TheJulia | just after dtantsur and I saw the whole argument in front of the TC about it's memory consumption, I'm a little skittish | 14:22 |
| iurygregory | yeah, I only heard about it =( (I dropped from the meeting before that) | 14:22 |
| opendevreview | Merged openstack/bifrost master: Remove the keystone admin endpoint https://review.opendev.org/c/openstack/bifrost/+/806207 | 14:31 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Refresh supported operating systems list https://review.opendev.org/c/openstack/bifrost/+/818780 | 14:42 |
| dtantsur | here goes ^^^ | 14:42 |
| dtantsur | I guess we should start thinking towards supporting Stream 9? | 14:42 |
| dtantsur | iurygregory: now that iSCSI deploy is gone, do we actually do many privileged things in Ironic? | 14:48 |
| dtantsur | I see that we have some code to mount a deploy ISO | 14:50 |
| dtantsur | which, I guess, could be done without mounting | 14:50 |
| iurygregory | dtantsur, so since privsep will take place of rootwrap we have two filters and some commands https://github.com/openstack/ironic-lib/blob/master/etc/ironic/rootwrap.d/ironic-lib.filters https://github.com/openstack/ironic/blob/master/etc/ironic/rootwrap.d/ironic-utils.filters | 14:50 |
| dtantsur | ironic-lib can be ignored IMO, it's mostly used in IPA nowadays | 14:51 |
| iurygregory | hummm | 14:51 |
| iurygregory | I was starting things in ironic-lib because there was a lot of filters there... | 14:51 |
| dtantsur | we only use qemu_img_info and convert_image from disk_utils | 14:51 |
| dtantsur | yeah, my point being: maybe we should go the direction of dropping privileged operations from Ironic itself | 14:52 |
| opendevreview | Merged openstack/sushy master: Migrate System constants to enums https://review.opendev.org/c/openstack/sushy/+/816717 | 14:52 |
| iurygregory | ok, that is something interesting | 14:52 |
| dtantsur | I'm pretty sure you can unpack an ISO without using mount | 14:52 |
| dtantsur | we even have pycdlib in requirements | 14:54 |
| iurygregory | so we wouldn't have to enable privsep at all if we can convert the calls we have | 14:55 |
| dtantsur | yep. and drop rootwrap because IPA always uses root. | 14:55 |
| rpittau | dtantsur: btw we'll soon have FC35 | 14:55 |
| dtantsur | ansible deploy needs double-checking, but I don't think any of the privileged commands are run on the controller | 14:55 |
| dtantsur | rpittau: this is one of the reasons I want to deprecate Fedora support :D | 14:55 |
| rpittau | :D | 14:55 |
| rpittau | ok, got my double espresso, ready for the 2 parallel meetings! | 14:56 |
| dtantsur | ouch, forgot coffee, may be slightly late | 14:57 |
| iurygregory | #startmeeting ironic | 15:00 |
| opendevmeet | Meeting started Mon Nov 22 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 o/ | 15:00 |
| ajya | o/ | 15:00 |
| iurygregory | Welcome to our weekly meeting \o/ | 15:00 |
| TheJulia | o/ | 15:00 |
| rpioso | \o | 15:00 |
| rloo | o/ | 15:01 |
| rpittau | o/ | 15:02 |
| iurygregory | you can find our agenda in the wiki | 15:02 |
| iurygregory | #link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting | 15:02 |
| iurygregory | #topic Announcements / Reminder | 15:02 |
| erbarr | o/ | 15:02 |
| iurygregory | #info Tested Runtimes for Yoga - Python 3.8 and 3.9 | 15:02 |
| iurygregory | #link http://lists.openstack.org/pipermail/openstack-discuss/2021-November/025881.html | 15:03 |
| stendulker | o/ | 15:03 |
| arne_wiebalck_ | o/ | 15:03 |
| iurygregory | #link https://governance.openstack.org/tc/reference/runtimes/yoga.html | 15:03 |
| iurygregory | #info First bugfix branch should be created by the end of next week | 15:03 |
| iurygregory | #info PTL on PTO during whole December basically | 15:04 |
| dtantsur | PTO++ | 15:04 |
| dtantsur | You can't even imagine how much work we'll assign to you in the meantime :) | 15:05 |
| iurygregory | Next week 29,30 and 03Dec I'm on PTO and the whole month... | 15:05 |
| iurygregory | dtantsur, omg :D | 15:05 |
| iurygregory | well my PTO starts this thursday afternoon :D | 15:05 |
| dtantsur | heh, nice | 15:05 |
| iurygregory | but I will be on irc/twitter | 15:05 |
| dtantsur | I'm available till at least mid-December, back in early January | 15:06 |
| iurygregory | if you need something let me know =) | 15:06 |
| iurygregory | we should evaluate how we will do the upstream meetings in December | 15:06 |
| TheJulia | I'll likely be around the first three weeks of december | 15:06 |
| rpittau | well if we move the meeting 1 hour later I can take ownership of some of them | 15:06 |
| dtantsur | definitely not on the 27th and unlike on 3rd :) | 15:07 |
| TheJulia | I don't expect anything other than banging my head against nova/rbac/etc | 15:07 |
| iurygregory | dtantsur, yeah I agree with this dates (we can skip for sure) | 15:07 |
| dtantsur | rpittau: I have nothing against moving the meeting, except that it's probably going to be a pain to notify everyone, change eavesdrop.o.o, etc | 15:07 |
| dtantsur | iurygregory: so I assume you won't be here to handle what we just discussed re dropping rootwrap? | 15:08 |
| iurygregory | 29 I should be able to run the meeting (if the jetlag doesn't hit me very hard) | 15:08 |
| dtantsur | I can do it just as well, if needed | 15:09 |
| TheJulia | I suspect we will just handle it like we have past decembers | 15:09 |
| iurygregory | let's see, I think I will be able to online next monday | 15:09 |
| TheJulia | if we have critical mass, awesome, if not skip() | 15:09 |
| dtantsur | fair | 15:09 |
| iurygregory | ++ | 15:09 |
| TheJulia | it may mean stuff piles on to the agenda for January | 15:09 |
| iurygregory | does anyone have anything to announce/remind us of? | 15:10 |
| rpittau | my only problem is that I have double meetings every time (until next time change) and I can't really run the ironic one :/ | 15:10 |
| iurygregory | rpittau, 0% of chance to change the downstream one? (just wondering) | 15:11 |
| TheJulia | I don't think we can realistically change in december, but maybe starting in January? | 15:11 |
| iurygregory | maybe we can start a thread and open a doodle to see what people think about moving the time of the meeting for next year? | 15:12 |
| TheJulia | ++ | 15:13 |
| opendevreview | Merged openstack/ironic-python-agent stable/xena: Re-read the partition table with partx -a, part 2 https://review.opendev.org/c/openstack/ironic-python-agent/+/817419 | 15:14 |
| opendevreview | Verification of a change to openstack/ironic-python-agent bugfix/8.1 failed: Re-read the partition table with partx -a, part 2 https://review.opendev.org/c/openstack/ironic-python-agent/+/817420 | 15:14 |
| iurygregory | #action iurygregory to create a doodle for the new time so people can vote + email to openstack-discuss | 15:14 |
| iurygregory | we don't have any action items from previous meeting or the subteam status (skipping) | 15:15 |
| iurygregory | #topic Deciding on priorities for the coming week | 15:15 |
| iurygregory | #link https://review.opendev.org/q/status:open+hashtag:ironic-week-prio | 15:15 |
| rpittau | iurygregory: not really :/ | 15:16 |
| dtantsur | I have a few patches to make bifrost a happier place | 15:16 |
| iurygregory | \o/ | 15:16 |
| dtantsur | https://review.opendev.org/c/openstack/bifrost/+/818592 https://review.opendev.org/c/openstack/bifrost/+/818549 https://review.opendev.org/c/openstack/bifrost/+/817978 https://review.opendev.org/c/openstack/bifrost/+/818780 | 15:17 |
| iurygregory | I'm ok with it ( I can review them after dinner today :D) | 15:18 |
| TheJulia | https://review.opendev.org/c/openstack/ironic/+/818735 likely needs to be on that list | 15:19 |
| TheJulia | backport to train | 15:19 |
| iurygregory | ++ | 15:19 |
| iurygregory | I see that we have some backports in train that have -1 this will probably help =) | 15:19 |
| TheJulia | yup | 15:20 |
| iurygregory | anything else? | 15:21 |
| iurygregory | moving on =) | 15:22 |
| iurygregory | #topic Discussion | 15:23 |
| iurygregory | Let's talk about privsep \o/ | 15:23 |
| TheJulia | so, do we do it given memory footprints... I guess is my question | 15:24 |
| dtantsur | yep. has it occurred to you that ISO 9660 is a cursed technology? | 15:24 |
| * dtantsur reads about file name mangling and goes wtf | 15:24 | |
| dtantsur | anyway | 15:24 |
| dtantsur | we don't need root in ironic now that we don't have the iscsi deploy | 15:24 |
| TheJulia | dtantsur: an ancient and cursed technology, possibly developed by the grays to drive us crazy | 15:25 |
| dtantsur | there is only this awkward place in images.py that mounts an ISO, I think that's it | 15:25 |
| TheJulia | so as a service, does that really warrant privsep? | 15:25 |
| TheJulia | on the agent, we need root privs regardless | 15:25 |
| dtantsur | that's the question | 15:26 |
| dtantsur | okay, we also mount a vfat image, which I guess is not needed either (we could use mtools or anything else) | 15:26 |
| TheJulia | yeah | 15:26 |
| dtantsur | so my suggestion is: let's stop doing privileged operations in Ironic | 15:26 |
| dtantsur | and drop both rootwrap and privsep completely | 15:26 |
| TheJulia | dtantsur: I was about to suggest the same | 15:26 |
| iurygregory | kill with fire *-* | 15:26 |
| dtantsur | yah | 15:27 |
| dtantsur | we can start collecting a list of privileged operations to rewrite and assign people to them | 15:27 |
| iurygregory | this way we don't need to add privsep and we can also kill rootwrap | 15:27 |
| rloo | remind me, is/was privsep a community goal? | 15:27 |
| iurygregory | rloo, yup it was/is | 15:28 |
| rloo | thx. and i agree, if possible, don't use both. if not possible, i guess we'll need to use privsep... | 15:28 |
| iurygregory | we can avoid using, this is the point dtantsur is raising (if I understood correct =) ) | 15:29 |
| dtantsur | yep | 15:29 |
| rloo | right, we 'can'. we will see :D | 15:29 |
| dtantsur | I'll create a story and do some research. But I can use volunteers for actual coding. | 15:29 |
| iurygregory | dtantsur, you can count on me | 15:29 |
| rpittau | dtantsur: I can rpobably spend some time on that | 15:30 |
| iurygregory | since I was working on the privsep stuff I will work on this =) | 15:30 |
| iurygregory | so I think we are ok with this approach and let's see how it goes =) | 15:31 |
| * iurygregory will change the themes track this instead of privsep | 15:31 | |
| iurygregory | to track* | 15:31 |
| dtantsur | https://storyboard.openstack.org/#!/story/2009704 | 15:31 |
| dtantsur | I'll add subtasks as I find them (help welcome) | 15:31 |
| iurygregory | awesome | 15:32 |
| iurygregory | any questions about this? | 15:33 |
| iurygregory | ok, moving on | 15:35 |
| iurygregory | #topic Baremetal SIG | 15:35 |
| iurygregory | #link https://etherpad.opendev.org/p/bare-metal-sig | 15:35 |
| arne_wiebalck_ | The latest video is up, thanks again to stevebaker[m] for editing ! | 15:35 |
| arne_wiebalck_ | And I think that is mostly it for today :) | 15:36 |
| iurygregory | next meeting will be on Dec 14, with Overview of Metal3 by dtantsur =) | 15:36 |
| iurygregory | I don't think we have any RFE for review or topics for Open Discussion | 15:37 |
| iurygregory | skipping =) | 15:38 |
| iurygregory | #topic Who is going to run the next meeting? | 15:38 |
| dtantsur | I can do it to avoid jetlag problems :) | 15:38 |
| iurygregory | tks dtantsur =) | 15:39 |
| iurygregory | Thanks everyone! | 15:39 |
| dtantsur | thank you iurygregory! | 15:39 |
| TheJulia | Thanks! | 15:39 |
| iurygregory | #endmeeting | 15:39 |
| opendevmeet | Meeting ended Mon Nov 22 15:39:31 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:39 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2021/ironic.2021-11-22-15.00.html | 15:39 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2021/ironic.2021-11-22-15.00.txt | 15:39 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2021/ironic.2021-11-22-15.00.log.html | 15:39 |
| opendevreview | Merged openstack/ironic-python-agent stable/wallaby: Re-read the partition table with partx -a, part 2 https://review.opendev.org/c/openstack/ironic-python-agent/+/817421 | 15:53 |
| dtantsur | TheJulia: I assume the CI won't behave until we merge https://review.opendev.org/c/openstack/ironic/+/818458/ ? | 16:09 |
| dtantsur | iurygregory, rpittau or anyone still around ^^ | 16:09 |
| TheJulia | dtantsur: master branch, for the most part yes | 16:10 |
| dtantsur | k, let's see if someone can approve it now | 16:10 |
| TheJulia | dtantsur: I think I still may need to disable a job or re-tool but I lost track of mentally over the weekend | 16:10 |
| iurygregory | yup | 16:10 |
| dtantsur | well, the patch is green at least | 16:10 |
| TheJulia | Yeah, I also put a change into devstack last week | 16:11 |
| TheJulia | it is all a blur | 16:11 |
| iurygregory | lets see =) | 16:11 |
| dtantsur | https://review.opendev.org/c/openstack/devstack/+/818449 has merged | 16:11 |
| dtantsur | I'd also appreciate reviews on https://review.opendev.org/c/openstack/ironic/+/817386, it's blocking progress on ironic-sushy cross tests | 16:14 |
| rpittau | bye everyone! o/ | 16:22 |
| opendevreview | Dmitry Tantsur proposed openstack/sushy master: Migrate CompositionService, EventService and TaskService to enums https://review.opendev.org/c/openstack/sushy/+/818797 | 16:26 |
| *** sshnaidm is now known as sshnaidm|afk | 16:45 | |
| opendevreview | Dmitry Tantsur proposed openstack/sushy master: Migrate Manager constants to enums https://review.opendev.org/c/openstack/sushy/+/818806 | 16:48 |
| dtantsur | time for Glühwein, see you tomorrow :) | 16:50 |
| NobodyCam | Good Morning Ironic'ers | 17:08 |
| arne_wiebalck_ | dtantsur: enjoy ! | 17:11 |
| arne_wiebalck_ | Hi NobodyCam o/ | 17:11 |
| NobodyCam | hey hey arne_wiebalck_ :) | 17:11 |
| *** sshnaidm|afk is now known as sshnaidm | 17:16 | |
| * TheJulia screams into the void over our context hook code | 17:31 | |
| opendevreview | Merged openstack/ironic master: CI: Fix devstack plugin with RBAC changes https://review.opendev.org/c/openstack/ironic/+/818458 | 17:44 |
| opendevreview | Merged openstack/ironic master: Stop relying on explicit values of Redfish constants https://review.opendev.org/c/openstack/ironic/+/817386 | 17:50 |
| JayF | dtantsur: Did you know: you're the 4th most prolific reviewer all time in openstack per stackalytics https://www.stackalytics.io/?release=all | 17:55 |
| opendevreview | Arne Wiebalck proposed openstack/sushy master: [Trivial] Fix docstring typo: s/SECORE/SECURE/g https://review.opendev.org/c/openstack/sushy/+/818816 | 17:56 |
| * arne_wiebalck just realised the weekly prio list has multiple pages | 18:07 | |
| arne_wiebalck | I assume there is no way to remove merged changes from the prio list? Isn't there a risk that we lose prioritized items in the backlog? Sorry if that is all obvious ... | 18:10 |
| * arne_wiebalck looked at Redfish code too long today | 18:10 | |
| TheJulia | arne_wiebalck: add status:open | 18:14 |
| arne_wiebalck | TheJulia: life can be so easy sometimes :-D | 18:16 |
| arne_wiebalck | 18:16 | |
| arne_wiebalck | thx | 18:16 |
| *** lucasagomes_ is now known as lucasagomes | 18:22 | |
| arne_wiebalck | bye everyone o/ | 18:36 |
| stevebaker[m] | morning | 19:17 |
| *** sshnaidm is now known as sshnaidm|afk | 19:26 | |
| TheJulia | good morning | 19:30 |
| opendevreview | Merged openstack/ironic-python-agent master: Call execute from ironic-lib in hardware.py https://review.opendev.org/c/openstack/ironic-python-agent/+/816791 | 20:04 |
| TheJulia | stevebaker[m]: could you have a moment to review/approve https://review.opendev.org/c/openstack/ironic-python-agent/+/817019 | 20:11 |
| stevebaker[m] | TheJulia: sure thing. Is that depends-on still needed? its pointing to a stable/ussuri change anyway | 20:17 |
| TheJulia | stevebaker[m]: already merged, so doesn't matter at this point | 20:17 |
| stevebaker[m] | duh, I thought I was looking at a train change | 20:18 |
| TheJulia | heh | 20:19 |
| TheJulia | ahh, but we don't need the depends-on the train branch | 20:20 |
| * TheJulia fixes that | 20:20 | |
| TheJulia | oh | 20:20 |
| TheJulia | heh | 20:21 |
| TheJulia | no, we do, but the train one | 20:21 |
| TheJulia | what fun last week was | 20:21 |
| opendevreview | Merged openstack/bifrost master: Fix the DHCP port in iptables (68 vs 67 and TCP vs UDP) https://review.opendev.org/c/openstack/bifrost/+/818592 | 20:24 |
| stevebaker[m] | TheJulia: heh, thanks for fixing CI. again. | 20:25 |
| TheJulia | there is still more fallout to deal with | 20:25 |
| TheJulia | I just lost my brain on what should have been super easy on Friday, and haven't found my brain again | 20:25 |
| opendevreview | Julia Kreger proposed openstack/ironic master: WIP: Auto-populate lessee for deployments https://review.opendev.org/c/openstack/ironic/+/818641 | 21:10 |
| TheJulia | stevebaker[m]: https://review.opendev.org/c/openstack/ironic/+/818298 <-- should be a quick review | 21:11 |
| stevebaker[m] | that was indeed quick | 21:14 |
| TheJulia | \o/ | 21:17 |
| opendevreview | Merged openstack/ironic-python-agent stable/wallaby: Fix UEFI record regex https://review.opendev.org/c/openstack/ironic-python-agent/+/817014 | 21:27 |
| opendevreview | Merged openstack/ironic-python-agent stable/victoria: Fix UEFI record regex https://review.opendev.org/c/openstack/ironic-python-agent/+/817018 | 21:30 |
| opendevreview | Merged openstack/ironic-python-agent stable/ussuri: Output verbose info from efibootmgr https://review.opendev.org/c/openstack/ironic-python-agent/+/817019 | 21:32 |
| opendevreview | Merged openstack/ironic-python-agent stable/ussuri: Delete EFI boot entry duplicate labels first https://review.opendev.org/c/openstack/ironic-python-agent/+/817020 | 21:33 |
| opendevreview | Merged openstack/ironic-python-agent stable/ussuri: Fix UEFI record regex https://review.opendev.org/c/openstack/ironic-python-agent/+/817021 | 21:40 |
| opendevreview | Merged openstack/ironic master: Fix some of the SRBAC tests https://review.opendev.org/c/openstack/ironic/+/818298 | 21:40 |
| TheJulia | \o/ | 22:01 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/xena: Fix some of the SRBAC tests https://review.opendev.org/c/openstack/ironic/+/818741 | 22:02 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/wallaby: Fix some of the SRBAC tests https://review.opendev.org/c/openstack/ironic/+/818742 | 22:02 |
| TheJulia | just from a keeping things orderly standpoint | 22:02 |
| opendevreview | Merged openstack/ironic master: Use default test dir https://review.opendev.org/c/openstack/ironic/+/818524 | 22:10 |
| opendevreview | Merged openstack/bifrost master: Use UEFI by default and deprecate legacy boot https://review.opendev.org/c/openstack/bifrost/+/818549 | 22:48 |
| opendevreview | Merged openstack/bifrost master: Revert "Explicitly trap on ERR" and fix exit code hanlding https://review.opendev.org/c/openstack/bifrost/+/817978 | 22:52 |
| opendevreview | Julia Kreger proposed openstack/ironic master: WIP: Auto-populate lessee for deployments https://review.opendev.org/c/openstack/ironic/+/818641 | 22:53 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Add FIPS jobs https://review.opendev.org/c/openstack/ironic/+/797739 | 23:03 |
| opendevreview | Merged openstack/ironic stable/train: Fix redfish-virtual-media file permission https://review.opendev.org/c/openstack/ironic/+/818183 | 23:11 |
| opendevreview | Merged openstack/ironic master: Create node get_interface method https://review.opendev.org/c/openstack/ironic/+/817086 | 23:28 |
| janders | good morning Ironic o/ | 23:48 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!