Wednesday, 2022-07-27

opendevreviewLajos Katona proposed openstack/neutron stable/xena: Test: mock out _check_netfilter_for_bridges in unit tests  https://review.opendev.org/c/openstack/neutron/+/85106307:02
opendevreviewLajos Katona proposed openstack/networking-bagpipe master: DNM: test master  https://review.opendev.org/c/openstack/networking-bagpipe/+/85115907:10
opendevreviewLajos Katona proposed openstack/networking-sfc master: DNM: test master  https://review.opendev.org/c/openstack/networking-sfc/+/85116007:10
*** amoralej|off is now known as amoralej07:50
opendevreviewMamatisa Nurmatov proposed openstack/neutron master: Use neutron-lib method is_session_active  https://review.opendev.org/c/openstack/neutron/+/85119208:24
opendevreviewRodolfo Alonso proposed openstack/neutron-lib master: [sqlalchemy-20] Use session.transaction information to decide if active  https://review.opendev.org/c/openstack/neutron-lib/+/85119308:32
opendevreviewLajos Katona proposed openstack/neutron stable/wallaby: Test: mock out _check_netfilter_for_bridges in unit tests  https://review.opendev.org/c/openstack/neutron/+/85106409:21
slaweqykarel: ralonsoh lajoskatona can You check https://review.opendev.org/c/openstack/neutron/+/844335 ? I think it's ready to review now09:25
ralonsohlet me check09:25
lajoskatonaslaweq: ok09:26
slaweqthx09:26
ykarelslaweq, ack09:26
ykarelslaweq, the blacklist test no longer needed09:29
ykarelissue is fixed in libvirt09:29
slaweqykarel: ok, let me update the patch09:30
ykarelslaweq, ack added one more nit, so include that as well09:30
opendevreviewSharon Koech proposed openstack/neutron-lib master: Network Cascade Deletion Extension  https://review.opendev.org/c/openstack/neutron-lib/+/84904609:45
opendevreviewSlawek Kaplonski proposed openstack/neutron master: Switch Fedora based job to Centos Stream  https://review.opendev.org/c/openstack/neutron/+/84433510:36
opendevreviewLajos Katona proposed openstack/neutron stable/victoria: Test: mock out _check_netfilter_for_bridges in unit tests  https://review.opendev.org/c/openstack/neutron/+/85121211:11
*** amoralej is now known as amoralej|lunch11:23
opendevreviewLajos Katona proposed openstack/neutron stable/ussuri: Test: mock out _check_netfilter_for_bridges in unit tests  https://review.opendev.org/c/openstack/neutron/+/85106511:56
opendevreviewLajos Katona proposed openstack/neutron stable/ussuri: Test: mock out _check_netfilter_for_bridges in unit tests  https://review.opendev.org/c/openstack/neutron/+/85106512:04
opendevreviewLajos Katona proposed openstack/neutron stable/ussuri: Test: mock out _check_netfilter_for_bridges in unit tests  https://review.opendev.org/c/openstack/neutron/+/85106512:13
slaweqykarel: I addressed Your comments in https://review.opendev.org/c/openstack/neutron/+/844335 please check when You will have some time12:19
opendevreviewLajos Katona proposed openstack/tap-as-a-service master: Py3: Remove six  https://review.opendev.org/c/openstack/tap-as-a-service/+/84662412:20
opendevreviewliuxie proposed openstack/neutron-fwaas master: Support l3 firewall for ovn driver  https://review.opendev.org/c/openstack/neutron-fwaas/+/84575612:20
ykarelslaweq, done12:20
slaweqykarel: thx12:20
lajoskatonaralonsoh: Hi, could you please check this fwaas patch when you have few minutes: https://review.opendev.org/c/openstack/neutron-fwaas/+/846834 ?12:41
ralonsohsure12:41
ralonsohlajoskatona, but this is not needed again in 0.7.212:42
ralonsohisn't better to bump fwaas pyroute2 version?12:42
lajoskatonaralonsoh: hmmm, you are right, I check on one of the other patches there is no need for any pyroute2 related change12:43
opendevreviewLajos Katona proposed openstack/neutron-fwaas master: Remove usage of six  https://review.opendev.org/c/openstack/neutron-fwaas/+/84455112:46
lajoskatonaralonsoh: perhaps it fetches the latest, so no issue, let's see the results from https://review.opendev.org/c/openstack/neutron-fwaas/+/84455112:47
ralonsohsure12:47
lajoskatonaall: I shuffled the bug deputy list: https://wiki.openstack.org/wiki/Network/Meetings#Bug_deputy please check if it is ok for you13:13
lajoskatonabcafarel: you start the cycle on 8. August is that ok?13:14
bcafarellajoskatona: sounds good, I will be around!13:15
lajoskatonabcafarel: thanks13:16
opendevreviewSlawek Kaplonski proposed openstack/neutron-specs master: Add spec for Network cascade deletion  https://review.opendev.org/c/openstack/neutron-specs/+/81082213:40
slaweqmlavalle: ralonsoh: lajoskatona I just updated spec https://review.opendev.org/c/openstack/neutron-specs/+/810822 - please check it when You will have few minutes13:41
ralonsohslaweq, sure13:41
ralonsohlajoskatona, just a heads-up: fwaas is failing in py310 13:41
ralonsohfolks: https://review.opendev.org/c/openstack/neutron-lib/+/851193 (if we have time to release a new n-lib version)13:44
lajoskatonaralonsoh: thanks13:46
lajoskatonaralonsoh: final deadline is 26th August, so we have :-)13:46
ralonsohperfect!13:46
lajoskatonaslaweq: thanks for the update13:47
*** amoralej|lunch is now known as amoralej13:47
lajoskatonaskoech[m]: the net cascade delete spec ----^ (https://review.opendev.org/c/openstack/neutron-specs/+/810822 )13:48
ykarellajoskatona, please also add me to the bug deputy list13:53
lajoskatonaykarel: thanks, I will add you13:53
ykarelack Thanks13:53
opendevreviewRodolfo Alonso proposed openstack/neutron master: [OVN][QoS] Add minimum bandwidth rule support to ML2/OVN  https://review.opendev.org/c/openstack/neutron/+/84229214:01
opendevreviewMerged openstack/neutron master: Switch Fedora based job to Centos Stream  https://review.opendev.org/c/openstack/neutron/+/84433514:03
slaweqhaleyb: hi, can You check https://review.opendev.org/c/openstack/neutron-specs/+/832660/ when You will have few minutes? I would like to know Your opinion about it as IPv6 expert :)14:03
slaweqthx in advance14:03
mlavalleslaweq: will do14:04
slaweqthx mlavalle 14:04
opendevreviewMerged openstack/neutron-fwaas master: Remove usage of six  https://review.opendev.org/c/openstack/neutron-fwaas/+/84455114:27
*** dasm|off is now known as dasm14:42
opendevreviewMerged openstack/neutron stable/yoga: Test: mock out _check_netfilter_for_bridges in unit tests  https://review.opendev.org/c/openstack/neutron/+/85092015:45
opendevreviewMerged openstack/neutron stable/xena: Test: mock out _check_netfilter_for_bridges in unit tests  https://review.opendev.org/c/openstack/neutron/+/85106315:45
opendevreviewMerged openstack/neutron-lib master: [sqlalchemy-20] Use session.transaction information to decide if active  https://review.opendev.org/c/openstack/neutron-lib/+/85119315:51
*** amoralej is now known as amoralej|off16:35
opendevreviewMerged openstack/neutron master: Update the Ethernet card information  https://review.opendev.org/c/openstack/neutron/+/84892917:28
opendevreviewMerged openstack/neutron-specs master: [ovn]Floating IP adds distributed attributes  https://review.opendev.org/c/openstack/neutron-specs/+/84698717:49
opendevreviewSlawek Kaplonski proposed openstack/neutron master: Make iptables rules diff to be case insensitive  https://review.opendev.org/c/openstack/neutron/+/84458918:49
opendevreviewSlawek Kaplonski proposed openstack/neutron master: [WIP] Switch fullstack/functional fips jobs to 9-stream  https://review.opendev.org/c/openstack/neutron/+/84324518:50
opendevreviewMerged openstack/neutron stable/ussuri: Test: mock out _check_netfilter_for_bridges in unit tests  https://review.opendev.org/c/openstack/neutron/+/85106519:00
opendevreviewMerged openstack/neutron stable/victoria: Test: mock out _check_netfilter_for_bridges in unit tests  https://review.opendev.org/c/openstack/neutron/+/85121219:00
opendevreviewMerged openstack/neutron stable/wallaby: Test: mock out _check_netfilter_for_bridges in unit tests  https://review.opendev.org/c/openstack/neutron/+/85106419:09
*** dasm is now known as dasm|off21:02

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