Thursday, 2023-02-09

ralonsohstephenfin, hi! Will you have 1 min to review https://review.opendev.org/c/openstack/oslo.privsep/+/864254?09:17
ralonsohthanks in advance09:17
ralonsohI would like to add this patch to the next release (tomorrow)09:18
ralonsohhi folks, please check https://review.opendev.org/c/openstack/neutron/+/87264409:19
ralonsohlajoskatona, slaweq ^09:19
lajoskatonaralonsoh: thanks09:42
slaweqralonsoh I will review it in about 30 minutes09:46
ralonsohthank you folks09:47
opendevreviewLajos Katona proposed openstack/networking-bagpipe master: Change IPDB to NDB  https://review.opendev.org/c/openstack/networking-bagpipe/+/87323509:48
lajoskatonaralonsoh: I pushed a quick fix for the pyroute2 0.7.4 vs bagpipe issue: https://review.opendev.org/c/openstack/networking-bagpipe/+/873235 (from the req bump they removed pyroute2 bump, see: https://review.opendev.org/c/openstack/requirements/+/872065 )09:52
ralonsohlajoskatona, let me check09:52
lajoskatonaralonsoh: the issue was with IPDB, whcih is anyway deprecated in pyroute209:53
ralonsohlajoskatona, is IPDB removed from 0.7.4?09:53
ralonsohI know we migrated to NDB in other projects09:53
lajoskatonaNo, but by the deprecation msg it was in 0.7.1 :-)09:54
ralonsohok, in any case, if this patch is passing, I think we can bump to 0.7.4, right?09:54
lajoskatonathe issue was some terrible tracehttps://6fb4508c56cf42ef8c6e-33283944619588e5b231e2976fdac28e.ssl.cf1.rackcdn.com/872386/3/check/dnm-cross-networking-bagpipe-py310/eeb55e5/testr_results.htmlback finally in eventlet, see: 09:55
lajoskatonaralonsoh: try it again: https://6fb4508c56cf42ef8c6e-33283944619588e5b231e2976fdac28e.ssl.cf1.rackcdn.com/872386/3/check/dnm-cross-networking-bagpipe-py310/eeb55e5/testr_results.html09:55
ralonsohok, now with NDB it should works09:56
ralonsohlet's wait for the CI then09:56
lajoskatonayeah, as I see and you said we have no other projects with IPDB, and here is the deprecation warning: https://github.com/svinota/pyroute2/blob/master/pyroute2/ipdb/main.py#L836-L84409:57
opendevreviewMerged openstack/neutron-tempest-plugin master: Do not override nodeset for stable scenario jobs  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/87305910:16
opendevreviewMerged openstack/neutron-tempest-plugin master: Bump Cirros version used in CI jobs to 0.6.1  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/86915210:38
opendevreviewLajos Katona proposed openstack/neutron master: WIP: desperate try to make DVR functional tests find interface  https://review.opendev.org/c/openstack/neutron/+/87311110:47
opendevreviewRodolfo Alonso proposed openstack/neutron stable/wallaby: [Trunk] Update the trunk status with the parent status  https://review.opendev.org/c/openstack/neutron/+/87175910:51
ralonsohlajoskatona, thanks for https://review.opendev.org/c/openstack/networking-bagpipe/+/87323511:48
ralonsohslaweq, ^^ can you review it?11:49
ralonsohthis is needed for https://review.opendev.org/c/openstack/requirements/+/872065 (to allow pyroute2 0.7.4)11:49
opendevreviewRodolfo Alonso proposed openstack/neutron master: [OVN] Add missing LSP device_owner info in trunk driver  https://review.opendev.org/c/openstack/neutron/+/87324712:05
opendevreviewMerged openstack/neutron-tempest-plugin master: Drop Train designate scenario job  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/87306012:18
opendevreviewMerged openstack/neutron master: Add a method to retrieve and register "rpc_workers" config knob  https://review.opendev.org/c/openstack/neutron/+/87264412:58
opendevreviewSlawek Kaplonski proposed openstack/neutron stable/train: [Trunk] Update the trunk status with the parent status  https://review.opendev.org/c/openstack/neutron/+/87314413:08
slaweqralonsoh approved13:15
ralonsohthanks!13:37
opendevreviewSlawek Kaplonski proposed openstack/neutron stable/victoria: [Trunk] Update the trunk status with the parent status  https://review.opendev.org/c/openstack/neutron/+/87314613:40
opendevreviewSlawek Kaplonski proposed openstack/neutron stable/ussuri: [Trunk] Update the trunk status with the parent status  https://review.opendev.org/c/openstack/neutron/+/87314713:40
slaweqralonsoh can You then review https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/867518 ?13:45
slaweqthx in advance13:45
ralonsohsure13:45
*** dasm|offline is now known as dasm|rover13:47
slaweqralonsoh lajoskatona I would also like to get https://review.opendev.org/c/openstack/neutron/+/871983 merged in this cycle13:48
ralonsohright yes13:48
opendevreviewSlawek Kaplonski proposed openstack/neutron master: Deprecate allow_stateless_action_supported config option  https://review.opendev.org/c/openstack/neutron/+/87198313:48
*** dkehn_ is now known as dkehn14:18
opendevreviewRodolfo Alonso proposed openstack/neutron master: [OVN] Add missing LSP device_owner info in trunk driver  https://review.opendev.org/c/openstack/neutron/+/87324714:18
opendevreviewBence Romsics proposed openstack/neutron master: port-hints: api extension  https://review.opendev.org/c/openstack/neutron/+/87008114:25
opendevreviewBence Romsics proposed openstack/neutron master: WIP port-hint-ovs-tx-steering: agent side  https://review.opendev.org/c/openstack/neutron/+/87290514:25
opendevreviewBence Romsics proposed openstack/neutron master: WIP port-hint-ovs-tx-steering: shim extension  https://review.opendev.org/c/openstack/neutron/+/87311314:25
opendevreviewBence Romsics proposed openstack/neutron master: DNM debug logs and dev helper scripts  https://review.opendev.org/c/openstack/neutron/+/87290614:25
opendevreviewMerged openstack/networking-bagpipe master: Change IPDB to NDB  https://review.opendev.org/c/openstack/networking-bagpipe/+/87323514:40
opendevreviewMerged openstack/neutron stable/zed: Fullstack: Wait placement process fixtrue to really stop  https://review.opendev.org/c/openstack/neutron/+/87287814:47
*** dasm|rover is now known as dasm|bbiab15:25
opendevreviewRodolfo Alonso proposed openstack/neutron master: [OVN] Bump the port revision number in trunk driver  https://review.opendev.org/c/openstack/neutron/+/87329615:39
opendevreviewBence Romsics proposed openstack/neutron master: port-hint-ovs-tx-steering: agent side  https://review.opendev.org/c/openstack/neutron/+/87290515:47
opendevreviewBence Romsics proposed openstack/neutron master: WIP port-hint-ovs-tx-steering: shim extension  https://review.opendev.org/c/openstack/neutron/+/87311315:47
opendevreviewBence Romsics proposed openstack/neutron master: DNM debug logs and dev helper scripts  https://review.opendev.org/c/openstack/neutron/+/87290615:47
opendevreviewMerged openstack/neutron master: Remove ``L3Scheduler._bind_routers`` method  https://review.opendev.org/c/openstack/neutron/+/87310215:50
opendevreviewRodolfo Alonso proposed openstack/neutron master: Increase port name size and type to internal  https://review.opendev.org/c/openstack/neutron/+/87311815:51
opendevreviewRodolfo Alonso proposed openstack/neutron master: DNM - TESTING PATCH: neutron-functional-with-uwsgi  https://review.opendev.org/c/openstack/neutron/+/87097915:52
opendevreviewRodolfo Alonso proposed openstack/neutron master: Add debugging exception messages in ``test_ovs_and_ovs_events``  https://review.opendev.org/c/openstack/neutron/+/87312616:04
opendevreviewRodolfo Alonso proposed openstack/neutron master: DNM - TESTING PATCH: neutron-functional-with-uwsgi  https://review.opendev.org/c/openstack/neutron/+/87329816:05
ralonsohmnaser, hi, did you have time to update https://bugs.launchpad.net/neutron/+bug/1988574?16:21
*** dasm|bbiab is now known as dasm|rover16:28
opendevreviewRodolfo Alonso proposed openstack/neutron master: [OVN] Skip ports with VNIC type direct and no port binding  https://review.opendev.org/c/openstack/neutron/+/87273716:34
opendevreviewMerged openstack/neutron master: allow manila ports to do multiple port binding for ML2  https://review.opendev.org/c/openstack/neutron/+/86929517:14
ralonsohslaweq, https://lists.openstack.org/pipermail/openstack-discuss/2023-February/032122.html17:32
ralonsohjust a heads-up17:32
opendevreviewMerged openstack/neutron-tempest-plugin master: [Stateless SG] Add negative API tests for Stateless SG  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/87215317:36
opendevreviewRodolfo Alonso proposed openstack/neutron master: Add debugging exception messages in ``test_ovs_and_ovs_events``  https://review.opendev.org/c/openstack/neutron/+/87312617:48
opendevreviewRodolfo Alonso proposed openstack/neutron master: DNM - TESTING PATCH: neutron-functional-with-uwsgi  https://review.opendev.org/c/openstack/neutron/+/87329817:48
opendevreviewRodolfo Alonso proposed openstack/neutron master: Increase port name size and type to internal  https://review.opendev.org/c/openstack/neutron/+/87311817:59
opendevreviewRodolfo Alonso proposed openstack/neutron master: DNM - TESTING PATCH: neutron-functional-with-uwsgi  https://review.opendev.org/c/openstack/neutron/+/87097917:59
opendevreviewGhanshyam proposed openstack/neutron-tempest-plugin master: Fix t-u-v stable jobs  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/87332520:14
opendevreviewGhanshyam proposed openstack/neutron stable/victoria: DNM: testing neutron-tempest-plugin 1.8.0 tag  https://review.opendev.org/c/openstack/neutron/+/87332620:17
opendevreviewGhanshyam proposed openstack/neutron stable/ussuri: DNM: testing neutron-tempest-plugin 1.8.0 tag  https://review.opendev.org/c/openstack/neutron/+/87327420:17
opendevreviewGhanshyam proposed openstack/neutron stable/train: DNM: testing neutron-tempest-plugin 1.8.0 tag  https://review.opendev.org/c/openstack/neutron/+/87327520:17
opendevreviewGhanshyam proposed openstack/neutron stable/train: DNM: testing neutron-tempest-plugin 1.8.0 tag  https://review.opendev.org/c/openstack/neutron/+/87327520:19
*** dasm|rover is now known as dasm|off22:18

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