| opendevreview | Julia Kreger proposed openstack/ironic master: WIP: Special case lenovo boot setup https://review.opendev.org/c/openstack/ironic/+/908946 | 00:43 |
|---|---|---|
| opendevreview | Julia Kreger proposed openstack/ironic master: httpboot enabled CI stuffs https://review.opendev.org/c/openstack/ironic/+/908955 | 00:49 |
| opendevreview | Julia Kreger proposed openstack/ironic master: ci: phase out ironic-standalone CI job https://review.opendev.org/c/openstack/ironic/+/908959 | 00:50 |
| opendevreview | Julia Kreger proposed openstack/ironic master: ci: fix ipmi partition job name https://review.opendev.org/c/openstack/ironic/+/908956 | 00:50 |
| opendevreview | Jacob Anders proposed openstack/sushy-tools master: Add missing whitespaces in test_virtual_media_certificates() https://review.opendev.org/c/openstack/sushy-tools/+/909065 | 04:40 |
| rpittau | good morning ironic! o/ | 07:43 |
| rpittau | JayF: as I mentioned before starting the current cycle I intend to propose myself for PTL for the next one, any help/advice is very welcome :) | 07:44 |
| opendevreview | Damien RANNOU proposed openstack/ironic-python-agent master: USB device discovery https://review.opendev.org/c/openstack/ironic-python-agent/+/902769 | 08:41 |
| opendevreview | Merged openstack/sushy-tools master: Add missing whitespaces in test_virtual_media_certificates() https://review.opendev.org/c/openstack/sushy-tools/+/909065 | 08:43 |
| tkajinam | it seems https://review.opendev.org/c/openstack/networking-baremetal/+/906788 depends on an abandoned change so is not going to gate | 08:52 |
| tkajinam | I've seen the networking-baremetal job has been broken and we really have to restore https://review.opendev.org/c/openstack/networking-baremetal/+/906923 but I'm not too sure seeing that one is abandoned | 08:55 |
| *** jph5 is now known as jph | 13:45 | |
| JayF | rpittau: Do or do not. There is no try. 😂 | 13:58 |
| rpittau | JayF: sounds like a great advice! :) | 13:58 |
| JayF | Seriously I'm excited, you'll do great | 13:58 |
| rpittau | thanks1 | 13:58 |
| JayF | Do not take that as a discouragement to run if you're someone else seeing this. I think actually having opposed elections are super healthy for community | 13:58 |
| rpittau | as far as we don't end up rolling dice to decide, I'm totally up for it! :D | 13:59 |
| TheJulia | good morning | 14:06 |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal master: [codespell] Fixing Spelling Mistakes https://review.opendev.org/c/openstack/networking-baremetal/+/906788 | 14:07 |
| TheJulia | just removing the dependency ^ that tkajinam pointed out above. Since the break should have already been fixed | 14:08 |
| tkajinam | TheJulia, thanks, but I'm afraid the networking-baremetal-multitenant-vlans job is still blocked... | 14:40 |
| tkajinam | /opt/stack/ironic/devstack/lib/ironic:2421 ironic-neutron-agent did not report some nodes. | 14:40 |
| tkajinam | The error is the one I've seen consistently for a while, afair | 14:40 |
| tkajinam | though I could not yet find actual trigger of that failure. | 14:40 |
| TheJulia | I'm fairly sure we fixed that already, but I'll keep an eye on the job runs | 14:45 |
| TheJulia | Thanks | 14:46 |
| TheJulia | oh wow that was a fast result | 14:48 |
| TheJulia | hmmmmm | 14:50 |
| TheJulia | err, I think it is how the job is built which causes the fix not to get invoked | 14:55 |
| opendevreview | Julia Kreger proposed openstack/ironic master: ci: allow service role CI account usage to have elevated access https://review.opendev.org/c/openstack/ironic/+/909120 | 15:23 |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal master: [codespell] Fixing Spelling Mistakes https://review.opendev.org/c/openstack/networking-baremetal/+/906788 | 15:24 |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal master: [codespell] Fixing Spelling Mistakes https://review.opendev.org/c/openstack/networking-baremetal/+/906788 | 15:24 |
| * TheJulia waits | 15:24 | |
| opendevreview | Afonne-CID Paul Onyedikachi proposed openstack/ironic master: Improve clarity of exception message https://review.opendev.org/c/openstack/ironic/+/909123 | 15:48 |
| rpittau | bye everyone see you tomorrow o/ | 16:43 |
| masghar | o/ | 17:09 |
| jamesdenton | hello hello. For ironic-inspector, is there a built-in delay/wait for LLDP information to be gathered before moving on? | 17:57 |
| JayF | I'm not sure, lets ask the code... | 18:04 |
| JayF | https://opendev.org/openstack/ironic-python-agent/src/commit/44d9c2219fd6d17a0aec063331187f147ab4ab11/ironic_python_agent/netutils.py#L169 | 18:05 |
| JayF | a configurable timeout, CONF.lldp_timeout | 18:05 |
| JayF | one way to hook that in from Ironic (there may be a better path that I don't know), would be adding ipa-lldp-timeout to your kernel command line when pxe'ing, and you can customize it | 18:06 |
| JayF | (or bake in a config file) | 18:06 |
| jamesdenton | ok, i thought that would be it. I guess that's ipa-lldp-timeout from the kernel ? | 18:06 |
| jamesdenton | kk | 18:06 |
| JayF | https://opendev.org/openstack/ironic-python-agent/src/commit/44d9c2219fd6d17a0aec063331187f147ab4ab11/ironic_python_agent/config.py#L142 you can always get the CLI slug from the default= on the oslo config setup | 18:06 |
| JayF | essentially we hook in all the kernel CLI options as defaults, so anything Ironic sets or that's set explicitly in config in the image overrides the kernel setting | 18:07 |
| jamesdenton | oh interesting, ok | 18:07 |
| JayF | if it's interesting bad, that was written like, 10 years ago | 18:07 |
| JayF | if it's interesting good, I think it was by me :P | 18:07 |
| jamesdenton | i have ipa-lldp-timeout=30 and ipa-collect-lldp=1, but no collection is happening | 18:07 |
| jamesdenton | JayF you are brilliant :D | 18:08 |
| JayF | https://opendev.org/openstack/ironic-python-agent/src/commit/44d9c2219fd6d17a0aec063331187f147ab4ab11/ironic_python_agent/config.py#L153 | 18:08 |
| JayF | so two things: 1) are you sure it's not ipa-collect-lldp=true (I'm not sure what all oslo.config accepts as binary; I always use true/false) and 2) you should likely be doing something like ipa-inspector-collectors=lldp (?) | 18:09 |
| JayF | and I know some of this changes based on what version you're on (we are migrating from an indepedent inspector service to one integrated in) | 18:09 |
| JayF | but inspector is a very big blind spot for me; so if that pointer isn't enough you might wanna see if someone else has better ideas | 18:10 |
| jamesdenton | this is 2023.1 so i'm not sure if that new inspector thing is there | 18:10 |
| jamesdenton | no worries, i appreciate the tips i will give them a go | 18:10 |
| JayF | that puts you in a place, I believe, where some of the APIs are in Ironic but proxied to Inspector... but not super well documented because it wasn't ready yet. Not something you need to worry about at all :D | 18:14 |
| opendevreview | Julia Kreger proposed openstack/ironic master: ci: allow service role CI account usage to have elevated access https://review.opendev.org/c/openstack/ironic/+/909120 | 20:45 |
| NobodyCam | good afternoon Ironic folks | 22:43 |
| NobodyCam | I had a question about: https://review.opendev.org/c/openstack/ironic/+/883577 | 22:44 |
| NobodyCam | would that be needed on the conductor or api | 22:44 |
| TheJulia | Conductor, since it needs to know how to talk to cinder | 22:51 |
| NobodyCam | that what I thought | 22:59 |
| NobodyCam | are there any follow up patches required | 22:59 |
| TheJulia | NobodyCam: no | 23:37 |
| NobodyCam | hummm | 23:38 |
| TheJulia | errors? | 23:41 |
| NobodyCam | just running in debug mode now to get the details | 23:42 |
| NobodyCam | still getting `ironic.common.exception.StorageError: Failed to request detach for volume 7447d495-035d-41bc-828e-4c46c9d29d3c from cinder for node 291840de-cc2c-44b0-8bd8-4087b48fa605: Invalid volume: Unable to detach volume. Volume status must be 'in-use' and attach_status must be 'attached' to detach. (HTTP 400) (Request-ID: req-b8c25485-7eb8-4d6d-8dcc-6c6e0dc0b112) | 23:46 |
| TheJulia | NobodyCam: does your ironic config have https://review.opendev.org/c/openstack/ironic/+/883577/1/devstack/lib/ironic#1628 ? | 23:48 |
| TheJulia | oh, the volume is just not attached according to cinder | 23:49 |
| NobodyCam | oh its false... | 23:50 |
| NobodyCam | volume is attached ad working | 23:51 |
| TheJulia | ad? | 23:51 |
| NobodyCam | I didn't check the dev stack file | 23:51 |
| NobodyCam | :( | 23:51 |
| NobodyCam | :head desk: | 23:51 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!