Wednesday, 2025-05-07

rm_work[m]bah I wish search worked better in Element...02:31
rm_work[m]has there been discussion here that I should find and read regarding https://bugs.launchpad.net/nova/+bug/2106073 ?02:32
rm_work[m]TheJulia: not sure which side of the fence this is going to end up falling on (assuming the nova folks haven't completely washed their hands of it, which they may have), but it is affecting us apparently T_T02:33
rm_work[m]from reading the thread it seems like possibly one of those things that is a major architectural issue, not just "someone go fix it quick"02:34
rm_work[m]but have been trying to think of ways to actually get back to contributing upstream, and maybe could convince folks here to let me spend cycles on this if there's momentum to actually fix it02:34
JayFrm_work[m]: https://review.opendev.org/c/openstack/ironic/+/946378 (+the next one in that chain for a larger fix/improvement)02:42
rm_work[m]ah ok cool, so well in hand02:43
rm_work[m]... was that linked on there and I just missed it02:43
rm_work[m]almost assuredly yes02:43
rm_work[m]or maybe not. anywho, +2 already, nice02:44
rm_work[m]I wonder if I can get that pulled down and test it in our lab...02:45
rm_work[m]assuming it is ready for testing? I see the followup is still -1 but that could be anything without spending a bit more time diving in02:46
rpittaugood morning ironic! o/05:21
AmarachiOrdor[m]Good Morning rpittau, Good morning everyone05:37
arne_wiebalckGood morning, Ironic!06:28
abongaleGood Morning, Ironic!08:21
queensly[m]Good morning08:47
opendevreviewRiccardo Pittau proposed openstack/bifrost master: Default ansible to version 10.x  https://review.opendev.org/c/openstack/bifrost/+/94824509:26
freemanboss[m]Good morning everyone 10:03
opendevreviewDoug Goldstein proposed openstack/python-ironicclient master: fix: portgroup create with uuid for idempotency  https://review.opendev.org/c/openstack/python-ironicclient/+/94903213:15
TheJuliagood morning13:38
TheJuliarm_work[m]: yeah, I'm working on it since we have customers who do all sorts of spine/leaf configurations which could likely run up against this. It actually explains some of the reports of issues we've not really understood over the past couple of years13:39
TheJuliawhich is unfortunate, but yeah. Anyhow, it is not a fun issue and I'm really hopeful to have a fully testable fix this week. Unfortunately nova sends Ironic pre-mastered configuration drives which means we really need to unpack/extract the contents, patch, and repackage13:41
TheJuliaWheeeee ironic networking meeting in like 2013:42
TheJuliaerr, that is an old meeting, not the new meeting13:57
* TheJulia consumes more coffee13:57
cardoenetworking meeting?14:12
cardoeisn't that in an hour and 45?14:13
cardoeSo I had a stupid idea with OVN router flavors and creating a generic bare metal flavor as a reference. Not sure if that should live in networking-generic-switch or if I should submit that against Neutron proper.14:15
TheJuliaYeah, it is14:37
TheJuliaI had a bad invite on my calendar from ages ago14:37
TheJuliawhich got me confuzzled14:37
shermanmcan I ask where the ironic networking meeting schedule is written down?14:39
shermanmon https://etherpad.opendev.org/p/ironic-networking it still says Jan 15th14:39
cardoehttps://etherpad.opendev.org/p/ironic-standalone-networking is the new one I got in the invite.14:45
shermanmcardoe: thanks!  I remember now that we'd planned on two "tracks" for networking , standalone and non-14:55
TheJuliashermanm: for integrated cases I think we're going to try and take the specs process to try and effectuate the needful15:12
shermanmthejulia: +1, just making sure I wasn't totally out of the loop :) 15:14
TheJulia https://meet.google.com/rct-jvgq-qrw?authuser=016:00
opendevreviewElod Illes proposed openstack/bifrost unmaintained/zed: [CI][UM-only] Remove not defined kolla-ansible job  https://review.opendev.org/c/openstack/bifrost/+/94905716:51
TheJuliaalegacy: template is https://opendev.org/openstack/ironic-specs/src/branch/master/specs/template.rst17:01
cardoeI hope my feedback helped and didn’t detract.17:44
JayFhonestly it's good you brought up what you did17:54
JayFa good check to ensure we're not gobbling up all the scope in the universe17:54
cardoeI just see some of the asks around NGS stuff like making it an agent that knows the state of the switches, etc.18:04
cardoeAnd my thought is that there's open source tools for this written by people that mess with those devices day in and day out.18:05
mnasiadkaHello18:05
mnasiadkaTheJulia: Is there an option to get some re-review on this IPA patch https://review.opendev.org/c/openstack/ironic-python-agent/+/816685 ?18:05
cardoeThe better approach would likely be to tell people to run one of those things and then make it easy to get the requested operation fired off to that system.18:05
TheJuliacardoe: I think you were making a really good point, thanks!18:06
alegacycardoe: ya, I do like the idea of being able to point a driver to an existing automation system.  Can you remind me of the ones you mentioned?  I was scribbling down notes as you were talking... I got autobot, dev42, netbox... correct me if I'm in the weeds.18:14
opendevreviewMerged openstack/ironic master: Add an index on ports.node_id  https://review.opendev.org/c/openstack/ironic/+/94843118:18
opendevreviewJay Faulkner proposed openstack/ironic master: Revert "Add an index on ports.node_id"  https://review.opendev.org/c/openstack/ironic/+/94906818:36
JayFcid and I were doing more performance testing18:37
JayFand we are now reasonably convinced that the preexisting CONSTRAINT `ports_ibfk_1` FOREIGN KEY (`node_id`) REFERENCES `nodes` (`id`) is providing an index somehow?18:37
JayFI may not fully understand all the moving parts; but a "for node in nodes; dbapi.get_port_by_node_id(node.id)" actually gets slightly slower with the index18:37
JayFand an explain pre-running migration seems to show that we're arleady using an index18:38
JayFTheJulia: dtantsur: ^ sanity check?18:38
TheJuliawhenever you apply a constraint, it requires the db to maintain an index18:38
JayFyeah18:39
JayFso the index is a noop/dupe18:39
JayFand the revert should be landed :( 18:39
TheJuliaSo, technically. The key is if your doing any sort of join or ... sorry searching for the word in the context switch18:39
TheJuliajoin or... query with additional parameters you'll want to match it if possible18:39
JayFoh,  I get what you're saying18:40
TheJuliai.e. if your doing "give me the ports which match this node *and* some other value, idealy you want a composite index18:40
JayFif we have a query that gets address/name/extra from port18:40
TheJuliaThe index, should match what you query18:40
TheJuliawell, query to *match*18:40
JayFan index of node_id -> portid/address/name/extra would be valuable18:40
JayFyes?18:40
TheJuliaSomewhat, the DB inherently indexes the pkey + any other single column indexed field18:41
JayFis the action for now to revert that simple index the right one though?18:41
JayFeven if there may be /an/ index that improves things, this likely isn't it18:41
JayFI feel like for a db migration there's a timer18:42
TheJuliaif your doing select from blah where field_a='a_value' and field_b != NULL; then you'll want the index composited on field_a and field_b18:42
JayFyep, I think that matches my understanding18:42
JayFwhich means a fkey port.node_id -> node.id18:42
TheJuliawhich is different from select field_a, field_b18:42
JayFand an index port.node_id -> node.id18:42
JayFis the same thing18:42
JayFand we'd need a better index that uses the methods/calls in question 18:42
TheJuliawell, fkey reference is the direct link, but yeah18:42
TheJuliaI guess, I'm talking theory and resulting practice, what is the driving observation?18:44
JayFthe driving observation is I can't find a benchmark this improves18:44
JayFor a DB query this changes the explain on 18:44
JayFand the explain showing me the fkey/contraint existed pre-index is a red flag18:44
JayFcid: https://opendev.org/openstack/networking-baremetal/src/branch/master/networking_baremetal/agent/ironic_neutron_agent.py#L221 is what the report was timing out at high scale18:46
JayFbut it's not even doing a join in the DB, it's just getting all the ports and mapping later18:46
JayFI don't think there's anything we can do DB-side to speed that up18:46
TheJuliaQuery patterning is also a death by a thousand cuts, The key is to do as much as you can as close to the db and not go back to the db18:46
TheJuliawhich is kind of the challenge I'm trying to understand18:47
dxterslabstevebaker[m]: do you know of any redfish API that provides a websocket for SOL access?18:47
JayFyeah the tl;dr on the root ask in that ironic_client.get_ports(detail=True) call is timing out in networking-baremetal in some cases18:47
JayFbut looking at the query, I don't think we can do crap on the ironic side to speed it up18:47
TheJuliadxterslab: afaik, no. from what I've seen is some just point at ipmi with the suggestion to take that path instead18:47
JayFit really is a shard/conductor group awareness thing potentially needed18:47
TheJuliaso its a joined query thing to do it, right now we're likely checking/reconciling it upfront18:48
TheJuliaerr18:48
TheJuliaafter the fact, not upfront18:48
dxterslabTheJulia: yeah... the noVNC approach looks to be non-trivial at all and probably proxying IPMI to websocket will be easier18:48
TheJuliasomebody did that, but its really just not ideal and web socketing it is apparently awkward18:49
TheJulia(and they didn't post the code, insert grumpy upstream maintainer comments)18:49
TheJuliaHey, folks... Any thoughs on dialing back logging on  keystoneauth.identity.v3.base ?18:50
dxterslabTheJulia: what challenges do you for see that I might not thought about?18:51
TheJuliadxterslab: the comments which were made with me on a call were "this is awkward and painful", but the folks built a third party api/surface to facilitate it separately18:52
dxterslabTheJulia: I guess I get to FAFO then lol18:53
TheJuliasorry :(18:53
TheJuliaI like the idea of having a proxy there for it, but $something does need to sit in that middle ground18:53
TheJuliaalso, apparently ipmitool doesn't have support for the flow control on the ipmi protocol18:54
TheJuliaso, super easy to stall out interactions18:54
TheJulia(I know some folks that want to use serial command multiplexing for $reasons)18:55
dxterslabright now i am looking to just support a single session and no multiplexing but vnc has presented to be more challenging than expected18:55
TheJuliaunderstandable18:56
dxterslabTheJulia: where the comments based on ipmitool or pyghmi? i am planning to use pyghmi to do this19:06
TheJuliaipmitool, I'd avoid pyghmi since it is pretty much a single maintainer thing at this point19:07
TheJuliaand... really only gets tested/developed against lenovo gear19:07
JayFpyghmi is great 90% of the time and nobody runs it because of the 10%19:07
dxterslabreassuring :D!19:08
opendevreviewMerged openstack/ironic master: Trivial: Fix gap in microversion sequence  https://review.opendev.org/c/openstack/ironic/+/94882919:13
rm_work[m]TheJulia: thanks for your work on that! when you're good to go if you want a tester, you can ping me, i'll see if we can pull that down in our Lab env20:46
opendevreviewJay Faulkner proposed openstack/ironic master: Inspection throws exception on CPU-less systems  https://review.opendev.org/c/openstack/ironic/+/94909021:18
opendevreviewJulia Kreger proposed openstack/ironic master: provide host_id to neutron early on  https://review.opendev.org/c/openstack/ironic/+/94637821:23
opendevreviewJulia Kreger proposed openstack/ironic master: WIP: Patch configdrive metadata  https://review.opendev.org/c/openstack/ironic/+/94667721:23
TheJuliarm_work[m]: if you want to give the two patches I just posted a spin, by all means. I still need to write a bunch of tests for the latter but the same basic code just seemed to work when I short circuited validation on network_data.json out of nova21:24
TheJuliaI just revised the commit message to lay out slight metadata differences, likely also just highlighted *another couple bugs*21:33
opendevreviewJulia Kreger proposed openstack/ironic master: WIP: Patch configdrive metadata  https://review.opendev.org/c/openstack/ironic/+/94667721:33
opendevreviewJay Faulkner proposed openstack/ironic master: Inspection throws exception on CPU-less systems  https://review.opendev.org/c/openstack/ironic/+/94909021:33
opendevreviewMerged openstack/ironic-python-agent master: netutils: Use ethtool ioctl to get permanent mac address  https://review.opendev.org/c/openstack/ironic-python-agent/+/94656221:53
opendevreviewMerged openstack/ironic master: Revert "Add an index on ports.node_id"  https://review.opendev.org/c/openstack/ironic/+/94906822:48
opendevreviewMerged openstack/ironic master: Remove tags from README  https://review.opendev.org/c/openstack/ironic/+/94847922:48

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