| opendevreview | morice proposed openstack/neutron master: [ovn] AZs distribution in L3 port scheduler https://review.opendev.org/c/openstack/neutron/+/892604 | 06:17 |
|---|---|---|
| opendevreview | Fernando Royo proposed openstack/neutron master: Add constant to identify OVN LB HM ports https://review.opendev.org/c/openstack/neutron/+/897345 | 08:23 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider master: Change device_owner used for OVN LB HM port https://review.opendev.org/c/openstack/ovn-octavia-provider/+/897348 | 08:27 |
| grishatsu | Is anyone familiar with neutron-linuxbridge-agent failing to run on a container on top of mac m1 because Cmd: ['iptables-save', '-t', 'raw']; Stdin: ; Stdout: ; Stderr: iptables-save v1.4.21: Cannot initialize: iptables who? (do you need to insmod?) | 08:32 |
| grishatsu | I wonder if it is even supported to run these in a container with a macOs host | 08:33 |
| frickler | I don't think anything is getting supported on macos | 08:52 |
| grishatsu | figures | 08:53 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Set the Neutron port status based on "lsp.up" and "lsp.enabled" https://review.opendev.org/c/openstack/neutron/+/896939 | 10:00 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [UT] Add chassis.external_ids dict in ``TestOvnSbIdlNotifyHandler`` https://review.opendev.org/c/openstack/neutron/+/897407 | 10:02 |
| opendevreview | Fernando Royo proposed openstack/neutron master: Add constant to identify OVN LB HM ports https://review.opendev.org/c/openstack/neutron/+/897345 | 10:04 |
| opendevreview | Thomas Goirand proposed openstack/neutron-vpnaas-dashboard master: Use repr() for object tests https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/897408 | 10:23 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Fix the ``log.setup`` method call with "fix_eventlet=False" https://review.opendev.org/c/openstack/neutron/+/897332 | 10:35 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Replace "tenant_id" with "project_id" in IPAM engine https://review.opendev.org/c/openstack/neutron/+/877533 | 10:37 |
| ralonsoh | ykarel, I've seen this twice | 10:42 |
| ralonsoh | https://08a3be70b8bf624f00d5-8466237625318fdb84c00b23a21e0718.ssl.cf5.rackcdn.com/883907/14/check/neutron-functional-with-uwsgi/97280d5/testr_results.html | 10:42 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider master: Change device_owner used for OVN LB HM port https://review.opendev.org/c/openstack/ovn-octavia-provider/+/897348 | 10:42 |
| ralonsoh | in the patch un-patching the evenlet issue and in this other one, not related | 10:42 |
| ralonsoh | ebtables is failing somehow | 10:42 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Add "jammy" distribution release to the legacy ebtables installation https://review.opendev.org/c/openstack/neutron/+/897412 | 10:52 |
| ykarel | ralonsoh, looks like a new issue, started seeing multiple times since today as per opensearch | 10:53 |
| ralonsoh | ^^ I've pushed this patch | 10:53 |
| ykarel | ahkk god, thanks | 10:54 |
| ykarel | but why it started happening today | 10:54 |
| ralonsoh | no idea, to be honest | 10:54 |
| ykarel | ok let's see if this helps | 10:54 |
| ralonsoh | let me check if there is a new nftables version | 10:54 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron stable/2023.1: Use safer methods to get security groups on security group logging https://review.opendev.org/c/openstack/neutron/+/896614 | 11:20 |
| opendevreview | Michel Nederlof proposed openstack/neutron master: [OVN] Delete entries for virtual ip from Mac_Binding table https://review.opendev.org/c/openstack/neutron/+/896884 | 12:02 |
| opendevreview | Michel Nederlof proposed openstack/neutron master: [OVN] Update lsp host id when virtual parent changes https://review.opendev.org/c/openstack/neutron/+/896883 | 12:05 |
| ykarel | ralonsoh, doesn't look any change in nftables/ebtables versions | 12:58 |
| ykarel | but there are some kernel updates which looks realted https://launchpad.net/ubuntu/+source/linux/5.15.0-86.96 | 12:59 |
| ralonsoh | yes, I saw this and that could be related | 12:59 |
| ralonsoh | I'm going to push a patch with several FT CI to test this patch | 13:00 |
| ykarel | so it was just working without ebtables-legacy but with that kernel update it needs some fixes or use ebtables-legacy | 13:00 |
| ykarel | considering consistent failure it looks good fix as job passing | 13:01 |
| ykarel | but may be we have to fix such that it works with non-legacy as it used to work like before | 13:02 |
| ralonsoh | that implies moving everything to nftables | 13:03 |
| ralonsoh | instead of using the legacy api | 13:03 |
| ralonsoh | and that... is a titanic job | 13:03 |
| ykarel | ohkk got it, now i recall some past efforts for this | 13:03 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: DNM functional and fullstack testing https://review.opendev.org/c/openstack/neutron/+/897427 | 13:06 |
| ykarel | reported https://bugs.launchpad.net/neutron/+bug/2038541 | 13:10 |
| mnederlof | ralonsoh, in regard to https://review.opendev.org/c/openstack/neutron/+/896883; i'm starting to think we might need to revisit the match method too, as it seems to me that the `old` variable holds the changed fields. So if the field is not present, there is not a change init, and we should not trigger on it. | 13:10 |
| mnederlof | also, upon a IP failover, it the virtual-parent field that is changed, not the virtual-parents | 13:11 |
| ralonsoh | sorry, in a meeting now | 13:11 |
| mnederlof | ok | 13:11 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Fix the ``log.setup`` method call with "fix_eventlet=False" https://review.opendev.org/c/openstack/neutron/+/897332 | 13:16 |
| *** mnederlof_ is now known as mnederlof | 13:35 | |
| opendevreview | Michel Nederlof proposed openstack/neutron master: [OVN] Delete entries for virtual ip from Mac_Binding table https://review.opendev.org/c/openstack/neutron/+/896884 | 14:31 |
| ralonsoh | lajoskatona, ykarel https://review.opendev.org/c/openstack/neutron/+/897332 (the ebtables patch tested in https://review.opendev.org/c/openstack/neutron/+/897427 is going very well) | 14:33 |
| ralonsoh | and this patch is in top of it | 14:33 |
| ykarel | ++ | 14:35 |
| opendevreview | Grisha Tsukerman proposed openstack/neutron stable/yoga: Fix router port deletion issue in DHCP agent https://review.opendev.org/c/openstack/neutron/+/897435 | 14:35 |
| opendevreview | Grisha Tsukerman proposed openstack/neutron stable/yoga: Adds protection in agent worker for failed jobs https://review.opendev.org/c/openstack/neutron/+/897436 | 14:35 |
| lajoskatona | ralonsoh: cool, I had no time since morning to check it | 14:39 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2023.2: Add "jammy" distribution release to the legacy ebtables installation https://review.opendev.org/c/openstack/neutron/+/897438 | 14:41 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2023.2: Fix the ``log.setup`` method call with "fix_eventlet=False" https://review.opendev.org/c/openstack/neutron/+/897439 | 14:42 |
| opendevreview | Luis Tomas Bolivar proposed openstack/neutron master: Add support for FDB aging https://review.opendev.org/c/openstack/neutron/+/893333 | 14:44 |
| ykarel | lajoskatona, please also check https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/897308 | 14:45 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2023.1: Add "jammy" distribution release to the legacy ebtables installation https://review.opendev.org/c/openstack/neutron/+/897440 | 14:45 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2023.1: Fix the ``log.setup`` method call with "fix_eventlet=False" https://review.opendev.org/c/openstack/neutron/+/897462 | 14:46 |
| grishatsu | Is there an automatic CI that runs tests that includes my changes? or is it something that I need to trigger manually? how do I monitor this? | 14:46 |
| ralonsoh | grishatsu, we have zuul and you can see the results in the gerrit patch once executed | 14:47 |
| ralonsoh | or you can monitor then in https://zuul.opendev.org/t/openstack/status | 14:47 |
| lajoskatona | grishatsu: could you copy-paste the link to your patch ? | 14:47 |
| grishatsu | https://review.opendev.org/c/openstack/neutron/+/897436 | 14:48 |
| grishatsu | https://review.opendev.org/c/openstack/neutron/+/897435 | 14:48 |
| ralonsoh | --> https://zuul.opendev.org/t/openstack/status#897435 | 14:48 |
| grishatsu | oh sweet so same ID like the review | 14:49 |
| ralonsoh | grishatsu, btw, please propose these patches for master first, not for a stable branch | 14:49 |
| grishatsu | np | 14:50 |
| lajoskatona | grishatsu: thanks, yes please propose to master, btw the bug is private at the moment, I am not sure what is the process in this case | 14:54 |
| mnederlof | ralonsoh, is there anything i can show you to explain better what happens in regard to the VIP failover or are you testing it yourself as well? | 14:56 |
| opendevreview | Grisha Tsukerman proposed openstack/neutron master: Adds protection in agent worker for failed jobs https://review.opendev.org/c/openstack/neutron/+/897463 | 14:56 |
| opendevreview | Grisha Tsukerman proposed openstack/neutron master: Fix router port deletion issue in DHCP agent https://review.opendev.org/c/openstack/neutron/+/897464 | 14:56 |
| ralonsoh | mnederlof, yes, you can implement functional tests instead of unit tests. UT don't prove anything | 14:57 |
| mnederlof | hmm, ok that would be a first for me to implement, so.. any pointers? | 14:59 |
| grishatsu | lajoskatona, ralonsoh thanks i resubmitted and i'll follow up for updates | 15:02 |
| mnederlof | nvm, iĺl try to see if i can deduce something from the existing tests | 15:04 |
| opendevreview | Grisha Tsukerman proposed openstack/neutron master: Adds protection in agent worker for failed jobs https://review.opendev.org/c/openstack/neutron/+/897463 | 15:06 |
| opendevreview | Grisha Tsukerman proposed openstack/neutron master: Fix router port deletion issue in DHCP agent https://review.opendev.org/c/openstack/neutron/+/897464 | 15:06 |
| ralonsoh | mnederlof, you can check TestNBDbMonitor.test_virtual_port_host_update and create new tests | 15:11 |
| mnederlof | awesome thanks, will do :) (though it will be after the weekend..) | 15:11 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron stable/zed: [OVN] Fix rate and burst for stateless security groups https://review.opendev.org/c/openstack/neutron/+/894846 | 15:51 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron stable/zed: Use safer methods to get security groups on security group logging https://review.opendev.org/c/openstack/neutron/+/897470 | 15:51 |
| opendevreview | Grisha Tsukerman proposed openstack/neutron master: Adds protection in agent worker for failed jobs https://review.opendev.org/c/openstack/neutron/+/897463 | 16:02 |
| opendevreview | Grisha Tsukerman proposed openstack/neutron master: Fix router port deletion issue in DHCP agent https://review.opendev.org/c/openstack/neutron/+/897464 | 16:02 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Drop unused tables from the Neutron database https://review.opendev.org/c/openstack/neutron/+/897472 | 16:09 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Drop unused tables from the Neutron database https://review.opendev.org/c/openstack/neutron/+/897472 | 16:22 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Drop unused tables from the Neutron database https://review.opendev.org/c/openstack/neutron/+/897472 | 16:24 |
| opendevreview | Merged openstack/neutron master: Add "jammy" distribution release to the legacy ebtables installation https://review.opendev.org/c/openstack/neutron/+/897412 | 16:40 |
| opendevreview | Merged openstack/neutron master: [UT] Add chassis.external_ids dict in ``TestOvnSbIdlNotifyHandler`` https://review.opendev.org/c/openstack/neutron/+/897407 | 16:40 |
| mnaser | https://bugs.launchpad.net/neutron/+bug/1889388 i have the same exact issue here except the answer to rodolfo's question of "are they on the same host" is yea | 17:28 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: ovn: don't set (empty) requested-chassis= key for ports https://review.opendev.org/c/openstack/neutron/+/897489 | 20:18 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!