Tuesday, 2024-07-30

opendevreviewyatin proposed openstack/neutron stable/2024.1: [2024.1 only] Switch to 2024.1 neutron-tempest-plugin jobs  https://review.opendev.org/c/openstack/neutron/+/92515904:53
ralonsohslaweq, ^^^ this patch is fixing 2024.1 (the tempest tests)05:35
opendevreviewRodolfo Alonso proposed openstack/neutron stable/2024.1: Initialize the policy enforcer for the "tagging" service plugin  https://review.opendev.org/c/openstack/neutron/+/92507705:36
opendevreviewLiushy proposed openstack/neutron master: [OVN] Support address group for ovn driver  https://review.opendev.org/c/openstack/neutron/+/85150906:18
opendevreviewRodolfo Alonso proposed openstack/neutron master: [WSGI] Move all OVN jobs to use WSGI API module  https://review.opendev.org/c/openstack/neutron/+/92431707:13
ykarelslaweq, when around please check https://review.opendev.org/c/openstack/neutron/+/92515907:57
opendevreviewRodolfo Alonso proposed openstack/neutron stable/2024.1: Do not release the executor inside ``_check_child_processes``  https://review.opendev.org/c/openstack/neutron/+/92517408:45
opendevreviewRodolfo Alonso proposed openstack/neutron stable/2023.2: Do not release the executor inside ``_check_child_processes``  https://review.opendev.org/c/openstack/neutron/+/92517508:45
opendevreviewRodolfo Alonso proposed openstack/neutron stable/2023.1: Do not release the executor inside ``_check_child_processes``  https://review.opendev.org/c/openstack/neutron/+/92517608:46
opendevreviewLiushy proposed openstack/neutron master: [OVN] Support address group for ovn driver  https://review.opendev.org/c/openstack/neutron/+/85150908:54
opendevreviewRodolfo Alonso proposed openstack/neutron master: Use the RBAC actions field for "network" and "subnet"  https://review.opendev.org/c/openstack/neutron/+/91537009:23
opendevreviewSlawek Kaplonski proposed openstack/neutron stable/2023.2: Add has_lock_periodic decorator for OVN Maintenance  https://review.opendev.org/c/openstack/neutron/+/92517809:35
opendevreviewSlawek Kaplonski proposed openstack/neutron stable/2023.1: Add has_lock_periodic decorator for OVN Maintenance  https://review.opendev.org/c/openstack/neutron/+/92517909:39
opendevreviewyatin proposed openstack/neutron master: [ovs-master jobs] Move OVS_BRANCH to branch-3.4  https://review.opendev.org/c/openstack/neutron/+/92518410:53
opendevreviewyatin proposed openstack/neutron master: [DNM] check ovn/ovs source job ovs_branch=3.4  https://review.opendev.org/c/openstack/neutron/+/92518711:06
ralonsohykarel, https://review.opendev.org/c/openstack/neutron/+/92509111:30
ralonsohthat patch was proposed yesterday11:30
ykarelralonsoh, thx i missed that, let's take that in , will abandon mine11:31
ralonsohbut the dnm is not passing11:31
ralonsohhttps://review.opendev.org/c/openstack/neutron/+/925187?tab=change-view-tab-header-zuul-results-summary11:31
ralonsohso there is something else that is needed11:32
ykarelyes looking into that now11:33
ykarelseems change is not picked11:34
ykarelas seeing 2024-07-30 11:11:11.902321 | controller | +++ OVS_BRANCH=branch-3.311:34
opendevreviewyatin proposed openstack/neutron master: [DNM] check ovn/ovs source job ovs_branch=3.4  https://review.opendev.org/c/openstack/neutron/+/92518711:38
ralonsohykarel, qq, if you know11:43
ralonsohhave you seen this error before: https://paste.opendev.org/show/bwsUPO7SEpDxoUXGpjAB/11:43
ralonsohI'm having this error in the ML2/OVN jobs with WSGI (actually there is one only, in the experimental queue)11:43
ralonsohand started to fail 7 days ago11:43
ralonsohI thought it was because of the RPC stuff in devstack, but it was working fine (with the RPC worker) before that11:44
ykarelralonsoh, nope i don't recall seeing those errors recently apart from the bug you recently reported11:45
ralonsohI think this is because the IPv6 address but I need to justify it11:45
ykarelyes likely11:47
ykarelwrt change not picked for ovs-master job, it's because stable/2024.1 variant being used in the job, but with above dnm it should work. and we could drop those variants from stable branches as relevant for master only11:48
ralonsohykarel, are you talking about https://review.opendev.org/c/openstack/neutron/+/925184?11:49
ralonsohI don't understand why the DNM patch PS1 was not working11:49
ralonsohwith PS2 is using the correct ovs branch11:49
ykarelyes talking about dnm patch11:49
ykarelabove is the reason for not working in PS111:49
ralonsohsorry, I still don't get it11:50
ralonsohyou are testing a master job, neutron-ovn-tempest-ipv6-only-ovs-master11:50
ykarelyes let me share reference with that it will be more clear11:53
ralonsohthanks!11:53
ykarelsee https://4cdf01f2d535f547e363-277fe61cde1e0fafc7c227e50deabdc0.ssl.cf1.rackcdn.com/925187/1/check/neutron-ovn-tempest-ipv6-only-ovs-master/0a82387/zuul-info/inventory.yaml11:54
ykarelspecifically _inheritance_path: 11:54
ralonsohno way...11:54
ykarelthere it matches source: openstack/neutron/zuul.d/tempest-singlenode.yaml@stable/2024.1#773>'11:54
ralonsohright11:54
ralonsohwhat I don't understand is how a inherited job can affect the parent too11:55
ralonsohwell, the stable branch one11:55
ykarelzuul pick jobs across branches, and we have branches: master set on these jobs11:56
ykareli don't know how it prioritizes one over the other though11:56
ykarelmay be frickler or fungi can tell that11:57
ralonsohpffff "the ways of zuul are unsearchable"11:57
ralonsohI didn't know that but that's very good to know it11:57
opendevreviewMerged openstack/neutron stable/2024.1: [2024.1 only] Switch to 2024.1 neutron-tempest-plugin jobs  https://review.opendev.org/c/openstack/neutron/+/92515912:00
fricklerykarel: ralonsoh: iiuc the "branches: ^master$" will only affect where the job will be run, not where the job definition is getting taken from. so I think you'll want to really delete all the *-master job definitions from all stable branches and repeat that process when new branches are being made12:04
ralonsohahhh but -master in this case refers to ovs-master12:05
opendevreviewyatin proposed openstack/neutron stable/2024.1: [stable/2024.1 only] Drop -master jobs  https://review.opendev.org/c/openstack/neutron/+/92519312:05
ralonsohno, I said nothing12:05
ykarelfrickler, yeap just pushed ^, the part that is not clear is the same job appears in both master and stable/2024.1, when the job is triggered it's picking the job from stable/2024.1 not the the one from master, how this prioritizes one over the other?12:07
ykarelso with the above patch we can avoid this to happen atleast12:07
ralonsohwe were using the suffix -master to refer to other libraries, like sqlalchemy or n-lib12:08
ralonsohbut I think these tests don't make sense in stable branches12:08
fricklerykarel: looking at that inheritance_path, the job definitions seem to get stacked, I'm not sure if zuul makes any guarantee on the ordering of that stack, might be random or alphabetic12:12
ralonsohbcafarel, please check https://review.opendev.org/c/openstack/neutron/+/92519312:15
bcafarelralonsoh: sure, I thought I had checked this one earlier, probably missed the button12:23
bcafarelah no that was the Switch to 2024.1 neutron-tempest-plugin jobs (same kind of new stable fixes)12:24
opendevreviewSlawek Kaplonski proposed openstack/neutron master: Lower spacing time of the OVN maintenance tasks which should be run once  https://review.opendev.org/c/openstack/neutron/+/92519412:30
opendevreviewSlawek Kaplonski proposed openstack/neutron master: Use has_lock_periodic decorator for the tasks which require ovn db lock  https://review.opendev.org/c/openstack/neutron/+/92519712:53
ykarelfrickler, thx for checking13:20
ykarelralonsoh, bcafarel when you get chance check https://review.opendev.org/c/openstack/neutron/+/925184 too13:20
ralonsohykarel, done but then I'll ask the other folk to abandon his patch13:21
ykarelah sorry13:21
ykarellet's take other one then i missed that again13:21
ykarelas that was proposed first13:21
ralonsohcool13:21
bcafarelwhich one should we review then?13:29
ralonsohbcafarel, https://review.opendev.org/c/openstack/neutron/+/92509113:32
opendevreviewRodolfo Alonso proposed openstack/neutron master: DNM == Test neutron-tempest-plugin-api-ovn-wsgi  https://review.opendev.org/c/openstack/neutron/+/92520513:34
haleyb#startmeeting networking14:00
opendevmeetMeeting started Tue Jul 30 14:00:17 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
mlavalle\o14:00
elvirao/14:00
ykarelo/14:00
ralonsohhi14:00
bcafarelo/14:00
slaweqo/14:01
haleybhi everyone14:01
haleyb#topic announcements14:01
haleybWe are now in Dalmatian release week (R - 9)14:01
rubasovo/14:01
haleybNon-client library freeze: August 22nd, 2024 (R-6 week)14:01
haleybClient library freeze: August 29th, 2024 (R-5 week)14:01
haleybDalmatian-3 milestone: August 29th, 2024 (R-5 week)14:02
haleybso we are pretty close to the end of D14:02
haleybhopefully we have a happy gate so we can merge code :-)14:03
haleybReminder: If you have a topic for the drivers meeting on Friday, please add it to the wiki @ https://wiki.openstack.org/wiki/Meetings/NeutronDrivers14:03
haleybanything crazy happen last week?14:04
ralonsohI think most of the CI problems are addressed14:04
mlavallewe didn't wreck the place \o/14:04
haleybgood to know :)14:05
haleybi am still digging out from a backlog, so feel free to ping me on irc for anything important14:05
obondarevo/14:05
haleyb#topic bugs14:06
haleybbcafarel was the deputy last week, here is a link to his report14:06
haleyb#link https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/NZKXVL2DHNFLKEEXETJ2JZYMQM5DALCZ/14:06
haleybthere are a few without assignee14:07
bcafarelon the 3 unassigned bugs, I see you checked the low-haning fruit one, also thanks ralonsoh for checking first one14:07
haleyb#link https://bugs.launchpad.net/neutron/+bug/207420714:07
haleybDHCP agent makes endless attempts to configure a network with MTU < 128014:07
haleybi thought this was related to a previous bug, but it's slightly different14:07
haleybralonsoh: just noticing you added a comment14:08
ralonsohI think it makes sense not to add the ipv6 metadata address if there are no ipv6 subnets14:08
ralonsohin any case, if you have ipv6 enabled in your env, you should not used smaller mtus14:08
ralonsohbut don't fail because of us14:08
haleybralonsoh: ack, i couldn't tell whether you thought it was invalid, but i agree we could fix this with a check14:09
haleybi can take it if noone else wants it as i worked on the last one14:09
haleybi don't know why people use small mtus like that, but...14:10
haleybnext bug is14:11
haleyb#link https://bugs.launchpad.net/neutron/+bug/207387214:11
haleybNeutron L3 agent not create ECMP in HA router but in single mode is ok14:11
haleyblooks like l3-ha differs from "regular" l3 in adding/deleting routes14:12
slaweqI can try to maybe reproduce it and look quickly if I will have some time, but not promissing anything for now14:13
haleybthis is reminding me of a previous bug where we had issues with overlapping routes, in this case it's just the gateway14:13
haleybslaweq: thanks, i'll watch it14:14
haleyblast unassigned is14:14
haleyb#link https://bugs.launchpad.net/neutron/+bug/207405614:14
haleybInvalid documented security group rule protocol "any"14:14
haleybI added a comment here based on what i knew14:14
haleybthe "any" keyword is only actually valid when using OSC, not the API14:15
haleybbut the API ref says "any" is valid14:15
slaweqIMO if Openstackclient accepts 'any' as valid value, we may accept it on the server side as well to be more consistent with the cli tool14:16
slaweqit should be easy to do14:16
haleybWe can clean-up the doc to say "use None or 0", but i was wondering if anyone thought supporting "any" as an alias in the API was a good idea14:16
haleybyou type faster than me14:17
slaweq:)14:17
ralonsohactually the value in the server is "ip"14:17
ralonsohfor protocol 014:17
haleybralonsoh: yes, i pointed that out in the bug, so there's almost too many choices14:18
ralonsohright, I'm now reading your comment14:18
ralonsohwe should not change the CLI (or deprecate "any" with time)14:18
ralonsohor accept both "any" or "ip" (and the deprecate "any")14:19
haleybright, i think i've used 'any' in scripts myself14:19
haleybi think it should be easy enough to support 'any' as an alias on the server side14:20
slaweq+114:20
ykarel+114:21
haleybthe only thing that might be an issue is what does the POST return for protocol?14:21
slaweqwhat it returns now for None and 0 values?14:21
slaweqI guess it should be the same and we can clearly document that 'any' is just alias for the value which is returned from API14:22
haleybyes, i just didn't know if someone would be confused if it didn't reutrn "any"14:22
haleybslaweq: yes, that sounds good14:22
haleybi will add that info to the bug, and should be able to work on it eventually14:23
haleybi had filed a bug over the weekend, one of those "this is an annoying traceback" seen in the server logs14:24
haleyb#link https://bugs.launchpad.net/neutron/+bug/207431214:25
haleybof course my quick fix proposal didn't work14:25
haleybhas anyone else noticed those tracebacks?14:25
haleybtitle was " DB semantic violation seen in neutron-tempest-plugin-openvswitch logs deleting router "14:25
haleybThis handler is supposed to handle AFTER events, as in 'AFTER it's committed', not BEFORE. Offending resource event: port, after_delete14:26
ralonsohyes when implementing the new db facade14:26
ralonsohI think this is because we are creating a port inside a transaction14:26
ralonsohsorry, deleting the port14:26
haleybi found the code that should have disabled the warning but it's not working in this case14:26
ralonsohis that somethign recurrent?14:26
haleybthere is also one in create_security_group() that doesn't have the "disable the warning" code14:26
haleybevery neutron-tempest-openvswitch run has occurrences14:27
haleybneutron-tempest-plugin-openvswitch14:27
haleybi didn't check other logs14:27
haleybone question i had was is it possible to move the code? i figured it would have been done by now if simple14:28
haleybi.e. save the ports and do in a second transaction14:28
haleybralonsoh: feel free to add a comment to the bug if you have a suggestion, it's not critical just an annoying traceback14:30
ralonsohwe can try that, we are deleting the GW port before that txn14:30
ralonsohI'll push a patch but we need to be cautious leaving leftovers 14:31
haleybthere are at least two TODOs for that same issue, luckily finding GUARD_TRANSACTION is easy14:31
haleybok, i had no other bugs14:32
haleybany others to discuss?14:32
haleybThis week elvira is the bug deputy, next week will be slaweq14:32
haleybis that ok for both?14:32
elvirasure!14:32
haleybCurrent bug count this week: 721, up 14 from last run14:33
haleyb#topic community-goals14:34
slaweqgood for me14:34
haleybWe merged the MANAGER role and added documentation, so good there, thanks slaweq !14:35
slaweqyeah, this can be IMO consider as done finally14:35
haleyblajos continues to work on neutronclient deprecation14:36
haleyb#link https://review.opendev.org/q/topic:%22bug/1999774%2214:36
haleybfixes all for horizon14:37
haleybralonsoh: and it looks like you just have one change left for evenlet deprecation?14:38
haleyb#link https://review.opendev.org/c/openstack/neutron/+/92431714:38
ralonsohI'm currently working in an issue related to wsgi and OVN14:38
haleyb#link https://review.opendev.org/q/topic:%22bug/2069581%2214:38
ralonsohthat is part of the eventlet server deprecation14:38
ralonsohyeah, ml2/ovn won't work right now with wsgi (it was 1 week before)14:39
haleyb:(14:39
ralonsohso just investigating https://bugs.launchpad.net/neutron/+bug/207514714:39
ralonsohand that's all14:39
haleyband i do see the governance change merged14:39
haleyback, thanks14:39
haleyb#link https://review.opendev.org/c/openstack/governance/+/902585 (merged)14:40
haleyb#topic on-demand14:40
haleybwe already talked about the dhcp-agent looping bug14:41
haleybany other topics?14:41
ralonsohnot from me14:41
haleybykarel: CI meeting in-person today?14:42
ykarelhaleyb, sent a cancellation mail for today14:42
haleybykarel: ack14:42
haleybok, if nothing else we are done, have a good week everyone!14:43
haleyb#endmeeting14:43
opendevmeetMeeting ended Tue Jul 30 14:43:09 2024 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)14:43
opendevmeetMinutes:        https://meetings.opendev.org/meetings/networking/2024/networking.2024-07-30-14.00.html14:43
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/networking/2024/networking.2024-07-30-14.00.txt14:43
opendevmeetLog:            https://meetings.opendev.org/meetings/networking/2024/networking.2024-07-30-14.00.log.html14:43
mlavalle\o14:43
ralonsohbye14:43
ykarelralonsoh, can you re +W https://review.opendev.org/c/openstack/neutron/+/92447114:43
ykarelnova patch merged14:43
ralonsohsure14:43
ykarelthx14:44
opendevreviewMerged openstack/ovn-octavia-provider master: [OVN][CI] Update OVS_BRANCH to be compatible with ovn main  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/92509215:00
opendevreviewSlawek Kaplonski proposed openstack/neutron master: Lower spacing time of the OVN maintenance tasks which should be run once  https://review.opendev.org/c/openstack/neutron/+/92519415:03
opendevreviewSlawek Kaplonski proposed openstack/neutron master: Lower spacing time of the OVN maintenance tasks which should be run once  https://review.opendev.org/c/openstack/neutron/+/92519415:04
opendevreviewRodolfo Alonso proposed openstack/neutron master: DNM == Test neutron-tempest-plugin-api-ovn-wsgi  https://review.opendev.org/c/openstack/neutron/+/92520515:06
opendevreviewPierre Riteau proposed openstack/ovn-octavia-provider master: Maintenance task: do not change IPv4 ip_port_mappings  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/92319615:09
opendevreviewMerged openstack/neutron master: [OVN][CI] Update OVS_BRANCH to be compatible with ovn main  https://review.opendev.org/c/openstack/neutron/+/92509115:39
opendevreviewJakub Libosvar proposed openstack/ovn-bgp-agent master: Remove dead code from get_port_chassis function  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/92467316:18
opendevreviewMerged openstack/neutron stable/2024.1: [stable/2024.1 only] Drop -master jobs  https://review.opendev.org/c/openstack/neutron/+/92519316:34
opendevreviewMerged openstack/neutron stable/2023.1: Don't print traceback if standard attr is missing on update  https://review.opendev.org/c/openstack/neutron/+/92447116:37
opendevreviewAdam Oswick proposed openstack/neutron master: Conditionally run address scope affinity checks  https://review.opendev.org/c/openstack/neutron/+/92498417:24
opendevreviewMiro Tomaska proposed openstack/neutron master: Only query for port do not get directly  https://review.opendev.org/c/openstack/neutron/+/92503919:30
opendevreviewMerged openstack/neutron master: Use has_lock_periodic decorator for the tasks which require ovn db lock  https://review.opendev.org/c/openstack/neutron/+/92519719:56
opendevreviewMerged openstack/neutron stable/2023.2: Add has_lock_periodic decorator for OVN Maintenance  https://review.opendev.org/c/openstack/neutron/+/92517819:56
opendevreviewTerry Wilson proposed openstack/neutron stable/2024.1: Use oslo_service's SignalHandler for signals  https://review.opendev.org/c/openstack/neutron/+/92318720:51
opendevreviewTerry Wilson proposed openstack/neutron stable/2024.1: Actualy set global "removal limit" options  https://review.opendev.org/c/openstack/neutron/+/92530120:57
opendevreviewTerry Wilson proposed openstack/neutron stable/2024.1: Actualy set global "removal limit" options  https://review.opendev.org/c/openstack/neutron/+/92530120:59
opendevreviewMerged openstack/neutron master: Do not release the executor inside ``_check_child_processes``  https://review.opendev.org/c/openstack/neutron/+/92458121:50

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