opendevreview | Miguel Lavalle proposed openstack/neutron-tempest-plugin master: Use OVN version 22.06 https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/848198 | 00:40 |
---|---|---|
opendevreview | Merged openstack/neutron-fwaas master: CI: add neutron and neutron-lib to docs required-projects https://review.opendev.org/c/openstack/neutron-fwaas/+/873376 | 00:58 |
opendevreview | Merged openstack/neutron-fwaas master: Add get_firewall_groups_count method https://review.opendev.org/c/openstack/neutron-fwaas/+/873360 | 01:01 |
opendevreview | liuyulong proposed openstack/neutron master: Pass physical bridge informations to OVS agent extension API https://review.opendev.org/c/openstack/neutron/+/866635 | 01:29 |
opendevreview | liuyulong proposed openstack/neutron master: Add host metadata haproxy manager https://review.opendev.org/c/openstack/neutron/+/864649 | 01:30 |
opendevreview | ZhouHeng proposed openstack/neutron-fwaas master: [WIP]test multinode https://review.opendev.org/c/openstack/neutron-fwaas/+/873088 | 03:20 |
*** ministry is now known as __ministry | 03:22 | |
opendevreview | Merged openstack/networking-bgpvpn master: Fix some pylint indentation warnings https://review.opendev.org/c/openstack/networking-bgpvpn/+/864397 | 03:28 |
opendevreview | ZhouHeng proposed openstack/neutron-fwaas master: [WIP]test multinode https://review.opendev.org/c/openstack/neutron-fwaas/+/873088 | 06:17 |
opendevreview | ZhouHeng proposed openstack/neutron-fwaas master: [WIP]test multinode https://review.opendev.org/c/openstack/neutron-fwaas/+/873088 | 08:09 |
opendevreview | Slawek Kaplonski proposed openstack/neutron-tempest-plugin master: [Secure RBAC] Add scope enforcement enabled job for master branch https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/867518 | 08:17 |
opendevreview | yatin proposed openstack/neutron master: [DNM] Check ovn jammy ipv6 job failure https://review.opendev.org/c/openstack/neutron/+/873684 | 08:24 |
opendevreview | Slawek Kaplonski proposed openstack/neutron master: Remove tripleo related jobs from neutron queues https://review.opendev.org/c/openstack/neutron/+/873685 | 08:42 |
ralonsoh | ^^ reviewing | 08:43 |
opendevreview | Rodolfo Alonso proposed openstack/neutron stable/zed: [OVN] Add missing LSP device_owner info in trunk driver https://review.opendev.org/c/openstack/neutron/+/873608 | 08:51 |
opendevreview | Rodolfo Alonso proposed openstack/neutron stable/yoga: [OVN] Add missing LSP device_owner info in trunk driver https://review.opendev.org/c/openstack/neutron/+/873609 | 08:51 |
opendevreview | Rodolfo Alonso proposed openstack/neutron stable/xena: [OVN] Add missing LSP device_owner info in trunk driver https://review.opendev.org/c/openstack/neutron/+/873610 | 08:52 |
opendevreview | Rodolfo Alonso proposed openstack/neutron stable/wallaby: [OVN] Add missing LSP device_owner info in trunk driver https://review.opendev.org/c/openstack/neutron/+/873611 | 08:52 |
opendevreview | Rodolfo Alonso proposed openstack/neutron stable/wallaby: [OVN] Bump the port revision number in trunk driver https://review.opendev.org/c/openstack/neutron/+/873616 | 08:52 |
opendevreview | Rodolfo Alonso proposed openstack/neutron stable/zed: [OVN] Bump the port revision number in trunk driver https://review.opendev.org/c/openstack/neutron/+/873612 | 08:52 |
opendevreview | Rodolfo Alonso proposed openstack/neutron stable/yoga: [OVN] Bump the port revision number in trunk driver https://review.opendev.org/c/openstack/neutron/+/873613 | 08:53 |
opendevreview | Rodolfo Alonso proposed openstack/neutron stable/xena: [OVN] Bump the port revision number in trunk driver https://review.opendev.org/c/openstack/neutron/+/873614 | 08:53 |
*** gibi_ is now known as gibi | 09:00 | |
opendevreview | Lajos Katona proposed openstack/neutron master: WIP: desperate try to make DVR functional tests more stable https://review.opendev.org/c/openstack/neutron/+/873111 | 09:30 |
opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] Add helper for retrieving LR associated with LRP https://review.opendev.org/c/openstack/neutron/+/873698 | 11:23 |
opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] Apply soft anti-affinity for LRs with multiple LRPs when scheduling https://review.opendev.org/c/openstack/neutron/+/873699 | 11:23 |
opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider master: Avoid use of ovn metadata port IP for HM checks https://review.opendev.org/c/openstack/ovn-octavia-provider/+/873426 | 11:42 |
opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider master: Avoid use of ovn metadata port IP for HM checks https://review.opendev.org/c/openstack/ovn-octavia-provider/+/873426 | 11:48 |
*** dasm|off is now known as dasm | 13:10 | |
opendevreview | Arkady Shtempler proposed openstack/neutron-tempest-plugin master: Using "Designate API" in addition to DNS query client check https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/859871 | 13:32 |
opendevreview | yatin proposed openstack/neutron-tempest-plugin master: [Wallaby] Fix tempest_exclude for ovs iptables_hybrid https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/873708 | 13:56 |
ykarel | lucasagomes, hi, can you check https://bugs.launchpad.net/neutron/+bug/2007167 related to ovn dhcp baremetal | 14:00 |
ykarel | thx in advance | 14:00 |
opendevreview | Merged openstack/neutron-tempest-plugin master: Clean definitions of the CI jobs for EOL'ed branches https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/872736 | 14:01 |
ralonsoh | ping bcafarel, elvira, frickler, mlavalle, mtomaska, obondarev, sahid, slawek, tobias-urdin, ykarel, lajoskatona, jlibosva, averdagu | 14:01 |
ralonsoh | #startmeeting networking | 14:01 |
opendevmeet | Meeting started Tue Feb 14 14:01:27 2023 UTC and is due to finish in 60 minutes. The chair is ralonsoh. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:01 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:01 |
opendevmeet | The meeting name has been set to 'networking' | 14:01 |
elvira | o/ | 14:01 |
ralonsoh | hello all | 14:01 |
obondarev | hi | 14:01 |
lajoskatona | o/ | 14:01 |
frickler | \o | 14:01 |
sahid | o/ | 14:02 |
bcafarel | o/ | 14:02 |
ykarel | o/ | 14:02 |
ralonsoh | ok, let's start | 14:02 |
* mlavalle is an internal company meeting. will lurk in the background | 14:02 | |
ralonsoh | #topic announcements | 14:02 |
ralonsoh | Antelope / 2023.1 schedule: https://releases.openstack.org/antelope/schedule.html | 14:03 |
ralonsoh | this week is R-5 | 14:03 |
slaweq | o/ | 14:03 |
ralonsoh | and that is antelope-3 milestone | 14:03 |
ralonsoh | remember this he final release for client libraries | 14:03 |
ralonsoh | and the feature freeze | 14:03 |
ralonsoh | the RC1 will be in 2 weeks | 14:04 |
ralonsoh | also remember that you have time to propose your candidacy for Neutron PTL (and other projects) | 14:04 |
ralonsoh | please check the calendar in https://governance.openstack.org/election/ | 14:04 |
ralonsoh | and a couple of needed links for the next PTG | 14:05 |
ralonsoh | Bobcat etherpad: https://etherpad.opendev.org/p/neutron-bobcat-ptg | 14:05 |
lajoskatona | \o/ | 14:05 |
ralonsoh | please start adding your topics to the Bobcat Neutron etherpad | 14:05 |
lajoskatona | +1 | 14:05 |
ralonsoh | during the next meetings, we'll check them and make the schedule | 14:06 |
ralonsoh | I'll remember this link every week | 14:06 |
ralonsoh | I also want to talk about the Vancouver meeting | 14:07 |
ralonsoh | but, if you don't mind, I'll move this topic to the "on demand" section | 14:07 |
lajoskatona | ok, I anyway added the advertisement for Vancouver forum proposals | 14:07 |
ralonsoh | lajoskatona, of course! | 14:07 |
lajoskatona | I mean I added to the on-demand agenda | 14:07 |
ralonsoh | right, I didn't see that | 14:08 |
ralonsoh | cool | 14:08 |
ralonsoh | and as usual, please check https://openinfra.dev/live/#all-episodes | 14:09 |
ralonsoh | ok, I think I'm not missing anything | 14:10 |
ralonsoh | related to the release schedule or anything | 14:10 |
ralonsoh | ok, let's move to the next topic then | 14:10 |
ralonsoh | #topic bugs | 14:10 |
ralonsoh | last week report is from elvira | 14:11 |
ralonsoh | #link https://lists.openstack.org/pipermail/openstack-discuss/2023-February/032167.html | 14:11 |
ralonsoh | so far, there are some bugs still not assigned | 14:11 |
ralonsoh | #link https://bugs.launchpad.net/neutron/+bug/2006953 | 14:11 |
ralonsoh | I think ykarel has sent a patch recently | 14:11 |
ykarel | yes | 14:12 |
ralonsoh | right: https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/873708 | 14:12 |
ralonsoh | ok thanks! | 14:12 |
ralonsoh | ahhh ok, I see the issue, thanks! | 14:12 |
ralonsoh | then next one is | 14:13 |
ralonsoh | #link https://bugs.launchpad.net/neutron/+bug/2006683 | 14:13 |
ralonsoh | but I think ykarel sent another patch | 14:13 |
ykarel | no lajoskatona sent it | 14:13 |
ralonsoh | ah yes | 14:13 |
ralonsoh | and is merged now: https://review.opendev.org/c/openstack/neutron-fwaas/+/873360 | 14:13 |
ralonsoh | (cool) | 14:13 |
ykarel | but not full fix | 14:13 |
ralonsoh | what else is missing? | 14:14 |
ykarel | i commented there https://bugs.launchpad.net/neutron/+bug/2006683/comments/3, can you check if that makes sense | 14:14 |
lajoskatona | exactly, I saw this in the logs but I am sure that it will not solve all issues, just cleans th log before the exact traceback | 14:14 |
lajoskatona | ykarel: I check your comment later, I missed that | 14:15 |
ralonsoh | ok so we are still seeing txn errors | 14:15 |
ralonsoh | due to the slqalchemy migration | 14:15 |
lajoskatona | yeah, but as I see sqalchemy2 will not be "included", at least the req bump patch is still red | 14:16 |
lajoskatona | and bauzas from Nova asked to skip it | 14:16 |
ykarel | lajoskatona, ack, we can check few more runs if it's sees again with your fix included | 14:16 |
ykarel | yes the current failures are with sqlalchmey-1.4 | 14:16 |
ralonsoh | in any case, we need to check what is the txn that is failing, or the operation | 14:17 |
ralonsoh | let's keep an eye on this one | 14:17 |
ykarel | +1 | 14:17 |
lajoskatona | +1 | 14:17 |
ralonsoh | the last one I had is | 14:18 |
ralonsoh | #link https://bugs.launchpad.net/neutron/+bug/2006145 | 14:18 |
lajoskatona | this one is for neutron-dynamic-routing as I see | 14:18 |
ralonsoh | maybe ltomasbo ^^ can help us on this one | 14:18 |
frickler | I'll take a look, too. | 14:19 |
ralonsoh | but I would say that what they are testing could not be supported | 14:19 |
frickler | I see the use case, otoh the current system is kind of working as designed | 14:20 |
ralonsoh | they are manually stopping the RPC communication | 14:20 |
ralonsoh | frickler, it is? | 14:20 |
frickler | well the current implementation assumes that rmq is always available | 14:21 |
ralonsoh | right | 14:21 |
frickler | that's why we have HA for it | 14:21 |
frickler | but if operators really need to shut it down for upgrades, we could try to adapt for that | 14:21 |
ralonsoh | so that could be a RFE, not a bug | 14:21 |
frickler | yes, I agree to that | 14:22 |
ralonsoh | perfect, do you mind commenting on the bug? | 14:22 |
frickler | I also wonder how everything else in openstack handles an rmq outage | 14:22 |
ltomasbo | frickler, ralonsoh: I know when using FRR you have some options to keep those routes. As NDR has its own speaker implementation, I suppose either that needs to be enabled (or added if missing) | 14:22 |
frickler | I'll have a look at the code and then comment, yes | 14:22 |
ralonsoh | other agents (OVS, DHCP, etc) keep running if no info from the server is needed | 14:23 |
* mlavalle back in the meeting | 14:23 | |
ralonsoh | of course, if they need to request something from the server, the operation will stop | 14:23 |
frickler | don't they do periodic refresh, too? | 14:23 |
ralonsoh | they do heartbeats | 14:24 |
ralonsoh | but that doesn't stop the agent | 14:24 |
ralonsoh | if the need to do a resync, then yes | 14:24 |
ralonsoh | in any case, I would need to check the ndr agent code too | 14:25 |
frickler | yes, let's dig a bit and come back to it next week or outside the meeting | 14:25 |
ralonsoh | frickler, ltomasbo thanks for the info | 14:25 |
ralonsoh | This week ykarel is the deputy, next week will be bcafarel (mtomaska can't next week) | 14:26 |
ykarel | yes | 14:26 |
bcafarel | ralonsoh: ack thanks for the reminder! | 14:26 |
ralonsoh | just a heads-up: we are experiencing random issues in the CI, more frequent as before | 14:26 |
ralonsoh | we will talk about this during the CI meeting | 14:27 |
ralonsoh | I know ykarel has addressed some of them during this week | 14:27 |
ralonsoh | but we need, somehow, to stabilize the CI | 14:27 |
ralonsoh | ok, let's move to the next topic | 14:28 |
ralonsoh | #topic specs | 14:28 |
ralonsoh | the only one open (an active) is | 14:28 |
ralonsoh | #link https://review.opendev.org/c/openstack/neutron-specs/+/870030 | 14:28 |
ralonsoh | I've been following the discussion between Dmitrii and lajoskatona | 14:28 |
ralonsoh | I still need to check some of the new API changes and the architecture | 14:29 |
ralonsoh | but I don't think we should merge it in this cycle | 14:29 |
lajoskatona | as I see there are open topics for those items | 14:30 |
ralonsoh | right | 14:30 |
lajoskatona | agree, but the progress and responsibility is good as I see | 14:30 |
ralonsoh | for sure and that will be attended during the next cycle without nay doubt | 14:30 |
ralonsoh | IMO, the spec is close to be ready | 14:31 |
ralonsoh | but not this week | 14:31 |
ralonsoh | so I'll add a comment today | 14:31 |
ralonsoh | that's all on this topic | 14:31 |
ralonsoh | let's move to the next | 14:31 |
ralonsoh | next one | 14:31 |
ralonsoh | # | 14:32 |
ralonsoh | (uffff) | 14:32 |
ralonsoh | #topic community_goals | 14:32 |
ralonsoh | 1) Consistent and Secure Default RBAC | 14:32 |
ralonsoh | #link https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/867518 | 14:32 |
ralonsoh | slaweq has done a fantastic job during this release | 14:32 |
ralonsoh | should we merge this patch ^^ now? | 14:32 |
ralonsoh | or should we wait 2 weeks? | 14:33 |
ralonsoh | slaweq ^ | 14:33 |
ralonsoh | (I think he is in other meeting at the same time) | 14:34 |
bcafarel | how stable is the job? (or should be) | 14:34 |
bcafarel | mostly thinking of CI stability these days :) | 14:34 |
ralonsoh | it is quite stable | 14:34 |
ralonsoh | a couple of failures in fwaas and bgp | 14:34 |
ralonsoh | but seems stable | 14:35 |
lajoskatona | I think instability is not coming from this now but from general issues in all other jobs | 14:35 |
slaweq | this job is pretty stable | 14:35 |
slaweq | the same as all other similar jobs | 14:35 |
slaweq | yes, I agree | 14:35 |
ralonsoh | lajoskatona, right | 14:35 |
ralonsoh | ok, then I'll change my vote on this one | 14:35 |
slaweq | those neutron-tempest-plugin-xxx jobs are IMHO one of the most stable once recently :) | 14:35 |
ralonsoh | yes | 14:35 |
slaweq | ++ | 14:35 |
bcafarel | if it stable big +1 to have it running and seeing results already on these (before switching next cycle) | 14:35 |
slaweq | thx | 14:35 |
ralonsoh | perfect | 14:36 |
slaweq | will be good to have it now so we will also have it in stable/anthelope later :) | 14:36 |
ralonsoh | ok, I've increased the review priority | 14:36 |
slaweq | in next cycle we will change it to be only job running with "old default policies" as we will switch to use new ones by default | 14:37 |
ralonsoh | perfect so new ones will be the default ones | 14:37 |
slaweq | in 2023.2 yes | 14:38 |
slaweq | in 2023.1 we will have one job which will test them | 14:38 |
ralonsoh | just out of curiosity, what is the status of other projects? | 14:39 |
slaweq | gmann knows it the best but AFAIK nova, glance, cinder are all good to switch also | 14:39 |
slaweq | and keystone of course | 14:39 |
slaweq | I don't really know much about other projects | 14:39 |
ralonsoh | good to know that big projects are on track too | 14:40 |
slaweq | https://review.opendev.org/q/topic:secure-rbac | 14:40 |
slaweq | there is many patches in that topic :) | 14:40 |
ralonsoh | ok, in some weeks we'll swap to the new policies by default | 14:41 |
slaweq | yes | 14:41 |
ralonsoh | perfect, we'll check at this time then | 14:41 |
ralonsoh | thanks! | 14:41 |
ralonsoh | the next sub-topic is | 14:42 |
ralonsoh | 2) Neutron client deprecation | 14:42 |
ralonsoh | lajoskatona, we have new releases, right? | 14:42 |
lajoskatona | Actually no news from my side | 14:42 |
ralonsoh | so we can continue with some patches now | 14:42 |
lajoskatona | perhaps one is htat I was waiting for SDK release, but as it is now out the req bump is still missing | 14:42 |
ralonsoh | ah right | 14:42 |
ralonsoh | (I always forget that second step) | 14:43 |
ralonsoh | never mind | 14:43 |
ralonsoh | ok, thanks lajoskatona! | 14:43 |
lajoskatona | So that is true for other libs also, we have releases finally but we still waiting for he u-c files to be upgraded | 14:43 |
ralonsoh | and the last topic for today is | 14:44 |
ralonsoh | #topic on_demand | 14:44 |
ralonsoh | lajoskatona, please | 14:44 |
lajoskatona | I just wanted to advertise the possibilty to propose forum topics for the summit | 14:44 |
ralonsoh | #link https://etherpad.opendev.org/p/neutron-bobcat-ptg | 14:45 |
lajoskatona | I think the forum schedule will be finalized mid April, and I suppose we have topropose before that | 14:45 |
lajoskatona | https://openinfra.dev/summit/vancouver-2023/forum/ | 14:45 |
slaweq | good point lajoskatona | 14:45 |
mlavalle | Thanks for the reminder | 14:45 |
ralonsoh | yeah, aboyt this | 14:45 |
ralonsoh | one sec | 14:45 |
slaweq | maybe we can open another etherpad to collect some ideas there | 14:46 |
ralonsoh | I opened this second etherpad | 14:46 |
ralonsoh | https://etherpad.opendev.org/p/neutron-vancouver-2023 | 14:46 |
ralonsoh | ^^^ | 14:46 |
slaweq | ralonsoh++ | 14:46 |
lajoskatona | thanks, +1 | 14:46 |
ralonsoh | so that's the point: what kind of topics are we going to discuss in Vancouver (after the virtual PTG)? | 14:46 |
ralonsoh | the PTG is in March, Vancouver summit is in June (3 months) | 14:47 |
slaweq | ralonsoh I think that those are 2 different things: one is our "ptg" in vancouver and second are topics for the forum session, to e.g. collect feedback from operators, etc. | 14:47 |
slaweq | but we can use the same etherpad for now to collect ideas for both | 14:47 |
ralonsoh | right, we can open different topics in the agenda | 14:48 |
lajoskatona | agree, but I see the Vancouve mid-cycle meeting as a frre to use opportuninty for disccusion | 14:48 |
ralonsoh | in any case, I'll send a mail today with both links | 14:49 |
ralonsoh | just to make it public for everyone | 14:49 |
lajoskatona | but an agenda is good idea | 14:49 |
ralonsoh | to be honest (but this is my opinion) I don't see the relevance of the Vancouver PTG | 14:50 |
ralonsoh | anyway, as commented, I'll send a mail today with both agendas | 14:51 |
frickler | +1, excluding the non-travelling part of the community isn't a good idea IMO | 14:51 |
lajoskatona | +1 | 14:51 |
lajoskatona | yeah but the meeting opprotunity is good, and I hope there will be option to have it hybrid | 14:52 |
ralonsoh | I agree on this | 14:52 |
ralonsoh | ok folks, is there anything else to be discussed today? | 14:53 |
ralonsoh | (please remember the CI meeting in 6 minutes) | 14:54 |
slaweq | ++ | 14:54 |
slaweq | see You in few minutes on CI meeting | 14:54 |
ralonsoh | see you there! bye | 14:54 |
ralonsoh | #endmeeting | 14:54 |
opendevmeet | Meeting ended Tue Feb 14 14:54:39 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:54 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/networking/2023/networking.2023-02-14-14.01.html | 14:54 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/networking/2023/networking.2023-02-14-14.01.txt | 14:54 |
opendevmeet | Log: https://meetings.opendev.org/meetings/networking/2023/networking.2023-02-14-14.01.log.html | 14:54 |
lajoskatona | Bye | 14:54 |
ykarel | o/ | 14:54 |
ralonsoh | slaweq, video or IRC? | 14:54 |
ralonsoh | (I never remember) | 14:54 |
mlavalle | slaweq: video or irc today? | 14:54 |
slaweq | irc today | 14:54 |
ralonsoh | perfect | 14:54 |
mlavalle | ack | 14:55 |
bcafarel | :) | 14:55 |
ralonsoh | ykarel, I'm going to send a neutron patch on top of https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/873708 | 14:55 |
ralonsoh | CI is just checking docs and pep8 | 14:56 |
ykarel | ralonsoh, ohkk should work, but ok to confirm :) | 14:56 |
ykarel | just in case | 14:56 |
opendevreview | Rodolfo Alonso proposed openstack/neutron stable/wallaby: DNM Testing patch of https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/873708 https://review.opendev.org/c/openstack/neutron/+/873717 | 14:58 |
ykarel | thx | 14:59 |
slaweq | #startmeeting neutron_ci | 15:00 |
opendevmeet | Meeting started Tue Feb 14 15:00:13 2023 UTC and is due to finish in 60 minutes. The chair is slaweq. 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 |
mlavalle | o/ | 15:00 |
slaweq | ping bcafarel, lajoskatona, mlavalle, mtomaska, ralonsoh, ykarel, jlibosva | 15:00 |
bcafarel | o/ | 15:00 |
ykarel | o/ | 15:00 |
mtomaska | o/ | 15:00 |
slaweq | hi | 15:00 |
jlibosva | o/ | 15:00 |
ralonsoh | hi | 15:00 |
slaweq | Grafana dashboard: https://grafana.opendev.org/d/f913631585/neutron-failure-rate?orgId=1 | 15:00 |
slaweq | ok, lets start as we have many topics for today | 15:00 |
slaweq | #topic Actions from previous meetings | 15:00 |
slaweq | lajoskatona to check additional logs in failing dvr related functional tests | 15:01 |
lajoskatona | o/ | 15:01 |
lajoskatona | yes, I checked and I am still not closer | 15:02 |
lajoskatona | I pushed a patch with tries for this issue, and today it is in this state: | 15:02 |
lajoskatona | https://review.opendev.org/c/openstack/neutron/+/873111 | 15:02 |
lajoskatona | it is now to run part of these tests with concurrency=1 | 15:03 |
opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider master: Avoid use of ovn metadata port IP for HM checks https://review.opendev.org/c/openstack/ovn-octavia-provider/+/873426 | 15:04 |
lajoskatona | when I checked the functional failures were for other failures and not for the usual DVR related issues, but I have to check more this topic | 15:04 |
slaweq | what about execution time? it it much longer? | 15:04 |
lajoskatona | good question, I haven't checked it yet, but I see one issue which is "too many files open" | 15:05 |
lajoskatona | not always, but from this patch's 20 runs I saw once as I remember | 15:05 |
slaweq | :/ | 15:05 |
lajoskatona | which is strange as in serial I would expect less load on such things | 15:06 |
slaweq | maybe that's another issue and some files aren't closed properly? | 15:06 |
ralonsoh | during the config parsing, maybe | 15:06 |
lajoskatona | possible, I can check that | 15:07 |
slaweq | thx | 15:07 |
ralonsoh | can you open another bug for this issue? | 15:07 |
slaweq | I will add AI for You for next week to remember about it | 15:07 |
lajoskatona | ack | 15:07 |
slaweq | #action lajoskatona to continue checking dvr functional tests issues | 15:07 |
slaweq | next one | 15:09 |
slaweq | ralonsoh to try to store journal log in UT job's results to debug "no such table" issues | 15:09 |
ralonsoh | no, I didn't work on this one | 15:09 |
ralonsoh | I was working on the FT ones | 15:09 |
slaweq | #action ralonsoh to try to store journal log in UT job's results to debug "no such table" issues | 15:09 |
slaweq | lets keep it for next week then | 15:09 |
slaweq | next one | 15:09 |
slaweq | mtomaska to check failed test_restart_rpc_on_sighup_multiple_workers functional test | 15:09 |
mtomaska | I looked into it.Of course I am not able to reproduce it locally. I really think this failure happens because of stestr concurrency which I cant quite replicate on my dev machine. Adding more logs to the test wont help. Is there a way to mark the test with concurrency=1 permanently? | 15:10 |
ralonsoh | you can move this test to the second FT execution | 15:10 |
ralonsoh | where we use concurrency one | 15:10 |
slaweq | You can do it in a way like lajoskatona in https://review.opendev.org/c/openstack/neutron/+/873111 | 15:11 |
slaweq | but I would really like to avoid doing that for all tests :) | 15:11 |
ralonsoh | we can move one single test only | 15:11 |
ralonsoh | same as sql ones | 15:11 |
ralonsoh | or "test_get_all_devices" | 15:11 |
mtomaska | ok cool. ralonsoh can I ping you on how to later? | 15:11 |
ralonsoh | sure | 15:11 |
slaweq | but if we will be sure that this is an issue | 15:11 |
slaweq | next one | 15:12 |
slaweq | ralonsoh to check Can't find port binding with logical port XXX error | 15:12 |
ralonsoh | #link https://review.opendev.org/c/openstack/neutron/+/873126 | 15:13 |
ralonsoh | ^ this is the patch | 15:13 |
slaweq | great, at least one fixed :) | 15:13 |
slaweq | thx | 15:14 |
slaweq | next one | 15:14 |
slaweq | mlavalle to check LRP failed to bind failure | 15:14 |
mlavalle | I checked it here: https://zuul.opendev.org/t/openstack/builds?job_name=neutron-functional&branch=master&skip=0 | 15:14 |
mlavalle | didn't happen again since the occurence you reported a week ago | 15:14 |
mlavalle | so for now, we don't need to worry about it | 15:15 |
mlavalle | it was a one of | 15:15 |
lajoskatona | \o/ | 15:15 |
lajoskatona | good news | 15:15 |
slaweq | not really | 15:15 |
slaweq | I saw the same issue today :/ | 15:15 |
slaweq | https://32c2553a5b1b0d58bd31-9e074609c9fda723118db1a634ce1a12.ssl.cf5.rackcdn.com/873247/3/gate/neutron-functional-with-uwsgi/6195659/testr_results.html | 15:16 |
lajoskatona | ohhhh | 15:16 |
mlavalle | oh, I'm reporting from my last check last night | 15:16 |
mlavalle | and that is a different job I think | 15:16 |
mlavalle | than the one we looked at last week | 15:16 |
slaweq | yes, it's different | 15:17 |
slaweq | also functional tests but it's from gate queue | 15:18 |
mlavalle | yeap | 15:18 |
slaweq | last week it was in periodic | 15:18 |
mlavalle | I was looking at periodic | 15:18 |
mlavalle | so that tells us that test case is flaky | 15:19 |
slaweq | yeap | 15:19 |
mlavalle | acorss jobs | 15:19 |
mlavalle | across jobs | 15:19 |
slaweq | will You try to investigate again this week? | 15:19 |
mlavalle | I'm sorry. I can't. I will be off on PTO starting tomorrow until March 1st | 15:19 |
mlavalle | but if you are willing to wait, I'll be happy to look at it upon my return | 15:20 |
slaweq | oh, ok | 15:20 |
slaweq | I will open LP bug to not forget about it | 15:20 |
slaweq | and we will see how often it will be occuring | 15:20 |
mlavalle | assign it to me if nobody is going to look at it | 15:21 |
slaweq | #action slaweq to report bug about failed to bind LRP in functional tests | 15:21 |
slaweq | sure, thx mlavalle | 15:21 |
mlavalle | thank you! | 15:21 |
slaweq | and last one | 15:21 |
slaweq | slaweq to check fullstack failures in test_multi_segs_network | 15:21 |
slaweq | I spent some time investigating it | 15:21 |
slaweq | I reported bug https://bugs.launchpad.net/neutron/+bug/2007152 | 15:21 |
slaweq | it seems to be valid issue in the code, not test problem | 15:22 |
slaweq | but I don't know yet what's wrong exactly there | 15:22 |
slaweq | there is some race between creation of subnets in network and cleaning stale devices from the namespace | 15:22 |
slaweq | I assigned this bug to myself and I will continue working on it | 15:23 |
slaweq | any questions/comments? | 15:24 |
lajoskatona | nothing from me | 15:25 |
slaweq | if not, lets move on | 15:25 |
slaweq | #topic Stable branches | 15:25 |
* bcafarel hides | 15:25 | |
slaweq | bcafarel any updates? | 15:25 |
slaweq | :) | 15:25 |
bcafarel | good and bad this week :) | 15:25 |
bcafarel | ussuri grenade and train look good from a quick look thanks to ykarel, wallaby I opened a bug and also fix in progress by ykarel https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/873708 | 15:26 |
bcafarel | victoria, we have 2 backports failing on py38/cover with " pkg_resources.extern.packaging.version.InvalidVersion: Invalid version: '<MagicMock name='execute().split().__getitem__().__getitem__()' id='140203724932048'>'" | 15:26 |
bcafarel | https://zuul.opendev.org/t/openstack/build/8f14ad998131488680c8f2142d1eab94 | 15:27 |
bcafarel | this one seems familiar but I could not find anything relevant, does it ring a bell? (if not will open a lp to track it) | 15:27 |
bcafarel | (and other branches are good \o/) | 15:27 |
ralonsoh | ^^ we can force the _get_version() to return the value, not utils_exec | 15:29 |
ralonsoh | I'll push a patch | 15:29 |
slaweq | ++ | 15:29 |
slaweq | thx ralonsoh | 15:29 |
bcafarel | ralonsoh++ thanks | 15:29 |
slaweq | and thx bcafarel for updates | 15:30 |
slaweq | so lets move on | 15:31 |
slaweq | #topic Stadium projects | 15:31 |
slaweq | lajoskatona I see there is bunch of topics there in agenda | 15:31 |
slaweq | are those added by You? | 15:31 |
lajoskatona | most of them seems ok | 15:31 |
lajoskatona | no I think it was not me | 15:32 |
lajoskatona | I think it is for one of the not so green ones | 15:32 |
slaweq | ok, maybe ykarel added them | 15:32 |
lajoskatona | fwaas job is failing, and it is causing failures as with tempest-plugin it is executed | 15:33 |
slaweq | there added links to failing fwaas tests | 15:33 |
ykarel | fwaas one we discussed in previous meeting | 15:33 |
lajoskatona | this is the bug for this: https://bugs.launchpad.net/neutron/+bug/2006683 | 15:33 |
lajoskatona | for fwaas I mean | 15:33 |
slaweq | who is now fwaas liaison? | 15:34 |
lajoskatona | the other topic is for the cfg related issue (see https://review.opendev.org/c/openstack/neutron/+/872644 ) | 15:34 |
lajoskatona | I think neutron-dynamic-routing and bgpvpn was the last affected by this, I have to check the fresh runs for them | 15:35 |
ralonsoh | I don't remember where this list is | 15:35 |
slaweq | https://docs.openstack.org/neutron/latest/contributor/policies/neutron-teams.html | 15:36 |
slaweq | found it :) | 15:36 |
slaweq | so it is zhouhenglc | 15:36 |
slaweq | maybe we should ping him to check those failures? | 15:36 |
lajoskatona | +1 | 15:36 |
slaweq | and temporary we can propose to make fwaas jobs to be non-voting if he will not reply | 15:36 |
lajoskatona | I will ping him | 15:37 |
ralonsoh | thanks | 15:37 |
slaweq | lajoskatona++ thx | 15:37 |
slaweq | #action lajoskatona to talk with zhouhenglc about fwaas jobs issues | 15:37 |
slaweq | ok, anything else regarding stadium today? | 15:37 |
lajoskatona | nothing from me | 15:38 |
slaweq | so lets move on | 15:38 |
slaweq | #topic Grafana | 15:38 |
slaweq | here everything looks that are on high failure rate for me | 15:39 |
ralonsoh | don't check too much the numbers | 15:39 |
ralonsoh | tempest ipv6: failing because of OVN issue in Jammy | 15:39 |
ralonsoh | FT: failign because of the test error I introduced (fixed now) | 15:39 |
slaweq | yes, I also saw pretty many "DNM" or "WIP" patches where most of the jobs failed | 15:40 |
ralonsoh | and I've been playing with CIm using 20 FT jobs at the same time | 15:40 |
ralonsoh | yes | 15:40 |
ralonsoh | so please, don't focus this week on numbers | 15:40 |
slaweq | so those results may be a bit missleading | 15:40 |
slaweq | ok, so lets move on to the rechecks then | 15:40 |
slaweq | #topic Rechecks | 15:40 |
slaweq | +---------+----------+... (full message at <https://matrix.org/_matrix/media/v3/download/matrix.org/flyUncZffilepRknBJrkPGIz>) | 15:40 |
slaweq | last week it was still very high: 2.0 rechecks in average to get patch merged | 15:41 |
ralonsoh | yes but always the same errors | 15:41 |
slaweq | but it's better in e.g. last 5 days or so | 15:41 |
slaweq | so hopefully it will be going down once we will fix few issues mentioned already | 15:42 |
slaweq | and some which I have to discuss still today :) | 15:42 |
slaweq | #topic Unit tests | 15:42 |
slaweq | critical bug reported https://bugs.launchpad.net/neutron/+bug/2007254 | 15:42 |
ralonsoh | I'm on this one | 15:43 |
slaweq | You are working on it? | 15:43 |
ralonsoh | tyes | 15:43 |
ralonsoh | yes | 15:43 |
ralonsoh | I'll assign it to me | 15:43 |
slaweq | ok, please assign it to Yourself then :) | 15:43 |
slaweq | thx | 15:43 |
ykarel | k thx, i too was trying to reproduce it, but couldn't | 15:43 |
slaweq | next topic | 15:44 |
slaweq | #topic fullstack/functional | 15:44 |
slaweq | here I found only one new issue | 15:44 |
slaweq | https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_777/870081/6/check/neutron-functional-with-uwsgi/7775413/testr_results.html | 15:44 |
slaweq | something related to mysql errors | 15:44 |
ralonsoh | again? maybe those tests have been executed at the same time, using mysql | 15:45 |
ralonsoh | right, mysql | 15:45 |
ralonsoh | I would't investigate it, this is just a test clash | 15:46 |
mtomaska | a test called "test_1" should not be even considered as a real test :) . Thats just bad naming | 15:46 |
slaweq | mtomaska yeah :) | 15:47 |
slaweq | ralonsoh but both were run with "--concurrency 1" | 15:47 |
ralonsoh | my bad when I pushed this patch | 15:47 |
slaweq | 2023-02-10 11:29:15.131934 | controller | ======... (full message at <https://matrix.org/_matrix/media/v3/download/matrix.org/AJmoLvYurxYfOWemkJFUhUBQ>) | 15:48 |
ralonsoh | slaweq, yes, you are right | 15:48 |
slaweq | my bet is that it was oom-killer what caused failure of first test | 15:49 |
slaweq | so cleanup wasn't done | 15:49 |
slaweq | and that's why second failed | 15:49 |
ralonsoh | yes, the 2nd is caused by the first one | 15:49 |
slaweq | but I didn't checked it in journal logs | 15:49 |
ralonsoh | and the first because of a timeout | 15:50 |
ralonsoh | I wouldn't spend too much time on this one | 15:50 |
slaweq | yeah, lets move on to the other topics | 15:50 |
slaweq | #topic grenade | 15:50 |
slaweq | I added this today as we have pretty many failures there | 15:50 |
slaweq | first, added by ykarel | 15:51 |
slaweq | seeing pcp deploy issues in these jobs randomly, basically can be seen in jobs where dstat is enabled | 15:51 |
slaweq | ykarel do You think that disabling dstat in grenade jobs will fix that problem (or workaround)? | 15:51 |
ykarel | slaweq, workaroud | 15:51 |
ykarel | as actual fix would need in pcp package | 15:52 |
slaweq | ok | 15:52 |
slaweq | will You propose patch? | 15:52 |
ykarel | sure will do that | 15:52 |
ralonsoh | ykarel++ | 15:52 |
slaweq | thx a lot | 15:52 |
slaweq | another issue which I have seen in grenade this week was related to failed ping to server: | 15:52 |
slaweq | https://58140b717b0fd2108807-1424e0f18ed1d65bb7e7c00bb059b2d8.ssl.cf2.rackcdn.com/872830/1/gate/neutron-ovs-grenade-multinode/6b0f11b/controller/logs/grenade.sh_log.txt | 15:52 |
slaweq | https://bca8357d212b699db2ea-90e26954923e94ff0df2f7d3073d9fdf.ssl.cf2.rackcdn.com/871983/3/check/neutron-ovn-grenade-multinode-skip-level/fd1c1fa/controller/logs/grenade.sh_log.txt | 15:52 |
slaweq | https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_01d/873615/2/check/neutron-ovs-grenade-dvr-multinode/01d90b6/controller/logs/grenade.sh_log.txt | 15:52 |
slaweq | I found at least those 3 occurences | 15:53 |
slaweq | so I think we should investigate it | 15:53 |
slaweq | anyone wants to check it? | 15:53 |
ralonsoh | sorry, not this week | 15:53 |
slaweq | no need to sorry ralonsoh :) | 15:53 |
slaweq | if nobody have any cycles, I will report LP for it and we will see | 15:54 |
slaweq | #action slaweq to report bug with failed ping in grenade jobs | 15:54 |
ykarel | +1 | 15:54 |
slaweq | next topic | 15:54 |
slaweq | #topic Tempest/Scenario | 15:54 |
slaweq | here I saw bunch of tests failing with ssh timeout | 15:55 |
slaweq | like e.g. tempest.scenario.test_network_basic_ops.TestNetworkBasicOps.test_hotplug_nic: | 15:55 |
slaweq | https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_f1f/872737/3/gate/neutron-ovn-tempest-ipv6-only-ovs-release/f1f5212/testr_results.html | 15:55 |
slaweq | https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_18b/864649/7/check/neutron-ovn-tempest-ipv6-only-ovs-release/18b432e/testr_results.html | 15:55 |
slaweq | https://74b83773945845894408-fddd31f569f44d98bd401bfd88253d97.ssl.cf2.rackcdn.com/873553/1/check/neutron-ovn-tempest-ipv6-only-ovs-release/6800220/testr_results.html | 15:55 |
ralonsoh | ykarel, founf the error | 15:55 |
ralonsoh | https://review.opendev.org/c/openstack/neutron/+/873684 | 15:55 |
ralonsoh | I think we should mark it as unstable | 15:55 |
ralonsoh | until OVN is fixed in Jammy | 15:56 |
ykarel | yes from logs i found that metadata flows were missing | 15:56 |
slaweq | that's great | 15:56 |
ykarel | and issue is not happening in focal | 15:56 |
ykarel | so it's specific to ovn22.03.0 in ubuntu jammy | 15:56 |
slaweq | can we skip this one test only or it can happen in other tests too? | 15:56 |
ralonsoh | only this one | 15:56 |
ralonsoh | so far | 15:56 |
slaweq | so lets skip that test in the job's definiton temporary and keep job voting | 15:57 |
ykarel | yes seen in this only tests, but can happen to others too | 15:57 |
slaweq | if it will happen in other tests too, we can always mark job as non-voting temporary later | 15:57 |
slaweq | wdyt? | 15:57 |
ralonsoh | we can skip this test only, for now | 15:57 |
ykarel | +1 to what ralonsoh said, and monitor the job | 15:58 |
slaweq | ok, ykarel will You propose patch for that? | 15:58 |
ykarel | ok will do | 15:58 |
slaweq | thx | 15:58 |
slaweq | ok, we are almost on top of the hour | 15:58 |
slaweq | but I have one last thing for today | 15:58 |
slaweq | there is question from amorin in https://review.opendev.org/c/openstack/neutron/+/869741 | 15:59 |
amorin | hello! | 15:59 |
slaweq | it's related to CI and neutron-ovs-tempest-dvr-ha-multinode-full | 15:59 |
ralonsoh | yes, I talked to him about this | 15:59 |
slaweq | and I think it's valid question | 15:59 |
ralonsoh | dvr_snat is not valid in compute | 15:59 |
ralonsoh | but we are still configuring it | 15:59 |
slaweq | I would like obondarev to look into this one too | 15:59 |
slaweq | IMO we can change L3 agent's mode to "dvr" in compute in that job | 16:00 |
ykarel | fwiw this job used to be 3 node in past, when it was switched to 2 node dvr_snat was set | 16:00 |
slaweq | and keep "ha=True" so it will be "ha" routers with only one node | 16:00 |
ralonsoh | yeah... | 16:00 |
slaweq | the same as we are doing in neutron-tempest-plugin- jobs | 16:00 |
slaweq | code path for ha will be tested | 16:00 |
slaweq | there will be no failover only really but that's not something what we are testing really in this job | 16:01 |
slaweq | ok, we are out of time | 16:01 |
slaweq | please comment on this patch and we can discuss there | 16:01 |
ralonsoh | ok | 16:01 |
slaweq | thx for attending the meeting today | 16:01 |
slaweq | #endmeeting | 16:01 |
opendevmeet | Meeting ended Tue Feb 14 16:01:43 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:01 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/neutron_ci/2023/neutron_ci.2023-02-14-15.00.html | 16:01 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/neutron_ci/2023/neutron_ci.2023-02-14-15.00.txt | 16:01 |
opendevmeet | Log: https://meetings.opendev.org/meetings/neutron_ci/2023/neutron_ci.2023-02-14-15.00.log.html | 16:01 |
ralonsoh | bye | 16:01 |
mlavalle | o/ | 16:01 |
bcafarel | o/ | 16:01 |
ykarel | o/ | 16:01 |
lajoskatona | o/ | 16:02 |
mtomaska | o/ | 16:03 |
opendevreview | Lajos Katona proposed openstack/networking-bagpipe master: CI: Add periodic weekly job with sqlalchemy master https://review.opendev.org/c/openstack/networking-bagpipe/+/872408 | 16:07 |
opendevreview | Lajos Katona proposed openstack/networking-bgpvpn master: CI: add oslo_master and sqlalchemy to periodic weekly https://review.opendev.org/c/openstack/networking-bgpvpn/+/861960 | 16:08 |
opendevreview | Merged openstack/neutron master: Remove tripleo related jobs from neutron queues https://review.opendev.org/c/openstack/neutron/+/873685 | 16:25 |
opendevreview | yatin proposed openstack/neutron master: [ovn-ipv6] Temporary exclude test_hotplug_nic https://review.opendev.org/c/openstack/neutron/+/873684 | 16:28 |
ykarel | ralonsoh, slaweq when u get chance a backport https://review.opendev.org/c/openstack/neutron/+/871989 | 16:32 |
ykarel | thx in advance | 16:32 |
ralonsoh | sure | 16:35 |
opendevreview | Rodolfo Alonso proposed openstack/neutron master: Mock ``IptablesManager._get_version`` in ``test_floating_forward_rules`` https://review.opendev.org/c/openstack/neutron/+/873734 | 17:24 |
opendevreview | Rodolfo Alonso proposed openstack/neutron master: [UT] Mock the retry decorator when creating duplicated NDPProxy https://review.opendev.org/c/openstack/neutron/+/873739 | 18:02 |
gmann | slaweq: on RBAC default switch update, nova has done, glance patch under review but cinder is not known. I think they might do in next cycle as there were some system scope things to cleanup | 18:14 |
opendevreview | Brian Haley proposed openstack/neutron stable/xena: Fix deprecation warnings in pkg_resources https://review.opendev.org/c/openstack/neutron/+/873585 | 19:19 |
opendevreview | Brian Haley proposed openstack/neutron stable/wallaby: Fix deprecation warnings in pkg_resources https://review.opendev.org/c/openstack/neutron/+/873586 | 19:19 |
opendevreview | Brian Haley proposed openstack/neutron stable/victoria: Fix deprecation warnings in pkg_resources https://review.opendev.org/c/openstack/neutron/+/873587 | 19:23 |
opendevreview | Brian Haley proposed openstack/ovsdbapp master: Add more IDL condition comparisons https://review.opendev.org/c/openstack/ovsdbapp/+/873748 | 19:56 |
opendevreview | Merged openstack/neutron stable/ussuri: Never raise an exception in notify() https://review.opendev.org/c/openstack/neutron/+/871989 | 22:56 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!