Thursday, 2023-05-25

opendevreviewMiguel Lavalle proposed openstack/neutron master: [PoC][DNM] Router flavors and service type for OVN  https://review.opendev.org/c/openstack/neutron/+/88398800:10
opendevreviewZhouHeng proposed openstack/neutron-tempest-plugin master: Disassociate all ports before delete firewall group  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/88382600:55
opendevreviewZhouHeng proposed openstack/neutron-tempest-plugin master: [fwaas]Disassociate all ports before deleting firewall group  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/88382601:06
opendevreviewZhouHeng proposed openstack/neutron-tempest-plugin master: [fwaas]Disassociate all ports before deleting firewall group  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/88382602:14
opendevreviewyatin proposed openstack/neutron master: [DNM] reproduce grenade issue  https://review.opendev.org/c/openstack/neutron/+/88362903:53
opendevreviewFrode Nordahl proposed openstack/neutron master: [ovn] Honor `enable_default_route_ecmp` attribute  https://review.opendev.org/c/openstack/neutron/+/87853105:48
opendevreviewFrode Nordahl proposed openstack/neutron master: [ovn] Allow L3 scheduler to be aware of current transaction  https://review.opendev.org/c/openstack/neutron/+/87476005:48
opendevreviewFrode Nordahl proposed openstack/neutron master: [ovn] Add helper for retrieving LR associated with LRP  https://review.opendev.org/c/openstack/neutron/+/87369805:48
opendevreviewFrode Nordahl proposed openstack/neutron master: [ovn] Apply soft anti-affinity for LRs with multiple LRPs when scheduling  https://review.opendev.org/c/openstack/neutron/+/87369905:48
opendevreviewFrode Nordahl proposed openstack/neutron master: [ovn] Add support for enable_default_route_bfd attribute  https://review.opendev.org/c/openstack/neutron/+/87854305:48
opendevreviewZhouHeng proposed openstack/neutron-tempest-plugin master: [fwaas]Disassociate all ports before deleting firewall group  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/88382605:53
opendevreviewRodolfo Alonso proposed openstack/neutron master: Improve the ``PortBindingUpdateVirtualPortsEvent`` match filter  https://review.opendev.org/c/openstack/neutron/+/88368107:09
opendevreviewZhouHeng proposed openstack/neutron-tempest-plugin master: [fwaas]Disassociate all ports before deleting firewall group  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/88382607:57
opendevreviewLajos Katona proposed openstack/networking-odl master: Fix issues due to rcent RBAC changes  https://review.opendev.org/c/openstack/networking-odl/+/88325108:05
opendevreviewZhouHeng proposed openstack/neutron-fwaas master: Firewall group associated with ports is not allowed to be deleted  https://review.opendev.org/c/openstack/neutron-fwaas/+/88433308:09
ralonsohlajoskatona, hi, this is about https://review.opendev.org/c/openstack/networking-odl/+/88325108:10
ralonsohwe are going to deprecate the project08:10
ralonsohand remove the master branch08:10
ralonsohis that patch needed?08:10
opendevreviewMerged openstack/ovn-octavia-provider stable/2023.1: Add retry on case of sqlite3.InterfaceError on FT  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/88380108:10
ralonsohah can you also check https://review.opendev.org/c/openstack/neutron-fwaas/+/883653?08:12
opendevreviewZhouHeng proposed openstack/neutron-fwaas master: Remove Exception Definition in fwaas_v2 Extension  https://review.opendev.org/c/openstack/neutron-fwaas/+/88433408:18
opendevreviewZhouHeng proposed openstack/neutron-fwaas master: Firewall group associated with ports is not allowed to be deleted  https://review.opendev.org/c/openstack/neutron-fwaas/+/88433308:23
opendevreviewZhouHeng proposed openstack/neutron-fwaas master: Remove Exception Definition in fwaas_v2 Extension  https://review.opendev.org/c/openstack/neutron-fwaas/+/88433408:23
opendevreviewMerged openstack/ovn-octavia-provider stable/zed: Add retry on case of sqlite3.InterfaceError on FT  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/88380208:37
lajoskatonaralonsoh: for networking-odl I restored https://review.opendev.org/c/openstack/networking-odl/+/883251, it seems to be green in zuul, if ok we can merge it and the one from elodilles (https://review.opendev.org/q/Icff1b5737b4d2c52ead48e3700de668038f3c5c1 )09:06
lajoskatonaralonsoh: so it will look like an organized close not an abandonment :-)09:06
ralonsohperfect!!09:11
ralonsohthis is much more elegant09:11
ralonsohthank you09:11
opendevreviewMerged openstack/neutron stable/zed: Revert "Delete sg rule which remote is the deleted sg"  https://review.opendev.org/c/openstack/neutron/+/88420509:15
opendevreviewMerged openstack/neutron stable/yoga: Revert "Delete sg rule which remote is the deleted sg"  https://review.opendev.org/c/openstack/neutron/+/88420609:16
opendevreviewMerged openstack/neutron stable/xena: Revert "Delete sg rule which remote is the deleted sg"  https://review.opendev.org/c/openstack/neutron/+/88420709:16
opendevreviewRodolfo Alonso proposed openstack/neutron master: [OVN] The L3 scheduler does not use all chassis by default  https://review.opendev.org/c/openstack/neutron/+/88432309:45
opendevreviewMerged openstack/neutron-tempest-plugin master: Make neutron-tempest-plugin-bgpvpn-bagpipe non-voting  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/88427009:47
grami[m]I'm wondering if I can get some help one with getting some more reviews on the https://review.opendev.org/c/openstack/networking-generic-switch/+/881959 and if anyone has some ideas on unit tests as I'm not a python guru and never used unit test?09:48
ralonsohgrami[m], you can ping the project drivers09:49
ralonsohhttps://launchpad.net/~generic-switch-drivers/+members#active09:49
grami[m]thanks so much :)09:49
opendevreviewRodolfo Alonso proposed openstack/networking-odl stable/zed: Tox4: add allowlist_externals where necessary  https://review.opendev.org/c/openstack/networking-odl/+/88432509:51
opendevreviewRodolfo Alonso proposed openstack/networking-odl stable/yoga: Tox4: add allowlist_externals where necessary  https://review.opendev.org/c/openstack/networking-odl/+/88432609:51
opendevreviewRodolfo Alonso proposed openstack/networking-odl stable/xena: Tox4: add allowlist_externals where necessary  https://review.opendev.org/c/openstack/networking-odl/+/88432709:52
opendevreviewMerged openstack/neutron-fwaas master: Fix issues due to recent RBAC changes  https://review.opendev.org/c/openstack/neutron-fwaas/+/88365309:53
opendevreviewRodolfo Alonso proposed openstack/networking-odl stable/xena: Tox4: add allowlist_externals where necessary  https://review.opendev.org/c/openstack/networking-odl/+/88432709:55
opendevreviewRodolfo Alonso proposed openstack/networking-odl stable/yoga: Tox4: add allowlist_externals where necessary  https://review.opendev.org/c/openstack/networking-odl/+/88432609:55
opendevreviewRodolfo Alonso proposed openstack/networking-odl stable/wallaby: Tox4: add allowlist_externals where necessary  https://review.opendev.org/c/openstack/networking-odl/+/88432909:58
ralonsohlajoskatona, qq: why https://review.opendev.org/c/openstack/networking-odl/+/883251 is depending on bgpvpn patch?10:01
ralonsohsorry, maybe I'm missing something10:01
opendevreviewLajos Katona proposed openstack/networking-odl master: Fix issues due to rcent RBAC changes  https://review.opendev.org/c/openstack/networking-odl/+/88325110:10
lajoskatonaralonsoh: let me check it, ODL import bgpvpn as API implementation, I removed the depends-on perhaps not necessary10:11
ralonsohthanks10:11
opendevreviewSlawek Kaplonski proposed openstack/neutron-tempest-plugin master: New basic API tests for the default SG rules templates CRUDs  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/88355310:21
opendevreviewRodolfo Alonso proposed openstack/networking-odl stable/yoga: Tox4: add allowlist_externals where necessary  https://review.opendev.org/c/openstack/networking-odl/+/88432610:26
opendevreviewSlawek Kaplonski proposed openstack/neutron master: Update api extension for default sg rules API  https://review.opendev.org/c/openstack/neutron/+/88326910:48
opendevreviewSlawek Kaplonski proposed openstack/neutron master: Add description field to the security_group_default_rules resource  https://review.opendev.org/c/openstack/neutron/+/88338610:48
opendevreviewSlawek Kaplonski proposed openstack/neutron master: Default SG api rules template - DB and OVO models  https://review.opendev.org/c/openstack/neutron/+/88324610:48
opendevreviewSlawek Kaplonski proposed openstack/neutron master: Default SG api rules template - DB and OVO models  https://review.opendev.org/c/openstack/neutron/+/88324611:03
opendevreviewRodolfo Alonso proposed openstack/networking-odl stable/yoga: Fix tox4 issues and define project upper constraints  https://review.opendev.org/c/openstack/networking-odl/+/88435911:16
opendevreviewFernando Royo proposed openstack/ovn-octavia-provider master: Apply admin_state_up on a new member creation  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/88073711:24
opendevreviewZhouHeng proposed openstack/neutron-tempest-plugin master: [fwaas]Disassociate all ports before deleting firewall group  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/88382611:30
lajoskatonaslaweq: Hi, sorry for disturbing, I try to find why bgpvpn job is failing, and I fixed few of the issues (not uahtorized for something typically): https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/88383811:57
lajoskatonaslaweq: but there's still some, example here: https://93ade36fb41a555ab6e4-6d75abbe2d1bba564eb60ae65c4ffb8a.ssl.cf2.rackcdn.com/883838/5/check/neutron-tempest-plugin-bgpvpn-bagpipe/d9e0bcc/testr_results.html11:57
lajoskatonaslaweq: in the q-svc logs I see things like this for some of the status: 403 responses for network association create :11:58
lajoskatonaMay 23 12:37:50.236490 np0034107136 neutron-server[59857]: INFO neutron.api.v2.resource [None req-36a291b7-c6b7-43b0-8d10-552f1cca00f5 tempest-TestBGPVPNBasic-996417847 tempest-TestBGPVPNBasic-996417847-project-admin] create failed (client error): Access was denied to this resource.11:59
lajoskatonaMay 23 12:37:50.242503 np0034107136 neutron-server[59857]: INFO neutron.wsgi [None req-36a291b7-c6b7-43b0-8d10-552f1cca00f5 tempest-TestBGPVPNBasic-996417847 tempest-TestBGPVPNBasic-996417847-project-admin] 10.208.224.24,10.208.224.24 "POST /networking/v2.0/bgpvpn/bgpvpns/1d8d4461-0c24-45ed-b51a-41e85509f678/network_associations HTTP/1.1" status: 403  len: 292 time: 0.166735911:59
lajoskatonaslaweq: from https://93ade36fb41a555ab6e4-6d75abbe2d1bba564eb60ae65c4ffb8a.ssl.cf2.rackcdn.com/883838/5/check/neutron-tempest-plugin-bgpvpn-bagpipe/d9e0bcc/controller/logs/screen-q-svc.txt11:59
lajoskatonaslaweq: what I can't understand, if it is project-admin, what is still missing to have access to create a net-association...... Perhaps you understand better than me these logs for new personas12:01
opendevreviewFernando Royo proposed openstack/neutron-lib master: Add FIPAssociated exception  https://review.opendev.org/c/openstack/neutron-lib/+/88390112:07
opendevreviewFernando Royo proposed openstack/neutron master: Return 409 Conflict to tenant user deleting port attached to FIP  https://review.opendev.org/c/openstack/neutron/+/88182712:09
opendevreviewRodolfo Alonso proposed openstack/networking-odl stable/yoga: Fix tox4 issues and define project upper constraints  https://review.opendev.org/c/openstack/networking-odl/+/88435912:49
opendevreviewRodolfo Alonso proposed openstack/neutron master: [OVN] The L3 scheduler does not use all chassis by default  https://review.opendev.org/c/openstack/neutron/+/88432313:26
opendevreviewFernando Royo proposed openstack/neutron master: Return 409 Conflict to tenant user deleting port attached to FIP  https://review.opendev.org/c/openstack/neutron/+/88182713:39
opendevreviewRodolfo Alonso proposed openstack/neutron master: WIP - Add ``OVNGatewayHAChassisGroup`` scheduler class  https://review.opendev.org/c/openstack/neutron/+/87203313:52
opendevreviewRodolfo Alonso proposed openstack/networking-odl stable/yoga: Fix tox4 issues and define project upper constraints  https://review.opendev.org/c/openstack/networking-odl/+/88435913:54
ralonsohhi lajoskatona, I'm having problems with n-odl, in stable yoga (and previous versions)14:07
ralonsohhttps://d887b8783462f4c7a515-9ac9734bde8524e2404b921e59789daf.ssl.cf2.rackcdn.com/884359/3/check/openstack-tox-py36/03eb7f6/job-output.txt14:07
ralonsohthis is during the pep8 dependencies installation14:07
ralonsohI'm forcing networking-l2gw>=12.0.0,<=18.0.014:07
ralonsohbut it is still installing master branch14:08
lajoskatonaralonsoh: I will check it14:09
ralonsohok, I think I know where to limit that14:09
ralonsohin the zuul definition14:09
opendevreviewRodolfo Alonso proposed openstack/networking-odl stable/yoga: Fix tox4 issues and define project upper constraints  https://review.opendev.org/c/openstack/networking-odl/+/88435914:30
opendevreviewRodolfo Alonso proposed openstack/networking-odl stable/yoga: Fix tox4 issues and define project upper constraints  https://review.opendev.org/c/openstack/networking-odl/+/88435914:50
opendevreviewMichal Nasiadka proposed openstack/neutron master: haproxy: Add support for configuring syslog  https://review.opendev.org/c/openstack/neutron/+/88440714:53
opendevreviewSlawek Kaplonski proposed openstack/neutron master: Default SG api rules template - DB and OVO models  https://review.opendev.org/c/openstack/neutron/+/88324614:54
slaweqralonsoh mlavalle lajoskatona haleyb hi, please look at patches https://review.opendev.org/q/topic:bug%252F1983053 when You will have time, I think those are now ready for review14:56
slaweqlajoskatona sorry, I was busy today, can You ping me tomorrow morning with this question You had earlier? I will check it then14:56
ralonsohsure, let me check them14:56
opendevreviewRodolfo Alonso proposed openstack/neutron master: Improve the ``PortBindingUpdateVirtualPortsEvent`` match filter  https://review.opendev.org/c/openstack/neutron/+/88368115:02
opendevreviewMichal Nasiadka proposed openstack/neutron master: haproxy: Add support for configuring syslog  https://review.opendev.org/c/openstack/neutron/+/88440715:09
opendevreviewJuan Pablo Suazo proposed openstack/tap-as-a-service master: Support for ovs-dpdk  https://review.opendev.org/c/openstack/tap-as-a-service/+/88423915:23
opendevreviewLajos Katona proposed openstack/neutron-tempest-plugin master: Adopt bgpvpn scenarios to recent sRBAC changes  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/88383815:56
opendevreviewMerged openstack/ovn-octavia-provider stable/2023.1: Fix update member action  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/88350916:15
opendevreviewMerged openstack/ovn-octavia-provider stable/zed: Fix update member action  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/88351016:40
opendevreviewMerged openstack/ovn-octavia-provider stable/yoga: Fix update member action  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/88351116:40
opendevreviewMerged openstack/ovn-octavia-provider stable/xena: Fix update member action  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/88351216:40
opendevreviewMerged openstack/ovn-octavia-provider stable/wallaby: Fix update member action  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/88351316:40
opendevreviewRodolfo Alonso proposed openstack/networking-odl stable/yoga: Fix tox4 issues and define project upper constraints  https://review.opendev.org/c/openstack/networking-odl/+/88435916:44
opendevreviewMerged openstack/neutron-lib stable/2023.1: Context.elevated() method sets all required roles for context object  https://review.opendev.org/c/openstack/neutron-lib/+/88339816:49
opendevreviewMerged openstack/neutron stable/zed: [S-RBAC] Fix new policies for get QoS rules APIs  https://review.opendev.org/c/openstack/neutron/+/88296116:51
opendevreviewRodolfo Alonso proposed openstack/networking-odl stable/yoga: Fix tox4 issues and define project upper constraints  https://review.opendev.org/c/openstack/networking-odl/+/88435917:30
opendevreviewMerged openstack/neutron stable/zed: [S-RBAC] Get QoS rule types API available for READER role  https://review.opendev.org/c/openstack/neutron/+/88296218:29
opendevreviewMerged openstack/neutron-lib stable/zed: Context.elevated() method sets all required roles for context object  https://review.opendev.org/c/openstack/neutron-lib/+/88339920:43
*** sfinucan is now known as stephenfin21:52
opendevreviewMerged openstack/neutron stable/2023.1: Use explicit inner join for networks in port query  https://review.opendev.org/c/openstack/neutron/+/88371423:15
opendevreviewMerged openstack/neutron stable/yoga: Use explicit inner join for networks in port query  https://review.opendev.org/c/openstack/neutron/+/88371223:15

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