Tuesday, 2023-02-28

opendevreviewMerged openstack/neutron-fwaas master: [sqlalchemy-20] The Connection.connect() method is considered legacy  https://review.opendev.org/c/openstack/neutron-fwaas/+/87521900:25
opendevreviewMerged openstack/neutron-fwaas master: Use upper-constraints in docs job  https://review.opendev.org/c/openstack/neutron-fwaas/+/87406200:25
opendevreviewMerged openstack/neutron master: Prevent router_ha_interface port from being removed via API  https://review.opendev.org/c/openstack/neutron/+/87493100:44
opendevreviewMiro Tomaska proposed openstack/neutron master: Reintroduce agent bridge resync test  https://review.opendev.org/c/openstack/neutron/+/87558603:21
opendevreviewMiro Tomaska proposed openstack/neutron master: Make retrieval of port mac column safe  https://review.opendev.org/c/openstack/neutron/+/87434203:48
opendevreviewLuis Tomas Bolivar proposed openstack/ovn-octavia-provider stable/zed: Ensure HM also apply to FIPs associated to LB VIPs  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/87559506:53
opendevreviewLuis Tomas Bolivar proposed openstack/ovn-octavia-provider stable/yoga: Ensure HM also apply to FIPs associated to LB VIPs  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/87559606:54
opendevreviewLuis Tomas Bolivar proposed openstack/ovn-octavia-provider stable/xena: Ensure HM also apply to FIPs associated to LB VIPs  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/87559706:54
opendevreviewLuis Tomas Bolivar proposed openstack/ovn-octavia-provider stable/wallaby: Ensure HM also apply to FIPs associated to LB VIPs  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/87559806:55
opendevreviewMamatisa Nurmatov proposed openstack/neutron master: Remove unused rbac rule  https://review.opendev.org/c/openstack/neutron/+/87530607:10
opendevreviewDmitrii Shcherbakov proposed openstack/neutron-lib master: ext-gw-multihoming: api-def and api-ref  https://review.opendev.org/c/openstack/neutron-lib/+/87088707:11
opendevreviewLuis Tomas Bolivar proposed openstack/neutron master: [OVN] Ensure traffic for provider vlan networks is not tunneled  https://review.opendev.org/c/openstack/neutron/+/87125207:20
pawan-guptahttps://www.irccloud.com/pastebin/FMR3QMWC/07:53
opendevreviewTakashi Kajinami proposed openstack/neutron master: Add missing osprofiler options  https://review.opendev.org/c/openstack/neutron/+/87562209:04
opendevreviewTakashi Kajinami proposed openstack/neutron master: Add missing osprofiler options  https://review.opendev.org/c/openstack/neutron/+/87562209:04
opendevreviewMerged openstack/neutron master: Normalise format of OVN agent heartbeat timestamp  https://review.opendev.org/c/openstack/neutron/+/87492609:05
opendevreviewTakashi Kajinami proposed openstack/neutron master: Add missing osprofiler options  https://review.opendev.org/c/openstack/neutron/+/87562209:06
opendevreviewRodolfo Alonso proposed openstack/neutron-vpnaas master: [sqlalchemy-20] Add reader context to ``VPNPluginDb`` get methods  https://review.opendev.org/c/openstack/neutron-vpnaas/+/87562609:35
slaweqralonsoh lajoskatona hi, I had to check (again) failed functional tests due to "interface not found" error. I did some analysis and wrote it in https://paste.opendev.org/show/b2a6RB41JeeEt5Rv51Of/ - please check it if You will have some time and tell me what You think about it09:41
ralonsohslaweq, let me check09:41
slaweqdoes it makes sense for You to add some "stupid" sleep(1) or something like that there?09:41
slaweqI know it would be ugly but I don't have any better idea TBH09:41
slaweqit's not urgent but would be good if You could check it before CI meeting today09:41
ralonsohso this is similar to the "shy ports" issue09:41
slaweqralonsoh thx09:42
ralonsohwe can discuss it today in the CI meeting09:42
slaweqI think it's still this "shy port"09:42
ralonsohbut yes, if a sleep(2) solves that, +2 from me09:42
slaweqyes, we can discuss it in the CI meeting but wanted to give You a heads up so You can prepare earlier :)09:42
ralonsohfor sure, thanks!09:43
lajoskatonaslaweq: let's discuss it this afternoon, sleep or busy_wait can be a good try09:45
lajoskatonaslaweq, ralonsoh: I played with it, see: https://review.opendev.org/c/openstack/neutron/+/873111/4/neutron/agent/linux/interface.py09:45
ralonsohah yes!09:46
lajoskatonaslaweq, ralonsoh: at that time it made no difference for me but perhaps it is just where we wait, perhaps I put it to the wrong place09:46
slaweqthx lajoskatona I added comment in Your patch09:47
opendevreviewRodolfo Alonso proposed openstack/neutron-vpnaas master: [sqlalchemy-20] Add reader context to ``VPNPluginDb`` get methods  https://review.opendev.org/c/openstack/neutron-vpnaas/+/87562610:06
pawan-guptaHi, I am trying to set up NAT networking on openstack instances/VMS.  1st vm is acting as a gateway with 2 interfaces. eth0 has public access and eth1 is connected to a private network.  Let's say these 2 IPs are 6X.2XX.1.2XX ( public IP) and 10.0.0.186 ( Private IP ). 2nd vm has only private IP on eth0 e.g. 10.0.0.251. This VM should be accessing the internet via gateway so I have set the default route as default 10:07
pawan-guptavia 10.0.0.186 dev eth0.  Enabled IPv4 packet forwarding and Masquerading on VM1 via the following `sysctl -w net.ipv4.conf.all.forwarding=1` and `iptables -t nat -A POSTROUTING -o eth0 -j MASQUERADE`. When I am trying to ping 8.8.8.8 from VM2,  I am not getting the packets back to this VM. I have installed tcpdump on VM1 to see the packets and noticed that packets are being sent to VM2 ( 10.0.0.251 ) but those are 10:07
pawan-guptanot reaching there as I notice 100% packet loss. I am not sure if DNAT is not happening properly. Any help in this will be greatly appreciated. below is the output of tcpdump10:07
pawan-guptaIP 10.0.0.251 > 8.8.8.8: ICMP echo request, id 1752, seq 11, length 6410:07
pawan-guptaIP 6X.2XX.1.2XX > 8.8.8.8: ICMP echo request, id 1752, seq 11, length 6410:07
pawan-guptaIP 8.8.8.8 > 6X.2XX.1.2XX: ICMP echo reply, id 1752, seq 11, length 6410:07
pawan-guptaIP 8.8.8.8 > 10.0.0.251: ICMP echo reply, id 1752, seq 11, length 6410:07
ralonsohpawan-gupta, this is not a Neutron routing problem. You can imitate how is done in the L3 code and what rules are added10:09
ralonsohor you can use a Neutron router instead10:09
pawan-gupta thanks @ralonsoh for the quick response. If I use a neutron router, the packets will flow with the ip configured there, I want to control the traffic from my own gateway. I will try to look into L3 code10:12
opendevreviewBodo Petermann proposed openstack/neutron-vpnaas master: VPNaaS support for OVN  https://review.opendev.org/c/openstack/neutron-vpnaas/+/76535310:15
opendevreviewRodolfo Alonso proposed openstack/neutron-vpnaas master: [sqlalchemy-20] Add writer context to ``BaseIPsecVPNDriver.create_vpnservice``  https://review.opendev.org/c/openstack/neutron-vpnaas/+/87562810:27
ralonsohlajoskatona, slaweq ^^ please check this patch and the parent one10:28
ralonsohin order to merge it asap and release the RC110:28
*** elodilles is now known as elodilles_afk10:28
opendevreviewMerged openstack/neutron-tempest-plugin master: Drop stateless_sg property check from base method  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/87410910:42
opendevreviewMerged openstack/neutron-vpnaas-dashboard master: Update master for stable/zed  https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/85766711:12
opendevreviewMerged openstack/neutron-vpnaas-dashboard master: Switch to 2023.1 Python3 unit tests and generic template name  https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/85766811:12
pawan-gupta@ralonsh, just to clarify, you meant `ip tables rules in neuton L3 agent code` for this `You can imitate how is done in the L3 code and what rules are added` or something else 11:22
pawan-gupta@ralonsoh just to clarify, you meant `ip tables rules in neuton L3 agent code` for this `You can imitate how is done in the L3 code and what rules are added` or something else 11:23
ralonsohyes, you can check how the routing is done in Neutron (legacy router code)11:24
ralonsohthis is just an example, you can check in google too11:24
ralonsohif needed, you can spawn a router with two interfaces11:25
ralonsohone private, one public (GW)11:25
ralonsohand check the iptables rules in the router namespace11:25
ralonsohthat will give you an idea of how to do the same in your VM11:25
opendevreviewMerged openstack/neutron-lib master: [sqlalchemy-20] Use Connection.execute() instead of Engine.execute()  https://review.opendev.org/c/openstack/neutron-lib/+/87486611:30
pawan-guptathanks @ralonsoh for your inputs11:32
ralonsohlucasagomes, hey! Do we need to backport https://review.opendev.org/c/openstack/neutron/+/875551/?12:01
opendevreviewSlawek Kaplonski proposed openstack/neutron-tempest-plugin master: [Stateless SG] Add test to check connectivity between vms  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/87482012:15
opendevreviewSlawek Kaplonski proposed openstack/neutron stable/zed: [Secure RBAC] Add shared_qos_policy rule  https://review.opendev.org/c/openstack/neutron/+/87560512:19
opendevreviewSlawek Kaplonski proposed openstack/neutron stable/zed: [Secure RBAC] Fix policy to get flavors  https://review.opendev.org/c/openstack/neutron/+/87560612:19
opendevreviewSlawek Kaplonski proposed openstack/neutron stable/zed: [Secure RBAC] Allow network owner to delete ports plugged to the network  https://review.opendev.org/c/openstack/neutron/+/87560712:19
opendevreviewSlawek Kaplonski proposed openstack/neutron stable/zed: Remove policy rule for get_network:router:external  https://review.opendev.org/c/openstack/neutron/+/87439812:20
opendevreviewSlawek Kaplonski proposed openstack/neutron-tempest-plugin master: [Secure RBAC] Add scope enforcement enabled job for Zed branch  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/87470912:21
opendevreviewSlawek Kaplonski proposed openstack/neutron master: [S-RBAC] Add release note about full support for new policies  https://review.opendev.org/c/openstack/neutron/+/87470612:28
opendevreviewFernando Royo proposed openstack/ovn-octavia-provider master: Remove HM uuiid from LB external_ids when the HM's pool is deleted  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/87540012:31
opendevreviewMerged openstack/networking-bgpvpn master: Remove TripleO job  https://review.opendev.org/c/openstack/networking-bgpvpn/+/84336212:39
opendevreviewFernando Royo proposed openstack/ovn-octavia-provider master: Remove HM uuiid from LB external_ids when the HM's pool is deleted  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/87540012:43
*** elodilles_afk is now known as elodilles12:44
opendevreviewMerged openstack/neutron-dynamic-routing master: CI: Add periodic weekly job with sqlalchemy master  https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/87256212:54
lucasagomesralonsoh, hi there, it would be good. At least down to wallaby if possible. Wanna me to propose it ?13:03
opendevreviewLucas Alvares Gomes proposed openstack/neutron stable/zed: Apply Ironic's server-ip-address as TFTP next-server  https://review.opendev.org/c/openstack/neutron/+/87566213:07
opendevreviewLucas Alvares Gomes proposed openstack/neutron stable/yoga: Apply Ironic's server-ip-address as TFTP next-server  https://review.opendev.org/c/openstack/neutron/+/87566313:07
ralonsohlucasagomes, thanks!13:07
opendevreviewLucas Alvares Gomes proposed openstack/neutron stable/xena: Apply Ironic's server-ip-address as TFTP next-server  https://review.opendev.org/c/openstack/neutron/+/87566413:07
opendevreviewLucas Alvares Gomes proposed openstack/neutron stable/wallaby: Apply Ironic's server-ip-address as TFTP next-server  https://review.opendev.org/c/openstack/neutron/+/87566513:08
ralonsohltomasbo, hi! please tell me what patches need to be merged in the next two days for the RC113:19
ralonsohhttps://review.opendev.org/q/project:openstack/ovn-octavia-provider+status:open13:19
ralonsoh--> https://review.opendev.org/c/openstack/releases/+/87545113:19
ltomasboralonsoh, let me see13:19
ltomasboralonsoh, these will be nice to have: https://review.opendev.org/c/openstack/ovn-octavia-provider/+/875400, https://review.opendev.org/c/openstack/ovn-octavia-provider/+/874609, the others I think can wait13:21
ltomasbofroyo ^13:21
ralonsohcool, I'll review them today13:22
ltomasboralonsoh thanks!13:22
froyoralonsoh, ltomasbo: yeah, also I will upload today a new one adding FTs13:23
ralonsohperfect13:23
opendevreviewBodo Petermann proposed openstack/neutron-vpnaas master: VPNaaS support for OVN  https://review.opendev.org/c/openstack/neutron-vpnaas/+/76535313:35
opendevreviewLajos Katona proposed openstack/python-neutronclient master: WIP: OSC: Remove BGPVPN calls to neutronclient  https://review.opendev.org/c/openstack/python-neutronclient/+/87572813:44
*** luis5tb is now known as ltomasbo14:00
opendevreviewMerged openstack/neutron master: Remove unused rbac rule  https://review.opendev.org/c/openstack/neutron/+/87530614:00
opendevreviewMerged openstack/neutron stable/yoga: Enable qos extension_driver in ovn_db_sync  https://review.opendev.org/c/openstack/neutron/+/87414314:00
opendevreviewLajos Katona proposed openstack/networking-odl master: DNM: Test master  https://review.opendev.org/c/openstack/networking-odl/+/87573114:01
ralonsoh#startmeeting networking14:02
opendevmeetMeeting started Tue Feb 28 14:02:51 2023 UTC and is due to finish in 60 minutes.  The chair is ralonsoh. Information about MeetBot at http://wiki.debian.org/MeetBot.14:02
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.14:02
opendevmeetThe meeting name has been set to 'networking'14:02
ralonsohping bcafarel, elvira, frickler, mlavalle, mtomaska, obondarev, sahid, slawek, tobias-urdin, ykarel, lajoskatona, jlibosva, averdagu 14:03
lajoskatonao/14:03
frickler\o14:03
rubasovo/14:03
obondarevhi14:03
bcafarelo/14:03
haleybo/14:03
ralonsohhello all14:04
ralonsohwe have a tight agenda, so let's begin14:04
ralonsoh#topic announcements14:04
ralonsohFirst of all, the OpenStack schedule14:04
ralonsoh#link https://releases.openstack.org/antelope/schedule.html14:04
ralonsohthis is the RC-1 week14:05
ralonsohand this is why we have a bunch of release patches to review14:05
ralonsoh#link https://paste.opendev.org/show/bVXlabBzaJ1OXCaMWNVo/14:05
ralonsohI've reviewed all of them here14:05
ralonsohhttps://etherpad.opendev.org/p/ralonsoh_cases14:05
ralonsohFor ovn-octavia-provider, there are a couple of patches that need to be merged in the next days14:06
slaweqo/14:06
ralonsohbut these patches are attended14:06
ralonsohabout vpnaas, I have more doubts14:06
ralonsohhttps://review.opendev.org/q/project:openstack/neutron-vpnaas+status:open14:06
ralonsohapart from the sqlalchemy patches14:07
ralonsohdo you know what should be merged before the RC1?14:07
lajoskatonagood question to discuss14:07
ralonsoh3 of them have a -W, I won't consider them14:08
ralonsohthe OVN patch cannot be merged 2 days before the release14:08
lajoskatonar1 is this week, so I would keep things back14:08
ralonsohand then we have older patches14:08
ralonsohso you propose to keep the current hash, right lajoskatona?14:09
ralonsohwell, we should merge the sqlalchemy ones14:09
slaweqif something is fix for critical issue we can always merge it next week, do backport to Antelope and have RC2 for that project14:09
ralonsohbecause those are legit bugs14:09
ralonsohhttps://review.opendev.org/c/openstack/neutron-vpnaas/+/87562814:09
ralonsohhttps://review.opendev.org/c/openstack/neutron-vpnaas/+/87562614:09
lajoskatonaralonsoh, slaweq: yes we can backport anyway14:10
ralonsohslaweq, ok (in any case, please check ^^ these patches)14:10
slaweqsure, I will14:10
ralonsohI'll update, if needed, the release patch hash14:10
ralonsohand about Neutron, I need to review the active patches14:10
ralonsohbut we should focus now in bug fixes, not features14:11
lajoskatonaRC1 release patches are out now: https://review.opendev.org/q/project:openstack/releases+RC1 (at least some)14:11
ralonsohI've sent the link14:12
ralonsoh#link https://paste.opendev.org/show/bVXlabBzaJ1OXCaMWNVo/14:12
ralonsohthis is the list of patches related to Neutron14:12
lajoskatonaohh, yeah the same but filtered :-)14:12
ralonsohplease don't hesitate to comment on them, I'll review them every day14:12
lajoskatonaack14:13
ralonsohand a kindly reminder for the Bobcat etherpad: https://etherpad.opendev.org/p/neutron-bobcat-ptg14:13
ralonsohplease add your topics for the next PTG14:13
ralonsohand check the openinfra videos: https://openinfra.dev/live/#all-episodes14:14
ralonsohtwo weeks ago we had a new one: VGPU Management by OpenStack Nova and Cyborg14:14
ralonsoh#link https://www.youtube.com/watch?v=WUdkS9558p814:14
ralonsohany other announcement?14:14
ralonsohlet's move then14:15
ralonsoh#topic bugs14:15
ralonsohlast week report is from bcafarel 14:16
ralonsoh#link https://lists.openstack.org/pipermail/openstack-discuss/2023-February/032453.html14:16
ralonsohanother busy week14:16
bcafarelsome people are quite productive :)14:16
ralonsohthe vpnaas bugs commented are already addressed14:16
bcafareland thanks ralonsoh for picking up the 2 vpnaas bugs14:16
ralonsohyw14:16
lajoskatona+114:16
ralonsohthe only one not assigned (or with a patch proposed) is14:17
ralonsoh#link https://bugs.launchpad.net/neutron/+bug/200806214:17
opendevreviewMerged openstack/neutron stable/wallaby: Enable qos extension_driver in ovn_db_sync  https://review.opendev.org/c/openstack/neutron/+/87414514:17
ralonsohbut maybe we'll go back to this one during the CI meeting (in 40 mins)14:18
ralonsohand this week lajoskatona is the deputy, next week will be slaweq14:18
slaweq++14:18
slaweqok14:18
lajoskatonaack14:18
ralonsohthere are no new patches in ryu/os-ken so we can skip this section14:19
ralonsohsame for the specs14:19
ralonsohso let's move to the next topic14:19
ralonsoh#topic community_goals14:19
ralonsoh1) Consistent and Secure Default RBAC 14:19
ralonsohhere I would like to bring the topic proposed by lajoskatona 14:20
ralonsohsRBAC patch backports to stable/zed 14:20
ralonsohthe community is asking us to backport the sRBAC functionality to Zed14:20
ralonsohslaweq, you have proposed most of the patches14:20
ralonsohbut there are some of then that should be discussed14:21
slaweqI proposed job for stable zed https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/87470914:21
slaweqand this patch depends on few backports14:21
ralonsohperfect, the commit message is a good summary for this14:21
slaweqif I'm not mistaken, that should be complete list of what's needed to make this job greep14:21
opendevreviewMerged openstack/neutron-lib master: port-hint-ovs-tx-steering: api definition  https://review.opendev.org/c/openstack/neutron-lib/+/87311214:21
slaweq*green14:21
lajoskatonayeah, I just brought it again as last week we discussed it and after more discussion with gmann it seems it is better to backport these14:21
slaweqbut lets see CI result first14:21
ralonsohperfect, so we all agree to backport this funcitonality to Zed?14:22
slaweqIMO it's good to backport those fixes14:22
ralonsoh(if CI is green and there are no conflicts with the current functionality)14:22
lajoskatona+1, from me14:22
slaweqofficially this new S-RBAC policies aren't supported in Zed but still some folks may want to use them14:22
slaweqlike e.g. mnaser 14:23
ralonsohI know, this is why I'm bringing this topic here14:23
slaweq:)14:23
ralonsohmy main concern is that we don't break anything in Zed14:23
slaweqno, it won't break anything for sure14:24
ralonsohperfect, so +1 from me. I'll review the patches during the week14:24
ralonsohand thanks for taking care of this14:24
ralonsohthe next topic is14:24
ralonsoh2) Neutron client deprecation 14:24
ralonsohlajoskatona, any comment on this?14:25
lajoskatonaopenstacksdk 1.0.1 is out now14:25
lajoskatonaplease check the etherpad: https://etherpad.opendev.org/p/python-neutronclient_deprecation14:25
lajoskatonasome ready to review patches: https://review.opendev.org/c/openstack/tap-as-a-service/+/86778614:25
lajoskatonahttps://review.opendev.org/c/openstack/python-neutronclient/+/86832114:26
lajoskatonaAnd just to try it I jumped into designate and octavia, and the designate one was an easy:14:26
lajoskatonahttps://review.opendev.org/c/openstack/designate/+/87499114:26
lajoskatonabut I have to discuss it with designate team14:26
lajoskatonaso some progress to start Bobcat slowly :-)14:27
ralonsohthat's a fantastic job14:27
ralonsohbtw, I'll add this etherpad to the meeting agenda14:27
lajoskatonathanks14:27
ralonsohand please, ^^^ @all check these patches14:27
lajoskatonagood idea to make it more visibl14:27
lajoskatonae----^14:27
ralonsohdone (including the active patches)14:29
ralonsohok, we are going faster than I expected (that's good)14:29
ralonsohlast section 14:29
ralonsoh#topic on_demand14:30
ralonsohI have a topic here14:30
ralonsohThere is a list of lieutenants for different areas 14:30
ralonsoh#link https://docs.openstack.org/neutron/latest/contributor/policies/neutron-teams.html14:30
ralonsohsome of them have only one single assignee (API, DB, control plane, client, docs, testing)14:31
ralonsohso the idea is to have at least two14:31
lajoskatona+114:31
slaweqI can help with some of them for sure14:32
ralonsohof course we can't force anyone to be there, but the idea is to propose yourself14:32
ralonsohslaweq, you can suggest any area14:32
ralonsohof course, the more people we have, the better14:32
slaweqYou can add me to API and Testing14:32
ralonsohand, btw, you don't need to be core14:33
slaweqand maybe Client if needed14:33
ralonsohslaweq, OK, I'll add you to these areas14:33
opendevreviewAnton Kurbatov proposed openstack/neutron stable/zed: Prevent router_ha_interface port from being removed via API  https://review.opendev.org/c/openstack/neutron/+/87573714:33
slaweq++14:33
slaweqthx14:33
lajoskatonaI can also add myself14:33
ralonsohlajoskatona, what areas?14:33
ralonsohbtw, you can send your own patch hehehe14:34
lajoskatonaI anyway play with client code recently, but API can also be 14:34
lajoskatonasure14:34
ralonsohcool!14:34
ralonsohand this also for all the people in the ping list14:35
opendevreviewAnton Kurbatov proposed openstack/neutron stable/yoga: Prevent router_ha_interface port from being removed via API  https://review.opendev.org/c/openstack/neutron/+/87573814:35
slaweqbtw. I'm not sure if yamamoto wants still to be there :)14:35
ralonsohso please, you just need to push a patch and add your name in this list14:35
slaweqhe's not active recently14:35
bcafarel+1 for folks in ping list especially as it does not require core14:35
ralonsohI'll add a separate patch removing him14:35
slaweqok, I will propose myself there14:35
ralonsohand I'll add him to the review14:35
ralonsohand then there is the release liaison14:36
opendevreviewAnton Kurbatov proposed openstack/neutron stable/xena: Prevent router_ha_interface port from being removed via API  https://review.opendev.org/c/openstack/neutron/+/87573914:36
ralonsohI talked to amotoki (slawek, lajoskatona and mlavalle were in this mail too)14:36
frickler"Infra" means opendev-infra?14:37
ralonsohand he said that it will be better to step down from being the release liaison14:37
ralonsohfrickler, yes and everything related to zuul and so14:37
fricklerralonsoh: ok, if you drop yamamoto there, you can add me instead, then14:38
ralonsohfrickler, I'll push a patch for that, thanks!14:38
* frickler can talk to self that way ;)14:38
bcafarelas long as you agree with yourself :)14:39
frickleroh, mostly I don't, sadly :D14:39
ralonsohso I'm going to propose a patch to drop amotoki from here14:39
ralonsoh#link https://opendev.org/openstack/releases/src/branch/master/data/release_liaisons.yaml#L8614:39
slaweqit will be like in this move "me, Irena and myself"14:39
amotokiI could not review release patches in time recently, so I think it is better to pass the release liaison role to someone14:39
slaweqbut instead of Irena there will be "Infra" :P14:39
ralonsohamotoki, thanks for all these years taking care of it14:39
bcafarel+10014:40
amotokiralonsoh: yw14:40
slaweqralonsoh and I can help with releases and be release liaison if it's ok for everyone14:40
slaweqamotoki++ thx for great work in that role for many years14:40
lajoskatonathanks amotoki :-)14:40
ralonsohslaweq, that will be perfect. I'm going to add me too (despite the PTL is always added by default to those patches)14:41
ralonsohwe can have more than one person14:41
amotokithe list of cross project liaisons are found here https://wiki.openstack.org/wiki/CrossProjectLiaisons14:42
slaweqralonsoh You don't need to be added as release liaison - PTL is treated in the same way in that case14:42
ralonsohuhh this is something I need to review too14:42
ralonsohslaweq, ok14:43
ralonsohok, I'll check https://wiki.openstack.org/wiki/CrossProjectLiaisons too, but after this meeting14:43
ralonsohamotoki, thanks for this!14:43
ralonsoh(too many lists!)14:43
amotokiI am not sure the list is well maintained, but it explains what is expected14:44
ralonsohok folks, this is all we have in the agenda14:45
ralonsohany other topic??14:45
opendevreviewMerged openstack/neutron master: Apply Ironic's server-ip-address as TFTP next-server  https://review.opendev.org/c/openstack/neutron/+/87555114:45
opendevreviewMerged openstack/neutron master: Remove two duplicated unit tests  https://review.opendev.org/c/openstack/neutron/+/87557114:46
slaweqralonsoh there is also https://github.com/openstack/releases/blob/master/data/release_liaisons.yaml14:46
opendevreviewMerged openstack/neutron master: Move register_common_config_options call to DietTestCase  https://review.opendev.org/c/openstack/neutron/+/87555214:46
slaweqwhich I think is used in the releases tools to add release liaisons to reviews14:46
ralonsohyes, this is the link I sent before, I think14:46
slaweqahh, ok14:46
ralonsoh(well, mine was from opendev)14:46
amotokislaweq: yes, the release liaisons are maintained in that file and it is mentioend in the wiki page too14:46
slaweqmaybe I missed it14:46
slaweqsorry14:46
ralonsohhehehe14:46
ralonsohdon't forget that we have the CI meeting in 15 mins in this channel (today via IRC, if I'm not wrong)14:47
ralonsohsee you all14:47
bcafarelo/14:47
ralonsoh#endmeeting14:47
opendevmeetMeeting ended Tue Feb 28 14:47:34 2023 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)14:47
opendevmeetMinutes:        https://meetings.opendev.org/meetings/networking/2023/networking.2023-02-28-14.02.html14:47
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/networking/2023/networking.2023-02-28-14.02.txt14:47
opendevmeetLog:            https://meetings.opendev.org/meetings/networking/2023/networking.2023-02-28-14.02.log.html14:47
lajoskatonao/14:47
slaweqyes, via IRC14:47
amotokio/14:47
opendevreviewBrian Haley proposed openstack/neutron master: [S-RBAC] Add release note about full support for new policies  https://review.opendev.org/c/openstack/neutron/+/87470614:47
bcafarellajoskatona: btw for neutron client deprecation, gthiemon told me it was almost done for octavia already (so you don't end up duplicating work) - and nice to see designate patch14:49
opendevreviewLuis Tomas Bolivar proposed openstack/neutron master: Update lieutenants for ovn-octavia-provider  https://review.opendev.org/c/openstack/neutron/+/87563714:50
opendevreviewMiro Tomaska proposed openstack/neutron master: Make retrieval of port mac column safe  https://review.opendev.org/c/openstack/neutron/+/87434214:50
ralonsohfolks: https://review.opendev.org/c/openstack/releases/+/87574014:51
ralonsohamotoki, slaweq ^^14:51
lajoskatonabcafarel: thanks,14:55
slaweq#startmeeting neutron_ci15:00
opendevmeetMeeting started Tue Feb 28 15:00:10 2023 UTC and is due to finish in 60 minutes.  The chair is slaweq. 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
slaweqping bcafarel, lajoskatona, mlavalle, mtomaska, ralonsoh, ykarel, jlibosva for CI meeting15:00
ralonsohhi15:00
slaweqo/15:00
*** kopecmartin_ is now known as kopecmartin15:00
slaweqGrafana dashboard: https://grafana.opendev.org/d/f913631585/neutron-failure-rate?orgId=115:00
mtomaskao/15:00
bcafarelo/15:01
opendevreviewDmitrii Shcherbakov proposed openstack/neutron master: Allow Multiple External Gateways  https://review.opendev.org/c/openstack/neutron/+/87359315:01
opendevreviewDmitrii Shcherbakov proposed openstack/neutron master: Add extra router attributes for ECMP and BFD  https://review.opendev.org/c/openstack/neutron/+/87479715:01
slaweqok, lets start15:01
slaweqit needs to be a bit shorter meeting than usual as I need to finish around 16:45 today15:02
slaweqsorry for that15:02
slaweq#topic Actions from previous meetings15:02
lajoskatonao/15:02
slaweqlajoskatona to continue checking dvr functional tests issues15:02
lajoskatonanot much progress, but we discussed today that some wait/sleep or busy-wait should be added to plug_new15:03
lajoskatonaso I will check it, thanks for the comment slaweq15:03
slaweqyeah, I was checking other issue, with interface not found and my analysis is here: https://paste.opendev.org/show/b2a6RB41JeeEt5Rv51Of/15:03
slaweqmaybe the same would fix that other DVR failures too15:03
lajoskatonaand the patch I used recently to try different aproaches: https://review.opendev.org/c/openstack/neutron/+/87311115:04
slaweqlajoskatona I will propose simple patch with sleep and will rebase Your test patch on top of it to test if that will help15:04
lajoskatonayes most probably as I found the same root method plug_new, so I hope we can catch it there15:04
slaweqok for You?15:04
lajoskatonayes let's do it15:04
slaweqthx15:04
slaweq#action slaweq to propose "sleep patch" to hopefully fix interface not found and dvr tests issues15:05
lajoskatonabut in recent runs I havent seen the issue, only other failures15:05
slaweqslaweq to check https://64e9807acd80d4cab1ab-851182d93d98b3728f798963c90c7371.ssl.cf5.rackcdn.com/periodic/opendev.org/openstack/neutron/master/neutron-functional-with-uwsgi-fips/1914083/testr_results.html15:06
slaweqit's the one which I already mentioned above15:06
slaweqso lets move on to the next one15:06
slaweqslaweq to report bug about macspoofing_port test in ovn-tempest-slow job15:06
slaweqok, next one15:06
slaweqbug reported https://bugs.launchpad.net/neutron/+bug/200806215:07
slaweqit's the only one not assigned to anyone15:07
slaweqas You mentioned in the previous meeting ralonsoh 15:07
ralonsohif I have time this week, I'll check it and assign it to myself15:08
ralonsohbut I can't promise that15:08
mtomaskaI can take a look next week15:08
slaweqthx15:08
mtomaskaso let me take it15:08
slaweqI didn't saw it this week15:08
slaweqso it's not that often15:08
slaweqthx mtomaska 15:09
slaweqI will assign AI for You for next week15:09
slaweq#action mtomaska to check https://bugs.launchpad.net/neutron/+bug/200806215:09
mtomaskaok15:09
slaweqand last one:15:09
slaweqralonsoh to check neutron-functional-with-sqlalchemy-master failures15:09
ralonsohyes, I've opened a bug for oslo.db15:09
ralonsohhttps://bugs.launchpad.net/oslo.db/+bug/200820915:09
ralonsohthere is a patch under review15:09
ralonsohand I've pushed this for Neutron15:09
ralonsohhttps://review.opendev.org/c/openstack/neutron/+/87477815:10
ralonsohthis patch should NOT be merged, but I'll rebase it frequently15:10
ralonsohto check the warnings still present in the code15:10
ralonsohthat's all15:10
slaweqthx for taking care of it ralonsoh 15:11
bcafarelgreat work there :)15:11
slaweqit's really important to make all those fixes and be ready for sqlalchemy 2.0 :)15:11
slaweqand also thx zzzeek for huge help with this issue15:11
ralonsohfor sure!15:11
slaweqI think we can now move on to the next topic15:12
slaweq#topic Stable branches15:12
slaweqbcafarel the floor is Yours :)15:12
opendevreviewChristian Rohmann proposed openstack/neutron-vpnaas master: Improve the reconciling for VPNaaS  https://review.opendev.org/c/openstack/neutron-vpnaas/+/87574515:13
opendevreviewRodolfo Alonso proposed openstack/neutron master: Add Slawomir Kaplonski as Lieutenant in API, Client and Testing areas  https://review.opendev.org/c/openstack/neutron/+/87574615:13
bcafarelI was away a few days last week so I may have missed a few things - but overall stable branches are in good shape15:13
bcafarelI wonder about ussuri 2 recent backports had a lot of failures https://review.opendev.org/c/openstack/neutron/+/873629 https://review.opendev.org/c/openstack/neutron/+/874979 but I did not have time to look into yet15:14
slaweqyeah, worth to check15:15
haleybbcafarel: i am interested as well so can probably look as one of those is from someone here, let me know15:15
bcafarelhaleyb: that would be great! I don't think I will have lots of time this week15:16
slaweqthx haleyb and bcafarel - if You will need any help, please let me know15:17
slaweqok, next topic15:18
slaweq#topic Stadium projects15:18
slaweqlajoskatona how things are going?15:18
slaweqI saw that networking-sfc and networking-odl were red this week15:18
lajoskatonayes, that is due to sqlalchemy2 with the new job, I will check it15:18
lajoskatonaI mean for sfc15:18
slaweqok15:18
opendevreviewChristian Rohmann proposed openstack/neutron-vpnaas master: Improve the reconciling for VPNaaS  https://review.opendev.org/c/openstack/neutron-vpnaas/+/87574515:18
lajoskatonafor ODL I am not sure, I started to check15:19
opendevreviewBrian Haley proposed openstack/neutron stable/ussuri: DNM: Testing Ussuri gate  https://review.opendev.org/c/openstack/neutron/+/87574715:19
slaweqthx lajoskatona 15:20
lajoskatonathat;s it from me15:20
opendevreviewRodolfo Alonso proposed openstack/neutron master: Add Jens Harbott as Lieutenants in Infra area  https://review.opendev.org/c/openstack/neutron/+/87574815:20
slaweqso lets move on15:20
slaweq#topic Grafana15:20
slaweq#link https://grafana.opendev.org/d/f913631585/neutron-failure-rate15:21
slaweqIMO things looks better this week15:21
slaweqthere are some failures, especially during weekend there were some spikes but I saw many patches marked as "WIP" this week and those were very red15:22
slaweqso I don't see there any specific issue this week15:22
ralonsohmuch better since last week fixes15:22
slaweqyeah15:22
slaweqI think we can smoothly go to the next topic which is15:23
slaweq#topic Rechecks15:23
slaweqand here it also seems that it's better this week15:23
slaweqwe are again below 1 recheck in average to merge patches15:23
ralonsohcool15:23
lajoskatona\o/15:24
slaweqthx all for hard work with the gate failures15:24
mtomaska\o\`15:24
slaweqI hope that if we will finally fix those functional tests issues it will be really good :)15:24
slaweqso now lets talk about particular failures :)15:25
slaweq#topic fullstack/functional15:25
slaweqhere I found few issues which we already discussed today, related to dvr lifecycle and interface not found issues15:25
slaweqbut except that I saw also:15:25
slaweqInitial state of ha router failed:15:25
slaweqhttps://45c66512638ca6be811a-ed2c0d9273209e4be8351fb65446e61d.ssl.cf1.rackcdn.com/periodic/opendev.org/openstack/neutron/master/neutron-functional-with-oslo-master/90c9e81/testr_results.html15:25
slaweqI think we saw that one few weeks ago too15:25
slaweqralonsoh do You maybe remember it?15:26
ralonsohyes but I didn't check it15:26
ralonsohthis code, to be honest, is not very stable15:27
ralonsoh(not the agent but the test itself)15:27
ralonsohI'll check it tomorrow15:27
slaweqthx a lot15:27
opendevreviewFernando Royo proposed openstack/ovn-octavia-provider master: Add new FTs for health monitoring basic operations  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/87574915:28
slaweq#action ralonsoh to check failed initail state of ha router test: https://45c66512638ca6be811a-ed2c0d9273209e4be8351fb65446e61d.ssl.cf1.rackcdn.com/periodic/opendev.org/openstack/neutron/master/neutron-functional-with-oslo-master/90c9e81/testr_results.html15:28
slaweqand next one15:28
slaweqneutron.tests.functional.agent.test_dhcp_agent.DHCPAgentOVSTestCase.test_good_address_allocation15:28
slaweqhttps://359f4eae1c9be4d72e49-0343763e072aec8a44b36b196f4e6ba5.ssl.cf2.rackcdn.com/periodic/opendev.org/openstack/neutron/master/neutron-functional-with-uwsgi-fips/ff7e0fd/testr_results.html15:28
slaweqalso I think we have seen it already before15:28
slaweqyeah, many times actually15:29
slaweqI see it in etherpad15:29
ralonsohbut I pushed a patch last week (I think so) related to this test15:29
opendevreviewMerged openstack/networking-sfc master: Update master for stable/zed  https://review.opendev.org/c/openstack/networking-sfc/+/85768315:29
slaweqso hopefully it will be fixed15:29
slaweqis Your patch merged?15:29
ralonsohchecking now15:29
slaweqmaybe we spot it before Your patch was merged15:29
ralonsohhttps://review.opendev.org/c/openstack/neutron/+/87465815:30
ralonsohand https://review.opendev.org/c/openstack/neutron/+/874654/1 (merged)15:30
slaweqok, but failure was on 26.0215:31
slaweqso before this patch was merged15:31
slaweqhopefully with those fixes we will not see that issue again :)15:31
ralonsohperfect15:31
slaweqeven 24.0215:31
slaweqbut still before patch was merged :)15:32
slaweqthx ralonsoh 15:32
slaweqand that's all what I have for today in the agenda15:32
slaweqnot many new failures this week, which is good15:32
slaweqanything else You want to discuss today as we have few more minutes left15:32
slaweq#topic On Demand15:33
opendevreviewMerged openstack/neutron master: [OVN] Ensure traffic for provider vlan networks is not tunneled  https://review.opendev.org/c/openstack/neutron/+/87125215:34
opendevreviewMerged openstack/neutron master: Change neutron-ovs-tempest-dvr-ha-multinode-full job's config  https://review.opendev.org/c/openstack/neutron/+/87453615:34
slaweqif not, I will give You some time back today15:34
mtomaskao/15:34
slaweqthx for attending the meeting and helping with CI stability :)15:34
ralonsohbye!15:34
slaweqhave a great day!15:34
lajoskatonao/15:34
slaweqo/15:34
slaweq#endmeeting15:34
opendevmeetMeeting ended Tue Feb 28 15:34:56 2023 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:34
opendevmeetMinutes:        https://meetings.opendev.org/meetings/neutron_ci/2023/neutron_ci.2023-02-28-15.00.html15:34
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/neutron_ci/2023/neutron_ci.2023-02-28-15.00.txt15:34
opendevmeetLog:            https://meetings.opendev.org/meetings/neutron_ci/2023/neutron_ci.2023-02-28-15.00.log.html15:34
opendevreviewMiro Tomaska proposed openstack/neutron master: RPC sighup test is failing, suspecting due to stestr concurrency  https://review.opendev.org/c/openstack/neutron/+/87434515:34
opendevreviewLuis Tomas Bolivar proposed openstack/neutron stable/zed: [OVN] Ensure traffic for provider vlan networks is not tunneled  https://review.opendev.org/c/openstack/neutron/+/87567315:36
opendevreviewLuis Tomas Bolivar proposed openstack/neutron stable/yoga: [OVN] Ensure traffic for provider vlan networks is not tunneled  https://review.opendev.org/c/openstack/neutron/+/87567415:37
opendevreviewLuis Tomas Bolivar proposed openstack/neutron stable/xena: [OVN] Ensure traffic for provider vlan networks is not tunneled  https://review.opendev.org/c/openstack/neutron/+/87567515:37
opendevreviewLuis Tomas Bolivar proposed openstack/neutron stable/wallaby: [OVN] Ensure traffic for provider vlan networks is not tunneled  https://review.opendev.org/c/openstack/neutron/+/87567615:38
opendevreviewMerged openstack/neutron stable/zed: Normalise format of OVN agent heartbeat timestamp  https://review.opendev.org/c/openstack/neutron/+/87548516:49
opendevreviewMerged openstack/neutron stable/yoga: Normalise format of OVN agent heartbeat timestamp  https://review.opendev.org/c/openstack/neutron/+/87548616:49
opendevreviewMerged openstack/neutron stable/zed: Format correctly (dialect=mac_unix_expanded) the MAC addresses  https://review.opendev.org/c/openstack/neutron/+/87553516:49
opendevreviewMerged openstack/neutron stable/yoga: Format correctly (dialect=mac_unix_expanded) the MAC addresses  https://review.opendev.org/c/openstack/neutron/+/87553616:49
opendevreviewSlawek Kaplonski proposed openstack/neutron master: Add sleep before checking if ovs port is in the namespace  https://review.opendev.org/c/openstack/neutron/+/87576717:32
opendevreviewSlawek Kaplonski proposed openstack/neutron master: WIP: desperate try to make DVR functional tests more stable  https://review.opendev.org/c/openstack/neutron/+/87311117:33
opendevreviewSlawek Kaplonski proposed openstack/neutron master: Add slaweq as API and Testing lieutenant  https://review.opendev.org/c/openstack/neutron/+/87577117:36
haleybslaweq: there was another lieutenant review i saw for you too, https://review.opendev.org/c/openstack/neutron/+/87574617:38
slaweqahh, ok17:38
slaweqthx haleyb 17:38
slaweqI though that ralonsoh told me to propose patch on my own17:39
slaweqbut that's fine too17:39
slaweqI will abandon mine17:39
haleybif you're happy with more roles i'll approve it :)17:40
ralonsohslaweq, sorry for the confusion 17:43
slaweqralonsoh: no problem at all17:43
opendevreviewRodolfo Alonso proposed openstack/neutron-vpnaas master: [sqlalchemy-20] Add reader context to ``VPNPluginDb`` get methods  https://review.opendev.org/c/openstack/neutron-vpnaas/+/87562617:46
opendevreviewRodolfo Alonso proposed openstack/neutron stable/wallaby: Format correctly (dialect=mac_unix_expanded) the MAC addresses  https://review.opendev.org/c/openstack/neutron/+/87553817:52
opendevreviewMerged openstack/neutron master: Add Slawomir Kaplonski as Lieutenant in API, Client and Testing areas  https://review.opendev.org/c/openstack/neutron/+/87574621:11
opendevreviewDmitrii Shcherbakov proposed openstack/neutron master: Allow Multiple External Gateways  https://review.opendev.org/c/openstack/neutron/+/87359321:14
opendevreviewDmitrii Shcherbakov proposed openstack/neutron master: Add extra router attributes for ECMP and BFD  https://review.opendev.org/c/openstack/neutron/+/87479721:14
opendevreviewDmitrii Shcherbakov proposed openstack/neutron master: [ovn] Allow L3 scheduler to be aware of current transaction  https://review.opendev.org/c/openstack/neutron/+/87476021:14
opendevreviewDmitrii Shcherbakov proposed openstack/neutron master: [ovn] Implement support for external-gateway-multihoming extension  https://review.opendev.org/c/openstack/neutron/+/87419921:14
opendevreviewDmitrii Shcherbakov proposed openstack/neutron master: [ovn] Add helper for retrieving LR associated with LRP  https://review.opendev.org/c/openstack/neutron/+/87369821:14
opendevreviewDmitrii Shcherbakov proposed openstack/neutron master: [ovn] Apply soft anti-affinity for LRs with multiple LRPs when scheduling  https://review.opendev.org/c/openstack/neutron/+/87369921:14
opendevreviewBrian Haley proposed openstack/neutron-tempest-plugin master: Do not use ubuntu minimal image in ussuri scenario jobs  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/87578622:05
opendevreviewBrian Haley proposed openstack/neutron stable/ussuri: DNM: Testing Ussuri gate  https://review.opendev.org/c/openstack/neutron/+/87574722:08
opendevreviewMerged openstack/neutron master: Make retrieval of port mac column safe  https://review.opendev.org/c/openstack/neutron/+/87434222:43
opendevreviewMerged openstack/neutron master: Add missing osprofiler options  https://review.opendev.org/c/openstack/neutron/+/87562223:54

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