| opendevreview | Seena Fallah proposed openstack/neutron master: ovn: remove first time condition from ChassisCreateEventBase https://review.opendev.org/c/openstack/neutron/+/818301 | 00:24 |
|---|---|---|
| opendevreview | Yi Yang proposed openstack/neutron master: Set network namespace and tap interface type in case of OVS DPDK https://review.opendev.org/c/openstack/neutron/+/818338 | 07:47 |
| lajoskatona | amotoki: Hi, regarding https://review.opendev.org/c/openstack/networking-midonet/+/798993 it seems some terrible pip resolver issue is there, to tell the truth I am lost there.... | 08:18 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/xena: Check interface presence in new namespace https://review.opendev.org/c/openstack/neutron/+/818340 | 08:41 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/wallaby: Check interface presence in new namespace https://review.opendev.org/c/openstack/neutron/+/818341 | 08:45 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/victoria: Check interface presence in new namespace https://review.opendev.org/c/openstack/neutron/+/818342 | 08:46 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/ussuri: Check interface presence in new namespace https://review.opendev.org/c/openstack/neutron/+/818343 | 08:46 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/train: Check interface presence in new namespace https://review.opendev.org/c/openstack/neutron/+/818344 | 08:46 |
| ralonsoh | slaweq, https://review.opendev.org/c/openstack/networking-ovn/+/812339 if you have time. Thanks in advance | 08:48 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/wallaby: Wait until workers have been launched https://review.opendev.org/c/openstack/neutron/+/818346 | 08:51 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/victoria: Wait until workers have been launched https://review.opendev.org/c/openstack/neutron/+/818347 | 08:51 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/ussuri: Wait until workers have been launched https://review.opendev.org/c/openstack/neutron/+/818348 | 08:52 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/train: Wait until workers have been launched https://review.opendev.org/c/openstack/neutron/+/818349 | 08:52 |
| slaweq | ralonsoh: sure, right now | 08:57 |
| slaweq | ralonsoh: commented there | 08:59 |
| slaweq | please correct me if I'm wrong but there is only UT part of the patch cherry-picked there | 08:59 |
| slaweq | is that correct? | 09:00 |
| ralonsoh | slaweq, yes, this is because of https://review.opendev.org/c/openstack/networking-ovn/+/812714 | 09:00 |
| ralonsoh | ^^ this patch implemented the fixes for this bug and 1942794 | 09:00 |
| ralonsoh | so the patch you reviewed is just implementing the testing part | 09:01 |
| slaweq | ahh, ok | 09:01 |
| slaweq | maybe it's worth to mention that in the commit message? | 09:01 |
| slaweq | wdyt? | 09:01 |
| ralonsoh | I know, that's the problem of migrating from neutron to n-ovn | 09:01 |
| ralonsoh | ok, I'll do it now | 09:01 |
| slaweq | to avoid such confusion in the future if someone will check git blame :) | 09:01 |
| slaweq | thx | 09:02 |
| opendevreview | Rodolfo Alonso proposed openstack/networking-ovn stable/train: Fix "_sync_metadata_ports" with no DHCP subnets https://review.opendev.org/c/openstack/networking-ovn/+/812339 | 09:06 |
| ralonsoh | slaweq, bcafarel ^ | 09:06 |
| ralonsoh | thanks in advance | 09:06 |
| bcafarel | ralonsoh: sure, added to my list | 09:08 |
| ralonsoh | note: I've just updated the commit message | 09:08 |
| opendevreview | Luis Tomas Bolivar proposed openstack/networking-bgpvpn master: WIP: Add OVN-based Neutron BGPVPN driver https://review.opendev.org/c/openstack/networking-bgpvpn/+/803161 | 09:08 |
| lajoskatona | slaweq, bcafarel:I have as well a stable patch for odl, please check it if you have some time: https://review.opendev.org/c/openstack/networking-odl/+/818154 | 09:10 |
| slaweq | ralonsoh: I quickly approved Your patch after commit message update | 09:12 |
| slaweq | lajoskatona: sure | 09:12 |
| ralonsoh | thanks! | 09:12 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-lib master: Add floating IP network QoS field to floating IP definition https://review.opendev.org/c/openstack/neutron-lib/+/817936 | 09:21 |
| fnordahl | Something like this data structure: https://pastebin.ubuntu.com/p/JrnQhGvTgM/ for OVN_PORT_BINDING_PROFILE_PARAMS would allow me to extend validate_and_get_data_from_binding_profile() to handle the various cases needed. Any thoughts? | 10:02 |
| bcafarel | lajoskatona: sure looking | 10:23 |
| ralonsoh | fnordahl, sorry I'm a bit lost with this | 10:48 |
| ralonsoh | this is for the smartnic patch, right? | 10:48 |
| fnordahl | ralonsoh: yes. the problem is once I add validation for something containing, let's say `pci_slot`, I need to cover all the cases (Legacy SR-IOV, HWOL with and without physnet and SmartNIC with and without physnet). | 10:51 |
| fnordahl | ralonsoh: the current code appears to assume the param_set is equal for all cases once a key is mentioned in it. | 10:52 |
| ralonsoh | we don't have legacy SRIOV in OVN | 10:52 |
| ralonsoh | those are considered external ports | 10:52 |
| ralonsoh | but we don;t plug SRIOV ports to OVN | 10:52 |
| ralonsoh | we have so far direct ports with swithdev caps | 10:52 |
| fnordahl | right, so as long as the validator won' be called for those that case is fine then. I'll double check that. | 10:53 |
| ralonsoh | same as in OVS | 10:53 |
| fnordahl | I think the validator will hit the switchdev ports though? | 10:53 |
| ralonsoh | let me ask you, what is the difference between smartnics, DPUs and HW offload (switchdev) ? | 10:54 |
| fnordahl | ralonsoh: The main difference is that the eswitch is only visible to the Arm cores on the NIC, so we need to run OVS and OVN there. The hypervisor host has no network agents. As a consequence when we bind a port we have both a hypervisor host and a smartnic control plane host to deal with. From Neutron and OVN's POV the port is bound to the smartnic control plane host. | 10:57 |
| ralonsoh | but this is DPU | 10:58 |
| fnordahl | yes | 10:58 |
| ralonsoh | and smartnic? | 10:58 |
| fnordahl | yes | 10:58 |
| ralonsoh | I don't get it. Smartnic and DPU is the same? | 10:59 |
| fnordahl | The terms are very overloaded and some use them interchanged. | 10:59 |
| ralonsoh | yeah, this is a problem | 10:59 |
| ralonsoh | there is a bit of confusion there | 10:59 |
| ralonsoh | anyway, let's consider smartnics as DPU | 10:59 |
| ralonsoh | I don't see anything in the middle right now | 11:00 |
| fnordahl | so to support the DPU case the port_binding data will have additional keys compared to the regular HWOL case. | 11:00 |
| fnordahl | A part of the work to support DPU is to use data in port_binding to look up the OVN chassis, so this is why I'm poking at the validator code. | 11:01 |
| ralonsoh | fnordahl, yes, OVNPortBindingProfileParamSet could be a valid solution | 11:03 |
| fnordahl | ralonsoh: great! Is it ok to define the namedtuple in constants.py or do we have a separate place for those? I guess it would be convenient to have it there as the consuming code would need access to its definition too. | 11:04 |
| ralonsoh | I think it is valid to define this tuple there, this is going to be a container for a set of constants (just in a more elegant way) | 11:06 |
| fnordahl | tip top, I'll move forward and we'll see how it turns out and can decide then. Thank you for your input! | 11:07 |
| opendevreview | Mamatisa Nurmatov proposed openstack/neutron-tempest-plugin master: Add Local IP API tests https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/814742 | 11:56 |
| opendevreview | Merged openstack/ovsdbapp stable/xena: tools: run functional tests against ovn/main branch https://review.opendev.org/c/openstack/ovsdbapp/+/813342 | 12:02 |
| opendevreview | Lajos Katona proposed openstack/neutron-tempest-plugin master: Add tap-as-a-service scenario tests https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/814876 | 12:12 |
| opendevreview | Lajos Katona proposed openstack/networking-odl stable/stein: Blacklist bandit, bump neutron-lib and retire neutron-lbaas https://review.opendev.org/c/openstack/networking-odl/+/818154 | 12:32 |
| opendevreview | Lajos Katona proposed openstack/networking-odl stable/stein: Blacklist bandit, bump neutron-lib and retire neutron-lbaas https://review.opendev.org/c/openstack/networking-odl/+/818154 | 12:33 |
| lajoskatona | slaweq: fyi: I pushed a grafana patch to have some zuul usage graphs: https://review.opendev.org/c/openstack/project-config/+/818230 | 12:36 |
| lajoskatona | slaweq: I have to ask around in infra how to include other projects like tripleo as that has many small projects for example, and I am not a grafana expert | 12:37 |
| opendevreview | Merged openstack/ovsdbapp stable/xena: Update .gitreview for stable/xena https://review.opendev.org/c/openstack/ovsdbapp/+/808269 | 12:47 |
| opendevreview | Merged openstack/ovsdbapp stable/xena: Update TOX_CONSTRAINTS_FILE for stable/xena https://review.opendev.org/c/openstack/ovsdbapp/+/808273 | 12:47 |
| frickler | lajoskatona: I don't know how to aggregate project either, you might also want to include neutron subproject maybe? left some other comments though | 12:48 |
| lajoskatona | frickler: thanks,I check it | 13:08 |
| opendevreview | Lajos Katona proposed openstack/neutron-tempest-plugin master: Add tap-as-a-service scenario tests https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/814876 | 13:12 |
| opendevreview | Merged openstack/networking-ovn stable/train: Fix "_sync_metadata_ports" with no DHCP subnets https://review.opendev.org/c/openstack/networking-ovn/+/812339 | 13:19 |
| opendevreview | Lajos Katona proposed openstack/networking-odl stable/stein: Blacklist bandit, bump neutron-lib and retire neutron-lbaas https://review.opendev.org/c/openstack/networking-odl/+/818154 | 13:27 |
| opendevreview | Lajos Katona proposed openstack/networking-odl stable/rocky: Blacklist bandit, bump neutron-lib and retire neutron-lbaas https://review.opendev.org/c/openstack/networking-odl/+/818384 | 13:34 |
| slaweq | lajoskatona: thx,that is really good idea | 13:56 |
| opendevreview | Lajos Katona proposed openstack/python-neutronclient master: DNM: test master https://review.opendev.org/c/openstack/python-neutronclient/+/818386 | 14:00 |
| opendevreview | Bence Romsics proposed openstack/neutron master: Disable tracebacks of eventlet.wsgi.server https://review.opendev.org/c/openstack/neutron/+/818391 | 14:12 |
| opendevreview | Lajos Katona proposed openstack/python-neutronclient master: DNM: test master https://review.opendev.org/c/openstack/python-neutronclient/+/818386 | 14:15 |
| opendevreview | Lajos Katona proposed openstack/python-neutronclient master: Fix lower-constraints job https://review.opendev.org/c/openstack/python-neutronclient/+/818386 | 14:36 |
| opendevreview | Terry Wilson proposed openstack/ovsdbapp master: Use setkey for DbSetCommand maps https://review.opendev.org/c/openstack/ovsdbapp/+/804252 | 14:53 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Add network QoS inheritance to floating IP https://review.opendev.org/c/openstack/neutron/+/817734 | 15:05 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Implement floating IP network QoS inheritance https://review.opendev.org/c/openstack/neutron/+/817790 | 15:05 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Add upgrade check for floating IP QoS inheritance from network https://review.opendev.org/c/openstack/neutron/+/818399 | 15:05 |
| opendevreview | Merged openstack/ovsdbapp master: Use py3 as the default runtime for tox https://review.opendev.org/c/openstack/ovsdbapp/+/787172 | 15:24 |
| opendevreview | Lajos Katona proposed openstack/python-neutronclient master: Fix lower-constraints and neutronclient-functional job https://review.opendev.org/c/openstack/python-neutronclient/+/818386 | 15:26 |
| opendevreview | Merged openstack/ovsdbapp master: Update master for stable/xena https://review.opendev.org/c/openstack/ovsdbapp/+/808276 | 15:26 |
| lajoskatona | slaweq: seems python-neutronclient master is broken: https://review.opendev.org/c/openstack/python-neutronclient/+/818386 | 15:27 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Remove functions which enable Neutron's QoS service https://review.opendev.org/c/openstack/neutron/+/815687 | 15:44 |
| opendevreview | Arnau Verdaguer proposed openstack/neutron master: Prevent deleting the only IP of a LRP https://review.opendev.org/c/openstack/neutron/+/817525 | 15:45 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: ovn: Bump OVN version to 21.09.1 https://review.opendev.org/c/openstack/neutron/+/818418 | 16:02 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: WIP ovn: Off-path SmartNIC DPU Port Binding with OVN https://review.opendev.org/c/openstack/neutron/+/808961 | 16:03 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: WIP ovn: Extend port binding parameter validation https://review.opendev.org/c/openstack/neutron/+/818420 | 16:03 |
| opendevreview | Merged openstack/ovsdbapp master: Add Python3 yoga unit tests https://review.opendev.org/c/openstack/ovsdbapp/+/808278 | 16:19 |
| opendevreview | Merged openstack/neutron master: [OVS][QOS] Dataplane enforcement is limited to min-bw egress direction https://review.opendev.org/c/openstack/neutron/+/816530 | 16:24 |
| opendevreview | Merged openstack/ovsdbapp master: nb: add support for lrp's gateway chassis API https://review.opendev.org/c/openstack/ovsdbapp/+/815723 | 16:24 |
| opendevreview | Merged openstack/ovsdbapp master: nb: provide 'discard' value for nexthop https://review.opendev.org/c/openstack/ovsdbapp/+/812484 | 16:29 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Handle OVN agents when "Chassis" register is deleted https://review.opendev.org/c/openstack/neutron/+/818132 | 16:30 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/xena: [OVS][QOS] Dataplane enforcement is limited to min-bw egress direction https://review.opendev.org/c/openstack/neutron/+/818436 | 16:34 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/wallaby: [OVS][QOS] Dataplane enforcement is limited to min-bw egress direction https://review.opendev.org/c/openstack/neutron/+/818437 | 16:34 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/victoria: [OVS][QOS] Dataplane enforcement is limited to min-bw egress direction https://review.opendev.org/c/openstack/neutron/+/818438 | 16:35 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/ussuri: [OVS][QOS] Dataplane enforcement is limited to min-bw egress direction https://review.opendev.org/c/openstack/neutron/+/818439 | 16:37 |
| ralonsoh | slaweq, lajoskatona rally jobs are failing 100% of times | 16:40 |
| ralonsoh | both ovn and ovs | 16:40 |
| ralonsoh | frickler, hi, do you why rally tasks could be failing? | 16:48 |
| ralonsoh | https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_803/817734/5/check/neutron-ovs-rally-task/803186b/job-output.txt | 16:48 |
| ralonsoh | all of them, OVS and OVN | 16:48 |
| ralonsoh | do you know* | 16:48 |
| frickler | ralonsoh: that'll very likely be fallout from https://review.opendev.org/c/openstack/devstack/+/780417 , I can take a closer look tomorrow | 16:51 |
| ralonsoh | frickler, thanks for the tip. I'll open a LP bug with this info | 16:52 |
| opendevreview | Merged openstack/neutron master: ovn: Filter ACL columns when syncing the DB https://review.opendev.org/c/openstack/neutron/+/818255 | 17:10 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron master: [OVN] Fix gateway_mtu option should not always be set https://review.opendev.org/c/openstack/neutron/+/818443 | 17:31 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron master: [OVN] Fix gateway_mtu option should not always be set https://review.opendev.org/c/openstack/neutron/+/818443 | 17:34 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron master: [OVN] Fix gateway_mtu option should not always be set https://review.opendev.org/c/openstack/neutron/+/818443 | 17:36 |
| *** elvira1 is now known as elvira | 17:36 | |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Prevent OVS to OVN migration if firewall "iptables_hybrid" https://review.opendev.org/c/openstack/neutron/+/818239 | 18:14 |
| opendevreview | Terry Wilson proposed openstack/ovsdbapp master: Add cooperative_yield() to OvsdbIdl https://review.opendev.org/c/openstack/ovsdbapp/+/818446 | 18:19 |
| opendevreview | Terry Wilson proposed openstack/ovsdbapp master: Use setkey for DbSetCommand maps https://review.opendev.org/c/openstack/ovsdbapp/+/804252 | 18:21 |
| *** tkajinam is now known as Guest6220 | 18:30 | |
| lajoskatona | haleyb: Hi, not sure if you agreed with ralonsoh for changing you bug deputy week, but the week after next (from 29 Nov) is mine so you can change with me as well, if that's ok for you | 19:22 |
| opendevreview | Lajos Katona proposed openstack/neutron master: Rally: provide auth_url for rally jobs https://review.opendev.org/c/openstack/neutron/+/818457 | 19:45 |
| opendevreview | Jakub Libosvar proposed openstack/neutron stable/xena: ovn: Filter ACL columns when syncing the DB https://review.opendev.org/c/openstack/neutron/+/818471 | 19:53 |
| opendevreview | Jakub Libosvar proposed openstack/neutron stable/wallaby: ovn: Filter ACL columns when syncing the DB https://review.opendev.org/c/openstack/neutron/+/818472 | 19:54 |
| opendevreview | Jakub Libosvar proposed openstack/neutron stable/victoria: ovn: Filter ACL columns when syncing the DB https://review.opendev.org/c/openstack/neutron/+/818473 | 19:54 |
| opendevreview | Jakub Libosvar proposed openstack/neutron stable/ussuri: ovn: Filter ACL columns when syncing the DB https://review.opendev.org/c/openstack/neutron/+/818474 | 19:56 |
| opendevreview | Jakub Libosvar proposed openstack/networking-ovn stable/train: ovn: Filter ACL columns when syncing the DB https://review.opendev.org/c/openstack/networking-ovn/+/818460 | 20:11 |
| opendevreview | Jakub Libosvar proposed openstack/networking-ovn stable/train: ovn: Filter ACL columns when syncing the DB https://review.opendev.org/c/openstack/networking-ovn/+/818460 | 20:37 |
| Satheesh_ | Greetings All - We are getting an issue when we have Control Plane in Ussuri version (21.2.10) and Compute plane is in Train (20.2.1) version, we are getting Virtual Interface Creation Error while provisioning the VM? nova.exception.VirtualInterfaceCreateException: Virtual Interface creation failed Did anyone faced such issue? ERROR neutron.plugins.ml2.drivers.agent._common_agent [req-3806c12d-6522-4b6a-b575-33d2faf0b859 - - - - -] Error in | 20:56 |
| Satheesh_ | nova-compute version is 20.4.2 and neutron-linuxbridge-agent version is 15.3.1.dev26 | 20:56 |
| Satheesh_ | Complete Error log snippet is attached here.. https://paste.opendev.org/show/811180/ | 20:56 |
| opendevreview | Mamatisa Nurmatov proposed openstack/neutron-tempest-plugin master: Add Local IP API tests https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/814742 | 21:16 |
| masterpe[m] | Satheesh_: This is a bug, I don't know what bug, but you need to change in file neutron/agent/linux/ipset_manager.py on line 9 change for ip, _mac in addresses: into for ip in addresses: on the Train computed nodes. | 22:01 |
| masterpe[m] | After upgrading the compute node to Ussuri, this line needs to be back to the default. | 22:01 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!