| opendevreview | Julia Kreger proposed openstack/ironic master: CI: Add logic for vxlan/geneve network types https://review.opendev.org/c/openstack/ironic/+/977080 | 00:33 |
|---|---|---|
| opendevreview | Julia Kreger proposed openstack/networking-baremetal master: WIP: CI: implement vxlan job https://review.opendev.org/c/openstack/networking-baremetal/+/977081 | 00:35 |
| opendevreview | Merged openstack/networking-generic-switch master: Avoid unnecessary functional test runs https://review.opendev.org/c/openstack/networking-generic-switch/+/977053 | 00:55 |
| opendevreview | Julia Kreger proposed openstack/ironic master: CI: Add logic for vxlan/geneve network types https://review.opendev.org/c/openstack/ironic/+/977080 | 01:53 |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal master: WIP: CI: implement vxlan job https://review.opendev.org/c/openstack/networking-baremetal/+/977081 | 01:57 |
| TheJulia | hopefully getting close | 01:59 |
| opendevreview | Syed Haseeb Ahmed proposed openstack/ironic master: Support pluggable API middleware via entry points https://review.opendev.org/c/openstack/ironic/+/976494 | 04:47 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Workaround for latest docker in metal3 job https://review.opendev.org/c/openstack/ironic/+/977047 | 08:52 |
| opendevreview | Syed Haseeb Ahmed proposed openstack/ironic master: Support pluggable API middleware via entry points https://review.opendev.org/c/openstack/ironic/+/976494 | 12:26 |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal master: vxlan: Fix ironic-neutron-agent reporting https://review.opendev.org/c/openstack/networking-baremetal/+/977218 | 13:07 |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal master: WIP: CI: implement vxlan job https://review.opendev.org/c/openstack/networking-baremetal/+/977081 | 13:07 |
| opendevreview | Riccardo Pittau proposed openstack/ironic bugfix/34.0: [bugfix only] remove metal3 integration job https://review.opendev.org/c/openstack/ironic/+/977224 | 14:15 |
| opendevreview | Merged openstack/networking-baremetal master: tox: Drop redundant injection of VIRTUAL_ENV variable https://review.opendev.org/c/openstack/networking-baremetal/+/976875 | 14:21 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Workaround for latest docker in metal3 job https://review.opendev.org/c/openstack/ironic/+/977047 | 15:12 |
| opendevreview | Merged openstack/ironic-python-agent master: tox: Drop redundant injection of VIRTUAL_ENV variable https://review.opendev.org/c/openstack/ironic-python-agent/+/976873 | 15:19 |
| cardoe | clif: I replied to your review | 15:21 |
| cardoe | TheJulia: on Haseeb's API middleware, I was going to as a follow on make the existing ones we had fall under his docs and be configured the same way if that makes sense. | 15:22 |
| clif | cardoe: Replied back. I hope what I'm saying makes sense. | 15:37 |
| cardoe | clif: No I don't follow. | 15:39 |
| TheJulia | cardoe: cool cool | 15:41 |
| cardoe | So you want an instance/class variable that's the string names of the fields? | 15:41 |
| cardoe | Cause those are available programmatically from the class definition. Otherwise keeping them as manual strings like that relies on humans to keep strings in sync with the names of parameters. | 15:42 |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal master: vxlan: Fix ironic-neutron-agent reporting https://review.opendev.org/c/openstack/networking-baremetal/+/977218 | 15:48 |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal master: WIP: CI: implement vxlan job https://review.opendev.org/c/openstack/networking-baremetal/+/977081 | 15:49 |
| TheJulia | so lets see if this works | 15:49 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: WIP: Crazy idea: NDFC support in NGS? https://review.opendev.org/c/openstack/networking-generic-switch/+/968484 | 16:35 |
| TheJulia | As an aside, I do wonder if nautobot or apstra might make a lot of sense to do something similar | 16:35 |
| clif | cardoe: I'm trying to say that ALL_KEYS and NECESSARY_KEYS should be defined as part of TraitAction so TraitAction.{ALL_KEYS,NECESSARY_KEYS} not defined in config_file.py | 17:52 |
| *** dking is now known as Guest3094 | 18:02 | |
| cardoe | clif: okay. They're computed off the class definition now. When you previously had them as just a list of arbitrary strings. | 18:02 |
| *** Guest3094 is now known as dking | 18:02 | |
| cardoe | I can compute them off the class definition and set them as class variables then? | 18:03 |
| cardoe | Functionally the code in config_file.py is just a very basic implementation of jsonschema | 18:03 |
| TheJulia | okay, where was my brain | 18:04 |
| cardoe | All I want is to not have the case where something changes a human has to update two places. Because that's how the inspection_rules where done. The config file parser had the ALL_KEYS and NECESSARY_KEYS thing as well. And then someone updated the fields in the class and didn't update the fields in the config file parser and it was broken. | 18:05 |
| cardoe | TheJulia: what do ya mean with Nautobot? | 18:05 |
| TheJulia | like, port config management, see the change | 18:05 |
| TheJulia | similar to ndfc | 18:05 |
| cardoe | Ultimately it's really too slow to do operations like that. | 18:05 |
| TheJulia | o'rly :( | 18:06 |
| TheJulia | Queue endless sadness | 18:06 |
| cardoe | We do the port operations ourselves with nornir/napalm and then let Nautobot's golden config and other stuff just act as a compliance checker | 18:06 |
| cardoe | But we use the same repo of templates so the end result is the same | 18:06 |
| TheJulia | Hmm | 18:07 |
| cardoe | Nautobot/NetBox take a change and generate a Job. They use Celery to farm out the job. But when you scale the Job stuff really becomes slow because you have to have a generic Job which then creates a pool specific Job which then runs your change. | 18:07 |
| * TheJulia starts heating up some lunch to reset brain from meeting | 18:07 | |
| cardoe | And the whole System Change management stuff they have in there must run and must capture the diffs | 18:08 |
| TheJulia | oh, ugh | 18:08 |
| TheJulia | okay | 18:08 |
| cardoe | Our folks reallly wanted us to have Nautobot drive it all. | 18:08 |
| cardoe | Plus Nautobot's expression of VNIs is... lacking... NetBox is much better. | 18:09 |
| TheJulia | so, it looks like I'm down to "why devstack do you not log or do anything around actual external_ids for bridge mappings with ovn | 18:09 |
| cardoe | Nidhi, that did the recent Redfish inspection changes, is actually writing the improvements to Nautobot's VNI handling as we speak. | 18:09 |
| cardoe | https://github.com/nautobot/nautobot/pull/8467 | 18:11 |
| TheJulia | oh well, anyway, oven is heating to make me a small personal pizza and I have the hope it brings happy belly and devstack clarity as in why does it hate me | 18:12 |
| clif | cardoe: but why wouldn't the config file use TraitActions.ALL_KEYS and TraitActions.NECESSARY_KEYS? I agree with not duplicating code or logic, which is why I think those constants should stay on TraitActions | 18:20 |
| clif | anything that needs to use them can refer directly to them instead of re-deriving them | 18:21 |
| cardoe | I can add them on there then. They weren’t ever constants before. You had them as instance variables. | 18:22 |
| opendevreview | cid proposed openstack/ironic stable/2025.1: Make RPCService picklable for oslo.service https://review.opendev.org/c/openstack/ironic/+/977261 | 18:54 |
| dking | dtantsur: Would you happen to still be around? | 19:25 |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal master: vxlan: Fix ironic-neutron-agent reporting https://review.opendev.org/c/openstack/networking-baremetal/+/977218 | 19:35 |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal master: WIP: CI: implement vxlan job https://review.opendev.org/c/openstack/networking-baremetal/+/977081 | 19:36 |
| dking | Maybe somebody else could help me. I'm receiving a cleaning error that driver_info is missing deploy_kernel and deploy_ramdisk. Those should be set, so I'm trying to troubleshoot. I'm using the metal3-io ironic image. the same settings work in a different environment, so I'm confused. | 19:37 |
| cardoe | cid: should that be against master? | 19:37 |
| cid | It should :D. | 19:51 |
| cid | I only noticed I haven't checkout out to master after the change was up. | 19:51 |
| dking | I see that there's an 'OS_CONTROLLER__DEPLOY_KERNEL' env variable, but I'm not sure how in the working environment that gets into the ironic configuration. | 19:51 |
| opendevreview | cid proposed openstack/ironic master: Make RPCService picklable for oslo.service https://review.opendev.org/c/openstack/ironic/+/977266 | 19:56 |
| opendevreview | cid proposed openstack/ironic master: Make RPCService picklable for oslo.service https://review.opendev.org/c/openstack/ironic/+/977266 | 20:04 |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal master: vxlan: Fix ironic-neutron-agent reporting https://review.opendev.org/c/openstack/networking-baremetal/+/977218 | 20:05 |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal master: WIP: CI: implement vxlan job https://review.opendev.org/c/openstack/networking-baremetal/+/977081 | 20:05 |
| TheJulia | cardoe: I feel like I'm chasing either broken ovn config or some weird slight incompatability | 20:06 |
| cardoe | Possible? | 20:08 |
| * TheJulia gets another data point | 20:09 | |
| cardoe | Logs on the ci job? | 20:10 |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal master: DNM: Disable segments to see if things play nice without them https://review.opendev.org/c/openstack/networking-baremetal/+/977267 | 20:10 |
| TheJulia | disable segments | 20:11 |
| TheJulia | So, there is like a double-edged sword configuration wise | 20:11 |
| TheJulia | if you want to dig, see: https://48f44a00bc5b9605050d-37d30e8ca4786c547186bac4ea4dd159.ssl.cf2.rackcdn.com/openstack/f358bff37ae54684a4e6cbbbfcba6126/controller/logs | 20:12 |
| TheJulia | Regarding the upcoming PTG, We should start gathering thoughts/plans/ideas | 20:37 |
| TheJulia | https://etherpad.opendev.org/p/ironic-ptg-2026.2 | 20:37 |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal master: WIP: CI: implement vxlan job https://review.opendev.org/c/openstack/networking-baremetal/+/977081 | 20:55 |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal master: DNM: Disable segments to see if things play nice without them https://review.opendev.org/c/openstack/networking-baremetal/+/977267 | 20:57 |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal master: vxlan: Fix mech looping and supported type declaration https://review.opendev.org/c/openstack/networking-baremetal/+/977218 | 22:03 |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal master: WIP: CI: implement vxlan job https://review.opendev.org/c/openstack/networking-baremetal/+/977081 | 22:03 |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal master: WIP: CI: implement vxlan job https://review.opendev.org/c/openstack/networking-baremetal/+/977081 | 23:16 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!