| daidv | haleyb, Hi, I have done the dns forwarder extension as spec we merged in previous cycle. Could you please to take a look on it in your free time, thank you alot in advanced. https://review.opendev.org/c/openstack/neutron/+/951390 | 04:22 |
|---|---|---|
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: DNM - Testing patch Neutron py310, py312 and py313 https://review.opendev.org/c/openstack/neutron/+/960876 | 07:12 |
| ralonsoh | slaweq, ykarel, lajoskatona hi folks | 08:22 |
| ralonsoh | So far, the testing patch https://review.opendev.org/c/openstack/neutron/+/960876 is passing | 08:22 |
| ralonsoh | please check the 5 patches under this one ^^^ | 08:22 |
| lajoskatona | ralonsoh: checking | 08:26 |
| ralonsoh | frickler, hello! just a qq: I'm trying to fix the issue with the eventlet removal from Neutron | 08:30 |
| ralonsoh | so, if possible now, I'm trying to remove any interaction with eventlet in the n-d-r code | 08:30 |
| ralonsoh | I've never tested this plugin | 08:30 |
| ralonsoh | I think, checking the local.conf from the tempest CI job, that we need only this | 08:31 |
| ralonsoh | https://zuul.opendev.org/t/openstack/build/9d28e54002674421ab8d38d59bbf4414/log/controller/logs/local_conf.txt#98 | 08:31 |
| ralonsoh | something else is needed? | 08:31 |
| ralonsoh | I would like to spawn the DR agent, that I think is not receiving any message | 08:31 |
| frickler | ralonsoh: at first glance, the pep8 error in https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/956747/5/neutron_dynamic_routing/services/bgp/agent/bgp_dragent.py seems relevant | 08:47 |
| ralonsoh | frickler, yes, a lot, I saw it | 08:48 |
| opendevreview | Merged openstack/os-ken master: hub: use native as default implementation https://review.opendev.org/c/openstack/os-ken/+/938337 | 08:53 |
| ralonsoh | haleyb, ^^ I think that deserves a new release | 09:08 |
| ralonsoh | I'll propose a patch in releases project | 09:08 |
| frickler | ralonsoh: in general I think n-d-r should just work, at least I've never needed to look into such low level details much. also thanks for working on this, I wasn't even aware that some eventlet removal is still needed for it | 09:14 |
| ralonsoh | frickler, yes, there are very few interaction (none in the running code) with eventlet | 09:14 |
| ralonsoh | If I make the tempest jobs run, the rest will be easier | 09:15 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [FT] Make ``test_find_child_pids`` more resilient https://review.opendev.org/c/openstack/neutron/+/966019 | 09:31 |
| opendevreview | Renjing Xiao proposed x/whitebox-neutron-tempest-plugin master: Remove some TODO comments https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/966148 | 09:40 |
| opendevreview | Brian Haley proposed openstack/neutron master: Remove mocking of eventlet.spawn in unit tests https://review.opendev.org/c/openstack/neutron/+/966149 | 09:52 |
| opendevreview | Merged openstack/neutron unmaintained/2024.1: [unmaintained-only] Periodic jobs will be weekly only https://review.opendev.org/c/openstack/neutron/+/965931 | 10:28 |
| opendevreview | Merged openstack/ovsdbapp master: Update to pylint 3.3.0 https://review.opendev.org/c/openstack/ovsdbapp/+/966059 | 11:31 |
| opendevreview | Merged openstack/neutron master: [FT] Initialize the segment ranges sequentially https://review.opendev.org/c/openstack/neutron/+/965933 | 11:39 |
| opendevreview | Jimin Shin proposed openstack/neutron-lib master: Add information to network ip availabilities https://review.opendev.org/c/openstack/neutron-lib/+/965087 | 12:12 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-dynamic-routing master: [eventlet-removal] Removal of eventlet usage in this repository https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/956747 | 12:20 |
| opendevreview | Merged openstack/neutron master: [FT] Fix ``test_ovsdb_monitor`` tests https://review.opendev.org/c/openstack/neutron/+/966011 | 12:26 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Add a post fork initialization method in OVN L3 https://review.opendev.org/c/openstack/neutron/+/964213 | 12:37 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] The external networks GW chassis must the same as the GW LRP https://review.opendev.org/c/openstack/neutron/+/962155 | 12:37 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Sync the LRP Gateway_Chassis with the network HCG https://review.opendev.org/c/openstack/neutron/+/964381 | 12:37 |
| opendevreview | Merged openstack/neutron master: Remove mocking of eventlet.spawn in unit tests https://review.opendev.org/c/openstack/neutron/+/966149 | 12:53 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [FT] Make ``test_find_child_pids`` more resilient https://review.opendev.org/c/openstack/neutron/+/966019 | 13:48 |
| opendevreview | Brian Haley proposed openstack/neutron master: Start running pylint on the functional test tree https://review.opendev.org/c/openstack/neutron/+/966182 | 14:00 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-dynamic-routing master: [eventlet-removal] Removal of eventlet usage in this repository https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/956747 | 14:02 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-vpnaas master: WIP == Fix tests https://review.opendev.org/c/openstack/neutron-vpnaas/+/966090 | 14:04 |
| ralonsoh | haleyb, I'm going to release a beta version of Neutron, that will include the UT/FT test changes | 14:44 |
| ralonsoh | that would be consumed by the stadium projects too | 14:44 |
| ralonsoh | is that ok? | 14:44 |
| ralonsoh | I'll wait for the in progress changes to stabilize the Neutron CI | 14:45 |
| lutimura | hey! i have a question about https://bugs.launchpad.net/neutron/+bug/2095590: for older releases of openstack, i.e. for those that still rely on eventlet, is there a workaround for it? or is replacing eventlet the only way here? | 14:47 |
| ralonsoh | lajoskatona, hi, last request for today: https://review.opendev.org/c/openstack/neutron/+/965783 | 14:48 |
| ralonsoh | thanks in advance! | 14:48 |
| ralonsoh | lutimura, this bug is not relevant for older releases, using the eventlet server | 14:49 |
| ralonsoh | that issue happened during the eventlet removal and the WSGI implementation | 14:49 |
| lajoskatona | ralonsoh: Hi, to be in balance please check these if you have some free time: https://review.opendev.org/q/topic:%22pyproject_toml%22 :-) | 15:03 |
| ralonsoh | lajoskatona, for sure! | 15:04 |
| opendevreview | Lajos Katona proposed openstack/neutron master: WIP: [eventlet]: Handle stop DHCP agent https://review.opendev.org/c/openstack/neutron/+/963386 | 15:08 |
| lajoskatona | ralonsoh: I pushed an update for the dhcp agent vs eventlet bug (https://launchpad.net/bugs/2126790 ): https://review.opendev.org/c/openstack/neutron/+/963386 | 15:38 |
| lajoskatona | ralonsoh: I added to the commit msg my last progress (thanks rubasov for the pairdebugging :-)), and now the dhcp agent master process is hanging only, in the final version I suppose some of the logic to handle thread joinings and so on will be unnecessary, but now I can't see what is the minimum that is needed :-( | 15:40 |
| lajoskatona | ralonsoh: there is one thing for example to close a hanging rootwrap process, which is weird why just hang there in the namespace I suppose.... | 15:41 |
| ralonsoh | lajoskatona, so the `self._process_monitor` we use to monitor dnsmasq (I'm correct?) is not stopping | 15:53 |
| ralonsoh | let me check this afternoon and tomorrow morning this, I'll ping you back | 15:53 |
| ralonsoh | lajoskatona, I think (of course I need confirmation) that we never call ProcessMonitor.stop | 15:55 |
| ralonsoh | in the DHCP agent we unregister the process to monitor | 15:55 |
| ralonsoh | but the thread monitoring the processes is still running | 15:56 |
| lajoskatona | ralonsoh: yes it monitors dnsmasq and haproxy, and by systemsct status printout the dnsmasq and haproxy processes are killed in this version of the patch | 15:56 |
| ralonsoh | let me try adding this `stop` call somewhere in the DHCP agent code | 15:58 |
| lajoskatona | ralonsoh: I added now the _process_monitor.stop, so some more magic is necessary, or perhaps just he order of things is not correct now.... | 15:58 |
| ralonsoh | lajoskatona, where did you add it? | 15:59 |
| ralonsoh | I'm checking the `DhcpAgentWithStateReport` and the child `DhcpAgent(manager.Manager)` class | 16:00 |
| ralonsoh | they don't implement the manager.Manager.stop method | 16:00 |
| ralonsoh | this could be the best place | 16:00 |
| ralonsoh | lajoskatona, I need to go now, I'll continue later today | 16:06 |
| opendevreview | Renjing Xiao proposed x/whitebox-neutron-tempest-plugin master: Remove some TODO comments https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/966148 | 16:33 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: bgp: Introduce Main router and chassis router commands https://review.opendev.org/c/openstack/neutron/+/959890 | 16:46 |
| opendevreview | Merged openstack/neutron master: [FT] Use an active wait in ``test__cleanup_previous_tags`` https://review.opendev.org/c/openstack/neutron/+/965783 | 17:01 |
| *** raineszm6 is now known as raineszm | 17:38 | |
| opendevreview | Merged openstack/neutron master: [UT] Mock the ``BatchNotifier`` call in the ML2 plugin tests https://review.opendev.org/c/openstack/neutron/+/966080 | 18:13 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: bgp: Introduce BGP chassis commands https://review.opendev.org/c/openstack/neutron/+/960422 | 19:04 |
| opendevreview | Mengyang Zhang proposed openstack/neutron-specs master: Create spec for RFE:project-specific qos controls https://review.opendev.org/c/openstack/neutron-specs/+/965946 | 21:39 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: bgp: OVN agent BGP extension https://review.opendev.org/c/openstack/neutron/+/960895 | 23:04 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: bgp: Detect BGP bridge creation https://review.opendev.org/c/openstack/neutron/+/964425 | 23:04 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!