| opendevreview | Steve Baker proposed openstack/ironic master: Add an autodetect deploy interface https://review.opendev.org/c/openstack/ironic/+/973187 | 00:27 |
|---|---|---|
| opendevreview | Steve Baker proposed openstack/ironic master: Replace ironic-tempest-bios-ipmi-direct with -autodetect https://review.opendev.org/c/openstack/ironic/+/974260 | 00:27 |
| opendevreview | Steve Baker proposed openstack/ironic master: Add bootc deploy to autodetect default https://review.opendev.org/c/openstack/ironic/+/975365 | 00:27 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: devstack: Ignore error with file existing on restack https://review.opendev.org/c/openstack/networking-generic-switch/+/973310 | 00:48 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: l2vni plug case with Cisco NXOS https://review.opendev.org/c/openstack/networking-generic-switch/+/968377 | 00:48 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: vxlan: Arista EOS and vendor neutral SONiC https://review.opendev.org/c/openstack/networking-generic-switch/+/972763 | 00:48 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: vxlan: Add Junos, Cumulus NVUE, and denote Dell OS10 as unsupported https://review.opendev.org/c/openstack/networking-generic-switch/+/972764 | 00:48 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: vxlan: OVS testing patch for 'vxlan' binding model https://review.opendev.org/c/openstack/networking-generic-switch/+/972765 | 00:48 |
| opendevreview | Merged openstack/networking-generic-switch master: devstack: Ignore error with file existing on restack https://review.opendev.org/c/openstack/networking-generic-switch/+/973310 | 04:21 |
| rpittau | good morning ironic! o/ | 08:11 |
| rpittau | gouthamr: just -1ed :) | 08:11 |
| rpittau | TheJulia: I was (still kind of am) sick :/ | 08:11 |
| janders | good morning Ironic o/ | 08:20 |
| janders | rpittau hope you feel better soon! | 08:21 |
| rpittau | thanks janders, I'm feeling better, I will be online today, just reduced capacity :) | 08:22 |
| janders | folks, after W+1 for https://review.opendev.org/c/openstack/ironic/+/974985 (hw health monitoring doco, has 2x +2) | 08:37 |
| janders | thanks in advance! | 08:37 |
| rpittau | janders: workflowed :D | 08:43 |
| janders | rpittau TY! | 08:44 |
| opendevreview | Merged openstack/ironic master: Add documentation for node health monitoring feature https://review.opendev.org/c/openstack/ironic/+/974985 | 09:17 |
| janders | ^ \o/ - last hardware health monitoring change for now | 09:22 |
| janders | api/client/sdk/doco DONE | 09:22 |
| janders | since it's a feature we put on the roadmap during the PTG, do I need to note anywhere this is now complete? | 09:23 |
| rpittau | janders: if we had a spec we can remove that from the not-implemented section | 09:41 |
| rpittau | but besides that, release notes already serve the scope, and we will mention it during the open infra live for Gazpacho :) | 09:41 |
| janders | thanks rpittau. There wasn't a spec, so I think we're all done. | 09:41 |
| janders | appreciate your assistance! :) | 09:41 |
| rpittau | anytime :) | 09:41 |
| opendevreview | Abhishek Bongale proposed openstack/python-ironicclient master: Fix field names for selection and json https://review.opendev.org/c/openstack/python-ironicclient/+/973948 | 10:36 |
| opendevreview | Abhishek Bongale proposed openstack/python-ironicclient master: Fix field names for selection and json https://review.opendev.org/c/openstack/python-ironicclient/+/973948 | 10:51 |
| opendevreview | Abhishek Bongale proposed openstack/python-ironicclient master: Fix field names for selection and json https://review.opendev.org/c/openstack/python-ironicclient/+/973948 | 12:27 |
| TheJulia | rpittau: eek :( Feel better! | 14:13 |
| rpittau | TheJulia: thanks :) | 14:13 |
| opendevreview | Merged openstack/ironic master: Add ironic-networking network interface https://review.opendev.org/c/openstack/ironic/+/966470 | 14:47 |
| opendevreview | Merged openstack/ironic-tempest-plugin master: Enable ironic-tempest-bios-ipmi-direct in check/gate https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/974780 | 14:57 |
| opendevreview | Jad Haj Yahya proposed openstack/ironic master: Fix Redfish boot mode handling for read-only BMCs https://review.opendev.org/c/openstack/ironic/+/975521 | 15:00 |
| opendevreview | Allain Legacy proposed openstack/ironic master: Improve exception handling in switch driver factory https://review.opendev.org/c/openstack/ironic/+/969852 | 15:41 |
| opendevreview | Allain Legacy proposed openstack/ironic master: Address remaining review comments for rpc methods https://review.opendev.org/c/openstack/ironic/+/971184 | 15:41 |
| opendevreview | Allain Legacy proposed openstack/ironic master: Add standalone networking service installation guide https://review.opendev.org/c/openstack/ironic/+/966471 | 15:41 |
| opendevreview | Jad Haj Yahya proposed openstack/ironic master: Fix Redfish boot mode handling for read-only BMCs https://review.opendev.org/c/openstack/ironic/+/975521 | 15:57 |
| alegacy | cid: thanks for reviews my remaining changes. saw your comment and updated with a new revision. | 16:08 |
| cid | Sure. I will take another look now. tks | 16:13 |
| alegacy | thanks, and unfortunately the others lost their +2 in the rebase as well. | 16:20 |
| opendevreview | Julia Kreger proposed openstack/ironic master: ci: Exclude test_baremetal_vm_multitenancy from multinode https://review.opendev.org/c/openstack/ironic/+/975543 | 16:56 |
| * cardoe looks around. | 17:01 | |
| cardoe | Hey there was an ironic meeting. | 17:01 |
| * TheJulia gives cardoe a cookie | 17:01 | |
| TheJulia | You okay? | 17:02 |
| cardoe | Just a little pre-PTO diet care of a stomach bug. | 17:02 |
| TheJulia | ugh :( | 17:03 |
| cardoe | Spent some time snuggling the porcelain yesterday and Sunday. | 17:03 |
| cardoe | I did speak with someone that's smarter than me on networking just a little bit ago and I'm diving into the VXLAN patches. | 17:04 |
| TheJulia | okay | 17:21 |
| TheJulia | Did that conversation include BUM traffic? | 17:21 |
| * dtantsur is a responsible adult and is not giggling at "bum traffic" | 17:22 | |
| TheJulia | oh, the timing on that is excellent | 17:22 |
| * TheJulia totally doesn't believe dtantsur... and concludes he is only pretending to be a responsible adult ;) | 17:22 | |
| dtantsur | :D | 17:24 |
| * TheJulia suddenly worries we might have broken cardoe somehow | 17:26 | |
| cardoe | Sorry. Context switching. | 17:27 |
| cardoe | Yeah I was asking about BUM traffic | 17:27 |
| TheJulia | So, broadcasts, unknowns, and multicast..... | 17:30 |
| * TheJulia dresses up as an awards announcer, and stands at a fancy podium and says "The award for killing the irc disucssion goes to... *opens envelop* ME!" | 17:35 | |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: vxlan: Arista EOS and vendor neutral SONiC https://review.opendev.org/c/openstack/networking-generic-switch/+/972763 | 17:44 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: vxlan: Add Junos, Cumulus NVUE, and denote Dell OS10 as unsupported https://review.opendev.org/c/openstack/networking-generic-switch/+/972764 | 17:44 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: vxlan: OVS testing patch for 'vxlan' binding model https://review.opendev.org/c/openstack/networking-generic-switch/+/972765 | 17:44 |
| TheJulia | so, on the cisco patch, I backed away from the physnet mapping to singular mcast group for bum traffic handling as its against best practices at this point. The others... are in various states. | 17:50 |
| cardoe | Yeah I'm not sure how to best expose that. | 17:53 |
| cardoe | So long term for a large scale I'd like a conductor groups, neutron XXXX, and nova cells? to maybe be the segmentation things. | 17:54 |
| TheJulia | ... to articulate, at some point the building block is the physnet | 17:55 |
| cardoe | So where do you see the recommendations for best practices and mcast groups btw? | 17:56 |
| JayF | .o(people use mcast groups still?!) | 17:56 |
| TheJulia | In that, and sort of disjointed in a way, is EVPN sufficient or is the expectation that disjointed patterns will be required with some management. I guess on the cisco side the limit is like 8k groupings for distinct routes, but I can't see mapping that to each VNI gets its own mcast address or group either. | 17:57 |
| TheJulia | cardoe: uhhh, hjensas left a comment with a link on a earlier cisco n-g-s revision | 17:57 |
| cardoe | Yeah I saw that. | 17:57 |
| cardoe | He used a version that was EOLed in 2019 and EOS in 2022. | 17:57 |
| TheJulia | https://review.opendev.org/c/openstack/networking-generic-switch/+/968377/5/networking_generic_switch/devices/netmiko_devices/cisco.py | 17:58 |
| TheJulia | good callouts | 17:58 |
| cardoe | So there's a design trade off as well. When you have a low number of VTEPs and a high number of VNIs then yes ingress replication. But in other scenarios multicast works out better. | 18:01 |
| cardoe | You definitely cannot give each VNI its own mcast address. | 18:01 |
| cardoe | Most of the hardware you're working with is going to be limited to 128 mcast groups for the ASIC | 18:02 |
| TheJulia | Looks like the docs have largely leaned *hardcore* over to just doing evpn | 18:02 |
| cardoe | But honestly this is why you're wanting to do BGP EVPN for updates and not a flood and learn model. | 18:03 |
| cardoe | Yeah. | 18:03 |
| TheJulia | Yeah, the math doesn't really work out there, and 128 mcast groups is also problematic. Seem slike just stripping out everything that is not evpn is sort of ideal | 18:03 |
| TheJulia | and if thats a limitation on the asics, that is likely across vendors because some of those same ASICs are from a 3rd party | 18:05 |
| cardoe | There's a lot of ways to go. I think the single use case initially makes more sense. | 18:05 |
| TheJulia | so, really the model is likely just "go evpn only" | 18:05 |
| cardoe | Yeah that's what I would say. | 18:05 |
| cardoe | Certainly can entertain other approaches as a 2.0 maybe? | 18:05 |
| TheJulia | The AI tools have largely gotten to pointing out issues with other vendors as reason to keep/lean on such, but strictly speaking, the evpn style is really best overall | 18:07 |
| * TheJulia wonders if a spell was cast that caused james to appear :) | 18:14 | |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: vxlan: Arista EOS and vendor neutral SONiC https://review.opendev.org/c/openstack/networking-generic-switch/+/972763 | 18:43 |
| JayF | alegacy: +2s throughout, including one with meaningful, actionable comments on the docs | 19:01 |
| jrosser | what actual switch ("Most of the hardwre youre working with") only supports 128 multicast groups? | 19:06 |
| jrosser | cardoe: ^ | 19:06 |
| cardoe | For BUM traffic for the underlay configuration. | 19:07 |
| cardoe | For VXLAN ingress replication | 19:07 |
| cardoe | Not generically multicast groups | 19:07 |
| jrosser | ah important caveat | 19:08 |
| alegacy | jayf: thanks. i've responded to your comments. you ok with a follow-up patch for these or re-spin? | 19:20 |
| JayF | up to you, I wouldn't have +2'd if I needed it before it landed | 19:20 |
| alegacy | k | 19:20 |
| jamesdenton | @TheJulia one unchecks 'Connect Securely' in Textual. ¯\_(ツ)_/¯ | 19:51 |
| JayF | join the cult of irccloud James :D | 19:53 |
| jamesdenton | i'd probably still be using IRCle if i could | 19:53 |
| TheJulia | Luckily irccloud is nicely insulated, thus preventing creation of a pluribus. | 20:08 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: vxlan: Arista EOS and vendor neutral SONiC https://review.opendev.org/c/openstack/networking-generic-switch/+/972763 | 20:11 |
| opendevreview | Verification of a change to openstack/ironic master failed: Improve exception handling in switch driver factory https://review.opendev.org/c/openstack/ironic/+/969852 | 20:15 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: vxlan: Add Junos, Cumulus NVUE, and denote Dell OS10 as unsupported https://review.opendev.org/c/openstack/networking-generic-switch/+/972764 | 20:25 |
| rm_work[m] | JayF: Yeah, we have that validation step before the boxes enter the "Ready" state in our inventory management, so I just use that filter and the difference between Current State and Ideal State would just be that instead of me running the "get Ready machines from IM" that Ironic does it. The script to pull them already exists and I do nothing except take the YAML and put it somewhere and run a manual enroll script | 21:20 |
| TheJulia | cardoe: https://review.opendev.org/q/topic:%22ironic-vxlan%22 | 21:26 |
| cardoe | Going through them. | 21:26 |
| * TheJulia goes on etsy to see if anyone has a cardboard cut-out of hypno-toad | 21:26 | |
| JayF | if there's not a "we make a die-cut cardboard cutout of your svg" service on etsy or the internet, someone is missing a market opportunity lol | 21:27 |
| opendevreview | Verification of a change to openstack/ironic master failed: Improve exception handling in switch driver factory https://review.opendev.org/c/openstack/ironic/+/969852 | 21:28 |
| TheJulia | well, die-cut might be a bit much for a one off to convince reviewers. | 21:28 |
| TheJulia | LASER! | 21:36 |
| opendevreview | Jay Faulkner proposed openstack/ironic-prometheus-exporter master: Add mypy infrastructure for type annotation migration https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/975578 | 21:53 |
| opendevreview | Jay Faulkner proposed openstack/ironic-prometheus-exporter master: Add type annotations to parsers/descriptions.py https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/975579 | 21:53 |
| JayF | These are the PRs we'll be pointing our MLH fellow at as examples for the type annotations work. A perfunctory review by interested parties (cc cardoe cid ) would be appreciated | 22:07 |
| JayF | please do not approve those until after the MLH fellow has gotten a chance to onboard with them though :D | 22:07 |
| cid | ++ | 23:31 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!