| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Catch when the process does not exist when killing it https://review.opendev.org/c/openstack/neutron/+/935046 | 07:06 |
|---|---|---|
| ralonsoh | ykarel, I was looking LP 2092567 and I saw https://github.com/svinota/pyroute2/issues/1225 | 07:38 |
| ralonsoh | are you on this? | 07:38 |
| ralonsoh | is the problem the table filter only? | 07:39 |
| ykarel | ralonsoh, just reported that | 07:39 |
| ykarel | ralonsoh, i see same with familty | 07:39 |
| ralonsoh | ok... so not only table | 07:39 |
| ralonsoh | let's wait for the pyroute2 fix then | 07:39 |
| ykarel | looks like pyroute2 going major restructuring | 07:40 |
| ralonsoh | if not, we can manually filter the returned list in the routes method | 07:40 |
| ralonsoh | yeah, I saw it, very big changes | 07:40 |
| ykarel | will check other issues related to namespace and report seperate issue for that | 07:41 |
| ralonsoh | thanks! | 07:41 |
| ykarel | like couple of tests failing on checking fip namespace self.assertFalse(self._namespace_exists(fip_ns)) | 07:42 |
| ykarel | k not just fip namespace | 07:43 |
| ykarel | same with router or snat namespaces, likely specific to remove netns | 07:43 |
| ralonsoh | ykarel, we reimplemented this method | 07:44 |
| ralonsoh | ah ah ah no, we deleted that re-implementation | 07:44 |
| ralonsoh | ykarel, is this failing with py3.12? | 07:47 |
| ykarel | yes noble py 3.12 | 07:47 |
| ralonsoh | we can try using | 07:47 |
| ralonsoh | ctypes.CDLL(ctypes.util.find_library('c'), use_errno=True) | 07:47 |
| ralonsoh | instead of | 07:47 |
| ralonsoh | ctypes.PyDLL(ctypes_util.find_library('c'), use_errno=True) | 07:47 |
| ralonsoh | only of the netns deletion | 07:47 |
| ykarel | ok no, i can see it in jammy too | 07:48 |
| ralonsoh | ok then | 07:48 |
| ralonsoh | in any case, we can try removing libc=priv_linux.get_cdll() in remove_netns | 07:49 |
| ykarel | atleast the test still fails even after removing ^ locally | 07:50 |
| sahid | ralonsoh: o/ | 08:51 |
| sahid | i'm currently add support of native threads for os-ken | 08:52 |
| sahid | https://opendev.org/openstack/os-ken/src/branch/master/os_ken/lib/hub.py | 08:52 |
| sahid | does that make sense in your pov? | 08:52 |
| ralonsoh | sahid, sorry, what does it mean this question? We need to migrate this project from eventlet, yes | 08:54 |
| ralonsoh | it makes sense because we need to remove eventlet | 08:54 |
| ralonsoh | what is the problem that you are facing? | 08:55 |
| sahid | ok, i don't know you may have discussed other solution, like perhaps using an other lib or I don't know, since is quite a big work I wanted to ensure I'm not doing something that will be not needed | 08:56 |
| ralonsoh | sahid, the only alternative to os-ken we ever had was using the ofctl commands directly | 08:58 |
| ralonsoh | that interface was removed some releases ago | 08:58 |
| ralonsoh | if the migration of os-ken to kernel threads is not possible, we can thing about restoring it | 08:58 |
| ralonsoh | but that also takes some time | 08:58 |
| sahid | ack thank you ralonsoh it was just to double check with you, let's try with adding the support of native thread to os-ken so | 09:00 |
| ralonsoh | sahid, in any case, I'll bring this topic to the next Neutron meeting | 09:01 |
| ralonsoh | next week | 09:01 |
| ralonsoh | if we can't migrate os-ken, we should think about bringing back the ofctl interface | 09:02 |
| sahid | ralonsoh: ack | 09:16 |
| sahid | i will try to push incrmeental changes soon as i can so we can stay in-sync | 09:17 |
| ykarel | ralonsoh, fyi reported https://github.com/svinota/pyroute2/issues/1226 for the namespace issue | 13:56 |
| ralonsoh | ykarel, thanks! | 14:19 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Synchronize the network segement range initialization https://review.opendev.org/c/openstack/neutron/+/938319 | 14:26 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Synchronize the network segement range initialization https://review.opendev.org/c/openstack/neutron/+/938319 | 15:24 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Synchronize the network segement range initialization https://review.opendev.org/c/openstack/neutron/+/938319 | 15:24 |
| opendevreview | Merged x/whitebox-neutron-tempest-plugin master: Ignore acl_log lines printed before a test SG logging starts https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/938178 | 16:24 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/os-ken master: hub: prepare support for native python libraries https://review.opendev.org/c/openstack/os-ken/+/938326 | 17:11 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/os-ken master: hub: refactor move Event and implement native Timeout https://review.opendev.org/c/openstack/os-ken/+/938327 | 17:11 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!