opendevreview | Dai Dang Van proposed openstack/neutron master: Add dns forwarder l2 extension https://review.opendev.org/c/openstack/neutron/+/951390 | 03:10 |
---|---|---|
opendevreview | Lajos Katona proposed openstack/neutron master: Delete tunnel endpoints if endpoint agent is deleted https://review.opendev.org/c/openstack/neutron/+/952988 | 08:07 |
lajoskatona | ralonsoh: Hi, regarding fullstack vs SDK see my patch: https://review.opendev.org/c/openstack/neutron/+/947851 | 09:00 |
lajoskatona | ralonsoh: and my chat with gtema: https://meetings.opendev.org/irclogs/%23openstack-sdks/%23openstack-sdks.2025-04-23.log.html | 09:00 |
ralonsoh | lajoskatona, thanks! I was writing the mail right now | 09:09 |
opendevreview | Rodolfo Alonso proposed openstack/networking-bagpipe master: Remove the fullstack job from the CI https://review.opendev.org/c/openstack/networking-bagpipe/+/953293 | 09:32 |
opendevreview | Rodolfo Alonso proposed openstack/neutron-fwaas master: Remove the fullstack job from the CI https://review.opendev.org/c/openstack/neutron-fwaas/+/953294 | 09:34 |
opendevreview | Rodolfo Alonso proposed openstack/neutron master: Remove the fullstack job from the CI https://review.opendev.org/c/openstack/neutron/+/953295 | 09:37 |
opendevreview | Lajos Katona proposed openstack/neutron master: [eventlet-removal] remove eventlet references from dhcp_agent https://review.opendev.org/c/openstack/neutron/+/950499 | 09:44 |
opendevreview | Rodolfo Alonso proposed openstack/neutron master: [eventlet-removal] Remove the usage of eventlet from the periodic worker https://review.opendev.org/c/openstack/neutron/+/952117 | 09:44 |
opendevreview | Rodolfo Alonso proposed openstack/neutron master: [eventlet-removal] Remove the eventlet WSGI Neutron server code https://review.opendev.org/c/openstack/neutron/+/952659 | 09:45 |
ralonsoh | bcafarel, hello! If you have less than 1 min: https://review.opendev.org/c/openstack/neutron/+/953064 | 09:50 |
ralonsoh | thanks! | 09:50 |
ralonsoh | this is a 2025.1 only patch | 09:51 |
opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Use stateless NAT rules for FIPs https://review.opendev.org/c/openstack/neutron/+/951511 | 10:38 |
opendevreview | Rodolfo Alonso proposed openstack/neutron master: Add non-repair mode to the existing method in OVN sync class https://review.opendev.org/c/openstack/neutron/+/953081 | 11:10 |
ralonsoh | slaweq, hello! if you have 1 min: https://review.opendev.org/c/openstack/neutron/+/952219 | 11:19 |
ralonsoh | thanks in advance | 11:19 |
opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: ovs: reimplement signals handling https://review.opendev.org/c/openstack/neutron/+/939321 | 11:20 |
opendevreview | Rodolfo Alonso proposed openstack/neutron master: ovs: remove the usage of eventlet in the OVS agent https://review.opendev.org/c/openstack/neutron/+/937765 | 11:21 |
opendevreview | Rodolfo Alonso proposed openstack/neutron master: [FT] Wait for the FIP Port_Binding event before checking MAC removal https://review.opendev.org/c/openstack/neutron/+/953195 | 11:26 |
opendevreview | Lajos Katona proposed openstack/neutron master: [eventlet-removal] remove eventlet references from dhcp_agent https://review.opendev.org/c/openstack/neutron/+/950499 | 12:31 |
bcafarel | ralonsoh: sure will check after my meetings! | 13:23 |
*** haleyb_ is now known as haleyb | 13:48 | |
opendevreview | Takashi Kajinami proposed openstack/neutron master: Fix missing [service_providers] service_provider https://review.opendev.org/c/openstack/neutron/+/953315 | 14:09 |
elvira | haleyb Hello!! I think you are one of the stable branches reviewers. For whenever you have the time, could you take a look at the rest of backports proposed for this? https://review.opendev.org/q/I00fa8332fdebc958ddb8f28c638670c75a70e0c5 | 14:17 |
elvira | Thanks a lot for the ones you already reviewed | 14:17 |
haleyb | elvira: hi, yes i can review, will do by eod | 14:21 |
ralonsoh | haleyb, hi! if you have 1 min: https://review.opendev.org/c/openstack/neutron/+/953195 | 14:59 |
ralonsoh | thanks in advance | 14:59 |
ralonsoh | btw, I'll check today evening the fullstack patch, I saw one comment from you | 15:00 |
haleyb | ralonsoh: ack | 15:04 |
ralonsoh | another btw, we are close to the cover percentage of 82% | 15:04 |
ralonsoh | https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_03f/openstack/03fb92a9a5c04cb0bd4a88a9124410c3/job-output.txt | 15:04 |
ralonsoh | and some cover jobs are failing | 15:04 |
ralonsoh | the problem is that, sometimes, we test using FTs, not UTs | 15:05 |
ralonsoh | cover job doesn't count that | 15:05 |
ralonsoh | we should lower a bit this number | 15:05 |
ralonsoh | maybe 75% | 15:05 |
ralonsoh | --> 2025-06-25 11:24:43.592366 | ubuntu-noble | Coverage failure: total of 81 is less than fail-under=82 | 15:05 |
ralonsoh | I need to leave now, I'll read you later | 15:06 |
haleyb | ralonsoh: oh, so we're adding more code than tests? maybe someone can look through the results and find a possible large area missing coverage | 15:06 |
haleyb | s/unit tests | 15:06 |
ralonsoh | that's the best approach, for sure | 15:06 |
ralonsoh | but consider that cover is only reading the UTs, but sometimes we only implement FTs | 15:07 |
haleyb | sometimes it's like 10 lines at a time, so slow work | 15:07 |
haleyb | yeah, and FTs are sometimes better. if we need to reduce to get CI green it's just something we have to do | 15:08 |
opendevreview | Merged openstack/neutron stable/2024.1: Improve ACL comparison efficiency https://review.opendev.org/c/openstack/neutron/+/952662 | 16:32 |
opendevreview | Merged openstack/neutron master: [FT] Wait for the FIP Port_Binding event before checking MAC removal https://review.opendev.org/c/openstack/neutron/+/953195 | 17:44 |
opendevreview | Merged openstack/neutron stable/2024.1: Consider logging options when using OVNdbsync https://review.opendev.org/c/openstack/neutron/+/952676 | 17:48 |
opendevreview | Brian Haley proposed openstack/neutron stable/2024.2: Bandit: add B311 to skiplist https://review.opendev.org/c/openstack/neutron/+/953341 | 18:34 |
opendevreview | Merged openstack/neutron stable/2025.1: Fix LoopingCallBase argument issue https://review.opendev.org/c/openstack/neutron/+/953064 | 21:44 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!