| opendevreview | Dmitrii Shcherbakov proposed openstack/ovn-bgp-agent master: Add BGP NIC IP cleanup logic for the OVN driver https://review.opendev.org/c/openstack/ovn-bgp-agent/+/940224 | 00:31 |
|---|---|---|
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [eventlet-removal][OVN] Require wsgi start-time in the config https://review.opendev.org/c/openstack/neutron/+/940123 | 07:14 |
| frickler | ralonsoh: do you know if a new neutron-lib release is planned or should I just propose one? seems https://review.opendev.org/c/openstack/neutron-vpnaas/+/898830 is blocked by that dependency | 07:32 |
| ralonsoh | frickler, let me check | 07:39 |
| ralonsoh | frickler, please, propose a new release | 07:41 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: common: fix wait_until_true to support native thread https://review.opendev.org/c/openstack/neutron/+/937843 | 07:56 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: polling: remove usage of eventlet.sleep() https://review.opendev.org/c/openstack/neutron/+/940136 | 07:56 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: ovs: reimplement signals handling https://review.opendev.org/c/openstack/neutron/+/939321 | 07:56 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: ovs: remove the usage of eventlet in the OVS agent https://review.opendev.org/c/openstack/neutron/+/937765 | 07:56 |
| sahid | o/ | 08:12 |
| sahid | anychance to have a second pass on those ones? https://review.opendev.org/c/openstack/neutron/+/939627/8 https://review.opendev.org/c/openstack/neutron/+/939348 | 08:12 |
| sahid | I fixed the test that was flaky | 08:12 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: ovs: reimplement signals handling https://review.opendev.org/c/openstack/neutron/+/939321 | 09:17 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: ovs: remove the usage of eventlet in the OVS agent https://review.opendev.org/c/openstack/neutron/+/937765 | 09:17 |
| sahid | ralonsoh: thank you for the reviews! regarding tests for that one https://review.opendev.org/c/openstack/neutron/+/939321 t've forgot to add the new file that articulate the change in the commit ;) | 09:20 |
| sahid | that is said I may need to add a bit more | 09:21 |
| ralonsoh | sahid, ok | 09:30 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Add limit of tags for every resource https://review.opendev.org/c/openstack/neutron/+/937887 | 10:53 |
| opendevreview | Lajos Katona proposed openstack/neutron master: WIP: [dhcp] Add DHCP domain_name option from network properties https://review.opendev.org/c/openstack/neutron/+/878820 | 11:02 |
| opendevreview | Lajos Katona proposed openstack/tap-as-a-service master: Doc: add documentation for usage and flow examples for OVS https://review.opendev.org/c/openstack/tap-as-a-service/+/828382 | 12:32 |
| opendevreview | Lajos Katona proposed openstack/tap-as-a-service master: Doc: add documentation for usage and driver details for SRIOV driver https://review.opendev.org/c/openstack/tap-as-a-service/+/881807 | 12:32 |
| opendevreview | Lajos Katona proposed openstack/neutron master: If OVS Manager creation failes retry to set values https://review.opendev.org/c/openstack/neutron/+/939117 | 12:45 |
| opendevreview | Lajos Katona proposed openstack/neutron-dynamic-routing master: DNM: test master branch https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/938442 | 12:45 |
| opendevreview | yatin proposed openstack/neutron master: [DNM] Check functional failure https://review.opendev.org/c/openstack/neutron/+/940128 | 13:09 |
| opendevreview | Lajos Katona proposed openstack/neutron-vpnaas master: Use eventlet.subprocess.Pope instead of Neutron utility method https://review.opendev.org/c/openstack/neutron-vpnaas/+/940241 | 13:36 |
| opendevreview | Lajos Katona proposed openstack/neutron-vpnaas master: Use eventlet.subprocess.Pipe instead of Neutron utility method https://review.opendev.org/c/openstack/neutron-vpnaas/+/940241 | 13:39 |
| opendevreview | Lajos Katona proposed openstack/neutron-vpnaas master: Use subprocess.Popen instead of Neutron eventlet utility method https://review.opendev.org/c/openstack/neutron-vpnaas/+/940243 | 13:48 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [eventlet-removal][OVN] Require wsgi start-time in the config https://review.opendev.org/c/openstack/neutron/+/940123 | 13:51 |
| opendevreview | Merged openstack/neutron stable/2024.1: Make sure that policy enforcer is initialized before use https://review.opendev.org/c/openstack/neutron/+/938912 | 13:59 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [eventlet-removal][OVN] Require wsgi start-time in the config https://review.opendev.org/c/openstack/neutron/+/940123 | 14:10 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: WIP == [eventlet-removal] OVN hash ring manager reimplementation https://review.opendev.org/c/openstack/neutron/+/940140 | 14:24 |
| ykarel | #startmeeting neutron_ci | 15:00 |
| opendevmeet | Meeting started Mon Jan 27 15:00:55 2025 UTC and is due to finish in 60 minutes. The chair is ykarel. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
| opendevmeet | The meeting name has been set to 'neutron_ci' | 15:00 |
| ykarel | Ping list: bcafarel, lajoskatona, slawek, mlavalle, mtomaska, ralonsoh, ykarel, jlibosva, elvira | 15:01 |
| mlavalle | \o | 15:01 |
| slaweq | hi | 15:01 |
| bcafarel | o/ | 15:01 |
| ralonsoh | hello | 15:01 |
| lajoskatona | o/ | 15:02 |
| ykarel | Hi everyone let's start with the topics | 15:02 |
| ykarel | #topic Actions from previous meetings | 15:02 |
| ykarel | ralonsoh to check https://bugs.launchpad.net/neutron/+bug/2085421 | 15:02 |
| ralonsoh | this is related to the hash ring issue, one sec | 15:03 |
| ralonsoh | https://bugs.launchpad.net/neutron/+bug/2095590 | 15:03 |
| ralonsoh | so I'm going to close it as duplicated | 15:03 |
| ykarel | ok good, you confirmed if this also fixing dynamic routing jobs? | 15:04 |
| mlavalle | one less bug, nice! | 15:04 |
| ralonsoh | not yet | 15:04 |
| ralonsoh | hmmm | 15:04 |
| ralonsoh | maybe we should keep it until we confirm that | 15:04 |
| ykarel | yes let's do that | 15:05 |
| ykarel | thx ralonsoh for taking care of these | 15:05 |
| ykarel | mlavalle to improve ipv6 test in https://bugs.launchpad.net/neutron/+bug/2095091 | 15:05 |
| mlavalle | I did look at it. Left a comment here: https://bugs.launchpad.net/neutron/+bug/2095091/comments/3 | 15:05 |
| mlavalle | since we have seen this only once, for now I propose doing nothing, unless the team feels we should address it sooner | 15:06 |
| ykarel | hmm we can watch for this, not seen it again yet | 15:08 |
| mlavalle | I can keep an eye on it | 15:08 |
| ykarel | mlavalle, have you checked what slaweq suggested if we really need the string checks? | 15:08 |
| mlavalle | I didn't look at that | 15:08 |
| ykarel | ack | 15:09 |
| mlavalle | looking at the strings is straightforward and if we need to improve the test it has an easy improvement | 15:09 |
| ykarel | ykarel to report the issue with neutron-functional-fips job | 15:09 |
| ykarel | reported https://bugs.launchpad.net/neutron/+bug/2095598 | 15:09 |
| ykarel | and was pointed out by ralonsoh that it might be fixed with a in review patch | 15:09 |
| ykarel | https://review.opendev.org/c/openstack/neutron/+/939627 | 15:10 |
| ralonsoh | yes, we saw that reviewing a related patch | 15:10 |
| ykarel | but test patch confirms it's not helping, so have to be relooked | 15:10 |
| ykarel | asked sahid if he can check that | 15:10 |
| ralonsoh | what is the testing aptch? | 15:10 |
| ralonsoh | https://review.opendev.org/c/openstack/neutron/+/940128 | 15:11 |
| ykarel | just running that test multiple times in https://review.opendev.org/c/openstack/neutron/+/940128 with depends on the possible fix | 15:11 |
| ykarel | yes that one | 15:11 |
| ralonsoh | let me check that, last week I spent sometime on this method | 15:11 |
| ralonsoh | is not very well implemented | 15:12 |
| ralonsoh | I'll assign it to me | 15:12 |
| ykarel | ralonsoh, ack thx | 15:12 |
| ykarel | #action ralonsoh to check https://review.opendev.org/c/openstack/neutron/+/940128 | 15:12 |
| ykarel | ralonsoh, just that we hitting it in fips jobs which runs on centos 9-stream | 15:12 |
| ykarel | i added what all changed since last success run there | 15:12 |
| ykarel | so good to check that as well what triggered this issue | 15:13 |
| ykarel | ok moving to next | 15:13 |
| ykarel | ykarel to send fix for job names in grafana dashboard | 15:13 |
| ykarel | pushed https://review.opendev.org/c/openstack/project-config/+/940065 | 15:13 |
| ykarel | already merged | 15:13 |
| ykarel | #topic Stable branches | 15:13 |
| ykarel | bcafarel, all yours ^ | 15:13 |
| bcafarel | quiet week, not too many backports, and CI looked stable there | 15:14 |
| bcafarel | I did not have time to check periodic though | 15:14 |
| ykarel | I checked periodic and it was good | 15:14 |
| ykarel | thx for the updates | 15:14 |
| bcafarel | nice :) | 15:14 |
| ykarel | #topic Stadium projects | 15:14 |
| ykarel | dynamic routing still broken https://zuul.openstack.org/build/cca145c899d84fd389aa911e3ddf60d1 | 15:14 |
| lajoskatona | vpnaas is failing | 15:15 |
| ykarel | - https://bugs.launchpad.net/neutron/+bug/2085421 | 15:15 |
| ykarel | vpnaas functional https://zuul.openstack.org/build/aaac7e51e607437b9095a085ed2d2a7e | 15:15 |
| ykarel | - broken with subprocess utility method removal https://review.opendev.org/c/openstack/neutron/+/939097 | 15:15 |
| lajoskatona | yes exactly | 15:15 |
| lajoskatona | I proposed patch for it: | 15:15 |
| lajoskatona | https://review.opendev.org/q/topic:%22bug/2087942%22+project:openstack/neutron-vpnaas | 15:15 |
| ykarel | thx lajoskatona | 15:16 |
| lajoskatona | one is to use eventlet.greendthread,subprocess the other to use native subprocess.popen, I suppose the last one is better in the long run, let's see if it runs into issue on CI first | 15:16 |
| ykarel | dynamic routing ralonsoh will be confirming if that also fixed with patch, then we should be good for stadiums | 15:17 |
| ralonsoh | ok | 15:17 |
| ralonsoh | lajoskatona, yes, much better the second | 15:17 |
| ykarel | #topic Rechecks | 15:17 |
| ykarel | We still have couple of rechecks for merging patches due to known intermittent issue, hopefully will be better once all current fixes get's in | 15:18 |
| ykarel | Bare rechecks wise quite good, no bare recheck out of total 38 rechecks | 15:18 |
| ykarel | #topic fullstack/functional | 15:19 |
| ykarel | - reproduced after logs added as part of https://bugs.launchpad.net/neutron/+bug/2079048 | 15:19 |
| ykarel | slaweq, if you could check if debug logs helped here ^? | 15:20 |
| ykarel | since you were looking that already :) | 15:20 |
| ykarel | can follow up on this later | 15:22 |
| ykarel | #topic Tempest/Scenario | 15:22 |
| slaweq | sure | 15:22 |
| ykarel | #undo | 15:23 |
| opendevmeet | Removing item from minutes: #topic Tempest/Scenario | 15:23 |
| ykarel | #action slaweq to check latest failure with logs added for https://bugs.launchpad.net/neutron/+bug/2079048 | 15:23 |
| ykarel | thx slaweq | 15:23 |
| ykarel | #topic Tempest/Scenario | 15:23 |
| ykarel | - we still have wsgi switch related issues | 15:23 |
| ykarel | hopefully will all be good once fixes from ralonsoh get's in | 15:24 |
| ykarel | thx again ralonsoh for working on these | 15:24 |
| ykarel | #topic grenade | 15:24 |
| ykarel | seen similar failure twice neutron-ovs-grenade-dvr-multinode, ping failure for FIP | 15:24 |
| ykarel | https://b3463c59c8f87072d3e9-87141d8cdbf10530595552debffaf82b.ssl.cf1.rackcdn.com/periodic/opendev.org/openstack/neutron/master/neutron-ovs-grenade-dvr-multinode/d964961/controller/logs/grenade.sh_log.txt | 15:24 |
| ykarel | https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_f66/periodic/opendev.org/openstack/neutron/master/neutron-ovs-grenade-dvr-multinode/f66a79c/controller/logs/grenade.sh_log.txt | 15:24 |
| ykarel | I recall seeing something similar in past too occassionally | 15:24 |
| ykarel | Will try to find and link that else open new one | 15:25 |
| ykarel | #action ykarel to check grenade failure | 15:25 |
| ykarel | #topic Grafana | 15:25 |
| ykarel | https://grafana.opendev.org/d/f913631585/neutron-failure-rate | 15:25 |
| ykarel | let's have a quick look here too | 15:25 |
| ykarel | ok in gate we can see some failures for fullstack | 15:27 |
| ykarel | those were related to l3 ha test, which i recall we have a known issue, will just link that | 15:28 |
| ykarel | and one was related to rabbitmq setup which we seen very rarely in past | 15:29 |
| ykarel | apart from that looks normal, can see spikes in check queues but those looks patches specific or known internmittent issues | 15:29 |
| ykarel | anything else to add? | 15:29 |
| mlavalle | not from me | 15:30 |
| lajoskatona | nothing from me | 15:30 |
| ralonsoh | no thanks | 15:30 |
| slaweq | ykarel just one thing about this metadata failure | 15:30 |
| slaweq | it failed on the patch for stable/2024.2 and my additional log was never backported there :/ | 15:31 |
| slaweq | I just checked that there is no info I wanted to check there | 15:31 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/2024.2: [Functional tests] Add logging router interfaces in metadata IPv6 tests https://review.opendev.org/c/openstack/neutron/+/940250 | 15:32 |
| ykarel | slaweq, ok thanks somehow i saw it was backported :) | 15:32 |
| ykarel | thx for relooking | 15:32 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/2024.1: [Functional tests] Add logging router interfaces in metadata IPv6 tests https://review.opendev.org/c/openstack/neutron/+/940251 | 15:32 |
| ykarel | #topic On Demand | 15:32 |
| ykarel | anything else you would like to add? | 15:32 |
| mlavalle | nothing from me | 15:32 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/2023.2: [Functional tests] Add logging router interfaces in metadata IPv6 tests https://review.opendev.org/c/openstack/neutron/+/940252 | 15:32 |
| slaweq | now backports are proposed ^^ | 15:32 |
| ykarel | ++ | 15:33 |
| slaweq | sorry that I forgo about them earlier | 15:33 |
| bcafarel | added to my pile | 15:33 |
| mlavalle | \o | 15:33 |
| ykarel | if nothing else, let's close early and have everyone 26 minutes back | 15:34 |
| ykarel | thx everyone for joining | 15:34 |
| ykarel | #endmeeting | 15:34 |
| opendevmeet | Meeting ended Mon Jan 27 15:34:27 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:34 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/neutron_ci/2025/neutron_ci.2025-01-27-15.00.html | 15:34 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/neutron_ci/2025/neutron_ci.2025-01-27-15.00.txt | 15:34 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/neutron_ci/2025/neutron_ci.2025-01-27-15.00.log.html | 15:34 |
| lajoskatona | \o/ | 15:34 |
| ralonsoh | bye | 15:34 |
| lajoskatona | Bye | 15:34 |
| ralonsoh | lajoskatona, hello! please check https://review.opendev.org/c/openstack/neutron/+/938659 | 16:06 |
| ralonsoh | and https://review.opendev.org/c/openstack/neutron/+/940123 is possible | 16:06 |
| ralonsoh | haleyb, ^ | 16:06 |
| ralonsoh | these are the first 2 patches to remove (and stabilize with ml2/ovn) the Neutron API | 16:07 |
| ralonsoh | to remove eventlet* | 16:07 |
| haleyb | ykarel: i'm unsure whether to approve https://review.opendev.org/c/openstack/neutron/+/939627/ based on your comments there | 16:13 |
| haleyb | ralonsoh: ack, just saw one release note typo in the second patch | 16:14 |
| ralonsoh | haleyb, thanks | 16:16 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [eventlet-removal][OVN] Require wsgi start-time in the config https://review.opendev.org/c/openstack/neutron/+/940123 | 16:18 |
| ralonsoh | haleyb, the patch and the fix is legit. It could not be fixing the open LP bug, but the fix is valid | 16:18 |
| lajoskatona | ralonsoh: checking | 16:19 |
| haleyb | ralonsoh: on the patch i asked yatin about? yes, i agree, we can merge and continue debugging that issue | 16:19 |
| ralonsoh | cool | 16:20 |
| opendevreview | Lajos Katona proposed openstack/neutron-vpnaas master: Use subprocess.Popen instead of Neutron eventlet utility method https://review.opendev.org/c/openstack/neutron-vpnaas/+/940243 | 17:07 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: WIP == Refresh host nodes before notifying https://review.opendev.org/c/openstack/neutron/+/940256 | 17:14 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: DNM - Test "neutron-ovn-tempest-ipv6-only-ovs*" with WSGI https://review.opendev.org/c/openstack/neutron/+/940257 | 17:16 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider master: Add LB sync logic https://review.opendev.org/c/openstack/ovn-octavia-provider/+/925324 | 18:51 |
| opendevreview | Merged openstack/neutron master: [eventlet-removal] Remove the usage of eventlet in the Neutron API https://review.opendev.org/c/openstack/neutron/+/938659 | 20:36 |
| opendevreview | Merged openstack/neutron stable/2024.1: [Functional tests] Add logging router interfaces in metadata IPv6 tests https://review.opendev.org/c/openstack/neutron/+/940251 | 20:36 |
| opendevreview | Jakub Libosvar proposed openstack/ovn-bgp-agent master: Change DVR FIP events to monitor the NAT table https://review.opendev.org/c/openstack/ovn-bgp-agent/+/940174 | 21:19 |
| *** bbezak is now known as Guest7191 | 23:32 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!