| opendevreview | Miguel Lavalle proposed openstack/neutron master: Fix support of IPv6 only networks in OVN metadata agent https://review.opendev.org/c/openstack/neutron/+/922264 | 00:41 |
|---|---|---|
| opendevreview | yatin proposed openstack/neutron master: [DNM] Check rally job fix https://review.opendev.org/c/openstack/neutron/+/926191 | 10:12 |
| opendevreview | yatin proposed openstack/neutron master: Adopt to StandardAttribute load method change to "selectin" https://review.opendev.org/c/openstack/neutron/+/923931 | 10:35 |
| opendevreview | yatin proposed openstack/neutron master: Temporary mark ovs-rally job as non-voting https://review.opendev.org/c/openstack/neutron/+/926196 | 11:21 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron master: Get ips from system dns resolver without scope https://review.opendev.org/c/openstack/neutron/+/926079 | 11:24 |
| lajoskatona | ykarel: Hi, today I can't attend the CI meeting, I opened a bug for the test_pagination... test failure: https://bugs.launchpad.net/neutron/+bug/2076328 | 11:24 |
| lajoskatona | ykarel: as comment #2 I added an idea for it but not sure if it could really work, please check if it is something that can help | 11:25 |
| ykarel | thx lajoskatona , ack will check | 11:27 |
| ykarel | lajoskatona, can you also check https://review.opendev.org/c/openstack/neutron/+/926196 to unblock neutron master gate | 11:27 |
| ykarel | lajoskatona, wrt c#2 seems those tests involve sorting, so having random bits may not help for such case? | 11:38 |
| ykarel | also it's failing at length check self.assertEqual(len(original), len(actual)), so not sure how random names will help in avoiding it | 11:42 |
| lajoskatona | ykarel: my idea was that the issue comes from parallel execution | 11:47 |
| lajoskatona | ykarel: 926196 => done | 11:49 |
| ykarel | lajoskatona, ack | 11:51 |
| opendevreview | yatin proposed openstack/neutron-tempest-plugin master: Filter resources in SearchCriteriaTest to avoid random failures https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/926201 | 12:47 |
| ykarel | lajoskatona, trying out ^ for those pagination failures | 12:47 |
| lajoskatona | ykarel: cool, checking | 12:48 |
| ykarel | hope it don't explode else can move the filtering only to the impacted pagination tests | 12:48 |
| opendevreview | yatin proposed openstack/neutron master: Adopt to StandardAttribute load method change to "selectin" https://review.opendev.org/c/openstack/neutron/+/923931 | 12:53 |
| haleyb | #startmeeting networking | 14:00 |
| opendevmeet | Meeting started Tue Aug 13 14:00:40 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 |
| lajoskatona | o/ | 14:00 |
| ihrachys | o/ | 14:00 |
| obondarev | hi | 14:00 |
| bcafarel | o/ | 14:01 |
| rubasov | o/ | 14:01 |
| ykarel | o/ | 14:01 |
| haleyb | i know a couple of people are on pto this week, so we can get started | 14:02 |
| haleyb | #topic announcements | 14:02 |
| elvira | o/ | 14:02 |
| haleyb | We are now in Dalmatian release week (R -7) | 14:02 |
| haleyb | Oslo libraries are entering feature freeze this week | 14:03 |
| haleyb | i know elvira had one change that would be good to merge regarding IPv6 addresses | 14:03 |
| haleyb | Non-client library freeze: August 22nd, 2024 (R-6 week) - next week | 14:04 |
| haleyb | Client library freeze: August 29th, 2024 (R-5 week) | 14:04 |
| haleyb | Dalmatian-3 milestone: August 29th, 2024 (R-5 week) | 14:04 |
| haleyb | Since we are at the end of the cycle, I would like to start using the priorities dashboard for patches in the "ready to merge" state. This could be older changes as well as new ones. | 14:05 |
| haleyb | Please just add them as RP+1, we will keep RP+2 for critical fixes | 14:05 |
| haleyb | i need to make a tinyurl for the dashboard, the full link is in the meeting page | 14:06 |
| haleyb | #link https://wiki.openstack.org/wiki/Network/Meetings | 14:06 |
| haleyb | so if people could take a look at that page once a day (and add things) it should help us close-out some long-running bugs | 14:07 |
| haleyb | it seemed better than adding yet another etherpad since it is dynamically updated based on states | 14:08 |
| mlavalle | +1 | 14:08 |
| haleyb | any other announcements? | 14:09 |
| haleyb | #topic bugs | 14:09 |
| haleyb | slaweq was the deputy last week, his report is at | 14:09 |
| haleyb | #link https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/message/S4TBXEFSCRVSOAAODUIIQTFG7ABCTCDX/ | 14:09 |
| haleyb | there was one critical bug | 14:10 |
| haleyb | #link https://bugs.launchpad.net/neutron/+bug/2076430 | 14:10 |
| haleyb | but it seems more we are the victim, fix is in oslo.serialization | 14:10 |
| haleyb | #link https://review.opendev.org/c/openstack/oslo.serialization/+/926172 | 14:11 |
| haleyb | and i see it just got pushed into the gate | 14:11 |
| haleyb | ykarel: that did seem to fix the issue, correct? | 14:11 |
| lajoskatona | https://review.opendev.org/c/openstack/oslo.serialization/+/926172 | 14:11 |
| lajoskatona | this one I think | 14:11 |
| lajoskatona | yes, sorry I missed the link | 14:11 |
| ykarel | haleyb, yes correct that fixes it | 14:11 |
| ykarel | we would also need to get it released and updated in u-c | 14:12 |
| ykarel | meanwhile we marking the job non-voting | 14:12 |
| ykarel | https://review.opendev.org/c/openstack/neutron/+/926196 | 14:12 |
| haleyb | ykarel: right. could we mark that version bad in requirements.txt? != type logic? i can never remember if gate likes that in all cases | 14:13 |
| haleyb | ykarel: ^^ ? | 14:16 |
| haleyb | we can come back to it | 14:16 |
| haleyb | next bug is | 14:16 |
| haleyb | #link https://bugs.launchpad.net/neutron/+bug/2076328 | 14:16 |
| haleyb | it's a periodic test failure, but ykarel posted a patch earlier | 14:17 |
| haleyb | #link https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/926201 | 14:17 |
| haleyb | next un-assigned is | 14:17 |
| haleyb | #link https://bugs.launchpad.net/neutron/+bug/2076181 | 14:18 |
| haleyb | "Accidentally no-allocation-pool for subnet leads to poor diagnosis of build-in dhcp service issues" | 14:18 |
| haleyb | slaweq marked as low-hanging-fruit | 14:18 |
| haleyb | last bug looks more like an rfe | 14:20 |
| haleyb | #link https://bugs.launchpad.net/neutron/+bug/2076122 | 14:20 |
| haleyb | dns integration: support multiple domains per instance | 14:20 |
| haleyb | i will take a look and triage for drivers meeting | 14:20 |
| haleyb | any other bugs to discuss? | 14:21 |
| haleyb | Current bug count this week: 720, up 2 from last week | 14:21 |
| haleyb | And this week I am the bug deputy, next week will be rubasov - does that work for you? | 14:21 |
| rubasov | ack, I'll be ooo on Mon-Tue, but I'll catch up when I'm back | 14:21 |
| opendevreview | Miguel Lavalle proposed openstack/neutron-tempest-plugin master: [DNM] Test metadata query over IPv6 only network https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/925928 | 14:22 |
| haleyb | #topic community-goals | 14:23 |
| haleyb | neutronclient deprecation is continuing | 14:23 |
| haleyb | #link https://review.opendev.org/q/topic:%22bug/1999774%22 | 14:23 |
| opendevreview | Miguel Lavalle proposed openstack/neutron-tempest-plugin master: [DNM] Test metadata query over IPv6 only network https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/925928 | 14:24 |
| haleyb | reviews are welcome, all in horizon | 14:24 |
| haleyb | regarding eventlet removal, we have one patch to merge | 14:24 |
| haleyb | #link https://review.opendev.org/q/topic:%22bug/2069581%22 | 14:24 |
| haleyb | #link https://review.opendev.org/c/openstack/neutron/+/924317 | 14:25 |
| mlavalle | added to my pile | 14:25 |
| haleyb | it was depending on a different patch that was failing the gate | 14:25 |
| mlavalle | I'll look at it later today | 14:25 |
| haleyb | ^that patch is -W until the other merges, trying to find it | 14:26 |
| haleyb | it was one monkey-patching stuff | 14:26 |
| mlavalle | ack. good to know | 14:26 |
| mlavalle | I think I saw the latter last week | 14:26 |
| haleyb | #link https://review.opendev.org/c/openstack/neutron/+/925376 | 14:27 |
| mlavalle | yeap but I think ykarel stopped it because it broke something | 14:28 |
| haleyb | well, we had rechecked enough that something is not right | 14:28 |
| ykarel | yes most such failures i saw against that patch so looked specific to patch | 14:29 |
| ykarel | but i didn't dig much on that | 14:30 |
| haleyb | i'll look a little, guess i can discount the rally failure at least | 14:30 |
| haleyb | #topic on-demand | 14:31 |
| haleyb | does anyone have a topic? i saw nothing in the agenda | 14:32 |
| mlavalle | yes, I have 2 or 3 things regarding accessing metadat over IPv6 only network: | 14:32 |
| haleyb | mlavalle: sure, go ahead | 14:32 |
| mlavalle | 1) This patch is ready to go https://review.opendev.org/c/openstack/neutron/+/922264 | 14:32 |
| mlavalle | 2) To verify the previous point I implemented a neutron-tempest-plugin test case: https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/925928 | 14:33 |
| mlavalle | 3) This test case now succeeds with the ML2/OVN backend. However, I discovered it fails with ML2/OVS and ML2/LB so I filed https://bugs.launchpad.net/neutron/+bug/2076916 | 14:35 |
| mlavalle | 4) I'm going to skip the test case for ML2/OVS and ML2/LB while I fix the new bug. So the test patch will be ready for reviws later today | 14:36 |
| mlavalle | that's all | 14:36 |
| haleyb | mlavalle: thanks for fixing that and finding the other bug, i could have sworn it worked when it originally merged | 14:37 |
| mlavalle | honestly, yesterday I thought it was somwething with my test case | 14:38 |
| mlavalle | it took me a while to convince myself that it was otherwise | 14:38 |
| haleyb | makes me think going with the link-local was a bad idea, since i'm assuming scope had to do with some of it, the %eth0 | 14:39 |
| mlavalle | the other thing that I want to mention is that there is also a bug in cloud-init | 14:39 |
| mlavalle | it doesn't know how to handle the %<interface> stuff | 14:40 |
| mlavalle | there is already a fix in cloud-init upstream | 14:40 |
| ykarel | https://github.com/canonical/cloud-init/commit/2ea276548e05dd1ee465bb71fc4c0f5305ff0bf4 | 14:41 |
| mlavalle | but it is cloud-init version 24.1.3-0ubuntu-20,04.5 | 14:41 |
| mlavalle | ykarel: yeap, that one | 14:42 |
| mlavalle | so even after we fix Neutron, we need the cloud-init fix if we want VM's to actually be able to boot with ipv6 only networks | 14:42 |
| haleyb | and i'm assuming cirros is using an old version of cloud-init? | 14:43 |
| mlavalle | the n-t-p test case I created uses a shell script injected with a config drive | 14:44 |
| mlavalle | the image I'm using is ubuntu | 14:44 |
| mlavalle | which has the old cloud-init version | 14:44 |
| mlavalle | ok, I think that is the complete picture of the metadata + ipv6 wreck | 14:45 |
| haleyb | :( | 14:45 |
| mlavalle | the thing is we had never tested metadat over ipv6 only networks before | 14:46 |
| mlavalle | that is why it is surfacing now | 14:46 |
| mlavalle | that we are actually testing | 14:46 |
| haleyb | i'm guessing when we tested manually we always used a %eth0 and claimed victory | 14:47 |
| mlavalle | and if was always a dual network with ipv4 | 14:47 |
| mlavalle | it was^^^ | 14:47 |
| haleyb | right. don't make me cry IPv6 tears they're twice as big :) | 14:48 |
| mlavalle | LOL | 14:48 |
| ykarel | :D | 14:48 |
| haleyb | IETF dork joke alert | 14:48 |
| mlavalle | I'll tell it to my wife and I'll get a dismissive stare | 14:49 |
| haleyb | i guess they're 4 times as big depends on what you're counting | 14:49 |
| haleyb | header versus address | 14:49 |
| haleyb | but i digress | 14:49 |
| haleyb | any other topics for discussion? | 14:50 |
| mlavalle | none from me | 14:50 |
| haleyb | otherwise we have the CI meeting in 10 minutes, online this time i believe | 14:50 |
| haleyb | #endmeeting | 14:51 |
| opendevmeet | Meeting ended Tue Aug 13 14:51:06 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:51 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/networking/2024/networking.2024-08-13-14.00.html | 14:51 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/networking/2024/networking.2024-08-13-14.00.txt | 14:51 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/networking/2024/networking.2024-08-13-14.00.log.html | 14:51 |
| haleyb | have a good week everyone | 14:51 |
| lajoskatona | haleyb, ykarel: I can't join today the CI meeting | 14:51 |
| ykarel | yes ci meeting over IRC in 9 minutes | 14:51 |
| ykarel | lajoskatona, ack | 14:51 |
| ykarel | Meeting 13.08.2024 | 15:01 |
| ykarel | #startmeeting neutron_ci | 15:01 |
| opendevmeet | Meeting started Tue Aug 13 15:01:34 2024 UTC and is due to finish in 60 minutes. The chair is ykarel. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:01 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:01 |
| opendevmeet | The meeting name has been set to 'neutron_ci' | 15:01 |
| ykarel | Ping list: bcafarel, lajoskatona, mlavalle, mtomaska, ralonsoh, ykarel, jlibosva, elvira | 15:01 |
| mlavalle | \o | 15:01 |
| haleyb | o/ | 15:01 |
| bcafarel | o/ | 15:02 |
| ykarel | i think we can start as couple of people are out on PTOs | 15:04 |
| ykarel | #topic Actions from previous meetings | 15:04 |
| ykarel | lajoskatona to check failures with test_list_pagination_page_reverse_with_href_links | 15:04 |
| ykarel | lajoskatona reported https://bugs.launchpad.net/neutron/+bug/2076328 | 15:04 |
| ykarel | i pushed a patch for this but would need more work, so will check tomorrow | 15:05 |
| ykarel | #topic Stable branches | 15:05 |
| ykarel | bcafarel any update here? | 15:05 |
| bcafarel | all good from what I saw! | 15:06 |
| ykarel | ok thx for the update, there are some periodic failures which we can see laters | 15:06 |
| ykarel | #topic Stadium projects | 15:07 |
| ykarel | rest all green apart from sfc https://bugs.launchpad.net/neutron/+bug/2068727 | 15:07 |
| ykarel | lajoskatona is already looking into it | 15:07 |
| ykarel | #topic Rechecks | 15:07 |
| ykarel | it was not bad in this too, just 1/7 bare rechecks | 15:08 |
| ykarel | let's check failures now | 15:08 |
| ykarel | #topic Rally jobs | 15:08 |
| ykarel | ovs-rally job is broken in master https://bugs.launchpad.net/oslo.serialization/+bug/2076430 | 15:08 |
| ykarel | Fixes proposed but that may time to be released, so temporary switching non-voting with https://review.opendev.org/c/openstack/neutron/+/926196 | 15:09 |
| ykarel | please review | 15:09 |
| ykarel | #topic Periodic | 15:09 |
| ykarel | postgres job failures https://bugs.launchpad.net/neutron/+bug/2072567 | 15:09 |
| ykarel | for this neutron-lib fixes are backported and released | 15:09 |
| ykarel | Would also needs https://review.opendev.org/c/openstack/neutron/+/923931 and backports | 15:10 |
| ykarel | please have a look at this too | 15:10 |
| ykarel | 9-stream jobs broken with recent devstack change | 15:10 |
| ykarel | https://zuul.openstack.org/builds?job_name=neutron-ovn-tempest-ovs-release-fips&project=openstack%2Fneutron&branch=master&skip=0 | 15:10 |
| ykarel | potential fixes https://review.opendev.org/q/topic:%22supportcheck%22 | 15:10 |
| ykarel | that's it on failures | 15:11 |
| ykarel | #topic Grafana | 15:11 |
| ykarel | https://grafana.opendev.org/d/f913631585/neutron-failure-rate | 15:11 |
| ykarel | let's have a quick look on it | 15:11 |
| ykarel | overall looks good, there are some spikes in check, ovs-rally known and other looks patch specifics. | 15:14 |
| ykarel | anything to add? | 15:14 |
| ykarel | will take it as all good :) | 15:15 |
| ykarel | #topic On Demand | 15:15 |
| bcafarel | overall looks good to me too | 15:15 |
| ykarel | anything else you would like to raise apart from topics? | 15:16 |
| mlavalle | not from me | 15:16 |
| haleyb | nothing here | 15:16 |
| bcafarel | same | 15:16 |
| ykarel | thx all for joining, have a nice rest of the week !! | 15:16 |
| ykarel | #endmeeting | 15:16 |
| opendevmeet | Meeting ended Tue Aug 13 15:16:49 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:16 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/neutron_ci/2024/neutron_ci.2024-08-13-15.01.html | 15:16 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/neutron_ci/2024/neutron_ci.2024-08-13-15.01.txt | 15:16 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/neutron_ci/2024/neutron_ci.2024-08-13-15.01.log.html | 15:16 |
| mlavalle | \o | 15:16 |
| bcafarel | o/ | 15:17 |
| ykarel | mlavalle, can you check https://review.opendev.org/c/openstack/neutron/+/926196 | 15:51 |
| ykarel | and also https://review.opendev.org/c/openstack/neutron/+/923931 | 15:51 |
| opendevreview | yatin proposed openstack/neutron-tempest-plugin master: Filter resources in pagination tests to avoid random failures https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/926201 | 16:16 |
| *** gibi_ is now known as gibi | 16:27 | |
| mlavalle | ykarel: done | 17:57 |
| opendevreview | Miguel Lavalle proposed openstack/neutron-tempest-plugin master: Test metadata query over IPv6 only network https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/925928 | 19:56 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!