clif | o/ | 00:00 |
---|---|---|
cid | clif, \o | 03:47 |
opendevreview | Syed Haseeb Ahmed proposed openstack/ironic master: control update ports logic with update_pxe_enabled config flag https://review.opendev.org/c/openstack/ironic/+/951114 | 07:02 |
opendevreview | Takashi Kajinami proposed openstack/python-ironicclient master: Exclude test code from coverage report https://review.opendev.org/c/openstack/python-ironicclient/+/951370 | 07:25 |
opendevreview | Takashi Kajinami proposed openstack/ironic-ui master: Exclude test code from coverage report https://review.opendev.org/c/openstack/ironic-ui/+/951374 | 07:28 |
adam-metal3 | Hello Ironic! | 07:32 |
adam-metal3 | I was wondering that if I would use https url for the http_url option of Ironic (in the config file), but the https server in question would use a self signed cert, is there an elegant way to specify what cert to use? because I don't see config options related to that, or will the http lib just pic it up from some default location? | 07:34 |
adam-metal3 | ir is it the one from the "default" group of configs "webserver_verify_ca" | 07:39 |
adam-metal3 | ? | 07:39 |
opendevreview | Syed Haseeb Ahmed proposed openstack/ironic master: control update ports logic with update_pxe_enabled config flag https://review.opendev.org/c/openstack/ironic/+/951114 | 10:09 |
opendevreview | Syed Haseeb Ahmed proposed openstack/ironic master: Control port updates with update_pxe_enabled flag https://review.opendev.org/c/openstack/ironic/+/951114 | 10:15 |
dtantsur | adam-metal3: hey, what are you going to pass the certificate *to*? the iPXE firmware will need it pre-baked, IPA to unless you use virtual media | 10:17 |
dtantsur | welcome (back?) clif! | 10:49 |
adam-metal3 | dtantsur, I want Ironic to use it when it verifys the kernel and initramfs urls in case the http server hosting them uses https, ipxe will take them other port and fir ipxe the cert will be baked in so it is just for ironic , specifically to be used in ipxe_utils cache_ramdisk_kernel | 12:17 |
dtantsur | adam-metal3: webserver_verify_ca should be the right option I think (it has nothing to do with http_url hence my confusion) | 12:17 |
adam-metal3 | dtantsur, for some resaon in the pxe_utils I think it always verifies against http_url by default because it tries to check the url an it seem to construct the url based on what I have in the "http_url" and e.g. if it fails it prints this "Validation of image href http://172.22.0.2:6180/images/ironic-python-agent.kernel failed," I don't have this url specified anywhere else in config | 12:20 |
adam-metal3 | in ironic config the kernel and the initramfs is only refferenced by "file://" | 12:21 |
dtantsur | adam-metal3: the URL is provided by BMO based on its environment variables. The fact that http_url has the same host is a coincidence. | 12:21 |
dtantsur | adam-metal3: I have a TODO item on BMO to stop requiring kernel/initramfs since, as you noticed, ironic-image already has defaults. It hasn't been done yet. | 12:22 |
adam-metal3 | dtantsur, ooo right bm also passes stuf to the API yes yes indeed srry for my confusion | 12:22 |
dtantsur | https://github.com/metal3-io/baremetal-operator/issues/2100 | 12:22 |
adam-metal3 | I might look into that issue then at some point I am doing stoff around this topic now anywas, but for my work I will need webserver_verify_ca anyhow, Tnaks for the clarification | 12:23 |
dtantsur | adam-metal3: if you do wire webserver_verify_ca in, please consider at least leaving an RFE on IrSO to wire the same | 12:25 |
dtantsur | so that we don't end up with feature regressions when (hopefully? ever?) we switch over to IrSO | 12:25 |
adam-metal3 | yes I will, it is mostly there, there is a corresponding env var in ironic-image already, I will start utilizing it from now on tho | 12:27 |
TheJulia | good morning | 13:05 |
* TheJulia sips coffee in a likely vain attempt to wake up | 13:06 | |
TheJulia | hmm, close on the multinode job but it appears we may be on track for tls fun next | 13:19 |
opendevreview | Julia Kreger proposed openstack/ironic master: DNM: CI Science - Expand the multinode job https://review.opendev.org/c/openstack/ironic/+/950206 | 13:20 |
opendevreview | Mithun Krishnan Umesan proposed openstack/networking-generic-switch master: Improve NGS documentation. Some minor nitpicks and changes https://review.opendev.org/c/openstack/networking-generic-switch/+/951022 | 13:22 |
* TheJulia makes more coffee | 13:37 | |
opendevreview | Julia Kreger proposed openstack/ironic master: Patch configdrive metadata https://review.opendev.org/c/openstack/ironic/+/946677 | 14:40 |
opendevreview | Julia Kreger proposed openstack/ironic master: DNM: CI Science - Expand the multinode job https://review.opendev.org/c/openstack/ironic/+/950206 | 14:53 |
TheJulia | okay, lets see if that works... or explodes | 14:53 |
TheJulia | well, seems to accept it, so that is a good sign | 14:54 |
opendevreview | Julia Kreger proposed openstack/ironic master: Consider missing MTU invalid metadata https://review.opendev.org/c/openstack/ironic/+/949385 | 15:07 |
opendevreview | Verification of a change to openstack/ironic-python-agent stable/2025.1 failed: netutils: Use ethtool ioctl to get permanent mac address https://review.opendev.org/c/openstack/ironic-python-agent/+/950489 | 15:47 |
opendevreview | Merged openstack/python-ironic-inspector-client master: Drop license classifier https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/951309 | 15:48 |
opendevreview | Merged openstack/python-ironicclient master: Exclude test code from coverage report https://review.opendev.org/c/openstack/python-ironicclient/+/951370 | 15:52 |
opendevreview | Jay Faulkner proposed openstack/ironic master: Update devstack-guide to use neutron- not q- https://review.opendev.org/c/openstack/ironic/+/951424 | 16:09 |
opendevreview | Merged openstack/ironic-ui master: Exclude test code from coverage report https://review.opendev.org/c/openstack/ironic-ui/+/951374 | 16:18 |
JayF | +++/opt/stack/ironic/devstack/lib/ironic:create_bridge_and_vms:2469 oscwrap --os-cloud devstack-admin network show private -c id -f value | 16:25 |
JayF | public endpoint for compute service in RegionOne region not found | 16:25 |
JayF | why does oscwrap network show want a compute service? | 16:25 |
JayF | this is using the standalone Ironic devstack config | 16:25 |
JayF | I figured it out; tldr it helps if you run *neutron*-api instead of *neurton*-api (or however I typo'd it) and apparently osc falls back to compute endpoint if network isn't there (assuming nova-network compat?) | 16:39 |
* TheJulia blinks | 17:14 | |
TheJulia | fun | 17:14 |
opendevreview | Merged openstack/ironic master: Add port/portgroup list conductor groups filter https://review.opendev.org/c/openstack/ironic/+/862292 | 17:16 |
opendevreview | Merged openstack/ironic master: Control port updates with update_pxe_enabled flag https://review.opendev.org/c/openstack/ironic/+/951114 | 17:16 |
opendevreview | Verification of a change to openstack/ironic-tempest-plugin master failed: Remove stable/2023.2 job https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/951229 | 17:19 |
TheJulia | looks like defaults have also been changed to force ovn | 17:20 |
JayF | TIL that it seems the order of enable_service commands can cause changes in behavior | 17:20 |
JayF | at least that's the existing hypothesis | 17:20 |
TheJulia | multinode jobs are now erroring trying to install/use ovn becaues the dhcp agent is present and its not a devstack change | 17:20 |
JayF | I have the opposite problem, working on documented devstack contributor configs and dhcp agent is *not* running | 17:20 |
JayF | with ovs | 17:20 |
JayF | I'm retrying now with ordering changes | 17:20 |
TheJulia | nothing like https://zuul.opendev.org/t/openstack/build/9ce1fd40397e427680ea77634ab44bc3 ? | 17:21 |
JayF | mine bailed during setup unable to find the network namespace | 17:21 |
TheJulia | Fun | 17:22 |
JayF | I checked and it was no agent started yet per logs | 17:22 |
JayF | and the logs were right; so I'm guessing maybe neutron-dhcp being before neutron-api is meaningful? I hope? | 17:22 |
JayF | TheJulia: that "no dhcp agent with ovn" is straight up incorrect in the ironic use case, yeah? | 17:22 |
TheJulia | Yeah, that sounds like an entirely different issue | 17:22 |
TheJulia | Strait up incorrect and wrong | 17:23 |
JayF | OK and it's in neutron then | 17:23 |
JayF | wait, q-agt/neturon-agent | 17:25 |
JayF | is not dhcp | 17:25 |
JayF | neutron-dhcp / q-dhcp is dhcp | 17:25 |
JayF | what is neutron-agent then? | 17:25 |
JayF | a52041cd3 (Slawek Kaplonski 2022-11-18 11:39:56 +0100 363) if is_service_enabled q-agt neutron-agent; then | 17:25 |
JayF | since 2022 so there's something else afoot | 17:26 |
TheJulia | Neutron l3-agent perhaps | 17:26 |
JayF | l3 agent is separate | 17:27 |
JayF | neutron-l3 | 17:27 |
JayF | TheJulia: this implies to me not running agent in ovn setups is accurate https://usercontent.irccloud-cdn.com/file/GXPRPAZK/image.png | 17:29 |
TheJulia | oh, I see what is going on with that job | 17:29 |
TheJulia | at least in my case | 17:29 |
JayF | I think mine is ordering | 17:30 |
opendevreview | Merged openstack/networking-generic-switch master: Improve NGS documentation. Some minor nitpicks and changes https://review.opendev.org/c/openstack/networking-generic-switch/+/951022 | 17:30 |
TheJulia | mine is layering of config | 17:30 |
TheJulia | because I have to have a group definition, and then host deifnitions | 17:30 |
TheJulia | definitions | 17:30 |
opendevreview | Julia Kreger proposed openstack/ironic master: DNM: CI Science - Expand the multinode job https://review.opendev.org/c/openstack/ironic/+/950206 | 17:31 |
TheJulia | lets see if that gets us closer | 17:31 |
JayF | same, I've got another cycle running | 17:32 |
TheJulia | cool cool | 17:33 |
JayF | I'm all the sudden /very very glad/ I took time yesterday to automate my VM creation on my hypervisor lol | 17:33 |
TheJulia | heh | 17:43 |
JayF | whelp, that failed in an identical way | 17:44 |
JayF | I think something is weird about how things are ordering but I'm not 100% sure | 17:44 |
JayF | TheJulia: you have time now to talk thru this with me? | 17:44 |
TheJulia | yeah | 17:46 |
JayF | https://us06web.zoom.us/j/85779536876?pwd=2337Vv7tiShA3QQX6ktuRiTcpSnDhR.1 | 17:47 |
opendevreview | Jay Faulkner proposed openstack/ironic master: Update devstack-guide to use neutron- not q- https://review.opendev.org/c/openstack/ironic/+/951424 | 18:38 |
opendevreview | Julia Kreger proposed openstack/ironic master: DNM: CI Science - Expand the multinode job https://review.opendev.org/c/openstack/ironic/+/950206 | 19:07 |
opendevreview | Merged openstack/ironic-python-agent stable/2025.1: netutils: Use ethtool ioctl to get permanent mac address https://review.opendev.org/c/openstack/ironic-python-agent/+/950489 | 19:46 |
opendevreview | Julia Kreger proposed openstack/ironic master: DNM: CI Science - Expand the multinode job https://review.opendev.org/c/openstack/ironic/+/950206 | 20:53 |
*** darmach9 is now known as darmach | 21:05 | |
opendevreview | Julia Kreger proposed openstack/ironic master: CI: remove legacy devstack baremetal admin and observer role usage https://review.opendev.org/c/openstack/ironic/+/951445 | 21:13 |
opendevreview | Julia Kreger proposed openstack/ironic master: DNM: CI Science - Expand the multinode job https://review.opendev.org/c/openstack/ironic/+/950206 | 23:43 |
TheJulia | I guess time will tell if that does it... but close. | 23:43 |
opendevreview | minwoo seo proposed openstack/ironic master: Add `api-call` action for ironic inspection rule https://review.opendev.org/c/openstack/ironic/+/946741 | 23:51 |
opendevreview | minwoo seo proposed openstack/ironic master: Add `api-call` action for ironic inspection rule https://review.opendev.org/c/openstack/ironic/+/946741 | 23:54 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!