| opendevreview | Merged openstack/neutron master: Revert "[DHCP] Lock the execution of ``_dhcp_ready_ports_loop``" https://review.opendev.org/c/openstack/neutron/+/924213 | 06:12 |
|---|---|---|
| opendevreview | Merged openstack/neutron-tempest-plugin master: Temporary switch lb/ovs/ovn scenario jobs to cirros-0.5.3 https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/924124 | 06:12 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [DHCP] Lock the execution of ``_dhcp_ready_ports_loop`` https://review.opendev.org/c/openstack/neutron/+/924300 | 07:11 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [DHCP] Lock the execution of ``_dhcp_ready_ports_loop`` https://review.opendev.org/c/openstack/neutron/+/924300 | 07:12 |
| opendevreview | yatin proposed openstack/neutron master: [DNM] Check linuxbridge jobs https://review.opendev.org/c/openstack/neutron/+/924305 | 08:17 |
| ralonsoh | slaweq, trivial patch (if you don't mind): https://review.opendev.org/c/openstack/neutron-lib/+/924041 | 08:26 |
| SvenKieske | Hi, https://review.opendev.org/c/openstack/neutron-vpnaas/+/765353/36/etc/neutron/rootwrap.d/vpnaas.filters#16 unconditionally sets ipv4 and ipv6 sysctls, which might cause issues. e.g. when a host has ipv6 support disabled | 09:17 |
| SvenKieske | question: would it be appropriate to write a bugreport (and patch) for that or do you consider hosts with ipv6 disabled as "unsupported usecase"? :) | 09:17 |
| SvenKieske | nvm, that's only the filter that allows to set this.. let me check the code more thorough where(if) this is actually called by the new agent. | 09:20 |
| SvenKieske | alright, the code handles it: https://opendev.org/openstack/neutron-vpnaas/src/commit/256464aea691f8b4957ba668a117963353f34e4c/neutron_vpnaas/services/vpn/device_drivers/ovn_ipsec.py#L194 | 09:23 |
| SvenKieske | nice :) | 09:24 |
| SvenKieske | now I'm wondering if it's correct though to set this inside the network namespace, unfortunately the kernel docs don't say if this sysctl is global or namespaced (in general afaik most net* sysctls are namespaced). | 09:32 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [WSGI] Move all OVN jobs to use WSGI API module https://review.opendev.org/c/openstack/neutron/+/924317 | 10:10 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [WSGI] Move all OVN jobs to use WSGI API module https://review.opendev.org/c/openstack/neutron/+/924317 | 10:14 |
| opendevreview | yatin proposed openstack/neutron master: [DNM] Check scenario jobs https://review.opendev.org/c/openstack/neutron/+/924305 | 10:19 |
| opendevreview | yatin proposed openstack/neutron master: [DNM] Check scenario jobs https://review.opendev.org/c/openstack/neutron/+/924305 | 10:29 |
| opendevreview | Merged openstack/neutron master: Limit the worker type in neutron-periodic-workers service https://review.opendev.org/c/openstack/neutron/+/923500 | 11:17 |
| opendevreview | Merged openstack/neutron master: [WSGI] Move all OVS jobs to use WSGI API module https://review.opendev.org/c/openstack/neutron/+/923711 | 11:45 |
| *** gthiemon1e is now known as gthiemonge | 11:46 | |
| opendevreview | Merged openstack/neutron-lib master: Missing required extension in 'subnet-external-network' https://review.opendev.org/c/openstack/neutron-lib/+/924041 | 11:55 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [FT] Make LR name random https://review.opendev.org/c/openstack/neutron/+/924334 | 13:43 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: DNM - TESTING PATCH: neutron-functional-with-uwsgi https://review.opendev.org/c/openstack/neutron/+/924335 | 13:45 |
| haleyb | SvenKieske: the l3-agent does set the ipv6 sysctl in a namespace | 14:43 |
| slaweq | ralonsoh sorry, I missed Your morning ping, but I already approved Your patch and it is merged :) | 14:46 |
| ralonsoh | slaweq, thanks! | 14:46 |
| SvenKieske | haleyb: I inferred that much from the code, thank you :) was just wondering if it's correct to set this inside the namespace, some sysctls are global, even if you set them inside a namespace. it's not documented for this specific one. | 15:12 |
| haleyb | SvenKieske: i hope it's correct, i know some of the conntrack ones are global | 15:13 |
| SvenKieske | haleyb: yes, it's unfortunate that this is not properly documented for every sysctl. I guess that's life. :) | 15:37 |
| opendevreview | Merged openstack/neutron master: Use parent's project id in the quota enforcement for some resources https://review.opendev.org/c/openstack/neutron/+/922716 | 17:44 |
| opendevreview | Merged openstack/neutron master: [FT] Make LR name random https://review.opendev.org/c/openstack/neutron/+/924334 | 19:44 |
| opendevreview | Merged openstack/neutron-dynamic-routing master: Add pyproject.toml to support pip 23.1 https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/916418 | 19:48 |
| opendevreview | Terry Wilson proposed openstack/neutron master: Actualy set global "removal limit" options https://review.opendev.org/c/openstack/neutron/+/924295 | 21:40 |
| opendevreview | Brian Haley proposed openstack/neutron master: Use convert_version_to_tuple() instead of pkg_resources https://review.opendev.org/c/openstack/neutron/+/924374 | 21:53 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!