| opendevreview | nikhil kshirsagar proposed openstack/neutron stable/ussuri: [OVN] Fix port disable security dead when run neutron-ovn-db-sync-util https://review.opendev.org/c/openstack/neutron/+/874979 | 04:50 |
|---|---|---|
| opendevreview | nikhil kshirsagar proposed openstack/neutron stable/ussuri: [OVN] Fix port disable security dead when run neutron-ovn-db-sync-util https://review.opendev.org/c/openstack/neutron/+/874979 | 05:25 |
| opendevreview | nikhil kshirsagar proposed openstack/neutron stable/ussuri: [OVN] Fix port disable security dead when run neutron-ovn-db-sync-util https://review.opendev.org/c/openstack/neutron/+/874979 | 05:33 |
| opendevreview | nikhil kshirsagar proposed openstack/neutron stable/ussuri: [OVN] Fix port disable security dead when run neutron-ovn-db-sync-util https://review.opendev.org/c/openstack/neutron/+/874979 | 05:48 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [sqlalchemy-20] Use the .begin() method of Engine https://review.opendev.org/c/openstack/neutron/+/874867 | 08:45 |
| opendevreview | Merged openstack/neutron master: Format correctly (dialect=mac_unix_expanded) the MAC addresses https://review.opendev.org/c/openstack/neutron/+/874654 | 09:19 |
| opendevreview | Merged openstack/os-vif master: Update master for stable/2023.1 https://review.opendev.org/c/openstack/os-vif/+/875097 | 10:04 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider master: Remove HM uuid from LB external_ids when the HM is deleted https://review.opendev.org/c/openstack/ovn-octavia-provider/+/875400 | 11:17 |
| opendevreview | Merged openstack/os-vif stable/2023.1: Update .gitreview for stable/2023.1 https://review.opendev.org/c/openstack/os-vif/+/875095 | 12:28 |
| opendevreview | Merged openstack/os-vif stable/2023.1: Update TOX_CONSTRAINTS_FILE for stable/2023.1 https://review.opendev.org/c/openstack/os-vif/+/875096 | 12:28 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/zed: Normalise format of OVN agent heartbeat timestamp https://review.opendev.org/c/openstack/neutron/+/875485 | 12:42 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/yoga: Normalise format of OVN agent heartbeat timestamp https://review.opendev.org/c/openstack/neutron/+/875486 | 12:43 |
| *** haleyb_ is now known as haleyb | 14:36 | |
| *** dansmith_ is now known as dansmith | 14:44 | |
| opendevreview | Merged openstack/ovn-octavia-provider master: Ensure HM also apply to FIPs associated to LB VIPs https://review.opendev.org/c/openstack/ovn-octavia-provider/+/873860 | 15:42 |
| jamesdenton_ | lucasagomes re: https://bugs.launchpad.net/neutron/+bug/2007167 - I was able to hand jam next_server into OVN DHCP_options to get TFTP working and a successful baremetal deploy. There does not appear to be any service, Ironic or Neutron, attempting to insert that option | 15:46 |
| lucasagomes | jamesdenton_, do you have this patch on https://review.opendev.org/c/openstack/neutron/+/840316/9/neutron/common/ovn/constants.py ? | 15:53 |
| jamesdenton_ | looking | 15:53 |
| lucasagomes | jamesdenton_, cause, IIRC, Ironic should send the server-ip-address option | 15:53 |
| lucasagomes | which goes to the next-server one for OVN | 15:53 |
| jamesdenton_ | yes, i have that patch | 15:57 |
| lucasagomes | jamesdenton_, I see, I need to check the ironic flow. I was expecting it to send it | 15:58 |
| jamesdenton_ | lucasagomes https://github.com/openstack/ironic/blob/master/ironic/common/pxe_utils.py#L62 -- Option 255? | 16:01 |
| lucasagomes | jamesdenton_, yeah, looks dnsmasq specific... I think we could either make ML2/OVN understand that option and convert it to next_server or perhaps change ironic to send the "next_server" option directly ? | 16:06 |
| jamesdenton_ | i started working on the latter, but only got so far. Seems there's a DHCP option filter in place, but i lost track of that | 16:07 |
| lucasagomes | jamesdenton_, in OVN that next-server is DHCP 253, which is for private use | 16:10 |
| lucasagomes | https://www.iana.org/assignments/bootp-dhcp-parameters/bootp-dhcp-parameters.xhtml | 16:10 |
| lucasagomes | just like the 255 that dnsmaq implements I guess | 16:11 |
| lucasagomes | but yeah, having Ironic to send it seems to be the right solution to work with both DHCP servers | 16:11 |
| opendevreview | Merged openstack/neutron-lib master: Remove unused rbac rule https://review.opendev.org/c/openstack/neutron-lib/+/875305 | 16:11 |
| jamesdenton_ | yep, ok. I've got Ironic adding 253 to the neutron port, but OVN is not picking it up. I thought i recall seeing some dhcp option filtering in the code, but that was saturday and i've slept twice since then | 16:12 |
| jamesdenton_ | oh yeah. nevermind, that is/was the filtering via SUPPORTED_DHCP_OPTS_MAPPING. I will look again | 16:15 |
| lucasagomes | jamesdenton_, it's possible we don't have that option as a number in the "known" option list for ML2/OVN. Can you try sending the "server-ip-address" ? | 16:16 |
| lucasagomes | that said, we can add 253 there in ML2/OVN too | 16:16 |
| lucasagomes | yup that's the mapping | 16:16 |
| jamesdenton_ | sure, i can try that | 16:17 |
| ralonsoh | lajoskatona, if you have 1 min: https://review.opendev.org/c/openstack/neutron/+/874867 | 16:23 |
| ralonsoh | thanks in advance | 16:23 |
| opendevreview | Merged openstack/neutron master: ml2 plugin: use const from neutron-lib https://review.opendev.org/c/openstack/neutron/+/874631 | 16:44 |
| opendevreview | Merged openstack/neutron master: Reduce number of routers created in ``test_gateway_chassis_rebalance`` https://review.opendev.org/c/openstack/neutron/+/874765 | 16:44 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider master: Reset member provisioning status to NO_MONITOR when a HM is deleted https://review.opendev.org/c/openstack/ovn-octavia-provider/+/874609 | 16:57 |
| opendevreview | Merged openstack/neutron stable/zed: Enable qos extension_driver in ovn_db_sync https://review.opendev.org/c/openstack/neutron/+/874142 | 17:00 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider master: Reset member provisioning status to NO_MONITOR when a HM is deleted https://review.opendev.org/c/openstack/ovn-octavia-provider/+/874609 | 17:03 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider master: Remove HM uuid from LB external_ids when the HM is deleted https://review.opendev.org/c/openstack/ovn-octavia-provider/+/875400 | 17:04 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/zed: Format correctly (dialect=mac_unix_expanded) the MAC addresses https://review.opendev.org/c/openstack/neutron/+/875535 | 17:04 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/yoga: Format correctly (dialect=mac_unix_expanded) the MAC addresses https://review.opendev.org/c/openstack/neutron/+/875536 | 17:04 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/xena: Format correctly (dialect=mac_unix_expanded) the MAC addresses https://review.opendev.org/c/openstack/neutron/+/875537 | 17:04 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/wallaby: Format correctly (dialect=mac_unix_expanded) the MAC addresses https://review.opendev.org/c/openstack/neutron/+/875538 | 17:05 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider master: Remove HM uuid from LB external_ids when the HM is deleted https://review.opendev.org/c/openstack/ovn-octavia-provider/+/875400 | 17:21 |
| opendevreview | Merged openstack/neutron stable/xena: Enable qos extension_driver in ovn_db_sync https://review.opendev.org/c/openstack/neutron/+/874144 | 17:53 |
| -opendevstatus- NOTICE: The Gerrit service on review.opendev.org experienced severe performance degradation between 17:50 and 19:45 due to excessive API query activity; the addresses involved are now blocked but any changes missing job results from that timeframe should be rechecked | 19:55 | |
| opendevreview | James Denton proposed openstack/neutron master: Apply Ironic's server-ip-address as TFTP next-server https://review.opendev.org/c/openstack/neutron/+/875551 | 20:48 |
| opendevreview | Brian Haley proposed openstack/neutron master: Move register_common_config_options call to DietTestCase https://review.opendev.org/c/openstack/neutron/+/875552 | 20:51 |
| opendevreview | Merged openstack/neutron master: [sqlalchemy-20] Use the .begin() method of Engine https://review.opendev.org/c/openstack/neutron/+/874867 | 21:07 |
| opendevreview | Brian Haley proposed openstack/neutron master: Remove two duplicated unit tests https://review.opendev.org/c/openstack/neutron/+/875571 | 21:46 |
| opendevreview | Dmitrii Shcherbakov proposed openstack/neutron-lib master: ext-gw-multihoming: api-def and api-ref https://review.opendev.org/c/openstack/neutron-lib/+/870887 | 22:46 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!