| opendevreview | Merged openstack/neutron master: agent/linux: remove eventlet usage from ip_lib https://review.opendev.org/c/openstack/neutron/+/940686 | 01:02 |
|---|---|---|
| opendevreview | Rico Lin proposed openstack/neutron-vpnaas master: Use write connection for get_vpn_services_on_host https://review.opendev.org/c/openstack/neutron-vpnaas/+/943359 | 01:07 |
| opendevreview | Merged openstack/neutron-vpnaas master: reno: Update master for unmaintained/zed https://review.opendev.org/c/openstack/neutron-vpnaas/+/917564 | 01:28 |
| opendevreview | Merged openstack/neutron-vpnaas master: Update jobs based on testing runtime for 2025.1 https://review.opendev.org/c/openstack/neutron-vpnaas/+/931504 | 02:28 |
| opendevreview | Rico Lin proposed openstack/neutron-vpnaas master: Use write connection for get_vpn_services_on_host https://review.opendev.org/c/openstack/neutron-vpnaas/+/943359 | 06:55 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: DNM - Test errors in ``neutron-functional-with-pyroute2-master`` https://review.opendev.org/c/openstack/neutron/+/942615 | 07:32 |
| vsaienko | Hello Neutron community, please add to you review queue small improvement in the installation guides related to OVN VLAN and DVR https://review.opendev.org/c/openstack/neutron/+/939961 | 07:37 |
| opendevreview | Vasyl Saienko proposed openstack/neutron master: Update OVN installation guide with tunings for VLAN + DVR https://review.opendev.org/c/openstack/neutron/+/939961 | 07:56 |
| vsaienko | ralonsoh: thanks for review, I've pushed an update https://review.opendev.org/c/openstack/neutron/+/939961 | 07:57 |
| vsaienko | ralonsoh: can you please have a look at my last comment https://bugs.launchpad.net/neutron/+bug/1995078 it seems we may close this bug and related SRIOV https://bugs.launchpad.net/neutron/+bug/1875852 and Baremetal bugs https://bugs.launchpad.net/ironic/+bug/2052468 as they should be fixed by https://bugs.launchpad.net/neutron/+bug/2052821 | 07:59 |
| opendevreview | Vasyl Saienko proposed openstack/neutron master: Update OVN installation guide with tunings for VLAN + DVR https://review.opendev.org/c/openstack/neutron/+/939961 | 08:09 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: ovs: remove the usage of eventlet in the OVS agent https://review.opendev.org/c/openstack/neutron/+/937765 | 08:22 |
| opendevreview | Lajos Katona proposed openstack/neutron master: WIP: [eventlet-removal] Remove the usage of eventlet in the SR-IOV agent https://review.opendev.org/c/openstack/neutron/+/942884 | 08:59 |
| lajoskatona | ralonsoh: Hi, for the sriov vs eventlat patch (https://review.opendev.org/c/openstack/neutron/+/942884 ) I was able to test it, booted VM, but I still has issues to have IP for the VMs (with master also, so have to debug my env more) | 09:00 |
| lajoskatona | ralonsoh: another thing which I have to check that stopping the service is really slow (like systemctl restart devstack@neutron-sriov-agt.service ) | 09:01 |
| *** ralonsoh_ is now known as ralonsoh | 09:03 | |
| ralonsoh | lajoskatona, did you check see the DHCP requests? | 09:04 |
| ralonsoh | lajoskatona, with the VM booted, can you use virsh to access it and execute the dhcpclient? | 09:04 |
| lajoskatona | ralonsoh: on my way to check what's happening | 09:10 |
| vsaienko | rolonsoh: please disregard my comment about https://bugs.launchpad.net/neutron/+bug/1995078, I've looked at the patches https://bugs.launchpad.net/neutron/+bug/2052821 they does not fix issue https://bugs.launchpad.net/neutron/+bug/1995078 | 09:12 |
| ralonsoh | vsaienko, because LP#2052821 is not related to the other bug | 09:15 |
| ralonsoh | these are you different issues | 09:15 |
| ralonsoh | actually LP#205282 is a new feature | 09:15 |
| vsaienko | yes, you are completely correct | 09:15 |
| vsaienko | we need to implement this https://bugs.launchpad.net/neutron/+bug/2092271 | 09:16 |
| ralonsoh | yes, this is a tech debt that should be solved | 09:17 |
| ralonsoh | once implemented, everything will use HA_Chassis_Group | 09:17 |
| ralonsoh | not like now, that we are using HCG and gateway_chassis | 09:17 |
| opendevreview | Elod Illes proposed openstack/networking-bagpipe unmaintained/yoga: [CI] Remove periodic stable jobs from unmaintained branch https://review.opendev.org/c/openstack/networking-bagpipe/+/943367 | 09:28 |
| opendevreview | Vasyl Saienko proposed openstack/neutron master: Introduce noop OVN Router scheduler https://review.opendev.org/c/openstack/neutron/+/943372 | 10:26 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [QoS] QoS rule check also considers the direction https://review.opendev.org/c/openstack/neutron/+/943373 | 10:31 |
| opendevreview | Dmitriy Chubinidze proposed openstack/neutron master: Update CIDR notation for loopback and multicast address according to actual RFC (https://datatracker.ietf.org/doc/html/rfc5735) https://review.opendev.org/c/openstack/neutron/+/942763 | 10:41 |
| opendevreview | Ivan Anfimov proposed openstack/neutron master: Update CIDR notation for loopback and multicast address according to actual RFC https://review.opendev.org/c/openstack/neutron/+/942763 | 10:41 |
| opendevreview | Ivan Anfimov proposed openstack/neutron master: Update CIDR notation for loopback and multicast address https://review.opendev.org/c/openstack/neutron/+/942763 | 10:43 |
| *** ykarel_ is now known as ykarel | 10:44 | |
| opendevreview | Maor Blaustein proposed x/whitebox-neutron-tempest-plugin master: Test verifies BZ#2214566/OSPRH-13533 doesn't regress https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/941930 | 10:51 |
| opendevreview | Merged openstack/neutron master: [eventlet-removal] Remove eventlet in the Metadata Agent https://review.opendev.org/c/openstack/neutron/+/942916 | 10:52 |
| opendevreview | Maor Blaustein proposed x/whitebox-neutron-tempest-plugin master: Test verifies BZ#2214566/OSPRH-13533 doesn't regress https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/941930 | 11:06 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Add router ``PUT`` external gateways actions policies https://review.opendev.org/c/openstack/neutron/+/941511 | 11:46 |
| opendevreview | Maor Blaustein proposed x/whitebox-neutron-tempest-plugin master: Test verifies BZ#2214566/OSPRH-13533 doesn't regress https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/941930 | 12:02 |
| opendevreview | Maor Blaustein proposed x/whitebox-neutron-tempest-plugin master: Test verifies BZ#2214566/OSPRH-13533 doesn't regress https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/941930 | 12:05 |
| opendevreview | Elod Illes proposed openstack/networking-bagpipe unmaintained/yoga: [CI] Remove periodic stable jobs from unmaintained branch https://review.opendev.org/c/openstack/networking-bagpipe/+/943367 | 12:08 |
| lajoskatona | ralonsoh: seems that the sriov is working with the patch, the only thing is that systemctl stop sriov-agt process is really low, I suppose the signal is not handled well, I have to check that | 12:10 |
| ralonsoh | lajoskatona, let me check one thing | 12:12 |
| ralonsoh | lajoskatona, we are not handling the signals in the sriov agent | 12:14 |
| opendevreview | Maor Blaustein proposed x/whitebox-neutron-tempest-plugin master: Test verifies BZ#2214566/OSPRH-13533 doesn't regress https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/941930 | 12:16 |
| ralonsoh | lajoskatona, we can use oslo.service.Service, that handles the signals (but is still using eventlet) | 12:17 |
| ralonsoh | or implement something there | 12:17 |
| ralonsoh | lajoskatona, in the OVS agent there is a basic signal handler ``_check_and_handle_signal`` | 12:19 |
| lajoskatona | I added a handler for sigterm: https://review.opendev.org/c/openstack/neutron/+/942884/2/neutron/plugins/ml2/drivers/mech_sriov/agent/sriov_nic_agent.py#471 in the patch | 12:22 |
| ralonsoh | lajoskatona, I think this thread is the culprit | 12:26 |
| ralonsoh | you added a wait time | 12:26 |
| ralonsoh | report_event.wait(max(0, report_interval - exec_time)) | 12:26 |
| ralonsoh | so it will wait the report interval and then it will end the thread | 12:27 |
| lajoskatona | let me check, as I remember I have it from the ovs-agent patches | 12:27 |
| ralonsoh | lajoskatona, nonono | 12:28 |
| ralonsoh | this is an event | 12:28 |
| ralonsoh | if the condition is set, the wait will end | 12:28 |
| ralonsoh | when you receive the sigterm signal, does the agent log it immediately? | 12:30 |
| ralonsoh | LOG.info("SIGTERM received, stopping SRIOV agent reporting.") | 12:30 |
| ralonsoh | another test, remove the thread join | 12:31 |
| ralonsoh | self.heartbeat['thread'].join() | 12:31 |
| ralonsoh | I'm going for lunch now | 12:33 |
| opendevreview | Vasyl Saienko proposed openstack/neutron master: Update OVN installation guide with tunings for VLAN + DVR https://review.opendev.org/c/openstack/neutron/+/939961 | 13:11 |
| opendevreview | Lajos Katona proposed openstack/neutron master: If OVS Manager creation failes retry to set values https://review.opendev.org/c/openstack/neutron/+/939117 | 13:30 |
| opendevreview | Vasyl Saienko proposed openstack/neutron master: Introduce noop OVN Router scheduler https://review.opendev.org/c/openstack/neutron/+/943372 | 13:48 |
| opendevreview | Vasyl Saienko proposed openstack/neutron master: WIP: Switch OVN Router scheduler to noop https://review.opendev.org/c/openstack/neutron/+/943392 | 13:48 |
| opendevreview | Vasyl Saienko proposed openstack/neutron master: Update OVN installation guide with tunings for VLAN + DVR https://review.opendev.org/c/openstack/neutron/+/939961 | 13:51 |
| lajoskatona | ralonsoh: yes the agent logs the sigterm immedietly, and Itested now without the join but the agent rpc loop is still running, Icheck few other things | 14:00 |
| sahid | lajoskatona: I think the issue is line 213 as in the pointer in entering here you will have to wait for the loop to finish | 14:09 |
| sahid | s/in/if | 14:09 |
| ralonsoh | no, this event wait should finish if set | 14:13 |
| sahid | ralonsoh: yes you are right | 14:16 |
| lajoskatona | sahid, ralonsoh: yes here in ovs-agent the same is present: https://review.opendev.org/c/openstack/neutron/+/941429/4/neutron/plugins/ml2/drivers/openvswitch/agent/ovs_neutron_agent.py#417 | 14:19 |
| lajoskatona | I am thinking to change the while true to something like while heartbeat['event'].is_set():..... | 14:19 |
| lajoskatona | that seems to be working, I push a new patch with that | 14:28 |
| ralonsoh | nice! | 14:34 |
| sahid | lajoskatona: cool I will have problably to align mine with your patch, right? | 14:51 |
| otherwiseguy | haleyb: ralonsoh: anyone else seeing ERROR neutron.plugins.ml2.ovo_rpc RuntimeError: cannot notify on un-acquired lock when doing operations in neutron/devstack/uwsgi? | 14:53 |
| otherwiseguy | It was a failed install that made it to neutron trying to create a router and I was just trying to figure out why things died. | 14:54 |
| ralonsoh | otherwiseguy, that happened only when using eventlet uwsgi | 14:54 |
| ralonsoh | otherwiseguy, are you using the latest version? | 14:54 |
| ralonsoh | master version | 14:54 |
| otherwiseguy | yeah | 14:55 |
| otherwiseguy | latest commit was Mar 3 | 14:55 |
| otherwiseguy | Mar 4 | 14:55 |
| ralonsoh | can you paste the rpc_workers, api_workers, rpc_state_report_workers config values? | 14:55 |
| otherwiseguy | ralonsoh: sure, one sec | 14:56 |
| otherwiseguy | ralonsoh: https://paste.opendev.org/show/b6ieoDNHC4cK897mqOKk/ | 15:00 |
| otherwiseguy | (processes = 2) | 15:00 |
| ralonsoh | otherwiseguy, I don't understand | 15:01 |
| ralonsoh | can you paste the full trace? | 15:02 |
| ralonsoh | if rpc_workers is None or rpc_workers >= 1: | 15:03 |
| ralonsoh | the OVOServerRpcInterface class will run only if you have rpc_workers | 15:03 |
| otherwiseguy | ralonsoh: https://paste.opendev.org/show/bFxE0tZxMvLnCXQZsuKA/ | 15:03 |
| ralonsoh | hold on | 15:04 |
| ralonsoh | the default is not zero, is the min | 15:04 |
| ralonsoh | so you are calling it | 15:04 |
| ralonsoh | you should explicitly set it to 0 | 15:04 |
| ralonsoh | anyway, in case of using sriov + ovn, we should need this | 15:05 |
| ralonsoh | otherwiseguy, please open a LP bug, this is something that must be investigated | 15:05 |
| haleyb | i don't remember seeing that failure before | 15:08 |
| otherwiseguy | ralonsoh: ok, will do. pretty standard/basic devstack values. https://paste.opendev.org/show/bt3Kt6OBqxvcSvluOHjD/ Thanks for the help. | 15:08 |
| ralonsoh | otherwiseguy, qq | 15:09 |
| ralonsoh | when that happens? | 15:09 |
| ralonsoh | ok, creating a router | 15:09 |
| opendevreview | Elod Illes proposed openstack/networking-bagpipe unmaintained/yoga: [CI] Remove periodic stable jobs from unmaintained branch https://review.opendev.org/c/openstack/networking-bagpipe/+/943367 | 15:11 |
| opendevreview | Lajos Katona proposed openstack/neutron master: WIP: [eventlet-removal] Remove the usage of eventlet in the SR-IOV agent https://review.opendev.org/c/openstack/neutron/+/942884 | 15:11 |
| ralonsoh | otherwiseguy, last question, to be sure: do you have https://review.opendev.org/c/openstack/neutron-lib/+/930716? | 15:12 |
| ralonsoh | check the n-lib version | 15:12 |
| otherwiseguy | when doing anything I tried. creating networks, etc. | 15:12 |
| * otherwiseguy looks | 15:13 | |
| lajoskatona | sahid: not sure, in ovs-agent there is the _check_and_handle_signal method that probably covers it, but please check | 15:17 |
| opendevreview | Vasyl Saienko proposed openstack/ovn-octavia-provider master: Prepare to handle ha_chassis_group for LRP https://review.opendev.org/c/openstack/ovn-octavia-provider/+/943243 | 15:18 |
| otherwiseguy | ralonsoh: 3.18.2 in the data/venv, which the tag seems to contain the above patch. | 15:19 |
| ralonsoh | so the bug is legit, I'll check it tomorrow morning | 15:19 |
| ralonsoh | folks, we have a method to check if something is true or false, including strings "true" "False", etc | 15:20 |
| ralonsoh | do you remember the name?? | 15:20 |
| ralonsoh | bool_from_string | 15:24 |
| vsaienko_ | https://github.com/openstack/oslo.utils/blob/master/oslo_utils/strutils.py#L142 | 15:26 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: WIP == [OVN] Make the LSP up/down events more robust https://review.opendev.org/c/openstack/neutron/+/943404 | 15:49 |
| *** gthiemon1e is now known as gthiemonge | 15:59 | |
| opendevreview | Elod Illes proposed openstack/networking-bagpipe unmaintained/yoga: [CI] Remove periodic stable jobs from unmaintained branch https://review.opendev.org/c/openstack/networking-bagpipe/+/943367 | 15:59 |
| opendevreview | Merged openstack/networking-bagpipe master: Drop remaining code related to linux bridge driver https://review.opendev.org/c/openstack/networking-bagpipe/+/943126 | 16:37 |
| opendevreview | Maor Blaustein proposed x/whitebox-neutron-tempest-plugin master: Timeout before getting logs https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/943410 | 16:53 |
| opendevreview | Maor Blaustein proposed x/whitebox-neutron-tempest-plugin master: Fetch only SGL log entries, drop count assertion https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/943411 | 16:53 |
| opendevreview | Merged openstack/neutron master: Add router ``PUT`` external gateways actions policies https://review.opendev.org/c/openstack/neutron/+/941511 | 16:56 |
| opendevreview | Alexey Stupnikov proposed openstack/neutron stable/2024.2: [ovn][trivial] Add 'empty_string_filtering' extension to OVN https://review.opendev.org/c/openstack/neutron/+/943417 | 17:25 |
| opendevreview | Alexey Stupnikov proposed openstack/neutron stable/2024.1: [ovn][trivial] Add 'empty_string_filtering' extension to OVN https://review.opendev.org/c/openstack/neutron/+/943418 | 17:27 |
| opendevreview | Alexey Stupnikov proposed openstack/neutron stable/2023.2: [ovn][trivial] Add 'empty_string_filtering' extension to OVN https://review.opendev.org/c/openstack/neutron/+/943419 | 17:27 |
| opendevreview | Merged openstack/neutron-vpnaas master: Use write connection for get_vpn_services_on_host https://review.opendev.org/c/openstack/neutron-vpnaas/+/943359 | 17:50 |
| opendevreview | Maor Blaustein proposed x/whitebox-neutron-tempest-plugin master: Fetch only SGL log entries, drop count assertion https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/943411 | 21:03 |
| opendevreview | Maor Blaustein proposed x/whitebox-neutron-tempest-plugin master: Fetch only SGL log entries, drop count assertion https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/943411 | 21:05 |
| opendevreview | Maor Blaustein proposed x/whitebox-neutron-tempest-plugin master: Fetch only SGL log entries, drop count assertion https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/943411 | 21:13 |
| opendevreview | Brian Haley proposed openstack/neutron master: DNM: Remove override of interface_driver in test_ip_lib https://review.opendev.org/c/openstack/neutron/+/943495 | 21:41 |
| opendevreview | Miro Tomaska proposed openstack/neutron master: [WIP] Just prototype for making sure that neutron:cidrs and device id is set. https://review.opendev.org/c/openstack/neutron/+/943501 | 22:06 |
| opendevreview | Merged openstack/neutron master: Fullstack case for metadata path https://review.opendev.org/c/openstack/neutron/+/888098 | 23:19 |
| atmark | Hello, is it possible to run an older version of the L3 agent, such as Neutron Server on Yoga, while the L3 agent is running on Xena? I'm encountering a bug and I'm unsure if it's related to the Yoga version or not | 23:25 |
| opendevreview | Merged openstack/networking-bagpipe master: Drop of_interface option https://review.opendev.org/c/openstack/networking-bagpipe/+/943128 | 23:51 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!