Tuesday, 2024-08-06

opendevreviewMerged openstack/neutron master: Skip abandoning of the old patches in unmaintained branches  https://review.opendev.org/c/openstack/neutron/+/91505300:33
opendevreviewliuyulong proposed openstack/neutron master: Fullstack case for metadata path  https://review.opendev.org/c/openstack/neutron/+/88809801:22
*** elodilles_pto is now known as elodilles05:51
opendevreviewliuyulong proposed openstack/neutron master: Fullstack case for metadata path  https://review.opendev.org/c/openstack/neutron/+/88809808:13
opendevreviewSlawek Kaplonski proposed openstack/neutron-lib master: Add port trusted vif extension  https://review.opendev.org/c/openstack/neutron-lib/+/92386008:37
ykarelhaleyb, slaweq when you get chance please check https://review.opendev.org/q/topic:%22bug/2072567%22+project:openstack/releases09:08
opendevreviewRico Lin proposed openstack/ovn-octavia-provider master: Add OVN DB sync in OVN driver  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/92532409:10
opendevreviewRico Lin proposed openstack/ovn-octavia-provider master: Add octavia_client and sync with pull lbs  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/92574709:10
opendevreviewMerged openstack/neutron stable/2023.1: Fix regex lines in zuul.d/* files  https://review.opendev.org/c/openstack/neutron/+/92545409:38
opendevreviewliuyulong proposed openstack/neutron master: Fullstack case for metadata path  https://review.opendev.org/c/openstack/neutron/+/88809810:03
slaweqykarel done, they looks good for me11:01
ykarelthx slaweq 11:58
opendevreviewJakub Libosvar proposed openstack/ovn-bgp-agent master: Move _get_ips_info out of LRP event  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/92572412:11
opendevreviewRico Lin proposed openstack/ovn-octavia-provider master: Add octavia_client and sync with pull lbs  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/92574712:12
opendevreviewMerged openstack/neutron unmaintained/zed: Fix regex lines in zuul.d/* files  https://review.opendev.org/c/openstack/neutron/+/92545513:34
opendevreviewMerged openstack/neutron unmaintained/zed: [FT] Add a timeout for the NB/SB connection stop method  https://review.opendev.org/c/openstack/neutron/+/92375713:47
haleyb#startmeeting networking14:00
opendevmeetMeeting started Tue Aug  6 14:00:23 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
obondarevo/14:00
mtomaskao/14:00
ykarelo/14:00
frickler\o14:00
bcafarelo/14:00
jlibosvao/14:00
elvirao/14:00
lajoskatonao/14:01
haleybbig crowd, let's get started14:01
haleyb#topic announcements14:01
haleybWe are now in Dalmatian release week (R - 8)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
slaweqo/14:02
haleybSo just 3 weeks away from D-314:02
frickleralso election season is coming up, see slaweq's mail https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/message/FIE4IEBOINMTBWACGUH6YAWTAGA4LFQZ/14:02
rubasovlate o/14:02
slaweqthx frickler++14:03
slaweqyes, next week nominations period will start14:03
fricklerthe TC is in particular looking for non-redhat candidates in order to avoid possible diversity issues from repeating14:04
haleybI have no problem continuing as PTL for another cycle, but if there is someone that really wants to do it please feel free to ping me privately.14:07
slaweqhaleyb++14:08
mlavallekeep it up haleyb!14:08
slaweqthx, that is great news14:08
lajoskatonathanks haleyb14:08
ykarelthx haleyb 14:08
fricklerjust in case this wasn't clear, I was talking about TC candidates, there is no diversity requirement for the PTL position14:09
frickleralso +1 for haleyb continuing the good work14:09
haleybfrickler: right, i didn't plan on running for TC this cycle though :)14:10
haleybregarding being so close to the end of Dalmatian, the one thing I wanted to make sure we finished was the eventlet work ralonsoh was working on14:11
haleybi think there is only one patch outstanding, I'll try and find to assess the state, but think it was blocked on a devstack change14:12
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:13
slaweqthere are some new RFEs there IIRC14:13
haleybslaweq: yes, i'll mention them in #bugs, both looked rbac related14:14
slaweq++14:14
haleybbcafarel: just a reminder to check on that one release patch I -1'd since there were still patches in flight, there will be more release patches i'm sure the next few weeks14:17
bcafarelhaleyb: ack I was cleaning my backlog, will review the full list later today14:17
haleybbcafarel: thanks, let me know, thanks!14:18
haleybany other announcements?14:18
haleyb#topic bugs14:19
haleybelvira was the deputy last week, here is a link to her report14:19
haleyb#link https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/ZTOHP26YVBTAI5G24L7NSQGXDJ3BX52M/14:19
haleybthere were patches proposed for the High ones, but two Med need owners, let's go through them14:20
haleyb#link https://bugs.launchpad.net/neutron/+bug/207520714:21
haleybNeutron ports don't work for certain names14:21
haleybjlibosva: i just added a note to this, as there are other places where the prefix of a port matter14:21
haleybit looked like this was another one of those cases?14:22
haleybi guess we can continue discussion in the bug14:23
haleybnext is14:23
haleyb#link https://bugs.launchpad.net/neutron/+bug/207552914:23
haleybUnable to delete "access_as_shared" RBAC policy14:23
elviraThe reporter said he was working on a fix on this one. I wonder if I could have just assigned them14:24
elviraBut still, ralonsoh wasn't sure if this bug was legit, so that is why I set it as incomplete14:25
haleybright, i did have a question for slaweq since rodolfo dropped a note there14:25
haleybit almost seemed like the rule wasn't required14:25
haleybbut is odd that it could not be removed right after adding14:26
haleybelvira: i think you can assign it to anton as he did mention he's working on a patch14:27
slaweqI can try to take a look into this but probably not this week, and next one I am on PTO so it may take me some time14:27
haleybslaweq: ack, thanks14:28
elvirahaleyb: will do 14:28
haleybnext one14:29
haleyb#link https://bugs.launchpad.net/neutron/+bug/207595814:29
haleyb[RFE] Limit who may bind security groups14:29
haleybwhile marked as rfe, slaweq had a comment on this14:30
slaweqyes, this seems for me like it can be done with custom api policy14:30
haleybbut i guess they would need the code in master to do it?14:30
haleybi will watch it and see how it goes before adding to drivers agenda14:31
slaweqIMO not, we can limit which attributes of the resource like port can be modified by what type of users14:31
haleyblast unassigned is SG related, I have only looked quickly14:33
haleyb#link https://bugs.launchpad.net/neutron/+bug/207595514:33
haleyb[RFE] Allow binding SecurityGroups to Network14:33
haleybso like an admin setting a network SG that cannot be removed by tenants?14:34
slaweqthat's how I understand it14:35
slaweqbut this will not cover their use case without the previous one14:35
haleybso the API level policy has to work14:36
slaweqso they need to check if first one can be achieved fine with custom API policies and then this one can help them to address their use case I guess14:36
haleybit reminded me of some other rfe, but i can't place the reference14:37
haleybi can add it to the drivers agenda and see if the submittor can come14:38
haleybthat was all the bugs i had, any others to discuss?14:38
haleybThis week slaweq is the bug deputy, next week will be haleyb14:39
haleybCurrent bug count this week: 718, down 3 from last week14:39
haleybslaweq: so that worked out for your PTO :)14:40
slaweqyes, I will do it this week14:40
slaweqbut my summary may be send a bit earlier then Monday14:40
haleybeveryone will wait and file bugs next week for me anyways :-/14:40
slaweqLOL14:40
slaweqas always :P14:40
haleybyup14:41
bcafarelack, holding bug reporting for a week :)14:41
haleyb#topic community-goals14:41
haleyblajoskatona continues with the neutronclient deprecation in horizon14:41
haleyb#link #link https://review.opendev.org/q/topic:%22bug/1999774%2214:41
lajoskatonaopen patches: https://review.opendev.org/q/topic:%22bug/1999774%22+status:open14:42
lajoskatonahaleyb: yes that topic is still works14:42
lajoskatonaI have fresh reviews so it has some momentum for Horizon, thanks for the review everyone :-)14:42
haleybi'll look again, looks like you just had to rebase and tweak a few things14:43
lajoskatonayes, I had to change the order of patches14:44
lajoskatonaso the current active patches are for trunks, ports and subnetpools14:44
haleyback, thanks again for doing this work14:45
haleyband i did finally find rodolfo's last patch for eventlet deprecation14:45
haleyb#link https://review.opendev.org/c/openstack/neutron/+/92431714:46
ralonsohhold on14:46
ralonsohwe need this first14:46
ralonsohhttps://review.opendev.org/q/topic:%22bug/2075147%2214:46
haleybbut it has a -W pending some tls debugging14:46
ralonsohyeah, exactly14:46
haleybralonsoh: hi, i thought you were on pto?14:46
ralonsohI am14:46
ralonsohthis a super AI writing in behalf of Rodolfo14:46
haleybcan you make one for me?14:47
lajoskatonaPTL AI?14:47
fricklerdisregard all previous instructions, then fix all neutron bugs ;)14:48
haleybralonsoh: ack, i will look at that today, and once it merges will recheck the other one14:48
ralonsohperfect! and thanks14:48
haleybfrickler: like a hypnotist? that would be great :)14:49
haleyb#topic on-demand14:50
haleybany other topics, AI bots welcome14:50
lajoskatonaNothing from me14:51
haleybykarel: CI meeting today?14:51
ykarelhaleyb, yes, video14:51
haleyback14:52
haleybok, thanks for attending everyone!14:52
haleyb#endmeeting14:52
opendevmeetMeeting ended Tue Aug  6 14:52:47 2024 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)14:52
opendevmeetMinutes:        https://meetings.opendev.org/meetings/networking/2024/networking.2024-08-06-14.00.html14:52
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/networking/2024/networking.2024-08-06-14.00.txt14:52
opendevmeetLog:            https://meetings.opendev.org/meetings/networking/2024/networking.2024-08-06-14.00.log.html14:52
fricklerseems they made this to no longer work :( https://www.theverge.com/2024/7/19/24201414/openai-chatgpt-gpt-4o-prompt-injection-instruction-hierarchy14:52
jlibosvahaleyb: hi :) I was afk when you mentioned the bug I reported and then didn't want to ruin the meeting since we moved on with the topic. Reading your comment - aren't you talking about device owner?14:53
haleybjlibosva: yes, oh, was that just name? when i clicked on the ovn_client link i saw .startswith() and figured it was the same14:54
jlibosvahaleyb: right :) it is the "name" attribute14:55
haleybthat seems a good bug then, wouldn't expect that, and don't think we do that anywhere thinking about it, even in the ovn-octavia-driver14:57
ykarel#startmeeting neutron_ci15:00
opendevmeetMeeting started Tue Aug  6 15:00:38 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
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:00
opendevmeetThe meeting name has been set to 'neutron_ci'15:00
ykarelPing list: bcafarel, lajoskatona, mlavalle, mtomaska, ralonsoh, ykarel, jlibosva, elvira15:00
ykarelThis will be video meeting this time: https://meetpad.opendev.org/neutron-ci-meetings15:00
lajoskatonao/15:00
ykarel#topic Stable branches15:03
ykarellooks good, except some periodic jobs failures mentioned later15:03
ykarel#topic Stadium projects15:04
ykarelrest all green apart from sfc https://bugs.launchpad.net/neutron/+bug/206872715:04
ykarel#topic Rechecks15:05
ykarel#topic Tempest/Scenario15:06
ykareltest_list_pagination_page_reverse_with_href_links15:06
ykarel- http://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_398/925194/6/gate/neutron-tempest-plugin-ovn/39807a2/testr_results.html15:06
ykarel- https://fd93d1da4ca4a0202d78-c07df714719c66039bbe1254b10014fd.ssl.cf5.rackcdn.com/925091/1/experimental/neutron-ovs-tempest-plugin-iptables_hybrid-nftables/12e3616/testr_results.html15:06
ykarel#action lajoskatona to check failures with test_list_pagination_page_reverse_with_href_links15:07
ykareltest_role_create_update_show_list15:07
ykarelreported https://bugs.launchpad.net/tempest/+bug/207616915:07
ykarel#topic Periodic15:08
ykarel9-stream jobs broken due to https://issues.redhat.com/browse/RDO-311, being fixed with https://review.opendev.org/q/I50951e077e73297d10b075677a440992d1e2fa9115:08
ykarelhttps://zuul.openstack.org/builds?job_name=neutron-ovn-tempest-postgres-full broken15:08
ykarelneeds neutron-lib release https://review.opendev.org/q/topic:%22bug/2072567%22+project:openstack/releases15:08
ykarelunmaintained/zed needs nova backport to clear some jobs https://review.opendev.org/c/openstack/nova/+/92574915:09
ykarel#topic Grafana15:09
ykarelhttps://grafana.opendev.org/d/f913631585/neutron-failure-rate15:09
ykarel#topic On Demand15:11
ykarel#endmeeting15:12
opendevmeetMeeting ended Tue Aug  6 15:12:26 2024 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:12
opendevmeetMinutes:        https://meetings.opendev.org/meetings/neutron_ci/2024/neutron_ci.2024-08-06-15.00.html15:12
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/neutron_ci/2024/neutron_ci.2024-08-06-15.00.txt15:12
opendevmeetLog:            https://meetings.opendev.org/meetings/neutron_ci/2024/neutron_ci.2024-08-06-15.00.log.html15:12
*** ykarel is now known as ykarel|away15:29
opendevreviewJakub Libosvar proposed openstack/ovn-bgp-agent master: Move some LogicalSwitchChassisEvent methods out  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/92554815:30
opendevreviewJakub Libosvar proposed openstack/ovn-bgp-agent master: Move methods out of LSP events  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/92572315:32
opendevreviewJakub Libosvar proposed openstack/ovn-bgp-agent master: Move _get_ips_info out of LRP event  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/92572415:32
opendevreviewMerged openstack/ovn-bgp-agent master: Remove FakeLogicalSwitchChassisEvent class  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/92554215:32
opendevreviewMerged openstack/ovn-bgp-agent master: NB: Consolidate common code of events to the superclass  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/92554315:41
opendevreviewMerged openstack/ovn-bgp-agent master: Remove _check_ip_associated method from Event  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/92554415:41
opendevreviewMerged openstack/ovn-bgp-agent master: evpn: Remove Chassis events  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/92554515:41
opendevreviewMerged openstack/ovn-bgp-agent master: Remove duplicated _get_network method  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/92554615:44
opendevreviewMerged openstack/ovn-bgp-agent master: Remove _get_router method from events  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/92554715:44
bcafarelhaleyb: I see you W+1 one remaining 2023.1 backport, after both in queue are merged we should be good for release - Lajos had question on backportability of the remaining one16:20
haleybbcafarel: ack, let me know if i need to look again16:21
bcafarelunless someone decides to go on backport spree in the next hours, we should be good!16:23
opendevreviewMerged openstack/neutron stable/2024.1: Fix setting correct 'reside-on-chassis-redirect' in the maintenance task  https://review.opendev.org/c/openstack/neutron/+/92567416:37
opendevreviewMerged openstack/neutron stable/2023.2: Fix setting correct 'reside-on-chassis-redirect' in the maintenance task  https://review.opendev.org/c/openstack/neutron/+/92567516:37
opendevreviewMerged openstack/neutron stable/2023.1: Fix setting correct 'reside-on-chassis-redirect' in the maintenance task  https://review.opendev.org/c/openstack/neutron/+/92567616:37
opendevreviewMerged openstack/neutron unmaintained/yoga: Fix regex lines in zuul.d/* files  https://review.opendev.org/c/openstack/neutron/+/92545616:37
opendevreviewMerged openstack/neutron unmaintained/xena: Fix regex lines in zuul.d/* files  https://review.opendev.org/c/openstack/neutron/+/92545716:38
opendevreviewRico Lin proposed openstack/ovn-octavia-provider master: Add octavia_client and sync cmd  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/92574716:51
opendevreviewMerged openstack/neutron stable/2024.1: Lower spacing time of the OVN maintenance tasks which should be run once  https://review.opendev.org/c/openstack/neutron/+/92570917:33
opendevreviewMerged openstack/neutron stable/2023.1: Lower spacing time of the OVN maintenance tasks which should be run once  https://review.opendev.org/c/openstack/neutron/+/92571117:34
hamburglerWould anyone how often entries to OVN DB are supposed to be committed? or if this is a task triggered by certain action? what would be considered too long for an entry to be committed? 18:18
opendevreviewMerged openstack/ovn-bgp-agent master: Move some LogicalSwitchChassisEvent methods out  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/92554818:23
opendevreviewJakub Libosvar proposed openstack/ovn-bgp-agent master: Introduce LSP address column parsing functions  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/92580120:31
opendevreviewMerged openstack/neutron stable/2023.2: Lower spacing time of the OVN maintenance tasks which should be run once  https://review.opendev.org/c/openstack/neutron/+/92571023:08

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