| opendevreview | Merged openstack/neutron stable/2024.1: Adopt to StandardAttribute load method change to "selectin" https://review.opendev.org/c/openstack/neutron/+/926629 | 00:11 |
|---|---|---|
| opendevreview | Merged openstack/neutron stable/2023.1: Adopt to StandardAttribute load method change to "selectin" https://review.opendev.org/c/openstack/neutron/+/926633 | 01:56 |
| *** elodilles_ooo is now known as elodilles | 06:44 | |
| opendevreview | Merged openstack/neutron stable/2023.2: Adopt to StandardAttribute load method change to "selectin" https://review.opendev.org/c/openstack/neutron/+/926631 | 08:17 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: DNM == Test neutron-tempest-plugin-openvswitch https://review.opendev.org/c/openstack/neutron/+/925205 | 08:19 |
| opendevreview | Merged openstack/ovn-octavia-provider master: Maintenance task: do not change IPv4 ip_port_mappings https://review.opendev.org/c/openstack/ovn-octavia-provider/+/923196 | 09:07 |
| ralonsoh | ykarel, https://review.opendev.org/c/openstack/neutron/+/925205/12..13/zuul.d/base.yaml | 09:20 |
| ralonsoh | I think I found the issue with the eventlet patch | 09:20 |
| ralonsoh | btw, this tool is implemented in devstack, using normal threading and not very eventlet friendly. I need to check it, but first I'll disable MYSQL_GATHER_PERFORMANCE for all Neutron jobs | 09:22 |
| opendevreview | Pierre Riteau proposed openstack/ovn-octavia-provider stable/2024.1: Maintenance task: do not change IPv4 ip_port_mappings https://review.opendev.org/c/openstack/ovn-octavia-provider/+/926713 | 09:23 |
| opendevreview | Pierre Riteau proposed openstack/ovn-octavia-provider stable/2023.2: Maintenance task: do not change IPv4 ip_port_mappings https://review.opendev.org/c/openstack/ovn-octavia-provider/+/926714 | 09:23 |
| opendevreview | Pierre Riteau proposed openstack/ovn-octavia-provider stable/2023.1: Maintenance task: do not change IPv4 ip_port_mappings https://review.opendev.org/c/openstack/ovn-octavia-provider/+/926715 | 09:24 |
| opendevreview | Pierre Riteau proposed openstack/ovn-octavia-provider master: Error log for missing certs with NB and SB DBs https://review.opendev.org/c/openstack/ovn-octavia-provider/+/924059 | 10:26 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Neutron quota engine checks the resource usage by default https://review.opendev.org/c/openstack/neutron/+/926725 | 10:40 |
| ralonsoh | slaweq, qq, this kind of changes should be done during a SLURP release or in a non-SLURP release? | 10:40 |
| frickler | ralonsoh: do you happen to have more information about how dbcounter is broken? I assume dansmith would be interested | 10:50 |
| ralonsoh | frickler, the patch is not working, at least for the native OVS cases (it is for the hybrid, but I need to recheck) | 10:51 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: Filter resources in pagination tests to avoid random failures https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/926201 | 10:59 |
| opendevreview | Pierre Riteau proposed openstack/ovn-octavia-provider stable/2024.1: [stable/2024.1 only] Drop -master jobs https://review.opendev.org/c/openstack/ovn-octavia-provider/+/926726 | 11:00 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Monkey patch the system libraries before calling them https://review.opendev.org/c/openstack/neutron/+/925376 | 11:33 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: DNM == Test neutron-tempest-plugin-openvswitch https://review.opendev.org/c/openstack/neutron/+/925205 | 11:33 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: DNM == Revert "Fix ObjectChangeHandler thread usage" https://review.opendev.org/c/openstack/neutron/+/926727 | 11:33 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: DNM == Revert "Fix ObjectChangeHandler thread usage" https://review.opendev.org/c/openstack/neutron/+/926727 | 11:57 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: DNM == Test neutron-tempest-plugin-openvswitch https://review.opendev.org/c/openstack/neutron/+/925205 | 11:57 |
| opendevreview | Merged openstack/ovn-octavia-provider master: Error log for missing certs with NB and SB DBs https://review.opendev.org/c/openstack/ovn-octavia-provider/+/924059 | 12:34 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: DNM == Revert "Fix ObjectChangeHandler thread usage" https://review.opendev.org/c/openstack/neutron/+/926727 | 13:08 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: DNM == Test neutron-tempest-plugin-openvswitch https://review.opendev.org/c/openstack/neutron/+/925205 | 13:08 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: DNM == Revert "Fix ObjectChangeHandler thread usage" https://review.opendev.org/c/openstack/neutron/+/926727 | 13:30 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: DNM == Test neutron-tempest-plugin-openvswitch https://review.opendev.org/c/openstack/neutron/+/925205 | 13:30 |
| ihrachys | haleyb: https://patchwork.ozlabs.org/project/ovn/patch/20240821085507.394179-1-amusil@redhat.com/ for nested snat | 13:32 |
| ralonsoh | haleyb, mlavalle slaweq lajoskatona: please check https://review.opendev.org/c/opendev/irc-meetings/+/926763 | 13:46 |
| racosta | ralonsoh, about your comment regarding the expiration date for the maintenance method. What do you suggest? I'm not very familiar with release cycles used there. https://review.opendev.org/c/openstack/neutron/+/907489/comment/99508a9d_ef8bb7ae/ | 13:58 |
| ralonsoh | racosta, your patch is updating the way of creating/updating the static routes | 14:03 |
| haleyb | ihrachys: thanks for that, i pinged martin as well since he was interested in the issue | 14:03 |
| ralonsoh | the maintenance method, that is run once, updates the existing registers to be in sync with the new fields added | 14:04 |
| lajoskatona | ralonsoh: cool, Monday seems good for CI meeting | 14:04 |
| ralonsoh | because we have a SLURP release cadence (https://releases.openstack.org/), we can ensure that in Epoxy (next release), this maintenance method will update all static routes | 14:05 |
| ralonsoh | and if anyone updates then to E+2, this method won't be needed anymore | 14:05 |
| ralonsoh | racosta, so this method should be removed in E+2 | 14:06 |
| ralonsoh | you have similar comments in other methods. E.g.: # TODO(ralonsoh): Remove this method in the E cycle (SLURP release) | 14:06 |
| ralonsoh | lajoskatona, cool thanks! | 14:06 |
| racosta | oh, I got it. That makes sense to use E+2. Thanks | 14:06 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-lib master: Add API extension ``quota-check-limit-default`` https://review.opendev.org/c/openstack/neutron-lib/+/926777 | 14:13 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-lib master: Add API extension ``quota-check-limit-default`` https://review.opendev.org/c/openstack/neutron-lib/+/926777 | 14:45 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: DNM == Test neutron-tempest-plugin-openvswitch (2) https://review.opendev.org/c/openstack/neutron/+/926788 | 15:01 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Neutron quota engine checks the resource usage by default https://review.opendev.org/c/openstack/neutron/+/926725 | 15:15 |
| ihrachys | haleyb: re ovn fix for nested. This is going to patch it for 23.09 but it won't apply to 22.03 (we actually don't know what "fixed it" in between 22.03 and 23.09. some digging would have to happen to understand the exact set of patches that could be backported to 22.03 to resolve it there too. And I don't think OVN folks are planning to do the digging. from redhat perspective, we are only | 15:44 |
| ihrachys | targeting this to clouds with ovn 23.09+ so it's not a big deal for us. For upstream CI validation, we'll pull a 23.09+ ovn from master in fullstack/functional jobs; but we won't be able to use the feature with stock canonical packages which I think are 22.03? | 15:44 |
| haleyb | ihrachys: ack, thanks. regarding upstream CI, yes I believe they are still on 22.03 except in the "build from source" jobs, hopefully that will change when we can move to a newer lts | 15:47 |
| ihrachys | haleyb: I'm ignorant... for your openstack *product*, do you also pull 22.03? | 15:49 |
| haleyb | ihrachys: we have a 22.03.x version of OVN in our archive that we support for both focal and jammy releases. if this snat change doesn't work there (don't think it does based on the bug), we might need to bisect to find what fixed it | 15:54 |
| ihrachys | haleyb: yes, bisect it is. (I'd probably start with validating that "commit prior to the one in 23.09 that presumably broke it" indeed is good. then bisect 22.03..<the-good-commit>. | 15:56 |
| ihrachys | because we are *told* that it is the one that broke it. but did it broke it or just exacerbated something that was still broken? that I don't know. :) | 15:56 |
| * haleyb will add it to his list | 15:58 | |
| opendevreview | Merged openstack/neutron master: Revert "Temporary mark ovs-rally job as non-voting" https://review.opendev.org/c/openstack/neutron/+/926613 | 16:56 |
| opendevreview | Ghanshyam proposed openstack/neutron master: Remove default override for RBAC config options https://review.opendev.org/c/openstack/neutron/+/926085 | 17:44 |
| gmann | ralonsoh: ^^ I rebased on top of your change, let's see | 17:45 |
| ralonsoh | gmann, I'll keep an eye on this patch | 17:45 |
| gmann | ralonsoh: thanks. | 17:45 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: DNM == Test neutron-tempest-plugin-openvswitch (2) https://review.opendev.org/c/openstack/neutron/+/926788 | 17:54 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: DNM == Test neutron-tempest-plugin-openvswitch (2) https://review.opendev.org/c/openstack/neutron/+/926788 | 17:54 |
| opendevreview | Mohammed Naser proposed openstack/neutron master: Fix MAC address filtering for ml2ovn-trace https://review.opendev.org/c/openstack/neutron/+/926840 | 18:49 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: DNM == Test neutron-tempest-plugin-openvswitch https://review.opendev.org/c/openstack/neutron/+/925205 | 20:32 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: DNM == Test neutron-tempest-plugin-openvswitch https://review.opendev.org/c/openstack/neutron/+/925205 | 20:45 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: DNM == Add new "tagging" API method: create (POST) https://review.opendev.org/c/openstack/neutron/+/924724 | 21:22 |
| opendevreview | Roberto Acosta proposed openstack/neutron master: [ML2/OVN] Add external_ids.neutron:is_static_route key for Static Routes https://review.opendev.org/c/openstack/neutron/+/907489 | 22:11 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: WIP Support nested SNAT for ml2/ovn https://review.opendev.org/c/openstack/neutron/+/926495 | 22:15 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: Support nested SNAT for ml2/ovn https://review.opendev.org/c/openstack/neutron/+/926495 | 23:51 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!