| opendevreview | Merged openstack/neutron stable/2023.1: Add scope ID to the "GROUP BY" clause in ``get_scoped_floating_ips`` https://review.opendev.org/c/openstack/neutron/+/892599 | 02:39 |
|---|---|---|
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/yoga: Set trunk parent port as access port in ovs to avoid loop https://review.opendev.org/c/openstack/neutron/+/907174 | 06:32 |
| ralonsoh | slaweq, good morning! If you have 1 min: https://review.opendev.org/c/openstack/neutron/+/903947/5 | 06:32 |
| ralonsoh | thanks! | 06:32 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Add ``subnetpool-prefix-ops`` extension to ML2/OVN mech driver https://review.opendev.org/c/openstack/neutron/+/906816 | 06:34 |
| ralonsoh | bcafarel, hello! Please, check https://review.opendev.org/q/I73b36ef161441b52922d888c11a144eafe8a7ed0 if you have 1 min | 06:45 |
| ralonsoh | thanks! | 06:45 |
| opendevreview | Lajos Katona proposed openstack/networking-bgpvpn master: Bump hacking https://review.opendev.org/c/openstack/networking-bgpvpn/+/907177 | 07:35 |
| slaweq | <ralonsoh> "slaweq, good morning! If you..." <- done | 08:51 |
| opendevreview | Takashi Kajinami proposed openstack/python-neutronclient master: Bump hacking https://review.opendev.org/c/openstack/python-neutronclient/+/906999 | 09:10 |
| opendevreview | Michel Nederlof proposed openstack/ovn-bgp-agent master: Fix address scope test and add address scope tests https://review.opendev.org/c/openstack/ovn-bgp-agent/+/907187 | 09:38 |
| opendevreview | Michel Nederlof proposed openstack/ovn-bgp-agent master: Fix address scope test and add address scope tests https://review.opendev.org/c/openstack/ovn-bgp-agent/+/907187 | 09:53 |
| opendevreview | Lajos Katona proposed openstack/networking-bgpvpn master: Bump hacking https://review.opendev.org/c/openstack/networking-bgpvpn/+/907177 | 10:04 |
| opendevreview | Lajos Katona proposed openstack/networking-bagpipe master: Bump hacking https://review.opendev.org/c/openstack/networking-bagpipe/+/907189 | 10:05 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: dhcp: fix usage of helper function to retrieve process name https://review.opendev.org/c/openstack/neutron/+/907191 | 10:48 |
| sahid | ralonsoh, lajoskatona o/ ...a quick one ^ if we can backport it until zed O:-) | 10:51 |
| sahid | wait... I should probably try to add a test for that | 10:52 |
| ralonsoh | sahid, why did you changed the process manager uuid from network.id to process id? | 10:57 |
| ralonsoh | https://review.opendev.org/c/openstack/neutron/+/840421/46/neutron/agent/linux/dhcp.py | 10:57 |
| ralonsoh | I don't see the rationale for that | 10:58 |
| opendevreview | Merged openstack/neutron master: [OVN] The OVN agent exposes the ``load_config`` method. https://review.opendev.org/c/openstack/neutron/+/903947 | 11:09 |
| opendevreview | Merged openstack/neutron master: [OVN] Add a start method to the OVN agent extensions https://review.opendev.org/c/openstack/neutron/+/903977 | 11:10 |
| opendevreview | Merged openstack/ovsdbapp master: Bump hacking https://review.opendev.org/c/openstack/ovsdbapp/+/906968 | 11:12 |
| sahid | ralonsoh: it's what is used when we do register | 11:18 |
| sahid | only using network.id is not enough since we have a dnsmask running per segment for a given network | 11:19 |
| ralonsoh | sahid, how many dnsmasq processes you have running per agent and node? | 11:22 |
| sahid | ralonsoh: one per network except for the network that are attached to multiple segments (multiple subnets) | 11:29 |
| sahid | in context of multisgement there si a dnsmask running for each segment | 11:30 |
| ralonsoh | sahid, but in the same host? | 11:32 |
| ralonsoh | a host can be connected to one single segment | 11:32 |
| SvenKieske | Hi, I can't change the status of this neutron bug from "won't fix", imho this at least needs some updated docs. I didn't know live migration with ovs is completely broken in neutron: https://bugs.launchpad.net/neutron/+bug/1970606 | 11:33 |
| sahid | ralonsoh: no no we have removed this limitation in zed for ovs agent, give me a moment I will share with you the spec | 11:35 |
| sahid | https://specs.openstack.org/openstack/neutron-specs/specs/zed/multiple-routed-provider-network-per-compute.html | 11:36 |
| sahid | SvenKieske: we are using live-migration with ovs intensivly, it's not completely broken, at least from our experience | 11:42 |
| SvenKieske | sahid: well if you read the bug report neutron apparently has no SLA with regards to how many packets might be dropped during live migration, but this is nowhere to be found in the docs, only in this obscure bug report. So all I'm asking would be to at least document that. I personally don't see a point advertising a feature and then saying "we can't promise it works" but ymmv | 11:48 |
| ralonsoh | SvenKieske, because live migration is not broken at all, there is no SLA for live migration, as I commented in the bug | 11:48 |
| ralonsoh | so your specific problem is that you have some network disruption during that process | 11:48 |
| ralonsoh | that is something expected, depending on multiple factors | 11:48 |
| ralonsoh | but saying that live migration is broken is false | 11:49 |
| SvenKieske | ralonsoh: that is not "a specific problem" but the only point for doing live migrations is to not have network disruptions. | 11:49 |
| ralonsoh | SvenKieske, sorry, that is not true | 11:49 |
| SvenKieske | to me it's true :) | 11:49 |
| ralonsoh | in a meeting now, please comment ion the bug | 11:49 |
| SvenKieske | and to many customers as well | 11:49 |
| SvenKieske | maybe someone can explain to me a usecase for live migration that is not served by cold migration if I can tolerate network disruptions. *shrugs* I don't currently see any, but that might be just my poor imagination. | 11:51 |
| SvenKieske | or maybe answer the reverse: what is the advantage of live migration in contrast to cold migration when it's not seamless network connections? | 11:52 |
| SvenKieske | okay, local batch jobs maybe. but I don't think that's the majority of users of such a feature. | 11:53 |
| SvenKieske | regardless of that: this is not documented anywhere, so I found that at least surprising. in my previous live migration tests also I had barely blips in network connectivity (somewhat short latency spikes maybe). | 11:54 |
| sahid | SvenKieske: there is a certain number of tweak nova is providing, there are related to libvirt to address cases where vm are intensivly using network during live migration | 11:54 |
| SvenKieske | so it works better than the "SLA" (whatever that means in an openstack context) | 11:54 |
| sahid | have you noticed them? | 11:54 |
| SvenKieske | you mean post-copy migration and slowing down of qemu etc. pp.? sure. | 11:55 |
| sahid | perhqps thqt cqn help | 11:55 |
| sahid | yes exactly | 11:55 |
| SvenKieske | my original issues was something quite different, I have a customer who needs to migrate from iptables firewall driver to ovs firewall driver and I wanted to know if there is the possibility of network interruptions during this process. then I was pointed at this bugreport which seems to indicate that packet delivery isn't guaranteed anyway. | 11:56 |
| opendevreview | Merged openstack/neutron master: [OVN] Add the bridge name and datapath type to the port VIF details https://review.opendev.org/c/openstack/neutron/+/903494 | 11:58 |
| sahid | Oh I see... so bascially I'm adding more confusion :-) | 11:58 |
| SvenKieske | not really, as I'm somewhat familiar with most of the live migration concepts and stuff. I'm not that deep into the neutron side of things but most things I discover I think are surprising and a little sad, like most IT is :) | 12:00 |
| SvenKieske | I mean most parts of this bug report seem to be only relevant to older openstack releases and seem to be fixed, if I follow enough links to different bug reports, so that's a plus. | 12:01 |
| SvenKieske | I guess `openflow_processed_per_port` was never defaulted to true later, was it? https://review.opendev.org/c/openstack/neutron/+/806246 seems like it would improve reliability a lot, from my superficial understanding | 12:03 |
| ralonsoh | no, this value is set to false by default | 12:07 |
| ralonsoh | this is because there are other deployments, with multuple ovs rules, that need to process these rules in batches | 12:07 |
| ralonsoh | of course, if you are migrating from iptables to ovs fw, that will speed up the FW setup in the destination host | 12:08 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-lib master: Skip the enforcer undefined rule check https://review.opendev.org/c/openstack/neutron-lib/+/906539 | 12:16 |
| sahid | ralonsoh: I have noticed you -1 on the change, is it the name of the helper function that is worry you? | 13:26 |
| sahid | because the function is basically returning the network_id as before | 13:27 |
| sahid | we needed some sort of salt in multisegment deployment to identify the different dnsmask process running for a given network | 13:27 |
| opendevreview | Lucas Alvares Gomes proposed openstack/ovn-bgp-agent master: Address the Load_Balancer's datapath_group column deprecation https://review.opendev.org/c/openstack/ovn-bgp-agent/+/904129 | 13:34 |
| opendevreview | Lajos Katona proposed openstack/tap-as-a-service master: Doc: add documentation for usage and flow examples for OVS https://review.opendev.org/c/openstack/tap-as-a-service/+/828382 | 13:37 |
| opendevreview | Lajos Katona proposed openstack/tap-as-a-service master: Doc: add documentation for usage and driver details for SRIOV driver https://review.opendev.org/c/openstack/tap-as-a-service/+/881807 | 13:37 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: dhcp: fix usage of helper function to retrieve process name https://review.opendev.org/c/openstack/neutron/+/907191 | 13:47 |
| haleyb | #startmeeting networking | 14:00 |
| opendevmeet | Meeting started Tue Jan 30 14:00:14 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 | 14:00 |
| mlavalle | \o | 14:00 |
| obondarev | o/ | 14:00 |
| lajoskatona | o/ | 14:00 |
| jlibosva | o/ | 14:00 |
| rubasov | o/ | 14:00 |
| ralonsoh | hello | 14:00 |
| slaweq | o/ | 14:01 |
| haleyb | ok, lets get started | 14:01 |
| haleyb | #topic announcements | 14:01 |
| bcafarel | o/ | 14:02 |
| frickler | \o | 14:02 |
| haleyb | #link https://releases.openstack.org/caracal/schedule.html | 14:02 |
| haleyb | Reminder we're in C-3, which ends week of Feb 26 | 14:02 |
| haleyb | Yoga "unmaintained" release patches up for review | 14:03 |
| haleyb | #link https://review.opendev.org/c/openstack/releases/+/906564 | 14:03 |
| haleyb | the series is linked from the topic there | 14:04 |
| *** ykarel__ is now known as ykarel | 14:04 | |
| haleyb | the ML email described the plans | 14:04 |
| frickler | also please don't merge any further changes to stable/yoga, else those will need updating | 14:04 |
| lajoskatona | there are zed release patches also | 14:04 |
| haleyb | #link https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/V7H6M5ANYOMFWSZ5BJSWAXVNQCUXI6D7/ | 14:04 |
| haleyb | frickler: right, that i wanted to discuss | 14:04 |
| lajoskatona | so if you have time check the zed patches also (ie.: https://review.opendev.org/c/openstack/releases/+/906863 ) | 14:05 |
| haleyb | #link https://review.opendev.org/q/project:openstack/neutron+branch:stable/yoga+status:open | 14:05 |
| ralonsoh | I'll abandon mines | 14:05 |
| haleyb | i think ihar proposed the other, i dropped a note there the other day | 14:06 |
| haleyb | so we are transitioning a lot of releases lately | 14:07 |
| ralonsoh | I'll ping him today | 14:07 |
| haleyb | lajoskatona: i will look at zed patches today | 14:08 |
| haleyb | #link https://review.opendev.org/q/project:openstack/neutron+branch:stable/zed+status:open | 14:08 |
| haleyb | i see just two | 14:08 |
| lajoskatona | haleyb: ok | 14:09 |
| ralonsoh | one merged today | 14:09 |
| haleyb | great | 14:09 |
| haleyb | any other releases i missed :) | 14:09 |
| haleyb | Xena, Wallaby, Victoria will transition to Unmaintained as well eventually fyi | 14:10 |
| haleyb | anyone have other announcements? | 14:10 |
| haleyb | #topic bugs | 14:11 |
| haleyb | amotoki was bug deputy last week, but i did not see an email | 14:11 |
| haleyb | i don't think there were many bugs, and they were all picked-up | 14:11 |
| mlavalle | and I'll take a look during my duty tour on last week's also | 14:12 |
| mlavalle | to make sure we are caught up | 14:13 |
| haleyb | mlavalle: ack, thanks | 14:13 |
| haleyb | i'll just mention one bug since i only quickly looked at it | 14:13 |
| haleyb | #link https://bugs.launchpad.net/neutron/+bug/2050236 | 14:13 |
| haleyb | Automatic L3 Agent Failover does not work for HA Routers | 14:13 |
| ralonsoh | I'm checking this one, in particular the sql query mentioned | 14:14 |
| ralonsoh | if I'm not wrong (I need to check that) | 14:14 |
| ralonsoh | this is an outerjoin with RouterExtraAttributes | 14:14 |
| ralonsoh | that means this query is getting all the routers (but I need to check twice) | 14:15 |
| haleyb | i believe it's true, they won't get rescheduled because there is >1 of them with l3-ha | 14:16 |
| haleyb | but i did not try and reproduce, just looked at code | 14:16 |
| haleyb | #link https://github.com/openstack/neutron/blob/master/neutron/objects/l3agent.py#L55 | 14:17 |
| haleyb | ralonsoh: thanks for looking into the query though, i might have mis-understood | 14:18 |
| ralonsoh | I didn't run the code yet | 14:18 |
| opendevreview | Merged openstack/ovn-bgp-agent master: Check for networks on router port in match_fn https://review.opendev.org/c/openstack/ovn-bgp-agent/+/907075 | 14:18 |
| ralonsoh | so just (mis)reading the code | 14:18 |
| haleyb | any other bugs to discuss? | 14:19 |
| haleyb | and as mentioned, mlavalle is the bug deputy, next week will be ralonsoh - does that work for you both? | 14:20 |
| ralonsoh | perfect | 14:21 |
| mlavalle | ack. on it | 14:21 |
| haleyb | i almost forgot the bug count | 14:21 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: dhcp: ensure that cleaning DHCP process with one segment happens first https://review.opendev.org/c/openstack/neutron/+/905617 | 14:21 |
| haleyb | Current bug count this week: 758, down 2 from last week | 14:21 |
| haleyb | we've been around 760 most of the cycle | 14:22 |
| haleyb | #topic specs | 14:22 |
| haleyb | #link https://review.opendev.org/q/project:openstack%252Fneutron-specs+status:open | 14:22 |
| opendevreview | Luis Tomas Bolivar proposed openstack/ovn-bgp-agent stable/2023.2: Check for networks on router port in match_fn https://review.opendev.org/c/openstack/ovn-bgp-agent/+/906896 | 14:23 |
| ralonsoh | I think https://review.opendev.org/c/openstack/neutron-specs/+/891204 is ready | 14:23 |
| ralonsoh | and has several +2 (not only this PS but previous ones) | 14:24 |
| haleyb | ack, i was able to look at https://review.opendev.org/c/openstack/neutron-specs/+/899209 last week as well and seemed ready too, i did not look at the third yet | 14:24 |
| ralonsoh | (I have this one open, I'll finish today) | 14:24 |
| haleyb | ralonsoh: great, thanks! | 14:25 |
| haleyb | #topic community_goals | 14:25 |
| haleyb | we only have one goal this cycle regarding neutronclient deprecation | 14:26 |
| haleyb | #link https://review.opendev.org/c/openstack/horizon/+/891205 | 14:26 |
| haleyb | i have not been able to review recently | 14:26 |
| haleyb | lajoskatona: any progress with the integrated test there? | 14:27 |
| lajoskatona | No I had no time last week for it, I have to go back and check again with Horion team | 14:28 |
| haleyb | ok, thanks | 14:29 |
| haleyb | onto last topic | 14:29 |
| haleyb | #topic on_demand | 14:29 |
| haleyb | i had added a topic on a long-running patch series | 14:30 |
| haleyb | #link https://review.opendev.org/q/topic:2023-aa-l3-gw-multihoming | 14:30 |
| lajoskatona | +1 | 14:31 |
| haleyb | i've been trying to keep it up-to-date, still hoping to get it all merged this cycle | 14:31 |
| haleyb | fnordahl: do you want me to do another rebase on that? | 14:31 |
| * haleyb will try regardless | 14:32 | |
| haleyb | does anyone have something to discuss? | 14:32 |
| haleyb | else will see you at CI meeting in :30 | 14:33 |
| lajoskatona | haleyb, ykarel: I can't join today the CI meeting :-( | 14:33 |
| mlavalle | \o | 14:34 |
| ykarel | lajoskatona, ack | 14:34 |
| mlavalle | see you at ci meeting | 14:34 |
| haleyb | ack, ok we are done then, have a good week everyone | 14:34 |
| haleyb | #endmeeting | 14:35 |
| opendevmeet | Meeting ended Tue Jan 30 14:35:02 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:35 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/networking/2024/networking.2024-01-30-14.00.html | 14:35 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/networking/2024/networking.2024-01-30-14.00.txt | 14:35 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/networking/2024/networking.2024-01-30-14.00.log.html | 14:35 |
| ralonsoh | byebye | 14:35 |
| slaweq | o/ | 14:35 |
| lajoskatona | o/ | 14:37 |
| opendevreview | Merged openstack/ovn-bgp-agent stable/2023.2: Check for networks on router port in match_fn https://review.opendev.org/c/openstack/ovn-bgp-agent/+/906896 | 14:56 |
| ykarel | #startmeeting neutron_ci | 15:00 |
| opendevmeet | Meeting started Tue Jan 30 15:00:51 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 |
| ralonsoh | hi | 15:01 |
| mlavalle | \o | 15:01 |
| ykarel | ping bcafarel, lajoskatona, mlavalle, mtomaska, ralonsoh, ykarel, jlibosva, elvira | 15:01 |
| haleyb | o/ | 15:01 |
| mlavalle | is it an irc meeting today? | 15:01 |
| slaweq | o/ | 15:01 |
| ykarel | yes | 15:01 |
| mlavalle | ack | 15:01 |
| bcafarel | o/ | 15:02 |
| ykarel | let's start with topics | 15:03 |
| ykarel | #topic Actions from previous meetings | 15:03 |
| ykarel | ralonsoh to check FOREIGN KEY constraint failed issue in functional test | 15:03 |
| ykarel | Logs: https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_2e7/903977/7/check/neutron-functional-with-uwsgi/2e76304/testr_results.html | 15:03 |
| ykarel | This is a sqlite error: the port binding register is created in the same trasaction as the port; the port ID must be on the transaction commited actions. | 15:03 |
| ykarel | This is not an issue related to the patch #903977 nor OVN. | 15:03 |
| ralonsoh | ^^ right, that was an issue with sqlite only | 15:04 |
| ralonsoh | not the Neutron code | 15:04 |
| ykarel | k thx for checking, let's watch if it reproduces again | 15:04 |
| ykarel | next one | 15:04 |
| ykarel | ralonsoh to check https://bugs.launchpad.net/neutron/+bug/1955008 | 15:04 |
| ralonsoh | --> https://review.opendev.org/c/openstack/neutron/+/906474 | 15:05 |
| ykarel | thx | 15:05 |
| slaweq | ralonsoh I just approved it | 15:05 |
| ralonsoh | thanks | 15:05 |
| ykarel | alonsoh to validate fullstack jobs with revert of https://review.opendev.org/q/I442587e7ef55917f4ea873e190bf8afbc0e911e1 | 15:06 |
| ralonsoh | --> https://review.opendev.org/q/topic:%22bug/1961740%22: reverts proposed and merged | 15:06 |
| ykarel | ralonsoh* | 15:06 |
| ykarel | thx, btw i saw similar failure in stable/2023.1 too | 15:06 |
| ralonsoh | really? pffff | 15:06 |
| ykarel | can look later in failures topic | 15:07 |
| ralonsoh | so I don't think this patch is good enough | 15:07 |
| ralonsoh | (well, at least for fullstack tests...) | 15:07 |
| ykarel | yes not seeing it in master/stable2023.2 | 15:08 |
| opendevreview | Lajos Katona proposed openstack/tap-as-a-service master: WIP: Documentation for tap-mirrors https://review.opendev.org/c/openstack/tap-as-a-service/+/907220 | 15:08 |
| ykarel | may be due to those tests are trimmed in master | 15:08 |
| ralonsoh | could be, right | 15:08 |
| ykarel | not sure if those patches from slaweq also in stable/2023.2 | 15:08 |
| ralonsoh | we can backport the fullstack optimizations if needed | 15:09 |
| ralonsoh | I think that could benefit stable branches CI too | 15:09 |
| bcafarel | +1 | 15:09 |
| ralonsoh | (I'll add this to my todo list) | 15:09 |
| ykarel | ok thx | 15:09 |
| ykarel | next one on me | 15:09 |
| ykarel | ykarel to check enabling debug logs for ovs for log size | 15:09 |
| ykarel | so i checked it with dnm patch https://review.opendev.org/c/openstack/neutron/+/907037 | 15:10 |
| ykarel | functional job -> 2MB --> 130 MB | 15:10 |
| ykarel | fullstack job -> 3MB --> 70 MB | 15:10 |
| ralonsoh | (zuul folks are going to kill us...) | 15:10 |
| ykarel | increase is significant so doesn't seem good to enable it for all such runs | 15:11 |
| slaweq | yes, it's better to not do that | 15:12 |
| ykarel | ok | 15:12 |
| ralonsoh | but we can document it for debugging purposes | 15:12 |
| ykarel | ykarel to check if internal ports removal/addition have impact on bridge add/delete | 15:12 |
| ykarel | ralonsoh, okk will check and report doc bug for it | 15:13 |
| ralonsoh | cool | 15:13 |
| ykarel | #action ykarel to report doc bug for ovs enable debug | 15:13 |
| slaweq | what if we would tar.gz those logs? | 15:14 |
| slaweq | it should be much smaller, no? | 15:14 |
| ralonsoh | 10-20 times smaller | 15:14 |
| ykarel | yes should be smaller, need to check how much exactly | 15:14 |
| slaweq | maybe worth to do one more experiment then? | 15:15 |
| ykarel | ok will do, but iirc if we tar we need to download those to check content or those still browsable? | 15:15 |
| slaweq | yes, we will need to download to check content | 15:16 |
| ykarel | ohkk | 15:16 |
| bcafarel | hmm if it is per single file gz, it should be browsable (may need some tweaks on server side for content-type) | 15:17 |
| bcafarel | tarball of all files indeed it would need download | 15:17 |
| ykarel | will update that patch to convert those to tar archive and confirm browsable aspects too | 15:17 |
| ykarel | bcafarel, thx will check | 15:17 |
| ykarel | ykarel to check if internal ports removal/addition have impact on bridge add/delete | 15:17 |
| ykarel | i was not able to reproduce that behavior with vsctl commands | 15:18 |
| ykarel | ovs command don't allow delete local internal port | 15:18 |
| ykarel | ovs-vsctl del-port test-br test-br | 15:18 |
| ykarel | fails as ovs-vsctl: cannot delete port test-br because it is the local port for bridge test-br (deleting this port requires deleting the entire bridge) | 15:18 |
| ykarel | so that add/delete/add of that internal port seems to be happing internally in ovs and it matches the shy ports issue we seen in past | 15:19 |
| ykarel | any idea to simulate this behavior? | 15:20 |
| slaweq | nope | 15:20 |
| ralonsoh | no sorry | 15:20 |
| ykarel | k thx let's move on | 15:21 |
| ykarel | #topic Stable branches | 15:21 |
| ykarel | bcafarel, any update | 15:21 |
| bcafarel | we already mentioned the revert for fullstack sleep, rest looks good | 15:21 |
| ykarel | thx | 15:21 |
| bcafarel | and with the unmaintained branches coming up we will have a shorter section soon :) | 15:21 |
| ykarel | cool :) | 15:22 |
| ykarel | #topic Stadium projects | 15:22 |
| ykarel | there was one failure in periodic | 15:22 |
| ykarel | for bgpvpn https://zuul.openstack.org/buildset/4bc10cd65b28461da28a76fa10553f07 | 15:22 |
| ykarel | Looks related to django update https://review.opendev.org/c/openstack/requirements/+/904202 | 15:23 |
| ykarel | lajos is out, so will check if there is already some fixes for it and create a lp | 15:23 |
| ykarel | #action ykarel to check and report lp bug for bgpvpn failures | 15:23 |
| opendevreview | Merged openstack/neutron stable/2023.2: Set trunk parent port as access port in ovs to avoid loop https://review.opendev.org/c/openstack/neutron/+/907142 | 15:24 |
| ykarel | #topic Rechecks | 15:24 |
| ykarel | overall good here, there was 0 bare rechecks out of 18 rechecks | 15:24 |
| ykarel | there were some known issues which could have resulted into higher rechecks | 15:24 |
| ykarel | #topic Unit tests | 15:25 |
| ykarel | https://launchpad.net/bugs/2051501 | 15:25 |
| ykarel | already Fixed with https://review.opendev.org/c/openstack/neutron/+/907040 | 15:25 |
| ykarel | thx ralonsoh | 15:25 |
| ralonsoh | yw | 15:25 |
| ykarel | #topic fullstack/functional | 15:25 |
| ykarel | https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_1bf/892599/4/gate/neutron-fullstack-with-uwsgi/1bf74e0/testr_results.html | 15:25 |
| ykarel | ^ is the failure i mentioned before in 2023.1 | 15:26 |
| ykarel | Looks same as for which zed/yoga revert https://review.opendev.org/q/topic:%22bug/1961740%22 | 15:26 |
| ralonsoh | no no, the error was different | 15:26 |
| ralonsoh | this is not the test case that was failing because of this patch | 15:26 |
| ralonsoh | (sorry, yes...) | 15:27 |
| ykarel | mmm but commit message says it | 15:27 |
| ralonsoh | my bad, I had other bug in mind | 15:27 |
| ykarel | ok :) | 15:27 |
| ykarel | so for fullstack we go with backport of optimization patches or do reverts | 15:28 |
| ralonsoh | let me push the backports tomorrow | 15:28 |
| ralonsoh | up to zed | 15:28 |
| ykarel | because i noticed higher fullstack failures in stable since the revert, so likely the failures for which those sleep patches pushed reappear? | 15:29 |
| ykarel | ok trimming those sounds better, thx | 15:29 |
| ykarel | #topic Periodic | 15:29 |
| ykarel | https://launchpad.net/bugs/2051501 | 15:30 |
| ykarel | this already Fixed with https://review.opendev.org/c/openstack/neutron/+/907040 | 15:30 |
| ykarel | https://zuul.openstack.org/builds?job_name=neutron-ovn-tempest-slow&project=openstack%2Fneutron&branch=master&skip=0 | 15:30 |
| ykarel | Looks related to recent changes https://review.opendev.org/q/topic:%222023-aa-l3-gw-multihoming%22+status:merged+project:openstack/neutron | 15:30 |
| ykarel | any volunteer to check/confirm these failures? | 15:31 |
| ralonsoh | I can check it | 15:31 |
| ykarel | thx | 15:31 |
| ykarel | #action ralonsoh to check failures with neutron-ovn-tempest-slow job in master | 15:31 |
| ykarel | https://zuul.openstack.org/builds?job_name=neutron-fullstack-with-uwsgi-fips&project=openstack%2Fneutron&branch=master&skip=0 | 15:31 |
| ykarel | Fails at dnf install -y kernel-modules-extra-5.14.0-410.el9.x86_64 | 15:32 |
| ykarel | it's happening due to Outdated infra mirrors http://mirror.iad.rax.opendev.org/centos-stream/9-stream/BaseOS/x86_64/os/Packages/ | 15:32 |
| ykarel | rpm exists at centos mirrors https://mirror.stream.centos.org/9-stream/BaseOS/x86_64/os/Packages/ | 15:32 |
| ralonsoh | then should we switch to this mirror? | 15:33 |
| ykarel | i think need to fix infra mirrors | 15:33 |
| ralonsoh | yeah, right, much better | 15:33 |
| ykarel | will check tomorrow with infra on this, likely the mirror which they follow is outdated | 15:33 |
| opendevreview | Merged openstack/neutron stable/2023.1: Set trunk parent port as access port in ovs to avoid loop https://review.opendev.org/c/openstack/neutron/+/907143 | 15:35 |
| ykarel | #action ykarel to reach out to infra for outdated mirror if it's not resolved in next periodic run | 15:35 |
| ykarel | #topic Grafana | 15:35 |
| ykarel | https://grafana.opendev.org/d/f913631585/neutron-failure-rate | 15:35 |
| ykarel | all good in gate queue, some spikes in check around weekend which we normally see due to less runs | 15:37 |
| ykarel | a spike is for sqlalchemy master job which already got fixed | 15:38 |
| ykarel | there are some recent spikes for tox-pep8 | 15:39 |
| ykarel | i haven't noticed any consistent error though, so may be patch specific | 15:39 |
| slaweq | but it's 1 failure out of 4 if I'm not mistaken for the pep8 | 15:40 |
| slaweq | so nothing really serious IMO | 15:40 |
| ykarel | yes right, then we are good | 15:41 |
| ykarel | #topic On Demand | 15:41 |
| ykarel | anything else you would like to raise here? | 15:41 |
| ralonsoh | nothing from me | 15:41 |
| bcafarel | all good here too | 15:41 |
| slaweq | nothing here | 15:41 |
| mlavalle | nothing from me | 15:42 |
| mlavalle | either | 15:42 |
| ykarel | k thanks everyone for joining | 15:42 |
| ykarel | let's close and have everyone 18 minutes back | 15:42 |
| ykarel | #endmeeting | 15:42 |
| opendevmeet | Meeting ended Tue Jan 30 15:42:33 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:42 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/neutron_ci/2024/neutron_ci.2024-01-30-15.00.html | 15:42 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/neutron_ci/2024/neutron_ci.2024-01-30-15.00.txt | 15:42 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/neutron_ci/2024/neutron_ci.2024-01-30-15.00.log.html | 15:42 |
| ralonsoh | bye! | 15:42 |
| slaweq | o/ | 15:42 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2023.2: [Fullstack] Remove unnecessary tests https://review.opendev.org/c/openstack/neutron/+/907223 | 15:42 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2023.2: [Fullstack] Remove test_port_shut_down module https://review.opendev.org/c/openstack/neutron/+/907224 | 15:42 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2023.2: [Fullstack] Remove SecurityGroupRulesTest.test_normalized_cidr_in_rule test https://review.opendev.org/c/openstack/neutron/+/907225 | 15:42 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2023.2: [Fullstack] Consolidate segmentation_id update tests into single test https://review.opendev.org/c/openstack/neutron/+/907226 | 15:42 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/zed: [Fullstack] Remove unnecessary tests https://review.opendev.org/c/openstack/neutron/+/907227 | 15:43 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/zed: [Fullstack] Remove test_port_shut_down module https://review.opendev.org/c/openstack/neutron/+/907228 | 15:43 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/zed: [Fullstack] Remove SecurityGroupRulesTest.test_normalized_cidr_in_rule test https://review.opendev.org/c/openstack/neutron/+/907229 | 15:43 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/zed: [Fullstack] Consolidate segmentation_id update tests into single test https://review.opendev.org/c/openstack/neutron/+/907230 | 15:43 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2023.1: [Fullstack] Remove unnecessary tests https://review.opendev.org/c/openstack/neutron/+/907231 | 15:43 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2023.1: [Fullstack] Remove test_port_shut_down module https://review.opendev.org/c/openstack/neutron/+/907232 | 15:43 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2023.1: [Fullstack] Remove SecurityGroupRulesTest.test_normalized_cidr_in_rule test https://review.opendev.org/c/openstack/neutron/+/907233 | 15:43 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2023.1: [Fullstack] Consolidate segmentation_id update tests into single test https://review.opendev.org/c/openstack/neutron/+/907234 | 15:43 |
| bcafarel | serial backporter :) | 15:43 |
| ralonsoh | I had that prepared hehehe | 15:43 |
| ralonsoh | I didn't push it during the meeting not to dirt the logs | 15:44 |
| ykarel | ralonsoh you used topic backport feature in gerrit? | 15:44 |
| ykarel | for these above backports | 15:44 |
| ralonsoh | I used the same one used before | 15:45 |
| ralonsoh | git fetch https://review.opendev.org/openstack/neutron refs/changes/74/906474/1 && git cherry-pick FETCH_HEAD | 15:45 |
| ralonsoh | sorry | 15:45 |
| ralonsoh | fullstack-cleanup | 15:45 |
| ykarel | ack | 15:45 |
| ralonsoh | ahh no no | 15:46 |
| ralonsoh | sorry, I didn't use gerrit | 15:46 |
| ralonsoh | I did it manually | 15:46 |
| ykarel | ok | 15:47 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2023.2: [OVN][FT] Retry in case of timeout when executing "ovsdb-client". https://review.opendev.org/c/openstack/neutron/+/907235 | 15:47 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2023.1: [OVN][FT] Retry in case of timeout when executing "ovsdb-client". https://review.opendev.org/c/openstack/neutron/+/907236 | 15:47 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/zed: [OVN][FT] Retry in case of timeout when executing "ovsdb-client". https://review.opendev.org/c/openstack/neutron/+/907237 | 15:47 |
| tkajinam | ralonsoh, `git review -x <id>` or `git review -X <id>` might be interesting to you | 16:00 |
| ralonsoh | checking that now | 16:01 |
| tkajinam | these are what I happened to learn today :-) | 16:04 |
| tkajinam | git checkout origin/stable/2023.2; git checkout -b tmp; git review -X 902633; git review -t fullstack-cleanup -f | 16:05 |
| opendevreview | Rodrigo Barbieri proposed openstack/neutron stable/ussuri: [OVN] Handle OVN agents when "Chassis" register is deleted https://review.opendev.org/c/openstack/neutron/+/907239 | 16:05 |
| tkajinam | would reproduce https://review.opendev.org/c/openstack/neutron/+/907223 | 16:05 |
| tkajinam | and the above fetch + cherry-pick may be replaced by `git review -x 906474` iiuc | 16:08 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: dhcp: fix usage of helper function to retrieve process name https://review.opendev.org/c/openstack/neutron/+/907191 | 16:19 |
| opendevreview | Merged openstack/neutron stable/2023.2: If method ``set_netns`` fails, restore previous device namespace https://review.opendev.org/c/openstack/neutron/+/906856 | 16:39 |
| opendevreview | Merged openstack/neutron stable/2023.1: If method ``set_netns`` fails, restore previous device namespace https://review.opendev.org/c/openstack/neutron/+/906855 | 16:39 |
| opendevreview | Merged openstack/neutron master: [OVN][FT] Retry in case of timeout when executing "ovsdb-client". https://review.opendev.org/c/openstack/neutron/+/906474 | 16:39 |
| opendevreview | Merged openstack/neutron stable/2023.1: For hosts in DVR mode, only fetch bound FIPs https://review.opendev.org/c/openstack/neutron/+/891398 | 18:13 |
| opendevreview | Lajos Katona proposed openstack/tap-as-a-service master: Bump hacking https://review.opendev.org/c/openstack/tap-as-a-service/+/907249 | 19:08 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: dhcp: fix dhcp cleaning stale devices process when enable action https://review.opendev.org/c/openstack/neutron/+/907250 | 19:39 |
| opendevreview | Brian Haley proposed openstack/neutron master: [ovn] Make scheduling of unhosted gateways aware of current transaction https://review.opendev.org/c/openstack/neutron/+/874760 | 20:07 |
| opendevreview | Brian Haley proposed openstack/neutron master: [ovn] Make scheduling of unhosted gateways aware of current transaction https://review.opendev.org/c/openstack/neutron/+/874760 | 21:42 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!