Wednesday, 2024-11-20

opendevreviewBrian Haley proposed openstack/neutron-lib master: Support project_id and project_name in ContextBase class  https://review.opendev.org/c/openstack/neutron-lib/+/93573100:04
opendevreviewMerged openstack/neutron master: Bump os-ken version to 2.11.2  https://review.opendev.org/c/openstack/neutron/+/93536401:33
opendevreviewLiushy proposed openstack/neutron-fwaas master: [OVN] Fix the provider error in devstack settings  https://review.opendev.org/c/openstack/neutron-fwaas/+/93410303:35
opendevreviewRodolfo Alonso proposed openstack/neutron master: Make grenade jobs work with Neutron API WSGI module  https://review.opendev.org/c/openstack/neutron/+/93566806:37
opendevreviewRodolfo Alonso proposed openstack/neutron master: Make grenade jobs work with Neutron API WSGI module  https://review.opendev.org/c/openstack/neutron/+/93566806:40
ralonsohslaweq, hello! As you mentioned yesterday: https://review.opendev.org/c/openstack/neutron/+/93566806:41
ralonsohwe still have "skip-level" jobs working with eventlet server because we implemented/refactored the wsgi module during the last cycle06:42
ralonsohso https://review.opendev.org/c/openstack/devstack/+/932203 will need to wait until 2025.206:42
slaweqralonsoh thx for checking this08:04
*** tv1 is now known as Kvisle08:11
KvisleI am quite deep in troubleshooting an issue with neutron-vpnaas (the OVN variant) and I wonder if it's a bug or not.  The mechanism that is looking for VPN Agents that are down are never identifying the agents.  When troubleshooting the python-code itself, I have understood that it will not list any OVN-based agents when it calls the core plugin (get_agents()).  It lists08:27
Kvisleonly the three agents in the database, which are the BGP Dragents.  The OpenStack-installation is working fine otherwise, as far as I can tell -- this problem seems to be limited to vpnaas.  Is this a bug, or a configuration issue?  Am I missing some glue somewhere?08:27
opendevreviewRodolfo Alonso proposed openstack/neutron-tempest-plugin master: Change the default FRR docker image repository  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/93574308:53
ralonsohKvisle, when using ML2/OVN, there are two searchs: the first one will find the OVN related agents and the second one will search in the DB09:11
ralonsohif the VPN agents are not in the database, this is because they are not registered09:12
ralonsohI'm not sure how the VPN agents do this, please check the neutron-vpnaas documentation and the check that you have enabled the RPC workers in the Neutron API09:12
ralonsoh(that you should because the dr-agents are there and I guess they are alive)09:13
opendevreviewRodolfo Alonso proposed openstack/neutron master: Filter out the floating IPs when removing a shared RBAC  https://review.opendev.org/c/openstack/neutron/+/93527809:48
opendevreviewRodolfo Alonso proposed openstack/neutron master: Migrate from tenant_id to project_id in ``test_network.py``  https://review.opendev.org/c/openstack/neutron/+/93528009:48
ralonsohslaweq, hello (again). If you have less than 1 min: https://review.opendev.org/c/openstack/neutron-lib/+/93458709:53
ralonsohthanks!09:53
opendevreviewRodolfo Alonso proposed openstack/neutron master: [OVN] Implement a utility method to retrieve the Neutron name  https://review.opendev.org/c/openstack/neutron/+/93561910:01
opendevreviewTakashi Kajinami proposed openstack/os-vif master: DNM: Testing 932436  https://review.opendev.org/c/openstack/os-vif/+/93575510:08
slaweqralonsoh done10:12
ralonsohslaweq, thanks!10:12
opendevreviewRodolfo Alonso proposed openstack/neutron master: Catch when the process does not exist when killing it  https://review.opendev.org/c/openstack/neutron/+/93504610:13
ralonsohhaleyb, lajoskatona hello! Please check https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/935743. For now and until we have a better alternative, we can pull from quay.io10:15
opendevreviewMerged openstack/ovn-bgp-agent stable/2024.1: Remove FakeLogicalSwitchChassisEvent class  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93519210:42
opendevreviewMerged openstack/ovn-bgp-agent stable/2023.2: Remove FakeLogicalSwitchChassisEvent class  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93519311:08
opendevreviewMerged openstack/ovn-bgp-agent stable/2024.1: NB: Consolidate common code of events to the superclass  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93519411:08
opendevreviewMerged openstack/ovn-bgp-agent stable/2023.2: NB: Consolidate common code of events to the superclass  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93519511:08
opendevreviewMerged openstack/ovn-bgp-agent stable/2023.2: Remove _check_ip_associated method from Event  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93520911:08
opendevreviewMerged openstack/ovn-bgp-agent stable/2024.1: Remove _check_ip_associated method from Event  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93519611:08
s3rj1kHi all, I noticed that in https://review.opendev.org/c/openstack/neutron-specs/+/935724 `openstack-tox-docs` failed due to warnings (as errors), but I don't see anything related to CR change, how should I proceed?11:39
opendevreviewMerged openstack/neutron master: Drop reference to removed services  https://review.opendev.org/c/openstack/neutron/+/93547611:51
Kvisleralonsoh: the RPC workers are enabled, and I can see that they are running in the process list.  if I do "openstack network agent list", I see everything.  However, this function (https://github.com/openstack/neutron-vpnaas/blob/master/neutron_vpnaas/db/vpn/vpn_agentschedulers_db.py#L328) - even if I call it without filters just to log the output, only finds the agents in11:52
Kvislethe db.11:52
ralonsohKvisle, as it should be11:56
ralonsohthe OVN agents are stored locally in a singleton class and the other agents are stored in the DB11:57
ralonsohthis is why, when using ML2/OVN, we call both method (the singleton class and the get_agents)11:57
opendevreviewRodolfo Alonso proposed openstack/neutron-specs master: DNM - Testing openstack-tox-docs  https://review.opendev.org/c/openstack/neutron-specs/+/93577612:00
opendevreviewMerged openstack/ovn-bgp-agent stable/2023.2: evpn: Remove Chassis events  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93521012:05
opendevreviewMerged openstack/ovn-bgp-agent stable/2024.1: evpn: Remove Chassis events  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93519712:05
opendevreviewMerged openstack/ovn-bgp-agent stable/2023.2: Remove duplicated _get_network method  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93521112:19
opendevreviewMerged openstack/ovn-bgp-agent stable/2024.1: Remove duplicated _get_network method  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93519812:26
opendevreviewMerged openstack/ovn-bgp-agent stable/2024.1: Remove _get_router method from events  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93519912:26
opendevreviewMerged openstack/neutron-lib master: Remove default override for config options policy_file  https://review.opendev.org/c/openstack/neutron-lib/+/93458712:30
opendevreviewMerged openstack/ovn-bgp-agent stable/2023.2: Remove _get_router method from events  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93521212:33
opendevreviewMerged openstack/ovn-bgp-agent stable/2024.1: Move some LogicalSwitchChassisEvent methods out  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93520012:34
opendevreviewMerged openstack/ovn-bgp-agent stable/2023.2: Move some LogicalSwitchChassisEvent methods out  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93521312:34
opendevreviewEduardo Olivares proposed openstack/ovn-bgp-agent stable/2024.2: Introduce LSP address column parsing functions  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93577912:35
opendevreviewEduardo Olivares proposed openstack/ovn-bgp-agent stable/2024.2: functional: Introduce a base class for OVN func testing  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93578012:39
Kvisleralonsoh: is that logic hidden behind the plugin-interface? or should I be able to find the call to the singleton class in the neutron_vpnaas-repository somewhere?12:45
ralonsohKvisle, did you check the code? Do you know how the vpnaas agent is registered?12:48
ralonsohI'm checking it and it seems that "register_vpn_agent" adds a new key in the chassis.external_ids12:48
Kvisleralonsoh: it's only in ovn as far as I can tell.12:48
ralonsohthen there is an agent_monitor that has an event OVNVPNAgent12:49
ralonsohaccording to ChassisVPNAgentWriteEvent, this agent monitor uses the same OVN agent singletong12:49
ralonsohhttps://github.com/openstack/neutron-vpnaas/blob/master/neutron_vpnaas/services/vpn/ovn/agent_monitor.py#L7312:50
ralonsohKvisle, so try to check where this agent_monitor is being executed, it seems that is a service provided by this repository12:51
ralonsohso maybe you need to call it in the service_plugins12:52
ralonsohthis is the first time I'm checkign this code12:52
ralonsohs3rj1k, it seems that the tox job is failing: https://review.opendev.org/c/openstack/neutron-specs/+/935776?tab=change-view-tab-header-zuul-results-summary12:53
ralonsohI'll open a bug12:53
opendevreviewMerged openstack/ovn-bgp-agent stable/2023.2: Move methods out of LSP events  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93521412:53
Kvisleralonsoh: thank you - I think I have something to go on perhaps.12:54
s3rj1kralonsoh: thanks12:54
opendevreviewMerged openstack/ovn-bgp-agent stable/2024.1: Move methods out of LSP events  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93520112:54
opendevreviewMerged openstack/ovn-bgp-agent stable/2024.1: Move _get_ips_info out of LRP event  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93520212:54
opendevreviewMerged openstack/ovn-bgp-agent stable/2023.2: Move _get_ips_info out of LRP event  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93521512:54
opendevreviewMerged openstack/ovn-bgp-agent stable/2023.2: Introduce LSP address column parsing functions  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93521612:59
opendevreviewMerged openstack/ovn-bgp-agent stable/2024.1: Introduce LSP address column parsing functions  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93520312:59
opendevreviewMerged openstack/ovn-bgp-agent stable/2024.1: functional: Introduce a base class for OVN func testing  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93520412:59
opendevreviewMerged openstack/ovn-bgp-agent stable/2023.2: functional: Introduce a base class for OVN func testing  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93521712:59
opendevreviewMerged openstack/ovn-bgp-agent stable/2023.2: Fix failures pointed by the flake8 run by the pre-commit script  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93518813:07
opendevreviewMerged openstack/ovn-bgp-agent stable/2024.1: Fix failures pointed by the flake8 run by the pre-commit script  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93518713:07
opendevreviewEduardo Olivares proposed openstack/ovn-bgp-agent stable/2024.2: Expose FIP if external_mac was set  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93578513:20
opendevreviewEduardo Olivares proposed openstack/ovn-bgp-agent stable/2024.2: nb driver: Don't expose FIP if the external_mac is not set  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93578613:20
opendevreviewEduardo Olivares proposed openstack/ovn-bgp-agent stable/2024.1: Expose FIP if external_mac was set  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93579113:59
opendevreviewEduardo Olivares proposed openstack/ovn-bgp-agent stable/2023.2: Expose FIP if external_mac was set  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93579214:00
opendevreviewEduardo Olivares proposed openstack/ovn-bgp-agent stable/2024.1: nb driver: Don't expose FIP if the external_mac is not set  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93579314:00
opendevreviewEduardo Olivares proposed openstack/ovn-bgp-agent stable/2023.2: nb driver: Don't expose FIP if the external_mac is not set  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93579414:01
opendevreviewMerged openstack/neutron-tempest-plugin master: Change the default FRR docker image repository  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/93574314:06
opendevreviewRodolfo Alonso proposed openstack/neutron master: Catch when the process does not exist when killing it  https://review.opendev.org/c/openstack/neutron/+/93504614:14
opendevreviewDmitriy Rabotyagov proposed openstack/ovn-bgp-agent master: Ensure that ARP/NDP is enabled for vlan devices  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93580115:06
opendevreviewRodolfo Alonso proposed openstack/neutron master: [OVN] Add a creation wait event for the PG drop tests  https://review.opendev.org/c/openstack/neutron/+/93580215:06
opendevreviewRodolfo Alonso proposed openstack/neutron master: [OVN] Use the MySQL backend for the ``TestOvnNbSync`` tests  https://review.opendev.org/c/openstack/neutron/+/93580415:31
opendevreviewDmitriy Rabotyagov proposed openstack/ovn-bgp-agent master: Ensure that ARP/NDP is enabled for vlan devices  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93580115:40
opendevreviewDmitriy Rabotyagov proposed openstack/ovn-bgp-agent master: Ensure that ARP/NDP is enabled for vlan devices  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93580115:55
opendevreviewDmitriy Rabotyagov proposed openstack/ovn-bgp-agent master: Handle trimming of vlan interface namings  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/91065416:46
opendevreviewDmitriy Rabotyagov proposed openstack/ovn-bgp-agent master: Use neutron_lib constants a device name limiter  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93432716:46
opendevreviewDmitriy Rabotyagov proposed openstack/ovn-bgp-agent master: Ensure that ARP/NDP is enabled for vlan devices  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/93580116:46
opendevreviewMerged openstack/python-neutronclient master: vpnaas: show external_vx_ip for vpn service show  https://review.opendev.org/c/openstack/python-neutronclient/+/93421517:34
opendevreviewMerged openstack/neutron master: Enable no-else-return pylint check  https://review.opendev.org/c/openstack/neutron/+/93420517:42
opendevreviewMerged openstack/neutron master: tests: Remove skip_if_ovs_older_than decorator from a test case  https://review.opendev.org/c/openstack/neutron/+/93525017:42
opendevreviewMerged openstack/neutron master: [OVN] The Port_Group deletion implies the ACLs deletion  https://review.opendev.org/c/openstack/neutron/+/93387718:06
opendevreviewMerged openstack/neutron master: Add new RBAC policies only for "target_project"  https://review.opendev.org/c/openstack/neutron/+/93495418:33
opendevreviewMerged openstack/neutron master: Set a new removal release for ``set_network_type``  https://review.opendev.org/c/openstack/neutron/+/93527218:34
opendevreviewsean mooney proposed openstack/os-vif master: address test stablity under load  https://review.opendev.org/c/openstack/os-vif/+/93565321:22
opendevreviewTakashi Kajinami proposed openstack/os-vif master: Clean up Windows support  https://review.opendev.org/c/openstack/os-vif/+/93243621:23
opendevreviewBrian Haley proposed openstack/neutron-specs master: Fix docs job errors and warnings  https://review.opendev.org/c/openstack/neutron-specs/+/93583621:53
opendevreviewBrian Haley proposed openstack/neutron-lib master: Support project_id and project_name in ContextBase class  https://review.opendev.org/c/openstack/neutron-lib/+/93573123:21

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!