| opendevreview | yatin proposed openstack/ironic unmaintained/2024.1: [Stable 2024.1 Only] pin ironic-tempest-plugin to last working tag https://review.opendev.org/c/openstack/ironic/+/979773 | 04:11 |
|---|---|---|
| opendevreview | yatin proposed openstack/ironic unmaintained/2024.1: [Stable 2024.1 Only] pin ironic-tempest-plugin to last working tag https://review.opendev.org/c/openstack/ironic/+/979773 | 06:58 |
| rpittau | good morning ironic! o/ | 07:51 |
| opendevreview | Verification of a change to openstack/networking-generic-switch master failed: docs: address nits for 978799 https://review.opendev.org/c/openstack/networking-generic-switch/+/979761 | 08:15 |
| kubajj | good morning rpittau, and Ironic! o/ | 09:02 |
| rpittau | hey kubajj :) | 09:02 |
| janders | hey Ironic'ers o/ | 11:29 |
| janders | o/ | 11:29 |
| janders | dtantsur cardoe when you're online it would be great to have a quick chat about outstanding comments in https://review.opendev.org/c/openstack/ironic/+/977569 | 11:30 |
| janders | it feels like we're close to be able to merge it but I'm not 100% sure if the outstanding observations/concerns need further revisions or are more lab-testing material | 11:31 |
| janders | would appreciate your insights! | 11:31 |
| opendevreview | Konstantinos Karampogias proposed openstack/ironic master: Skip redundant BIOS reboot when settings already match https://review.opendev.org/c/openstack/ironic/+/979828 | 12:31 |
| TheJulia | good morning | 13:08 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: fix(redfish): correct submission of DateTime to BMC https://review.opendev.org/c/openstack/ironic/+/979722 | 13:55 |
| opendevreview | Konstantinos Karampogias proposed openstack/ironic master: Skip redundant BIOS reboot when settings already match https://review.opendev.org/c/openstack/ironic/+/979828 | 15:01 |
| cardoe | janders: yeah I need to get back to testing it | 15:31 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Add [redfish]connect_timeout config for faster unreachable BMC detection https://review.opendev.org/c/openstack/ironic/+/979872 | 15:42 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Add [redfish]connect_timeout config for faster unreachable BMC detection https://review.opendev.org/c/openstack/ironic/+/979872 | 15:43 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Add [redfish]connect_timeout config for faster unreachable BMC detection https://review.opendev.org/c/openstack/ironic/+/979872 | 15:44 |
| cardoe | dtantsur, rpittau: https://review.opendev.org/c/openstack/ironic/+/979722 I think that's one you guys will see in Metal3 since I tried using that verify you've enabled there. | 16:23 |
| cardoe | I don't understand why its not commenting on the bug. | 16:24 |
| dtantsur | I don't think we use this particular step just yet, but thanks! | 16:25 |
| rpittau | cardoe: thanks! looks good besides the missing test | 16:36 |
| TheJulia | brraaaains | 17:32 |
| TheJulia | Anyone seen my brains today? | 17:32 |
| JayF | TheJulia: I just learned that vm brasseur was your old grandboss! I had no idea. | 17:35 |
| JayF | (she just sat for a gr-oss out interview) | 17:36 |
| TheJulia | She was not my grandboss. Separate management chain | 17:37 |
| JayF | either way, still neat to find openstack connections I didn't realize existed | 17:38 |
| TheJulia | cool cool | 17:38 |
| cardoe | dtantsur: so fwiw now that I am thinking about it. We should always strip microseconds. | 18:49 |
| cardoe | Per the spec they’re always optional. But if you are going to send them, you must match the precision supported by the hardware. Which can be 3 or 6 digits. | 18:50 |
| cardoe | The test always worked by not sending it. | 18:50 |
| *** mdfr5 is now known as mdfr | 19:39 | |
| TheJulia | +1 strip microseconds | 19:47 |
| TheJulia | because your going to drift throught the interaction, the most reliable thing is to add time but even then... yeah. | 19:48 |
| clif | JayF: do you know the 'right' place to teardown dynamic portgroups? | 19:51 |
| clif | vif_detach? | 19:54 |
| JayF | _do_node_tear_down | 19:54 |
| JayF | oooooh | 19:54 |
| JayF | that's probably a better answer | 19:54 |
| JayF | we need to ensure it gets run no matter what though, I wonder if we have strong assurances that gets called | 19:55 |
| clif | if the vif is attached to a dynamic_portgroup, detach it, then tear down the dynamic_portgroup? | 19:55 |
| JayF | yes | 19:55 |
| JayF | I'm just trying to think of any edge cases there | 20:01 |
| JayF | for servicing, we should be detaching and reattaching to the existing network, we should ensure that's doing a detach/attach | 20:02 |
| JayF | same idea for rescuing | 20:02 |
| opendevreview | Julia Kreger proposed openstack/ironic master: redfish: increase cache password hashing iterations https://review.opendev.org/c/openstack/ironic/+/979903 | 20:04 |
| TheJulia | yeah, tear down as part of tear down of the deployed instance | 20:05 |
| JayF | are you saying vif_detach wasn't the right answer? | 20:06 |
| TheJulia | oh | 20:06 |
| TheJulia | well, hold up | 20:06 |
| JayF | clif: I think vif_detach is wrong | 20:06 |
| TheJulia | no, vif_detach is good | 20:06 |
| JayF | it doesn't get called, e.g., in rescuing cases | 20:06 |
| TheJulia | its dynamic, so should be okay to it on attach/detach | 20:06 |
| TheJulia | no, it doesn't | 20:06 |
| JayF | but isn't that *wrong* for this case? | 20:06 |
| TheJulia | is this where we get a state diagram out | 20:06 |
| TheJulia | ? | 20:06 |
| JayF | here's what I'm trying to reconcile | 20:07 |
| JayF | vif_detach only gets called from nova/external api user afaict | 20:07 |
| JayF | in cases of things like rescuing or servicing where we tear down/swap network on an active server | 20:07 |
| JayF | is it OK that the dynamic portgroup (at least the data model for it) is still in place when it's on a service or rescue network | 20:08 |
| TheJulia | so vif_detach *is* dynamic in that you can ask for new things to get applied, I guess the question is "at deploy time" or "at attach/detach" time, detach is fine, but yeah. | 20:08 |
| TheJulia | yes, should be perfectly okay | 20:08 |
| TheJulia | just because it exists, doesn't mean it has to be used | 20:08 |
| JayF | okay good, from claude and my digging | 20:09 |
| JayF | for vif in vifs; do vif_detach; done gets called after unconfigure_tenant_networks removes the binding from neutron in _do_node_tear_down() | 20:09 |
| JayF | so vif_detach is the right place, and it's a feature-not-a-bug that we keep the dynamic portgroup in the data model (representing the port config if returned to active) | 20:10 |
| JayF | even if we're servicing/rescuing and therefore on a servicing/recusing network | 20:10 |
| JayF | clif: ^ tl;dr yes vif_detach is the right place, re-confirmed | 20:10 |
| TheJulia | yup | 20:10 |
| clif | ty I will proceed with vif_detach | 20:14 |
| TheJulia | So, feels like we need to figure out what date we want to speak about our advances? | 20:42 |
| JayF | I'm unsure what you mean? | 20:43 |
| JayF | I do owe a reponse to that ironic video chain, it's on my list for today | 20:43 |
| JayF | my task list has not gotten below "overflowing" since being back | 20:43 |
| TheJulia | Yeah, step 0 is sort of figuring out a date | 20:53 |
| JayF | ack; I was waiting to reply until I got slides started for us to start populating | 20:54 |
| JayF | but I went ahead and kicked that reply out | 20:54 |
| TheJulia | I think we're sort of in too many cooks trying to organize, but I think bullets on slides is a good starting point | 21:10 |
| JayF | I was literally just talking about taking the oif live standard slides -> clone -> s/openstack/ironic/ in a few places | 21:11 |
| JayF | "hey everyone here's a collab space" | 21:11 |
| JayF | it's just competing with a thousand other tiny things for my time | 21:11 |
| JayF | I'll get there today | 21:11 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: Fix port binding race by using the original segment data https://review.opendev.org/c/openstack/networking-generic-switch/+/979914 | 21:12 |
| TheJulia | ok | 21:15 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: vxlan: Multicast traffic options for cisco driver https://review.opendev.org/c/openstack/networking-generic-switch/+/979332 | 21:46 |
| TheJulia | cardoe: I feel you may have opinions on ^, lmk and we can discuss at some point | 21:47 |
| JayF | TheJulia: actually, I'm not doing that, because apparently the actual openinfra live slide decks aren't out yet :) | 21:49 |
| TheJulia | start a new deck maybe ? | 21:52 |
| JayF | the existing ones had starter slides that looked hard for me to replicate | 21:52 |
| TheJulia | float the idea to allison to give us a clean starting point | 21:52 |
| TheJulia | dunno | 21:52 |
| JayF | e.g. sponsor logos everywhere | 21:52 |
| JayF | floating the idea to allison is the right answer | 21:52 |
| TheJulia | ++ | 21:52 |
| TheJulia | okay, running into town then | 21:52 |
| cardoe | I’ll peek. | 23:00 |
| cardoe | stevebaker[m]: novnc doesn’t have to have ironic right? Like I can have novnc in a smaller container in the same pod as conductor? | 23:02 |
| opendevreview | Karan Anand proposed openstack/python-ironicclient master: Add type annotations to `ironicclient/common/http.py` https://review.opendev.org/c/openstack/python-ironicclient/+/978917 | 23:14 |
| opendevreview | Karan Anand proposed openstack/python-ironicclient master: Add type annotations to `ironicclient/v1/events.py` https://review.opendev.org/c/openstack/python-ironicclient/+/979924 | 23:20 |
| opendevreview | Karan Anand proposed openstack/python-ironicclient master: Add type annotations to `ironicclient/v1/shard.py` https://review.opendev.org/c/openstack/python-ironicclient/+/979925 | 23:23 |
| opendevreview | Karan Anand proposed openstack/python-ironicclient master: Add type annotations to `ironicclient/v1/utils.py` https://review.opendev.org/c/openstack/python-ironicclient/+/979926 | 23:24 |
| opendevreview | Karan Anand proposed openstack/python-ironicclient master: Add type annotations to `ironicclient/v1/conductor.py` https://review.opendev.org/c/openstack/python-ironicclient/+/979927 | 23:28 |
| opendevreview | Karan Anand proposed openstack/python-ironicclient master: Add type annotations to `ironicclient/v1/deploy_template.py` https://review.opendev.org/c/openstack/python-ironicclient/+/979928 | 23:32 |
| opendevreview | Karan Anand proposed openstack/python-ironicclient master: Add type annotations to `ironicclient/v1/inspection_rule.py` https://review.opendev.org/c/openstack/python-ironicclient/+/979929 | 23:37 |
| opendevreview | Karan Anand proposed openstack/python-ironicclient master: Add type annotations to `ironicclient/v1/runbook.py` https://review.opendev.org/c/openstack/python-ironicclient/+/979930 | 23:40 |
| opendevreview | Karan Anand proposed openstack/python-ironicclient master: Add type annotations to `ironicclient/v1/volume_connector.py` https://review.opendev.org/c/openstack/python-ironicclient/+/979931 | 23:45 |
| opendevreview | Karan Anand proposed openstack/python-ironicclient master: Add type annotations to `ironicclient/v1/volume_target.py` https://review.opendev.org/c/openstack/python-ironicclient/+/979932 | 23:46 |
| opendevreview | Karan Anand proposed openstack/python-ironicclient master: Add type annotations to `ironicclient/v1/client.py` https://review.opendev.org/c/openstack/python-ironicclient/+/979933 | 23:51 |
| opendevreview | Karan Anand proposed openstack/python-ironicclient master: Add type annotations to `ironicclient/v1/port.py` https://review.opendev.org/c/openstack/python-ironicclient/+/979934 | 23:54 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!