Thursday, 2022-02-10

*** pmannidi is now known as pmannidi|brb04:23
*** pmannidi|brb is now known as pmannidi05:20
opendevreviewVanou Ishii proposed openstack/ironic-python-agent master: Rescan device after filesystem creation  https://review.opendev.org/c/openstack/ironic-python-agent/+/82793606:40
arne_wiebalckGood morning, Ironic!07:31
rpittaugood morning ironic! o/08:18
jandershey arne_wiebalck rpittau and Ironic o/10:10
rpittauhey janders :)10:10
arne_wiebalckhey janders and rpittau o/10:12
rpittauhey arne_wiebalck :)10:12
dtantsurmorning folks11:07
opendevreviewMerged openstack/bifrost master: Enable epel repository only when needed  https://review.opendev.org/c/openstack/bifrost/+/82644211:08
iurygregorygood morning Ironic 11:09
jandershey dtantsur iurygregory o/11:10
opendevreviewVerification of a change to openstack/ironic master failed: Fix the benchmark job  https://review.opendev.org/c/openstack/ironic/+/82849811:43
opendevreviewMerged openstack/ironic master: Add idrac-redfish clean steps to not require ramdisk  https://review.opendev.org/c/openstack/ironic/+/82575312:03
opendevreviewMerged openstack/tenks master: Explicitly set boot_mode to legacy BIOS  https://review.opendev.org/c/openstack/tenks/+/82730412:28
opendevreviewVerification of a change to openstack/bifrost master failed: Do not clone repositories that are not used  https://review.opendev.org/c/openstack/bifrost/+/82629213:10
janderssee you tomorrow Ironic o/13:20
opendevreviewMerged openstack/ironic master: Fix the benchmark job  https://review.opendev.org/c/openstack/ironic/+/82849814:08
TheJuliagood morning14:10
* TheJulia wonders where the freight train came from14:10
MahnoorAsgharGood morning!14:21
*** pmannidi is now known as pmannidi|AFK14:25
dtantsurmorning TheJulia, MahnoorAsghar 14:41
dtantsurTheJulia: choochoopenstack? :)14:41
rpittaugood morning TheJulia dtantsur :)14:41
dtantsurnow that I've fixed the bifrost upgrade job, we're seeing some instability there..14:43
dtantsurUnable to establish connection to http://centos-8-stream-ovh-bhs1-0028399920:8089: HTTPConnectionPool(host='centos-8-stream-ovh-bhs1-0028399920', port=8089): Max retries exceeded with url: / (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x7f3b14890358>: Failed to establish a new connection: [Errno 101] ENETUNREACH',))14:44
dtantsurthis is interesting, ironic API cannot talk to the conductor on the same host14:44
dtantsurI wonder if it's v6-vs-v4 problem14:46
opendevreviewDmitry Tantsur proposed openstack/ironic-tempest-plugin master: Make the basic_ops tests friendlier to non-Cirros images  https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/82849215:07
jamesdentongood morning, all. I recently switched from file-backed to swift-backed (radosgw) images for Ironic (using tempurl), and am finding that while iPXE appears to download the kernel and ramdisk image OK, I get an error "Failed to open file: deploy_ramdisk". Just wondering if anyone has come across that. Thanks.15:55
*** priteau_ is now known as priteau16:11
*** tkajinam is now known as Guest21016:30
*** priteau is now known as priteau_16:48
*** priteau_ is now known as priteau16:48
dtantsurjamesdenton: where is this error happening, in iPXE?16:56
dtantsurI wonder if swift returns it something like an HTML page with HTTP 404 :)16:56
jamesdentoni don't think so, as i was able to d/l with curl OK. https://postimg.cc/754sF1Gt16:59
dtantsurjamesdenton: and it's the actual image, not something weird?16:59
dtantsurI mean, what you get with curl?17:00
jamesdentoni'll have to retry here shortly and i'll check the hash17:00
rpittaugood night! o/17:00
dtantsurjamesdenton: also paste the full iPXE script from pxelinux.cfg/blah-blah17:01
dtantsurhttps://forum.ipxe.org/showthread.php?tid=18186 may provide some useful background17:02
dtantsuryour images don't end with deploy_ramdisk, but rather with a UUID. I think that's the problem.17:03
opendevreviewDmitry Tantsur proposed openstack/ironic master: [WIP] Deprecate instance network boot  https://review.opendev.org/c/openstack/ironic/+/82874617:04
jamesdentonyes, i thought that might be an issue. https://paste.opendev.org/show/bcU5OrZxuKt0p9dEXXAZ/17:05
jamesdentoninitrd=deploy_ramdisk17:05
dtantsuryep. lemme think what it should be instead..17:05
jamesdentoni actually just finished updating the pxelinux.cfg to use the UUID of the glance image, and that worked17:05
dtantsurjamesdenton: if I make a patch, will you be able to test it?17:06
jamesdentoninitrd=98baaacc-f9a9-4c8c-ba0f-7e9eece1792117:06
jamesdentonyes17:06
dtantsurokie, give me 10 minutes17:06
jamesdentonsure thing17:06
opendevreviewDmitry Tantsur proposed openstack/ironic master: [WIP] Set correct initrd_filename for iPXE when using Swift  https://review.opendev.org/c/openstack/ironic/+/82877017:12
dtantsurjamesdenton: what about this ^^^?17:12
jamesdentonlooking17:12
jamesdentonlooks reasonable17:14
jamesdentonlemme patch it up and i'll let you know in about 10-1517:14
opendevreviewMerged openstack/bifrost master: Do not clone repositories that are not used  https://review.opendev.org/c/openstack/bifrost/+/82629217:18
jamesdentondtantsur looking good so far17:24
jamesdentonbooting the ramdisk nw17:24
jamesdentonthanks for the quick turnaround on that. glad i'm not crazy17:24
dtantsurjamesdenton: nice! I'll finish this patch later today17:28
jamesdentonthank you!17:31
opendevreviewDmitry Tantsur proposed openstack/ironic-tempest-plugin master: Make the basic_ops tests friendlier to non-Cirros images  https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/82849217:44
opendevreviewDmitry Tantsur proposed openstack/ironic master: Set correct initrd_filename for iPXE when using Swift  https://review.opendev.org/c/openstack/ironic/+/82877017:51
dtantsurjamesdenton: now with tests and a release note ^^17:51
jamesdentonwoot17:53
dtantsurI guess this issue means that we don't have coverage for ipxe_use_swift in the CI :-/17:54
TheJuliawhat?! I thought we did17:55
TheJuliathen again, it is entirely possible that it went away quietly at some point17:55
TheJuliaarne_wiebalck: do you folks use ironic-python-agent-builder from RDO rpms or from source?17:56
dtantsurwe don't, and I think configdrive_use_swift either17:56
arne_wiebalckTheJulia: from source17:56
TheJuliaarne_wiebalck: okay, so if I pin back the version to a more appropriate version in RDO, its not going to blow up for at least you folks who are the only ones who have added anything feature-ish into the builder since then17:57
TheJuliadtantsur: amazing how much has changed17:57
arne_wiebalckTheJulia: that should be correct17:58
dtantsurwe have so many options...17:58
dtantsurif I manage to reduce the number of jobs soon(ish), we may be able to increase coverage again17:58
TheJulia++17:58
arne_wiebalckdtantsur: options are the operators' friend (if the default are good :-D)17:58
TheJuliawe've got some overlap/duplication for reasons, I'd love to see some of it peeled back17:58
arne_wiebalck*defaults17:58
dtantsurTheJulia: we literally run the same test twice in non-standalone jobs :D17:59
TheJuliadtantsur: I know :(17:59
dtantsurand there is a duplication with standalone jobs..17:59
dtantsurI'm working on untangle this mess17:59
dtantsuro/19:00
TheJuliagoodnight19:16
stevebaker[m]good morning19:47
stevebaker[m]TheJulia: Hey could I trouble you for a review on this? Its a bootloader cleanup which I want to build on to fix the overcloud-full+centos9 https://review.opendev.org/c/openstack/diskimage-builder/+/79087819:56
TheJuliastevebaker[m]: approved20:10
stevebaker[m]thanks20:12
arne_wiebalckbye everyone, see you tomorrow o/20:19
*** osmanlicilegi is now known as Guest023:49

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