| *** bbezak is now known as Guest22493 | 01:19 | |
| opendevreview | Takashi Kajinami proposed openstack/neutron-fwaas-dashboard master: Use systemctl command to manage service https://review.opendev.org/c/openstack/neutron-fwaas-dashboard/+/955642 | 02:57 |
|---|---|---|
| opendevreview | Takashi Kajinami proposed openstack/neutron-vpnaas-dashboard master: Use systemctl command to manage service https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/955643 | 02:57 |
| opendevreview | Takashi Kajinami proposed openstack/neutron-fwaas-dashboard master: Use yaml format policy file https://review.opendev.org/c/openstack/neutron-fwaas-dashboard/+/955650 | 04:46 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [eventlet-removal] Don't use eventlet in the unit tests https://review.opendev.org/c/openstack/neutron/+/952258 | 07:50 |
| *** ykarel_ is now known as ykarel | 07:52 | |
| *** Guest22493 is now known as bbezak | 08:01 | |
| opendevreview | Lajos Katona proposed openstack/neutron stable/2025.1: Delete tunnel endpoints if endpoint agent is deleted https://review.opendev.org/c/openstack/neutron/+/955668 | 09:13 |
| opendevreview | Lajos Katona proposed openstack/neutron stable/2025.1: Delete tunnel endpoints if endpoint agent is deleted https://review.opendev.org/c/openstack/neutron/+/955668 | 09:15 |
| slaweq | lajoskatona ykarel hi, can you check https://review.opendev.org/c/openstack/neutron/+/954570 and https://review.opendev.org/c/openstack/neutron/+/955488/ when You will have time? | 10:16 |
| slaweq | ralonsoh you can also take a look at https://review.opendev.org/c/openstack/neutron/+/955488/ when you will have a minute | 10:16 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Add default value for the "context_with_global_access" policy https://review.opendev.org/c/openstack/neutron/+/954140 | 10:16 |
| slaweq | thx a lot for help with those | 10:16 |
| ykarel | slaweq, ack | 10:23 |
| ralonsoh | slaweq, sure | 10:24 |
| slaweq | and also https://review.opendev.org/c/openstack/neutron/+/954140 should be now ready once we will use neutron-lib 3.21.0 in neutron | 10:25 |
| slaweq | it is now in the check queue again but I hope it should be fine now | 10:25 |
| ralonsoh | sure, let me check this one too | 10:26 |
| slaweq | thank you | 10:26 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [eventlet-removal] Don't use eventlet in the unit tests https://review.opendev.org/c/openstack/neutron/+/952258 | 10:31 |
| opendevreview | Dr. Jens Harbott proposed openstack/neutron stable/2024.2: [DNM] Check designate issue with oslo.utils uc revert https://review.opendev.org/c/openstack/neutron/+/951494 | 10:44 |
| opendevreview | Stephen Finucane proposed openstack/neutron stable/2024.2: [DNM] Check designate issue with scripts fix https://review.opendev.org/c/openstack/neutron/+/951494 | 10:45 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [eventlet-removal] Don't use eventlet in the unit tests https://review.opendev.org/c/openstack/neutron/+/952258 | 11:13 |
| opendevreview | Stephen Finucane proposed openstack/neutron stable/2024.2: [DNM] Check designate issue with scripts fix https://review.opendev.org/c/openstack/neutron/+/951494 | 11:17 |
| opendevreview | Lajos Katona proposed openstack/neutron-fwaas master: Add N535 hacking check to ban eventlet imports https://review.opendev.org/c/openstack/neutron-fwaas/+/955689 | 11:56 |
| lajoskatona | ralonsoh: Hi, regarding the eventlet check (https://bugs.launchpad.net/neutron/+bug/2117373 ) luckily we already have a check for it in neutron-ib: https://opendev.org/openstack/neutron-lib/src/branch/master/neutron_lib/hacking/checks.py#L164 | 12:20 |
| lajoskatona | ralonsoh: so we can use that, I pushed a patch for fwaas (it fails of course :P) | 12:21 |
| lajoskatona | https://review.opendev.org/c/openstack/neutron-fwaas/+/955689 | 12:23 |
| opendevreview | Lajos Katona proposed openstack/neutron-fwaas master: Add N535 hacking check to ban eventlet imports https://review.opendev.org/c/openstack/neutron-fwaas/+/955689 | 12:26 |
| opendevreview | Merged openstack/neutron master: [ML2/OVN] Validate allowed address pairs and distributed ports https://review.opendev.org/c/openstack/neutron/+/954570 | 12:31 |
| opendevreview | Lajos Katona proposed openstack/neutron-vpnaas master: Add N535 hacking check to ban eventlet imports https://review.opendev.org/c/openstack/neutron-vpnaas/+/955693 | 12:32 |
| ralonsoh | lajoskatona, thanks! | 12:37 |
| ralonsoh | lajoskatona, btw, I'm going to change the LP name to N535 | 12:40 |
| ralonsoh | I added as example your patches | 12:41 |
| ralonsoh | thanks!! | 12:41 |
| ralonsoh | slaweq, https://review.opendev.org/c/openstack/neutron/+/954140 | 12:56 |
| ralonsoh | minor nits, apart from this, the patch is ready. Thanks for the documentation and testing | 12:56 |
| slaweq | thx | 12:59 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [eventlet-removal] Don't use eventlet in the unit tests https://review.opendev.org/c/openstack/neutron/+/952258 | 13:15 |
| cardoe | If you folks have time for https://review.opendev.org/c/openstack/neutron/+/955086 the queue, I'll work on the backports once the main one lands. | 13:50 |
| ralonsoh | slaweq, ^ if you have 2 mins | 13:55 |
| slaweq | approved | 14:03 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Add default value for the "context_with_global_access" policy https://review.opendev.org/c/openstack/neutron/+/954140 | 14:40 |
| opendevreview | Merged openstack/neutron master: [OVN]: fix ovn-db-sync to include gateway chassis https://review.opendev.org/c/openstack/neutron/+/955086 | 16:23 |
| opendevreview | Merged openstack/neutron master: Use ALLOWED_ADDRESS_PAIR resources constant from neutron_lib https://review.opendev.org/c/openstack/neutron/+/955488 | 17:58 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!