Tuesday, 2021-05-11

openstackgerritIan Wienand proposed openstack/diskimage-builder master: Add fedora-containerfile element  https://review.opendev.org/c/openstack/diskimage-builder/+/79036505:18
openstackgerritIan Wienand proposed openstack/diskimage-builder master: bootloader: disable BLS for Fedora  https://review.opendev.org/c/openstack/diskimage-builder/+/79054905:18
ianwstevebaker / cgoncalves: ^ i'm kind of surprised we haven't hit this in extant work05:18
ianwthe symptom is that you have /boot/loader/entries/* that have the wrong root device -- it seems "kernel-install" makes these entries and I'm really not sure how it chooses the root= , but it chooses wrong05:19
ianwthere's probably some way to override this05:19
*** cgoncalves has quit IRC05:27
*** cgoncalves has joined #openstack-dib05:27
*** prometheanfire has quit IRC05:27
*** prometheanfire has joined #openstack-dib05:28
openstackgerritIan Wienand proposed openstack/diskimage-builder master: bootloader: disable BLS for Fedora  https://review.opendev.org/c/openstack/diskimage-builder/+/79054906:49
openstackgerritIan Wienand proposed openstack/diskimage-builder master: Add fedora-containerfile element  https://review.opendev.org/c/openstack/diskimage-builder/+/79036506:49
cgoncalvesianw, thanks for the patch! I will have a look at it asap. The CentOS Stream 9 builds may also be hitting this as there was some issue at boot ("dracut-initqueue[246]: Warning: Could not boot") a couple of days ago07:17
*** rpittau|afk is now known as rpittau07:38
ianwcgoncalves: yay, got the containerfile job working -> https://zuul.opendev.org/t/openstack/build/8d7b05f13ba4400dbc5b8edfde552a7208:46
cgoncalvesianw, nice!! I was also able to create and boot a centos stream 9 image with your BLS patch slightly modified to apply to centos too08:49
*** dtantsur|afk is now known as dtantsur10:15
*** iurygregory has quit IRC14:22
*** iurygregory has joined #openstack-dib14:22
*** sdanni has joined #openstack-dib15:44
sdanniHi! I submitted a change to add a keylime-agent element recently (https://review.opendev.org/c/openstack/diskimage-builder/+/789601).  This change is related to ironic security interface spec (https://review.opendev.org/c/openstack/ironic-specs/+/576718) and TheJulia is the main reviewer. I got the feedback from reviewer lan that I should discuss with people involved if this change should be submitted here. I guess this16:07
sdanniis the right place to have a conversation about it?16:07
clarkbsdanni: yes this is the correct venue for discussion (either here or the mailing list). ianw is the nick of your reviewer16:14
clarkbsdanni: reading ianw's comment I think what ianw is saying is that you can have the element in ironic and not need to keep it maintained in dib16:15
clarkbthe trouble with keeping it in dib is the dib maintainers need to then keep it up todate and they are unlikely to be the best individuals as they are not coupled to the use case16:15
clarkbTheJulia: ^ fyi16:15
clarkbas an example we keep a number of use case specific elements at https://opendev.org/openstack/project-config/src/branch/master/nodepool/elements and don't put them in dib16:17
sdanniclarkb: thanks! Do you mean I should put the element in ironic-python-agent-builder?16:19
*** dtantsur is now known as dtantsur|afk16:32
*** rpittau is now known as rpittau|afk16:36
clarkbsdanni: yes that is what ianw suggests; however, ianw also notes that he doesn't want you to go in circles so we should have TheJulia weigh in too before you do that16:39
sdannicool!16:41
clarkbdtantsur|afk: ^ may also be able to weigh in18:18
*** yoctozepto has quit IRC19:13
*** yoctozepto6 has joined #openstack-dib19:13
*** dhill has quit IRC21:58
*** dhill has joined #openstack-dib22:00
stevebakersdanni: hey, this is where IPA specific dib elements live https://opendev.org/openstack/ironic-python-agent-builder/src/branch/master/dib22:03
ianwstevebaker: i have a "bad" image, i can try removing the option23:28
ianwi think we need to rewrite parts of the bootloader to use grubby and "grubby --update-kernel=ALL --args=..."23:29
stevebakerianw: ack23:45
ianwnow i think about it, i'm sure i had a bit of a bootloader cleanup patch at some point ...23:51
ianwhttps://review.opendev.org/c/openstack/diskimage-builder/+/54112923:54
ianwFeb 06, 2018 4:13 PM23:55

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!