opendevreview | Takashi Kajinami proposed openstack/tap-as-a-service master: Drop removed options from sample config https://review.opendev.org/c/openstack/tap-as-a-service/+/906014 | 02:09 |
---|---|---|
opendevreview | likui proposed openstack/os-ken master: Update python classifier in setup.cfg https://review.opendev.org/c/openstack/os-ken/+/906017 | 02:28 |
opendevreview | liuxie proposed openstack/neutron-tempest-plugin master: Add test job for firewall_v2 backend OVN https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/905421 | 02:59 |
opendevreview | liuxie proposed openstack/neutron-tempest-plugin master: Add test job for address_group api backend Ml2/OVN https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/906020 | 03:17 |
opendevreview | liuxie proposed openstack/neutron master: [OVN] Support address group for ovn driver https://review.opendev.org/c/openstack/neutron/+/851509 | 03:23 |
opendevreview | liuxie proposed openstack/neutron-tempest-plugin master: Add test job for address_group api backend Ml2/OVN https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/906020 | 08:43 |
ralonsoh | rubasov, https://review.opendev.org/c/openstack/neutron/+/905125 approved. Please confirm in your env that this patch works and fixes the issue | 09:11 |
ralonsoh | once done, we need to propose backports | 09:11 |
ralonsoh | and thanks a lot! | 09:11 |
rubasov | ralonsoh: will do that, thank you! | 09:12 |
lajoskatona | ralonsoh: regarding eventlet bump (https://review.opendev.org/c/openstack/neutron/+/905924 and all similar patches) I added this topic to drivers meeting agenda to have a broader discussion with the team (perhaps team meeting is also a good idea) | 09:23 |
ralonsoh | perfect | 09:26 |
opendevreview | Michel Nederlof proposed openstack/ovn-bgp-agent master: Fix FRR 9 mgmtd crashes when applying config https://review.opendev.org/c/openstack/ovn-bgp-agent/+/906031 | 09:26 |
ralonsoh | (btw, I won't attend tomorrow, I'll be in PTO, sorry) | 09:26 |
opendevreview | Michel Nederlof proposed openstack/ovn-bgp-agent master: Fix FRR 9 mgmtd crashes when applying config https://review.opendev.org/c/openstack/ovn-bgp-agent/+/906031 | 09:27 |
opendevreview | Merged openstack/ovn-bgp-agent master: Refactor ensure_routing_table_for_bridge https://review.opendev.org/c/openstack/ovn-bgp-agent/+/905795 | 09:37 |
opendevreview | Rodolfo Alonso proposed openstack/neutron master: DNM == WIP == Define the namespace for RPC metadata https://review.opendev.org/c/openstack/neutron/+/905309 | 09:42 |
lajoskatona | ralonsoh: when haleyb will apear I will ping him and we can discuss this next week also | 10:00 |
opendevreview | Michel Nederlof proposed openstack/ovn-bgp-agent master: Fix FRR 9 mgmtd crashes when applying config https://review.opendev.org/c/openstack/ovn-bgp-agent/+/906031 | 10:01 |
ralonsoh | thanks | 10:01 |
mnederlof | ralonsoh, with regard to frr9 issue :), i´ll try to see which version added this new flag, as the manpage of 7.2.1 does not mention it yet | 10:15 |
mnederlof | otherwise i´d have to check which version of frr we are running upon start to make it backward compatible :) | 10:16 |
ralonsoh | mnederlof, thanks, it will be needed to check the version | 10:17 |
ralonsoh | exactly | 10:17 |
opendevreview | Michel Nederlof proposed openstack/ovn-bgp-agent master: Fix FRR 9 mgmtd crashes when applying config https://review.opendev.org/c/openstack/ovn-bgp-agent/+/906031 | 10:26 |
mnederlof | it seems v7.3.0 already has it in the docker image though | 10:26 |
opendevreview | Rodolfo Alonso proposed openstack/neutron master: If method ``set_netns`` fails, restore previous device namespace https://review.opendev.org/c/openstack/neutron/+/905836 | 10:27 |
opendevreview | Rodolfo Alonso proposed openstack/neutron stable/yoga: DNM == Check error in TestSecurityGroupsSameNetwork.test_securitygroup(ovs-openflow) https://review.opendev.org/c/openstack/neutron/+/906034 | 10:32 |
mnederlof | ralonsoh, apparently it has been added quite some time ago (https://github.com/FRRouting/frr/commit/0846286b096be2cf437a9b95eb4c484156806df3) | 10:34 |
mnederlof | not sure how to check for backward compatibility, other than doing a vtysh --help and check output for option, or by creating config option | 10:35 |
mnederlof | one could argue it has always been included in frr, since the v2.0-rc1 tag also includes this -f option (https://github.com/FRRouting/frr/blob/frr-2.0-rc1/vtysh/vtysh_main.c) | 10:43 |
*** dmellado747 is now known as dmellado74 | 10:58 | |
opendevreview | Luis Tomas Bolivar proposed openstack/ovn-bgp-agent stable/2023.2: Refactor ensure_routing_table_for_bridge https://review.opendev.org/c/openstack/ovn-bgp-agent/+/905911 | 11:01 |
*** dmellado745 is now known as dmellado74 | 11:07 | |
opendevreview | Merged openstack/tap-as-a-service master: Bump hacking https://review.opendev.org/c/openstack/tap-as-a-service/+/905878 | 11:28 |
opendevreview | Merged openstack/ovn-bgp-agent stable/2023.2: Refactor ensure_routing_table_for_bridge https://review.opendev.org/c/openstack/ovn-bgp-agent/+/905911 | 11:52 |
opendevreview | Merged openstack/neutron stable/2023.1: Register Chassis_Private table in BaseOvnSbIdl https://review.opendev.org/c/openstack/neutron/+/905585 | 12:03 |
opendevreview | Merged openstack/neutron stable/zed: Register Chassis_Private table in BaseOvnSbIdl https://review.opendev.org/c/openstack/neutron/+/905586 | 12:03 |
opendevreview | Merged openstack/neutron stable/yoga: Register Chassis_Private table in BaseOvnSbIdl https://review.opendev.org/c/openstack/neutron/+/905587 | 12:03 |
*** priteau_ is now known as priteau | 12:14 | |
opendevreview | Merged openstack/neutron stable/wallaby: Register Chassis_Private table in BaseOvnSbIdl https://review.opendev.org/c/openstack/neutron/+/905589 | 12:47 |
opendevreview | Michel Nederlof proposed openstack/ovn-bgp-agent master: If the hostname is not configured, fallback to the system-id. https://review.opendev.org/c/openstack/ovn-bgp-agent/+/906043 | 13:01 |
opendevreview | Michel Nederlof proposed openstack/ovn-bgp-agent master: Fix startup if the hostname is not configured in OVS and use the system-id. https://review.opendev.org/c/openstack/ovn-bgp-agent/+/906043 | 13:03 |
opendevreview | liuxie proposed openstack/neutron-tempest-plugin master: Add test job for address_group api backend Ml2/OVN https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/906020 | 14:04 |
opendevreview | Harald Jensås proposed openstack/neutron stable/yoga: [OVN] DB sync host/physnet - filter on agent_type https://review.opendev.org/c/openstack/neutron/+/905913 | 14:20 |
haleyb | lajoskatona: i did appear a little while back, just in meetings | 16:21 |
haleyb | i'm guessing there was an eventlet issue after seeing the -W in the reviews | 16:21 |
haleyb | lajoskatona: oh, and if you're still around, can you see my comments in https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/894027 ? thanks | 16:23 |
lajoskatona | haleyb: Hi, this whole eventlet thing is an issue :-) | 16:51 |
lajoskatona | haleyb: in the mean time Harve answered on the fresh mail thread (https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/7PJZ4WYRS2B3PIZ5KYFL6V3JWYL4V5FQ/ ) | 16:52 |
lajoskatona | haleyb: as I see if not all projects bump lower eventlet limits to >3.4 than we should keep the current constraints to be in sync with everybody else, but I think it would be good to have a common opinion | 16:54 |
lajoskatona | haleyb: ralonsoh can't participate tomorrow, but as the merge of these patches are halted we have time I think | 16:54 |
lajoskatona | haleyb: I check the n-t-p patch, I totally forgot that since last time I see it :-) | 16:55 |
haleyb | lajoskatona: so regarding eventlet, there was a bug so we shouldn't merge, until a new release happens? or i guess we don't have to? i am a little confused | 16:57 |
lajoskatona | haleyb: the main issue with eventlet is that it is not py312 compatible (the new 0.34 solves that if I understand well), as py312 is not target for 2024.1 it is an issue for the coming release mostly | 17:03 |
lajoskatona | but eventlet lost its maintainers and python evolved another direction so the guys would like to change direction of Openstack as well and use something else like ayncio instead | 17:04 |
lajoskatona | haleyb: for that there is a (wip) resolution which is quite good to list all the issues with eventlet and a possible way to modernize openstack: https://review.opendev.org/c/openstack/governance/+/902585 | 17:05 |
lajoskatona | haleyb: the "only" problem is that in the current state of the community and Openstack it is enourmous work and hard to sync with all the projects | 17:06 |
haleyb | lajoskatona: ack, i'm watching that review. I was just trying to figure out why we shouldn't merge the requirements change (swift bug with it?), but i guess it should come automatically as UC moved. | 17:07 |
lajoskatona | haleyb: I lost this morning the track because I saw the same that the req patches should be merged, but perhaps TC has more details | 17:08 |
opendevreview | Fernando Royo proposed openstack/ovn-bgp-agent master: Fix OVN LB Delete events for NB driver https://review.opendev.org/c/openstack/ovn-bgp-agent/+/905783 | 17:15 |
opendevreview | Merged openstack/neutron stable/yoga: [OVN] DB sync host/physnet - filter on agent_type https://review.opendev.org/c/openstack/neutron/+/905913 | 17:54 |
opendevreview | Merged openstack/neutron stable/xena: Register Chassis_Private table in BaseOvnSbIdl https://review.opendev.org/c/openstack/neutron/+/905588 | 17:54 |
opendevreview | Brian Haley proposed openstack/neutron master: Fix keyword-arg-before-vararg warnings https://review.opendev.org/c/openstack/neutron/+/906006 | 20:16 |
haleyb | mlavalle: qq - in order to set the dns_name of a port, you need to have a dns domain defined, correct? i.e. not the default name. i always mess this up | 22:15 |
opendevreview | Merged openstack/neutron-lib stable/2023.1: Add api-ref about SG rule bulk create https://review.opendev.org/c/openstack/neutron-lib/+/905570 | 22:29 |
opendevreview | Brian Haley proposed openstack/neutron master: Remove string support in install_instructions https://review.opendev.org/c/openstack/neutron/+/905755 | 23:08 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!