*** pmannidi is now known as pmannidi|brb | 04:23 | |
*** pmannidi|brb is now known as pmannidi | 05:20 | |
opendevreview | Vanou Ishii proposed openstack/ironic-python-agent master: Rescan device after filesystem creation https://review.opendev.org/c/openstack/ironic-python-agent/+/827936 | 06:40 |
---|---|---|
arne_wiebalck | Good morning, Ironic! | 07:31 |
rpittau | good morning ironic! o/ | 08:18 |
janders | hey arne_wiebalck rpittau and Ironic o/ | 10:10 |
rpittau | hey janders :) | 10:10 |
arne_wiebalck | hey janders and rpittau o/ | 10:12 |
rpittau | hey arne_wiebalck :) | 10:12 |
dtantsur | morning folks | 11:07 |
opendevreview | Merged openstack/bifrost master: Enable epel repository only when needed https://review.opendev.org/c/openstack/bifrost/+/826442 | 11:08 |
iurygregory | good morning Ironic | 11:09 |
janders | hey dtantsur iurygregory o/ | 11:10 |
opendevreview | Verification of a change to openstack/ironic master failed: Fix the benchmark job https://review.opendev.org/c/openstack/ironic/+/828498 | 11:43 |
opendevreview | Merged openstack/ironic master: Add idrac-redfish clean steps to not require ramdisk https://review.opendev.org/c/openstack/ironic/+/825753 | 12:03 |
opendevreview | Merged openstack/tenks master: Explicitly set boot_mode to legacy BIOS https://review.opendev.org/c/openstack/tenks/+/827304 | 12:28 |
opendevreview | Verification of a change to openstack/bifrost master failed: Do not clone repositories that are not used https://review.opendev.org/c/openstack/bifrost/+/826292 | 13:10 |
janders | see you tomorrow Ironic o/ | 13:20 |
opendevreview | Merged openstack/ironic master: Fix the benchmark job https://review.opendev.org/c/openstack/ironic/+/828498 | 14:08 |
TheJulia | good morning | 14:10 |
* TheJulia wonders where the freight train came from | 14:10 | |
MahnoorAsghar | Good morning! | 14:21 |
*** pmannidi is now known as pmannidi|AFK | 14:25 | |
dtantsur | morning TheJulia, MahnoorAsghar | 14:41 |
dtantsur | TheJulia: choochoopenstack? :) | 14:41 |
rpittau | good morning TheJulia dtantsur :) | 14:41 |
dtantsur | now that I've fixed the bifrost upgrade job, we're seeing some instability there.. | 14:43 |
dtantsur | Unable 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 |
dtantsur | this is interesting, ironic API cannot talk to the conductor on the same host | 14:44 |
dtantsur | I wonder if it's v6-vs-v4 problem | 14:46 |
opendevreview | Dmitry 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/+/828492 | 15:07 |
jamesdenton | good 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 priteau | 16:11 | |
*** tkajinam is now known as Guest210 | 16:30 | |
*** priteau is now known as priteau_ | 16:48 | |
*** priteau_ is now known as priteau | 16:48 | |
dtantsur | jamesdenton: where is this error happening, in iPXE? | 16:56 |
dtantsur | I wonder if swift returns it something like an HTML page with HTTP 404 :) | 16:56 |
jamesdenton | i don't think so, as i was able to d/l with curl OK. https://postimg.cc/754sF1Gt | 16:59 |
dtantsur | jamesdenton: and it's the actual image, not something weird? | 16:59 |
dtantsur | I mean, what you get with curl? | 17:00 |
jamesdenton | i'll have to retry here shortly and i'll check the hash | 17:00 |
rpittau | good night! o/ | 17:00 |
dtantsur | jamesdenton: also paste the full iPXE script from pxelinux.cfg/blah-blah | 17:01 |
dtantsur | https://forum.ipxe.org/showthread.php?tid=18186 may provide some useful background | 17:02 |
dtantsur | your images don't end with deploy_ramdisk, but rather with a UUID. I think that's the problem. | 17:03 |
opendevreview | Dmitry Tantsur proposed openstack/ironic master: [WIP] Deprecate instance network boot https://review.opendev.org/c/openstack/ironic/+/828746 | 17:04 |
jamesdenton | yes, i thought that might be an issue. https://paste.opendev.org/show/bcU5OrZxuKt0p9dEXXAZ/ | 17:05 |
jamesdenton | initrd=deploy_ramdisk | 17:05 |
dtantsur | yep. lemme think what it should be instead.. | 17:05 |
jamesdenton | i actually just finished updating the pxelinux.cfg to use the UUID of the glance image, and that worked | 17:05 |
dtantsur | jamesdenton: if I make a patch, will you be able to test it? | 17:06 |
jamesdenton | initrd=98baaacc-f9a9-4c8c-ba0f-7e9eece17921 | 17:06 |
jamesdenton | yes | 17:06 |
dtantsur | okie, give me 10 minutes | 17:06 |
jamesdenton | sure thing | 17:06 |
opendevreview | Dmitry Tantsur proposed openstack/ironic master: [WIP] Set correct initrd_filename for iPXE when using Swift https://review.opendev.org/c/openstack/ironic/+/828770 | 17:12 |
dtantsur | jamesdenton: what about this ^^^? | 17:12 |
jamesdenton | looking | 17:12 |
jamesdenton | looks reasonable | 17:14 |
jamesdenton | lemme patch it up and i'll let you know in about 10-15 | 17:14 |
opendevreview | Merged openstack/bifrost master: Do not clone repositories that are not used https://review.opendev.org/c/openstack/bifrost/+/826292 | 17:18 |
jamesdenton | dtantsur looking good so far | 17:24 |
jamesdenton | booting the ramdisk nw | 17:24 |
jamesdenton | thanks for the quick turnaround on that. glad i'm not crazy | 17:24 |
dtantsur | jamesdenton: nice! I'll finish this patch later today | 17:28 |
jamesdenton | thank you! | 17:31 |
opendevreview | Dmitry 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/+/828492 | 17:44 |
opendevreview | Dmitry Tantsur proposed openstack/ironic master: Set correct initrd_filename for iPXE when using Swift https://review.opendev.org/c/openstack/ironic/+/828770 | 17:51 |
dtantsur | jamesdenton: now with tests and a release note ^^ | 17:51 |
jamesdenton | woot | 17:53 |
dtantsur | I guess this issue means that we don't have coverage for ipxe_use_swift in the CI :-/ | 17:54 |
TheJulia | what?! I thought we did | 17:55 |
TheJulia | then again, it is entirely possible that it went away quietly at some point | 17:55 |
TheJulia | arne_wiebalck: do you folks use ironic-python-agent-builder from RDO rpms or from source? | 17:56 |
dtantsur | we don't, and I think configdrive_use_swift either | 17:56 |
arne_wiebalck | TheJulia: from source | 17:56 |
TheJulia | arne_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 then | 17:57 |
TheJulia | dtantsur: amazing how much has changed | 17:57 |
arne_wiebalck | TheJulia: that should be correct | 17:58 |
dtantsur | we have so many options... | 17:58 |
dtantsur | if I manage to reduce the number of jobs soon(ish), we may be able to increase coverage again | 17:58 |
TheJulia | ++ | 17:58 |
arne_wiebalck | dtantsur: options are the operators' friend (if the default are good :-D) | 17:58 |
TheJulia | we've got some overlap/duplication for reasons, I'd love to see some of it peeled back | 17:58 |
arne_wiebalck | *defaults | 17:58 |
dtantsur | TheJulia: we literally run the same test twice in non-standalone jobs :D | 17:59 |
TheJulia | dtantsur: I know :( | 17:59 |
dtantsur | and there is a duplication with standalone jobs.. | 17:59 |
dtantsur | I'm working on untangle this mess | 17:59 |
dtantsur | o/ | 19:00 |
TheJulia | goodnight | 19:16 |
stevebaker[m] | good morning | 19: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/+/790878 | 19:56 |
TheJulia | stevebaker[m]: approved | 20:10 |
stevebaker[m] | thanks | 20:12 |
arne_wiebalck | bye everyone, see you tomorrow o/ | 20:19 |
*** osmanlicilegi is now known as Guest0 | 23:49 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!