| opendevreview | Takashi Kajinami proposed openstack/networking-sfc master: Fix compatibility with SQLAlchemy 2.0 https://review.opendev.org/c/openstack/networking-sfc/+/920046 | 02:32 |
|---|---|---|
| opendevreview | Takashi Kajinami proposed openstack/neutron-vpnaas master: do not use str(url) to stringify a URL for subsequent use https://review.opendev.org/c/openstack/neutron-vpnaas/+/920049 | 02:45 |
| opendevreview | liuyulong proposed openstack/neutron master: Revert "Fix ObjectChangeHandler thread usage" https://review.opendev.org/c/openstack/neutron/+/919917 | 03:51 |
| opendevreview | liuyulong proposed openstack/neutron master: Revert change futurist.ThreadPoolExcutor to eventlet.GreenPool https://review.opendev.org/c/openstack/neutron/+/919918 | 03:51 |
| opendevreview | yatin proposed openstack/neutron master: Do not fail on missing logical router ports https://review.opendev.org/c/openstack/neutron/+/919699 | 04:24 |
| opendevreview | Takashi Kajinami proposed openstack/networking-sfc master: Fix compatibility with SQLAlchemy 2.0 https://review.opendev.org/c/openstack/networking-sfc/+/920046 | 05:18 |
| opendevreview | yatin proposed openstack/neutron master: Update grenade_from_branch for skip level jobs https://review.opendev.org/c/openstack/neutron/+/920075 | 06:21 |
| opendevreview | Takashi Kajinami proposed openstack/networking-bgpvpn master: Fix compatibility with SQLAlchemy 2.0 https://review.opendev.org/c/openstack/networking-bgpvpn/+/920045 | 07:17 |
| opendevreview | Merged openstack/neutron-fwaas master: do not use str(url) to stringify a URL for subsequent use https://review.opendev.org/c/openstack/neutron-fwaas/+/919743 | 07:26 |
| opendevreview | Merged openstack/neutron-vpnaas master: do not use str(url) to stringify a URL for subsequent use https://review.opendev.org/c/openstack/neutron-vpnaas/+/920049 | 07:36 |
| opendevreview | Merged openstack/tap-as-a-service master: do not use str(url) to stringify a URL for subsequent use https://review.opendev.org/c/openstack/tap-as-a-service/+/920047 | 08:03 |
| opendevreview | Lajos Katona proposed openstack/tap-as-a-service master: GRE/ERSPAN mirroring for taas https://review.opendev.org/c/openstack/tap-as-a-service/+/885357 | 08:08 |
| opendevreview | Lajos Katona proposed openstack/tap-as-a-service master: CLI for Tap Mirrors https://review.opendev.org/c/openstack/tap-as-a-service/+/886085 | 08:08 |
| opendevreview | Lajos Katona proposed openstack/tap-as-a-service master: Tap Mirror OVS driver https://review.opendev.org/c/openstack/tap-as-a-service/+/893086 | 08:08 |
| opendevreview | Lajos Katona proposed openstack/tap-as-a-service master: TAAS tap-mirror OVN driver https://review.opendev.org/c/openstack/tap-as-a-service/+/893509 | 08:08 |
| opendevreview | Lajos Katona proposed openstack/tap-as-a-service master: Documentation for tap-mirrors https://review.opendev.org/c/openstack/tap-as-a-service/+/907220 | 08:08 |
| opendevreview | Merged openstack/networking-bagpipe master: do not use str(url) to stringify a URL for subsequent use https://review.opendev.org/c/openstack/networking-bagpipe/+/920050 | 08:17 |
| opendevreview | Merged openstack/networking-sfc master: Fix compatibility with SQLAlchemy 2.0 https://review.opendev.org/c/openstack/networking-sfc/+/920046 | 08:39 |
| opendevreview | Lajos Katona proposed openstack/tap-as-a-service master: bandit: add bandit and bashate checks for tox https://review.opendev.org/c/openstack/tap-as-a-service/+/915421 | 09:14 |
| opendevreview | Merged openstack/networking-bgpvpn master: Fix compatibility with SQLAlchemy 2.0 https://review.opendev.org/c/openstack/networking-bgpvpn/+/920045 | 10:51 |
| opendevreview | Lajos Katona proposed openstack/tap-as-a-service master: GRE/ERSPAN mirroring for taas https://review.opendev.org/c/openstack/tap-as-a-service/+/885357 | 11:38 |
| opendevreview | Lajos Katona proposed openstack/tap-as-a-service master: CLI for Tap Mirrors https://review.opendev.org/c/openstack/tap-as-a-service/+/886085 | 11:39 |
| opendevreview | Lajos Katona proposed openstack/tap-as-a-service master: Tap Mirror OVS driver https://review.opendev.org/c/openstack/tap-as-a-service/+/893086 | 11:39 |
| opendevreview | Lajos Katona proposed openstack/tap-as-a-service master: TAAS tap-mirror OVN driver https://review.opendev.org/c/openstack/tap-as-a-service/+/893509 | 11:39 |
| opendevreview | Lajos Katona proposed openstack/tap-as-a-service master: Documentation for tap-mirrors https://review.opendev.org/c/openstack/tap-as-a-service/+/907220 | 11:39 |
| fkr | ahoi | 11:54 |
| opendevreview | ZhouHeng proposed openstack/neutron-fwaas master: create default security group using an independent context https://review.opendev.org/c/openstack/neutron-fwaas/+/916968 | 12:07 |
| opendevreview | Liushy proposed openstack/neutron-fwaas master: Support l3 stateless firewall based on OVN https://review.opendev.org/c/openstack/neutron-fwaas/+/845756 | 12:08 |
| opendevreview | Merged openstack/ovn-octavia-provider stable/2024.1: Reduce coverage threshold on stable branches https://review.opendev.org/c/openstack/ovn-octavia-provider/+/920016 | 12:09 |
| opendevreview | Merged openstack/ovn-octavia-provider stable/2023.2: Reduce coverage threshold on stable branches https://review.opendev.org/c/openstack/ovn-octavia-provider/+/920017 | 12:09 |
| opendevreview | Merged openstack/ovn-octavia-provider stable/2023.1: Reduce coverage threshold on stable branches https://review.opendev.org/c/openstack/ovn-octavia-provider/+/920018 | 12:10 |
| opendevreview | Lajos Katona proposed openstack/neutron-tempest-plugin master: Add wsgi enabled OVS and OVN tempest jobs https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/919725 | 12:50 |
| opendevreview | yatin proposed openstack/neutron master: [functional tests] compatibility with ovsdbapp>=2.7.0 https://review.opendev.org/c/openstack/neutron/+/920101 | 13:40 |
| opendevreview | Lajos Katona proposed openstack/neutron-dynamic-routing master: netaddr1: except TypeError in validate_ip_addr https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/919790 | 13:51 |
| haleyb | #startmeeting networking | 14:00 |
| opendevmeet | Meeting started Tue May 21 14:00:30 2024 UTC and is due to finish in 60 minutes. The chair is haleyb. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:00 |
| opendevmeet | The meeting name has been set to 'networking' | 14:00 |
| haleyb | Ping list: bcafarel, elvira, frickler, mlavalle, mtomaska, obondarev, slaweq, tobias-urdin, ykarel, lajoskatona, jlibosva, averdagu, amotoki, haleyb, ralonsoh | 14:00 |
| mlavalle | \o | 14:00 |
| slaweq | o/ | 14:00 |
| frickler | \o | 14:00 |
| obondarev | o/ | 14:00 |
| ykarel | o/ | 14:00 |
| mtomaska | o/ | 14:00 |
| fkr | o/ | 14:00 |
| lajoskatona | o/ | 14:01 |
| haleyb | #topic announcements | 14:01 |
| ihrachys | o/ | 14:01 |
| haleyb | thanks for taking care of things last week, i had very little extra time | 14:02 |
| haleyb | and thanks mlavalle for running the meetings | 14:02 |
| mlavalle | :-) | 14:02 |
| haleyb | Last week was the Dalmatian-1 milestone, so you've probably seen the releases for libraries | 14:03 |
| haleyb | i didn't see one for neutron itself, but assume it will be soon | 14:03 |
| haleyb | this week is R - 19 | 14:04 |
| haleyb | #link https://releases.openstack.org/dalmatian/schedule.html | 14:04 |
| opendevreview | OpenStack Release Bot proposed openstack/networking-bgpvpn master: reno: Update master for unmaintained/wallaby https://review.opendev.org/c/openstack/networking-bgpvpn/+/911523 | 14:05 |
| haleyb | Reminder if you have a topic for the drivers meeting on Fridays, please add it to the wiki @ https://wiki.openstack.org/wiki/Meetings/NeutronDrivers | 14:05 |
| haleyb | there will be a meeting this week pending we have a topic | 14:05 |
| lajoskatona | ack | 14:06 |
| mlavalle | haleyb: a lot of us RH people will be off this Friday | 14:06 |
| mlavalle | that might impact quorum | 14:06 |
| haleyb | mlavalle: thanks for letting me know, i might just cancel then | 14:06 |
| haleyb | and depending on my schedule i might be out (again) next Tuesday, will let people know since that impacts this meeting | 14:07 |
| haleyb | last week neutron had an issue with sqlalchemy 2.0 (conf file url bug), i think i've seen related changes in other network-* repos recently, please prioritize those since i'm assuming those gates are red | 14:08 |
| haleyb | i didn't have any other announcements, did anyone else? | 14:09 |
| lajoskatona | yes that one hit neutron-tempest-plugin also, but most of them are merged I beleive | 14:09 |
| haleyb | lajoskatona: ack, thanks, if you know of others needing a +2 feel free to ping me | 14:10 |
| lajoskatona | we can imagine how an eventlet change will look like if we have such hours with such small thing like SQLalchemy :P | 14:10 |
| frickler | also the new ovsdbapp release seems to have some issue which ykarel is investigating? | 14:10 |
| mlavalle | haleyb: I'll be glad to run the meeting again next Tuesday, if needed | 14:10 |
| ihrachys | frickler test only and there's a neutron fix https://review.opendev.org/c/openstack/neutron/+/920101 | 14:11 |
| ykarel | yes reported https://bugs.launchpad.net/neutron/+bug/2066263 for ovsdbapp | 14:11 |
| haleyb | mlavalle: thanks, i will let you know | 14:11 |
| haleyb | ykarel: thanks for the bug | 14:13 |
| * haleyb is slow today | 14:13 | |
| haleyb | let's move onto bugs then | 14:13 |
| haleyb | #topic bugs | 14:13 |
| haleyb | i realize there was some coverage issues the past two weeks, thanks to mlavalle and rubasov for moving things around | 14:14 |
| haleyb | what that means is the current schedule on the wiki needs to change | 14:15 |
| haleyb | if it makes it easier, I can just do this week, which would leave the next few months the same, instead of moving everyone | 14:16 |
| mlavalle | thanks! | 14:16 |
| haleyb | since we did not hear from amotoki i will try and follow-up, but it might mean that we will lose one deputy | 14:17 |
| lajoskatona | thanks | 14:18 |
| haleyb | so i will update the schedule after this meeting to reflect that | 14:18 |
| haleyb | so for next week, deputy will be lucasagomes - does that work for you lucas? | 14:18 |
| haleyb | he might not be here | 14:19 |
| haleyb | lets talk about the bugs | 14:19 |
| mlavalle | haleyb: I'll ping him internally | 14:19 |
| haleyb | #link https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/QYRWPDRUP2ONHJXEH6GJPLT4EV524AWM/ | 14:19 |
| haleyb | email from Bence | 14:19 |
| haleyb | #link https://bugs.launchpad.net/neutron/+bug/2065821 | 14:20 |
| haleyb | cover job failures | 14:20 |
| haleyb | from the bug, this is a 50% failure rate at the moment | 14:20 |
| haleyb | sigh, what is the opensearch user/pass ? | 14:21 |
| haleyb | #link https://opensearch.logs.openstack.org/_dashboards/app/login?nextUrl=%2F_dashboards%2Fapp%2Fdata-explorer%2Fdiscover%2F#?_a=(discover:(columns:!(_source),isDirty:!t,sort:!()),metadata:(indexPattern:'94869730-aea8-11ec-9e6a-83741af3fdcd',view:discover))&_g=(filters:!(),refreshInterval:(pause:!t,value:0),time:(from:now-10d,to:now))&_q=(filters:!(),query:(language:lucene,query:'message:%22Killed%22%20AND%20build_name:%22op | 14:21 |
| haleyb | enstack-tox-cover%22')) | 14:21 |
| slaweq | openstack/openstack I guess | 14:22 |
| lajoskatona | openstack-openstack I think | 14:22 |
| lajoskatona | at least this morning that allowed me in :-) | 14:22 |
| haleyb | thanks, will need to save that | 14:22 |
| haleyb | i have not looked, has anyone else? | 14:23 |
| haleyb | maybe something for CI meeting, but is critical as affecting the gate | 14:23 |
| haleyb | not a useful failure, just "Killed" | 14:23 |
| haleyb | i will look after meeting | 14:24 |
| haleyb | next bug | 14:25 |
| haleyb | #link https://bugs.launchpad.net/neutron/+bug/2065743 | 14:25 |
| haleyb | CLI arguments for rbac create are misleading and possibly incorrect | 14:25 |
| haleyb | from triage looks like we might need both a neutron and client change | 14:26 |
| haleyb | can anyone take that one? | 14:27 |
| slaweq | I may try but not this week probably | 14:27 |
| haleyb | slaweq: ack, thanks | 14:28 |
| slaweq | I will assign it to myself | 14:28 |
| haleyb | the rest of the bugs seem to have owners and patches | 14:29 |
| haleyb | any other bugs to talk about? | 14:29 |
| lajoskatona | another one from last week not in the report: | 14:29 |
| lajoskatona | https://bugs.launchpad.net/neutron/+bug/2065823 (neutron.tests.functional.agent.l3.test_legacy_router.L3AgentTestCase.test_legacy_router_lifecycle failed in _assert_internal_devices ) | 14:29 |
| lajoskatona | it happened once as I seen so perhaps not high prio at the moment | 14:30 |
| opendevreview | yatin proposed openstack/neutron master: [functional tests] compatibility with ovsdbapp>=2.7.0 https://review.opendev.org/c/openstack/neutron/+/920101 | 14:31 |
| haleyb | lajoskatona: i see your last comment there, maybe this is some sql race condition, will need to keep an eye on it | 14:32 |
| haleyb | alright, next topic | 14:33 |
| haleyb | #topic specs | 14:33 |
| haleyb | #link https://review.opendev.org/q/project:openstack%252Fneutron-specs+status:open | 14:33 |
| lajoskatona | +1, I attached the logs of the failing test | 14:33 |
| haleyb | there is just one spec left, other change there is pre-commit hook | 14:34 |
| haleyb | and i think we just had nits | 14:34 |
| haleyb | maybe we can ping rbartzen to update? | 14:36 |
| mlavalle | +1 | 14:36 |
| haleyb | racosta: can you update your remaining BGP spec? ^^ | 14:37 |
| haleyb | moving on | 14:37 |
| haleyb | #topic community-goals | 14:38 |
| haleyb | slaweq or lajoskatona - any update? | 14:38 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: tests: Use `del_fake_chassis` when deleting a chassis https://review.opendev.org/c/openstack/neutron/+/920103 | 14:38 |
| slaweq | Nothing from me this week | 14:39 |
| lajoskatona | no news from me | 14:39 |
| haleyb | ack | 14:39 |
| racosta | Hi haleyb, sure. Sorry for the dalay in updating the spec, I'll update asap. | 14:39 |
| haleyb | racosta: no problem, thanks for doing it | 14:40 |
| haleyb | #topic on-demand | 14:40 |
| haleyb | i don't see anything on the wiki, does anyone have a topic? | 14:40 |
| mtomaska | I have one quick thing | 14:40 |
| mtomaska | In the last PTG, we discussed python type annotation | 14:40 |
| mtomaska | I started with getting some tooling in for neutron-lib | 14:41 |
| mtomaska | and I wondering if we ever discussed(maybe TCs?) which type annotation tool we want to use | 14:41 |
| mtomaska | mypy or pyright. Mypy is what I see other repos selected but I am also wondering if we ever considered pyright | 14:42 |
| mtomaska | it has some Cons and Pros... | 14:42 |
| haleyb | mtomaska: that's a good question and I don't know the answer, but have seen mypy mostly | 14:42 |
| ihrachys | I think mypy was discussed before. I have no idea what pyright is and why it's better. perhaps you have and it justifies going the other way than other projects? | 14:42 |
| ihrachys | patches are here btw in case others would like to see: https://review.opendev.org/q/topic:%22type_tooling%22 | 14:43 |
| mtomaska | Right, I have two patches just for tooling. They are basically similar. There are some pros and cons for Pyright | 14:44 |
| ihrachys | what I'm missing in the patches (regardless of the tool to be used) is I don't see how the tooling is actually used. can we pick a module and convert it to the typing and show how the new tools warn / fail on violations in tests or wherever? | 14:44 |
| mtomaska | yes ^, that is my next step | 14:44 |
| ihrachys | mtomaska perhaps you could write up for others a comparison and then we can discuss after reading it? | 14:45 |
| mtomaska | I can propose two patches. One against pyright and another mypy. Then we can decide which route to go | 14:45 |
| haleyb | mtomaska: do other repos have voting/non-voting jobs? | 14:45 |
| mtomaska | Excellent article here: https://github.com/microsoft/pyright/blob/main/docs/mypy-comparison.md | 14:45 |
| haleyb | i would just use mypy if noone else is using pyright | 14:45 |
| mtomaska | I am biased towards pyright but I dont want to go against what I see already in other openstack repos(mypy) | 14:46 |
| ihrachys | right. there should be a good reason to not follow the herd | 14:46 |
| ihrachys | a REALLY good reason | 14:46 |
| mtomaska | the best I have is that it is "faster" and works better with IDEs | 14:47 |
| mtomaska | thats is all | 14:47 |
| ihrachys | mtomaska do you have an idea on the first guinea module for conversion and tool demo? | 14:47 |
| slaweq | FWIW there is no any guidance about it from the TC. Projects can choose tools they want AFAIK | 14:47 |
| mtomaska | so not very STRONG reasons | 14:47 |
| haleyb | codesearch is spinning at the moment, but searching for pyright will find all the copyright strings | 14:47 |
| mtomaska | I already searched, and I dont see any pyright mentions in the code base | 14:48 |
| mtomaska | one disadvantage to pyright. Under the hood it is written in Javascript/TypeScript. We would still use the python module they provide but we will implicitly depend on npm... probably dont want that | 14:49 |
| lajoskatona | +1 against npm in our deps | 14:50 |
| mtomaska | mypy is pure python | 14:50 |
| haleyb | mtomaska: seems mypy is the way... and thanks for working on this :) | 14:50 |
| ihrachys | haha js hate ensues | 14:50 |
| mtomaska | ok then seems like mypy then. I will propose some "demo" patches... | 14:51 |
| haleyb | i have stayed away from js my entire career, no need to get closer now | 14:51 |
| mlavalle | lol | 14:51 |
| mtomaska | and I will keep pyright for personal use / future dreams :-) | 14:51 |
| mtomaska | haleyb: you dont need to touch JS | 14:52 |
| haleyb | ----------------------------------------------------------------| JS | 14:52 |
| mtomaska | from the use perspective you dont realize its JS under the hood. | 14:52 |
| mtomaska | ok thanks all. I will be pushing new patches to neutron-lib and making them Active | 14:53 |
| haleyb | ok, again thanks for the work | 14:53 |
| haleyb | any other topics? | 14:53 |
| mlavalle | can I get this reviewd, please: https://review.opendev.org/c/openstack/neutron/+/918151? | 14:53 |
| mtomaska | Ill update you on the next meeting | 14:53 |
| ihrachys | I raised it before, but... we still have to resolve netaddr issues after 1.0.0 bump. I have patches that I raised before that would be nice to get some reviews on. also lajoskatona seems to've stepped in too with some. all can be found here: https://review.opendev.org/q/topic:%22netaddr1%22 | 14:53 |
| haleyb | ihrachys: ack, will look | 14:54 |
| haleyb | ykarel: CI meeting online or in-person? | 14:54 |
| ykarel | irc | 14:54 |
| haleyb | great | 14:55 |
| haleyb | ok, thanks for all the discussions and have a good week everyone | 14:55 |
| haleyb | #endmeeting | 14:55 |
| opendevmeet | Meeting ended Tue May 21 14:55:36 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:55 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/networking/2024/networking.2024-05-21-14.00.html | 14:55 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/networking/2024/networking.2024-05-21-14.00.txt | 14:55 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/networking/2024/networking.2024-05-21-14.00.log.html | 14:55 |
| mlavalle | o/ | 14:55 |
| slaweq | o/ | 14:55 |
| obondarev | o/ | 14:55 |
| lajoskatona | o/ | 14:55 |
| ihrachys | ciao | 14:55 |
| mtomaska | o/ | 14:56 |
| opendevreview | Lajos Katona proposed openstack/neutron-dynamic-routing master: netaddr1: except TypeError in validate_ip_addr https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/919790 | 14:56 |
| ykarel | #startmeeting neutron_ci | 15:00 |
| opendevmeet | Meeting started Tue May 21 15:00:46 2024 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, mlavalle, mtomaska, ralonsoh, ykarel, jlibosva, elvira | 15:00 |
| mlavalle | o/ | 15:00 |
| slaweq | o/ | 15:02 |
| lajoskatona | o/ | 15:02 |
| haleyb | o/ but am double-booked | 15:03 |
| ykarel | k i think we cans start with topics | 15:03 |
| ykarel | #topic Actions from previous meetings | 15:04 |
| ykarel | mlavalle to check failures for test_subport_delete functional test | 15:04 |
| mlavalle | I did check it. There are several cases over the past week | 15:04 |
| mlavalle | it seems to me a race condition in the test | 15:04 |
| mlavalle | where we delete the subport using the trunks plugin | 15:05 |
| mlavalle | and there is a event notification that is processed byt the ovn driver | 15:05 |
| mlavalle | sometimes we go and check if the subport has been removed from the OVN NBDB | 15:06 |
| mlavalle | and the event hasn't been processed yet | 15:06 |
| mlavalle | so later today I'll add a delay before the checking | 15:06 |
| mlavalle | and see if that takes care of it | 15:06 |
| ykarel | thx mlavalle for looking into this, can you also take care of reporting lp and tracking fixes as part of that? | 15:07 |
| mlavalle | will do | 15:07 |
| mlavalle | for sure | 15:07 |
| ykarel | thx much | 15:07 |
| ykarel | next one was on me | 15:07 |
| ykarel | ykarel to check failure in test_virtual_port_host_update_upon_failover functional test | 15:08 |
| ykarel | i didn't get chance to look into this, so will check this week | 15:08 |
| ykarel | #action ykarel to check failure in test_virtual_port_host_update_upon_failover functional test | 15:08 |
| ykarel | #topic Stable branches | 15:08 |
| ykarel | bcafarel, around? | 15:08 |
| ykarel | Not much activity in stable, but overall looks good | 15:08 |
| ykarel | considering periodic runs | 15:09 |
| ykarel | k let's move to next | 15:09 |
| ykarel | #topic Stadium projects | 15:09 |
| ykarel | there were couple of failures | 15:09 |
| ykarel | functional jobs broken for vpnaas, bgpvpn, sfc, fwaas with sqlalchemy 2.0, need fixes similar to https://review.opendev.org/c/openstack/neutron/+/919594 | 15:09 |
| ykarel | there are fixes proposed already by tkajinam https://review.opendev.org/q/topic:%22fix_str_url%22 | 15:10 |
| lajoskatona | yes these are from sqlalchemy | 15:10 |
| ykarel | also https://review.opendev.org/q/topic:%22sqlalchemy-20%22 | 15:11 |
| ihrachys | no neutron patches that are not merged | 15:11 |
| ykarel | for networking i think we already have all needed merged | 15:11 |
| ykarel | so we should be good now | 15:11 |
| lajoskatona | As I see sfc and vpnaas alsa has this isseu I will check them after the meeting | 15:11 |
| ykarel | thx lajoskatona , but i saw patches in sfc and vpnaas atleast for functional failures | 15:12 |
| lajoskatona | ykarel: true, I checked only open patches, cool | 15:13 |
| ykarel | k then all good :), let's move to next | 15:13 |
| ykarel | #topic Rechecks | 15:13 |
| ykarel | we had comparitively more rechecs due to various issues | 15:14 |
| ykarel | sqlalchemy/pylint/tox-cover ones, and some random failures | 15:14 |
| ykarel | bare rechecks wise not bad, had 3/23 bare rechecks, let's keep doing it | 15:15 |
| lajoskatona | +1 | 15:15 |
| ykarel | #topic Unit tests/pep8 | 15:15 |
| ykarel | there was timeout in pep8, and pep8 timeouts fixed by https://review.opendev.org/c/openstack/neutron/+/919594 | 15:15 |
| ykarel | cover jobs are intermittently failing, ihrachys reported https://bugs.launchpad.net/neutron/+bug/2065821 | 15:16 |
| ykarel | it's not assigned yet, but i think haleyb mentioned he will take a look | 15:16 |
| haleyb | yes, i will take a look | 15:17 |
| ykarel | thx haleyb | 15:17 |
| ykarel | #topic fullstack/functional | 15:17 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/2023.2: Adding isolation to functional tests https://review.opendev.org/c/openstack/ovn-octavia-provider/+/920109 | 15:17 |
| opendevreview | Merged openstack/neutron master: Update grenade_from_branch for skip level jobs https://review.opendev.org/c/openstack/neutron/+/920075 | 15:17 |
| ykarel | fullstack jobs was broken with sqlalchemy 2.0, is now fixed by ixed by https://review.opendev.org/c/openstack/neutron/+/919594 | 15:17 |
| ykarel | functional tests broken with ovsdbapp 2.7.1 https://bugs.launchpad.net/neutron/+bug/2066263 | 15:18 |
| ykarel | ^ not impacting gate, was detected by pro active tests on requirements patch | 15:18 |
| ykarel | pushed https://review.opendev.org/c/openstack/neutron/+/920101 | 15:18 |
| ykarel | please review, this will unblock ovsdbapp release | 15:19 |
| ykarel | #topic grenade | 15:19 |
| ykarel | Skip level jobs in master periodic were broken | 15:20 |
| ykarel | #topic Periodic | 15:20 |
| ykarel | https://zuul.openstack.org/builds?job_name=neutron-ovn-grenade-multinode-skip-level&job_name=neutron-ovs-grenade-multinode-skip-level&project=openstack%2Fneutron&branch=master&skip=0 | 15:20 |
| ykarel | reported https://bugs.launchpad.net/neutron/+bug/2066221 | 15:20 |
| ykarel | fix https://review.opendev.org/c/openstack/neutron/+/920075 just merged, so should be good now | 15:20 |
| ykarel | k that's it on failures | 15:21 |
| ykarel | #topic Grafana | 15:21 |
| ykarel | https://grafana.opendev.org/d/f913631585/neutron-failure-rate | 15:21 |
| ykarel | let's have a quick look here | 15:21 |
| ykarel | gate queue looks good, there are some spikes in check for pep8/fullstack/cover and those are known issues | 15:24 |
| ykarel | anything to add here? | 15:24 |
| mlavalle | not from me | 15:24 |
| mlavalle | looks good | 15:24 |
| lajoskatona | nothing from me | 15:25 |
| ykarel | ack | 15:26 |
| ykarel | #topic On Demand | 15:26 |
| ykarel | anything else you would like to raise? | 15:26 |
| lajoskatona | I have few patches fro stadiums | 15:26 |
| lajoskatona | Testing runtime update: https://review.opendev.org/q/topic:%22update-testing-runtime%22 | 15:27 |
| lajoskatona | and for the pyproject.toml patches: https://review.opendev.org/q/topic:%22pip-23.1-support%22 | 15:27 |
| ykarel | thx lajoskatona | 15:27 |
| lajoskatona | this one is not filtered for networking | 15:27 |
| ykarel | lajoskatona, wrt pyproject.toml, that do not work with rhel9/centos 9 stream with GLOBAL_VENV=False, right? | 15:29 |
| ykarel | i recently saw an issue with it and tap-as-a-service | 15:29 |
| ykarel | as there it was already merged | 15:29 |
| lajoskatona | interesting | 15:29 |
| lajoskatona | this global_venv thing caused me nightmares anyway on ubuntu as well | 15:30 |
| mlavalle | lol | 15:30 |
| lajoskatona | ok, so that can be a topic how to do that, but perhaps not just for networking ? | 15:31 |
| ykarel | for tap-as-a-service i think as we not testing in centos-9-stream it was not caught | 15:32 |
| ykarel | not sure about other project where the patch is open | 15:32 |
| ykarel | for reference it failed as:- | 15:32 |
| ykarel | ++functions-common:_safe_permission_operation:2170 sudo chown -R stack '/opt/stack/tap-as-a-service/*.egg-info' | 15:33 |
| ykarel | chown: cannot access '/opt/stack/tap-as-a-service/*.egg-info': No such file or directory | 15:33 |
| ykarel | and dropping pyproject.toml from tap-as-a-service installation went fine with centos 9-stream + GLOBAL_VENV=false | 15:34 |
| lajoskatona | as a lot of these were merged openstack wide it can be a globa problem | 15:34 |
| lajoskatona | thanks ykarel for pointing to this issue with those patches | 15:35 |
| ykarel | yes and if those merges together with https://review.opendev.org/c/openstack/devstack/+/898845 i think that should catch some issues | 15:36 |
| lajoskatona | armageddon? | 15:36 |
| ykarel | i haven't checked where are we with full support of that, but something to check for | 15:36 |
| lajoskatona | that's it from me | 15:37 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/2023.1: Adding isolation to functional tests https://review.opendev.org/c/openstack/ovn-octavia-provider/+/920112 | 15:37 |
| ykarel | ok can check this offline, thx everyone for joining | 15:37 |
| ykarel | #endmeeting | 15:38 |
| opendevmeet | Meeting ended Tue May 21 15:38:04 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:38 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/neutron_ci/2024/neutron_ci.2024-05-21-15.00.html | 15:38 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/neutron_ci/2024/neutron_ci.2024-05-21-15.00.txt | 15:38 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/neutron_ci/2024/neutron_ci.2024-05-21-15.00.log.html | 15:38 |
| lajoskatona | Bye | 15:38 |
| ykarel | lajoskatona, can you also check https://review.opendev.org/c/openstack/neutron/+/920101 | 15:38 |
| mlavalle | o/ | 15:39 |
| ihrachys | ykarel zuul is complaining about regexes in jobs descriptions btw, e.g. here https://review.opendev.org/c/openstack/neutron/+/920075 | 15:40 |
| ykarel | ihrachys, ahh right, forgot to raise that in meeting, we need to take care for that | 15:41 |
| ykarel | likely some recent zuul release triggered this | 15:41 |
| ykarel | we in past hit it for branches: and now seems to have enabled for irrelevant-files | 15:42 |
| frickler | yes, see https://zuul.opendev.org/t/openstack/config-errors?project=openstack%2Fneutron&skip=250 | 15:42 |
| ihrachys | ack; I guess it's not a fire situation, just warnings. but annoying. | 15:43 |
| frickler | and https://review.opendev.org/c/zuul/zuul/+/916141 | 15:43 |
| ykarel | thx frickler ihrachys | 15:44 |
| ykarel | mlavalle, can you check https://review.opendev.org/c/openstack/neutron/+/920101 | 15:49 |
| opendevreview | Merged openstack/neutron-fwaas master: Add pyproject.toml to support pip 23.1 https://review.opendev.org/c/openstack/neutron-fwaas/+/916417 | 16:05 |
| opendevreview | Merged openstack/neutron-vpnaas master: Update jobs based on testing runtime for 2024.2 https://review.opendev.org/c/openstack/neutron-vpnaas/+/914280 | 16:40 |
| opendevreview | Merged openstack/networking-bgpvpn master: reno: Update master for unmaintained/wallaby https://review.opendev.org/c/openstack/networking-bgpvpn/+/911523 | 16:48 |
| opendevreview | Merged openstack/networking-bagpipe master: Update jobs based on testing runtime for 2024.2 https://review.opendev.org/c/openstack/networking-bagpipe/+/914272 | 16:59 |
| opendevreview | Merged openstack/tap-as-a-service master: Add py312 job to periodic based on testing runtime for 2024.2 https://review.opendev.org/c/openstack/tap-as-a-service/+/914643 | 17:08 |
| opendevreview | Brian Haley proposed openstack/neutron-tempest-plugin master: Remove old excludes https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/917700 | 17:18 |
| opendevreview | Brian Haley proposed openstack/neutron master: Remove old excludes https://review.opendev.org/c/openstack/neutron/+/917595 | 17:18 |
| ykarel | haleyb, can you also check https://review.opendev.org/c/openstack/neutron/+/920101 | 17:21 |
| haleyb | ykarel: ack, done | 17:25 |
| ykarel | thx haleyb | 17:33 |
| opendevreview | Merged openstack/networking-bgpvpn master: Update jobs based on testing runtime for 2024.2 https://review.opendev.org/c/openstack/networking-bgpvpn/+/914273 | 17:35 |
| opendevreview | Merged openstack/networking-sfc master: Add py312 job to periodic based on testing runtime for 2024.2 https://review.opendev.org/c/openstack/networking-sfc/+/914276 | 17:39 |
| opendevreview | Merged openstack/neutron master: [functional tests] compatibility with ovsdbapp>=2.7.0 https://review.opendev.org/c/openstack/neutron/+/920101 | 18:22 |
| opendevreview | Merged openstack/neutron master: Remove the Windows OS libraries https://review.opendev.org/c/openstack/neutron/+/914271 | 19:40 |
| opendevreview | Merged openstack/networking-bgpvpn master: Replace CRLF by LF https://review.opendev.org/c/openstack/networking-bgpvpn/+/906927 | 19:56 |
| opendevreview | Brian Haley proposed openstack/neutron master: DNM: Debug cover job failure https://review.opendev.org/c/openstack/neutron/+/920150 | 21:14 |
| opendevreview | Brian Haley proposed openstack/neutron master: DNM: Debug cover job failure https://review.opendev.org/c/openstack/neutron/+/920150 | 21:18 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!