Tuesday, 2024-09-03

*** elodilles_pto is now known as elodilles07:26
opendevreviewRodolfo Alonso proposed openstack/neutron master: DNM == Testing postgres error retrieving resource "standard_attr"  https://review.opendev.org/c/openstack/neutron/+/92778908:17
opendevreviewSlawek Kaplonski proposed openstack/neutron master: Add trusted vif api extension for the port  https://review.opendev.org/c/openstack/neutron/+/92606808:23
slaweqlajoskatona ykarel hi, can one of you check https://review.opendev.org/c/openstack/neutron/+/927707 ? Thx in advance08:24
opendevreviewMichel Nederlof proposed openstack/ovn-bgp-agent master: Fix race condition where an event could unexpose a network during sync  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/92779108:44
opendevreviewSlawek Kaplonski proposed openstack/neutron master: Remove default override for RBAC config options  https://review.opendev.org/c/openstack/neutron/+/92608508:55
opendevreviewMichel Nederlof proposed openstack/ovn-bgp-agent master: Fix race condition where an event could unexpose a network during sync  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/92779109:03
opendevreviewMichel Nederlof proposed openstack/ovn-bgp-agent master: Fix race condition where an event could unexpose a network during sync  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/92779109:04
opendevreviewMichel Nederlof proposed openstack/ovn-bgp-agent master: Fix race condition where an event could unexpose a network during sync  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/92779109:12
opendevreviewRodolfo Alonso proposed openstack/neutron master: [SR-IOV] The port status=DOWN has precedence in the VF link status  https://review.opendev.org/c/openstack/neutron/+/92779509:32
opendevreviewRodolfo Alonso proposed openstack/neutron stable/2024.1: DNM == Testing postgres error retrieving resource "standard_attr"  https://review.opendev.org/c/openstack/neutron/+/92779709:39
opendevreviewSlawek Kaplonski proposed openstack/neutron master: Add trusted vif api extension for the port  https://review.opendev.org/c/openstack/neutron/+/92606810:06
ralonsohhaleyb, lajoskatona hello! If you have 2 mins: https://review.opendev.org/c/openstack/neutron/+/92779510:19
ralonsohI would like to backport this patch too10:19
opendevreviewRodolfo Alonso proposed openstack/neutron master: Protect the "standardattr" retrieval from a concurrent deletion  https://review.opendev.org/c/openstack/neutron/+/92780110:34
ralonsohykarel, ^ please take a look10:35
ykarelralonsoh, ack10:42
ralonsohslaweq, hi! If you have less than 1 min: https://review.opendev.org/c/openstack/neutron/+/92722112:48
slaweqralonsoh done12:56
ralonsohthanks!12:56
*** obondarev_ is now known as obondarev13:03
opendevreviewRodolfo Alonso proposed openstack/neutron stable/2024.1: [SR-IOV] The port status=DOWN has precedence in the VF link status  https://review.opendev.org/c/openstack/neutron/+/92781513:12
opendevreviewMichel Nederlof proposed openstack/ovn-bgp-agent master: Fix cleanup of rules per evpn device  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/92781613:12
opendevreviewRodolfo Alonso proposed openstack/neutron stable/2023.2: [SR-IOV] The port status=DOWN has precedence in the VF link status  https://review.opendev.org/c/openstack/neutron/+/92781713:12
opendevreviewRodolfo Alonso proposed openstack/neutron stable/2023.1: [SR-IOV] The port status=DOWN has precedence in the VF link status  https://review.opendev.org/c/openstack/neutron/+/92781813:13
opendevreviewMerged openstack/neutron master: User defined router flavor driver with no LSP  https://review.opendev.org/c/openstack/neutron/+/91780013:15
opendevreviewRodolfo Alonso proposed openstack/neutron-lib master: Add new API extension ``uplink-status-propagation-updatable``  https://review.opendev.org/c/openstack/neutron-lib/+/92782013:37
opendevreviewMerged openstack/neutron master: [SR-IOV] The port status=DOWN has precedence in the VF link status  https://review.opendev.org/c/openstack/neutron/+/92779513:56
haleyb#startmeeting networking14:00
opendevmeetMeeting started Tue Sep  3 14:00:38 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
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.14:00
opendevmeetThe meeting name has been set to 'networking'14:00
haleybPing list: bcafarel, elvira, frickler, mlavalle, mtomaska, obondarev, slaweq, tobias-urdin, ykarel, lajoskatona, jlibosva, averdagu, amotoki, haleyb, ralonsoh14:00
ihrachyso/14:00
obondarevo/14:00
ralonsohhello14:00
mlavalle\o14:01
lajoskatonao/14:01
slaweqo/14:01
rubasovo/14:01
haleyb#topic announcements14:02
haleybWe are now in Dalmatian release week (R - 4)14:02
haleybIf there are any featureful requests that need merging we should discuss as an exception14:03
haleybcut/paste from elod's email14:03
haleybFocus should be on finding and fixing release-critical bugs, so that14:03
haleybrelease candidates and final versions of the 2024.2 Dalmatian14:03
haleybdeliverables can be proposed, well ahead of the final 2024.2 Dalmatian14:03
haleybrelease date.14:03
haleybI am fine landing any other bug fixes as well14:04
haleybstable/2024.2 branches should be created soon for all not-already-branched libraries14:05
haleybRC1 deadline: September 13th, 2024 (R-3 week)14:05
haleybFinal RC deadline: September 26th, 2024 (R-1 week)14:05
haleybFinal 2024.2 Dalmatian release: October 2nd, 202414:06
haleybthat's a lot of info, any questions or comments on the schedule?14:06
ralonsohhttps://releases.openstack.org/dalmatian/schedule.html#d-cycle-highlights14:06
ralonsohI think you need to send this patch14:07
haleybralonsoh: thanks for reminding me, i will propose it today or tomorrow. if anyone has a highlight feel free to ping me else i'll go through the commit messages, etc14:09
haleybas far as reviews, please add completed ones to the review dashboard by adding an RP +114:10
haleybthere are a few on the list at the moment14:10
haleybas it was a US holiday yesterday i have not looked since Friday14:11
lajoskatona+114:11
lajoskatonaI checked some from Liu but this week (and last one) I had no time for them14:11
lajoskatonabut those are big features14:12
haleyblajoskatona: thanks for checking those, i had been trying to make time for the metadata path extension changes as well14:12
haleyb#link https://review.opendev.org/q/topic:%22distributed_metadata_data_path%2214:13
haleybthe last three are the largest14:13
lajoskatonayes not easy ones14:14
ralonsohI have one from me (I've increased the RP because the n-lib patch is merged and the API ref released): https://review.opendev.org/c/openstack/neutron/+/92472414:15
ralonsohnot critical, of course14:15
slaweqalso https://review.opendev.org/q/topic:%22bug/2060916%22 are ready to review14:16
haleybonly other announcement from me today is i will be unavailable this afternoon, not feeling great at the moment and appointment later14:17
haleybalways seems to happen after a long weekend14:17
slaweqhaha, we call it hangover :P14:17
slaweqI also often have it after long weekend :D14:17
slaweqbut seriously, I hope You will feel better soon haleyb :)14:18
lajoskatona+114:18
opendevreviewIhar Hrachyshka proposed openstack/neutron-tempest-plugin master: Update snat_rules_apply_to_nested_networks with ovn details  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/92722014:19
haleybhah, you make a good point, but i did not drink at all yesterday...14:19
haleybany other announcements?14:20
opendevreviewRodolfo Alonso proposed openstack/neutron master: Neutron quota engine checks the resource usage by default  https://review.opendev.org/c/openstack/neutron/+/92672514:20
haleyb#topic bugs14:21
haleybmlavalle was the deputy last week, his report is at14:21
haleyb#link https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/ACLSJ2F6VBBBVOWDYNFZENSCEOTZACKO/14:21
haleybtwo of the high's have proposed fixes14:22
haleybthe other is rbac related14:22
ihrachysthe second is not a fix it seems14:22
haleyb#link https://bugs.launchpad.net/neutron/+bug/207851814:22
slaweqregarding designate scenario, I today proposed fix in designate devstack plugin14:22
ralonsohno, the second is not a fix but an extra check, the bug is still open14:23
slaweqI proposed patch https://review.opendev.org/c/openstack/designate/+/927792 which is tested in the https://review.opendev.org/c/openstack/neutron/+/926085 and neutron-tempest-plugin-designate-scenario job is passing now: https://zuul.opendev.org/t/openstack/build/092e8d0e2af24124bb873a4b9dc592aa14:23
slaweqso now johnsom and other folks from designate team will need to review it14:23
johnsomack14:23
slaweqit don't seems to be issue in neutron really, so I marked it as invalid for neutron14:24
slaweqthx johnsom 14:24
haleybslaweq: ah, thanks for tracking that down, we should make the job voting again once it merges14:24
ihrachysafaiu we disabled the job due to this. we'll need to re-enable when it's fixed in designate?14:24
slaweqyes, I will propose revert of that patch which made it non-voting14:25
haleybihrachys: yes, we made it non-voting to land the requirements bump14:25
opendevreviewSlawek Kaplonski proposed openstack/neutron-tempest-plugin master: Revert "Make neutron-tempest-plugin-designate-scenario non voting"  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/92783414:26
haleybralonsoh: and getting back to the functional failure, i see that is assigned to you let us know if you need any help looking14:27
opendevreviewSlawek Kaplonski proposed openstack/neutron-tempest-plugin master: Revert "Make neutron-tempest-plugin-designate-scenario non voting"  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/92783414:27
ralonsohI didn't close the LP bug, the patch is marked as related-bug14:27
slaweqrevert proposed ^^14:27
ralonsohIf I see again this issue, I'll have more info about the problem14:27
ralonsohit is not easy to debug this error14:27
haleybralonsoh: ack14:28
haleybbrb14:28
ralonsoh(during this break, please check this fix for postgresql: https://review.opendev.org/c/openstack/neutron/+/927801)14:29
haleybthe last but is another OVN nested router issue14:30
haleyb#link https://bugs.launchpad.net/neutron/+bug/207787914:30
haleybs/but/bug14:30
haleybihrachys: thanks for the notes there14:30
ihrachyshaleyb: this looks like an RFE for a different way of configuring nested snat14:31
ihrachysnot via subnets + static routes; but though gw_info on inner-router14:32
ihrachys(this is an invitation to tag it as such and let drivers team to consider UX for these scenarios)14:33
haleybi will have to look at the commands he ran, but can probably tag it as an RFE and discuss friday, and hope the submittor can work on it14:35
ihrachysthanks. fyi I don't have immediate plans to work on it :)14:35
haleybthe other issue 2+ of us see now is you can't add a FIP to a VM on a nested network, which looks like14:36
haleyb#link https://bugs.launchpad.net/neutron/+bug/207250514:36
haleybihrachys: understood14:36
haleybthose were the only bugs i had to discuss14:37
haleybihrachys is the deputy list week, lucasgomes next week14:38
haleybs/list/this can't type14:38
ihrachysack14:38
haleybis lucas around? i will look for him later this week14:40
mlavalleI can ping him internally to make sure he is aware14:40
haleybmlavalle: thanks14:41
haleybCurrent bug count this week: 711, down 6 from last week14:41
haleyb#topic community-goals14:41
haleyblajoskatona: i see a number of the horizon sdk patches merged?14:42
lajoskatonayes :-)14:42
lajoskatonaand thanks for the reviews, and for the ones that are for bumping netaddr in Horizon and Cinder14:43
lajoskatonahttps://review.opendev.org/q/topic:%22bump-netaddr%2214:44
haleyblajoskatona: np, i was surprised the netaddr bump hadn't merged, but maybe it's pushed to next cycle now?14:44
haleybah, looks like it14:44
lajoskatonaI don't know, just saw the hanging patch in requirements repo and thought I can push it perhaps14:44
haleybit's marked -214:45
lajoskatonaI don't feel the urge from Cinder and Horizon team (or anybody not that interested in networking) to fix something with netaddr14:45
fricklerreqs are frozen now, yes14:45
haleyblajoskatona: thanks for helpiing to fix those last projects14:45
* mlavalle confirmed with lucasgomes he is ok being bugs deputy next week14:45
haleybmlavalle: thanks14:45
haleyblajoskatona: :-/ is all i can say to that14:46
lajoskatonafrickler: thanks, we come back to it in Epoxy I suppose :-)14:47
haleybthe other community goal was eventlet removal, and i just see one patch left14:48
haleyb#link https://review.opendev.org/c/openstack/neutron/+/92431714:48
haleybralonsoh: is that ready to merge?14:49
ralonsohyes and that will be the last one until we have news from oslo.services and oslo.messaging14:49
ralonsohyes, is ready (and then I'll need to update all the docs and names, referring to wsgi/eventlet server)14:49
mlavallewas the failure in experimental expected?14:50
ralonsohwhich one?14:50
ralonsohwe can discuss it in the patch14:51
ihrachyssorry, I am clueless, butL what does it mean? are we ready to remove the dependency? or is it neutron-server specific?14:51
mlavallethe build failed (several jobs)14:51
ralonsohihrachys, I don't understand your question14:51
ralonsohmlavalle, I'll check them later again14:51
ihrachys"the community goal was eventlet removal", "and i just see one patch left" I interpret it as we removed eventlet completely? or I misread?14:52
ralonsohthis is not neutron-server specific, we are moving to wsgi to avoid using eventlet server and because almost all projects already move to wsgi14:52
haleyb#link https://review.opendev.org/c/openstack/governance/+/90258514:52
ihrachyscontext of the question is `git grep eventlet` shows a lot of stuff in agents14:52
haleybthat's the larger goal14:52
ralonsohno, there is one patch left for now, we are not even close to remove eventlet 14:52
ralonsohthe agent stuff should start with the oslo.service change, from using eventlet to other strategy (to be defined)14:53
ihrachysok ok. see, I'm clueless :) and I misread the "just one patch left" thing14:53
ihrachysand thanks for clarification14:54
slaweqyeah, this is much bigger goal and it will have multiple steps IIUC14:54
lajoskatonaralonsoh, ihrachys: thanks for the details and the good questions14:54
haleyb#topic on-demand14:55
opendevreviewMichel Nederlof proposed openstack/ovn-bgp-agent master: Fix cleanup of rules per evpn device  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/92781614:55
haleybi did not see any topics in the agenda for this week, but if anyone has one can discuss now14:55
mlavallecan I get another pair of eyes on https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/926503?14:55
mlavalleit's an easy one14:55
ihrachysi'll bite. we should promote ovs-lts jobs to gate, eventually. thoughts?14:56
ihrachys(as in - jobs that build ovs and ovn from latest LTS branches and run tempest against it)14:56
lajoskatonamlavalle: added to my list14:56
mlavallethanks!14:56
ihrachysthese are proposed as experimental here https://review.opendev.org/c/openstack/neutron/+/927221 but I'm looking if we should promote them to periodic and then checl14:57
ihrachys*check14:57
ralonsohwe'll have 6 more CI hours per patch14:57
ihrachyswe'll have better coverage too14:58
ralonsohbut makes sense to test OVN latest release14:58
ralonsohfor sure, I'm OK with this14:58
slaweqI'm fine adding them to the periodic queue but for gate/check I think we should maybe try to replace some exisitng jobs with those new ones?14:58
ralonsohcould be another option, to avoid being warned by zuul folks again14:59
ihrachysI am all for revisiting what we run in gate, or even in experimental (which seems like a graveyard for jobs)14:59
ihrachysralonsoh: were we warned? interesting14:59
slaweqI'm not even worried that much about resources usage but also about overall gate stability14:59
ralonsohihrachys, we check the periodic executions every week in the CI meeting14:59
slaweqwe have seen that in the past - we had many jobs there and even when each of them were failing 10% of times, it was almost impossible to merge anything15:00
ihrachyswhat I hear is I should start with periodics and build confidence :)15:00
haleybi was going to say the same thing, is there something we can remove?15:00
ralonsohihrachys, I think you can move these jobs to periodic too (actually by moving them to periodic we'll have them in experimental)15:01
slaweqso my proposal for now is: let's add them to the periodic queue, and discuss in some time during the drivers meeting maybe15:01
ihrachysralonsoh: ack I'll do it first thing and we can revisit gate question in several months or weeks15:01
ralonsoh+115:01
haleyb+1 from me15:02
haleybwe are over time so i'll end the meeting15:02
haleybthanks for all the discussions15:03
ihrachysfyi looked at a random patch and seeing 6 jobs tempest for ovs and 3 for ovn. priorities :)15:03
haleybihrachys: right, and they all took 2 hours each15:03
haleyb#endmeeting15:04
opendevmeetMeeting ended Tue Sep  3 15:04:15 2024 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:04
opendevmeetMinutes:        https://meetings.opendev.org/meetings/networking/2024/networking.2024-09-03-14.00.html15:04
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/networking/2024/networking.2024-09-03-14.00.txt15:04
opendevmeetLog:            https://meetings.opendev.org/meetings/networking/2024/networking.2024-09-03-14.00.log.html15:04
lajoskatonao/15:04
mlavalle\o15:04
ralonsohbye15:04
slaweqbye15:04
opendevreviewIhar Hrachyshka proposed openstack/neutron master: zuul: Move ovs-master and ovs-lts jobs to periodic queue  https://review.opendev.org/c/openstack/neutron/+/92784315:28
opendevreviewIhar Hrachyshka proposed openstack/neutron master: Degrade openstack-tox-py311-with-sqlalchemy-master to experimental  https://review.opendev.org/c/openstack/neutron/+/92784715:42
opendevreviewSebastian Lohff proposed openstack/neutron master: Allow ml2 drivers to start their own RPC listeners  https://review.opendev.org/c/openstack/neutron/+/91959015:46
opendevreviewIhar Hrachyshka proposed openstack/neutron master: DNM Collecting stats for packaged ovn test failures  https://review.opendev.org/c/openstack/neutron/+/92784915:59
opendevreviewIhar Hrachyshka proposed openstack/neutron-tempest-plugin master: Switch enforce-scope job to ovn  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/92785016:05
opendevreviewIhar Hrachyshka proposed openstack/neutron-tempest-plugin master: zuul: remove obsolete job definitions  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/92785116:10
gmannslaweq: maybe we can keep this bug open for neutron also. I commented on bug about how service-service interaction should happen https://bugs.launchpad.net/neutron/+bug/207851816:20
gmannplease check, this is the part missing on neutron side for service role implementation 16:20
samcat116Hi all, I have a question about the network_auto_schedule neutron config option. In a 2023.2 deployment, I am seeing errors similar to this bug: https://bugs.launchpad.net/networking-ovn/+bug/1647421. The fix for that it seems is to respect the network_auto_schedule config option. Should this be set to false for an OVN deployment with no other DHCP agents deployed?17:50
slaweqgmann I opened new one based on your comment17:54
gmannslaweq: ok, ++17:55
slaweqbut at least we are unblocked for now17:56
slaweqwith this job17:56
slaweqand it can be voting again17:56
gmannI am going to implement that 'sending service + user token for service to service interaction' in next cycle. I did not get time to do this cycle17:56
gmannslaweq: agree ++17:56
slaweqthx gmann 17:56
slaweq++17:56
gmannI missed to right to implement in Nova and that time we can do it for other services including neutron also17:57
gmannI faced the same issue as neutron designate interaction in nova cinder interaction also while using the service only token where many internal operations depends on admin token17:58
slaweqso this is already done in nova?17:59
slaweqor will you do it next cycle?17:59
gmannand then we discussed to solve this by sending both the token and service can use service token for only authentication and user token for actual operation 17:59
gmannslaweq: not yet, I started but could not complete due to my bandwidth in other work. https://review.opendev.org/q/topic:bp%252Fpolicy-service-role-default18:00
gmannI will do it very first thing in next cycle18:00
gmannhope that work and we do not get more issue but let's see18:01
gmannslaweq: this is good to go now https://review.opendev.org/c/openstack/neutron/+/92608519:08
slaweqgmann: approved19:09
gmannslaweq: thanks19:24
mnaserdoes anyone know how neutron + routed provider networks + external ports (aka baremetal provisioning) can work?19:52
mnaserusually you'd use an ip helper, but i'm not sure how that would work19:52
mnaserbecause ovn-controller is the one that is dealing with this19:52
opendevreviewMerged openstack/neutron-tempest-plugin master: Revert "Make neutron-tempest-plugin-designate-scenario non voting"  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/92783420:23
opendevreviewMerged openstack/neutron master: Introduce ovs-lts jobs that run against the latest ovn LTS branch  https://review.opendev.org/c/openstack/neutron/+/92722120:27
opendevreviewMerged openstack/neutron master: docs: fix neutron-ovn-tempest-with-neutron-lib-master description  https://review.opendev.org/c/openstack/neutron/+/92759320:27
opendevreviewMerged openstack/os-ken master: Python3.12: do not use ssl.wrap_socket  https://review.opendev.org/c/openstack/os-ken/+/92757621:20
opendevreviewMerged openstack/neutron master: Remove default override for RBAC config options  https://review.opendev.org/c/openstack/neutron/+/92608521:34
opendevreviewMerged openstack/networking-bagpipe master: Add pyproject.toml to support pip 23.1  https://review.opendev.org/c/openstack/networking-bagpipe/+/90021621:49
opendevreviewMerged openstack/neutron-tempest-plugin master: Update snat_rules_apply_to_nested_networks with ovn details  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/92722023:08
opendevreviewMerged openstack/neutron-tempest-plugin master: Revert "Switch networking-sfc job to be run with eventlet"  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/92710923:08

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!