Friday, 2022-02-11

opendevreviewZhouHeng proposed openstack/neutron-fwaas master: Revert "Retire neutron-fwaas project"  https://review.opendev.org/c/openstack/neutron-fwaas/+/82814900:47
opendevreviewMerged openstack/neutron stable/victoria: [OVN] Add the VIF details "connectivity" parameter  https://review.opendev.org/c/openstack/neutron/+/82644001:03
*** Guest2 is now known as prometheanfire01:28
opendevreviewliuyulong proposed openstack/neutron master: [Doc] packet rete limit user workflow and notes  https://review.opendev.org/c/openstack/neutron/+/82867302:51
opendevreviewliuyulong proposed openstack/neutron master: Fullstack tests of packet rate limit for ovs qos driver  https://review.opendev.org/c/openstack/neutron/+/81941802:51
opendevreviewMerged openstack/neutron stable/ussuri: [OVN] Add the VIF details "connectivity" parameter  https://review.opendev.org/c/openstack/neutron/+/82644105:24
opendevreviewZhouHeng proposed openstack/neutron-fwaas master: Revert "Retire neutron-fwaas project"  https://review.opendev.org/c/openstack/neutron-fwaas/+/82814908:15
opendevreviewZhouHeng proposed openstack/neutron-fwaas master: Revert "Retire neutron-fwaas project"  https://review.opendev.org/c/openstack/neutron-fwaas/+/82814908:23
bcafarelsigh, grenade jobs failing on train again https://zuul.opendev.org/t/openstack/build/e850cb27f72a42aaaf18b555a31f2eb508:57
bcafarelERROR: This script does not work on Python 3.6 The minimum supported Python version is 3.7 hmm09:02
ykarelbcafarel, seems need to backport https://review.opendev.org/c/openstack/devstack/+/827421 to stein09:06
bcafarelykarel++ sounds like it indeed!09:08
bcafarelok let's see how https://review.opendev.org/c/openstack/devstack/+/828769 goes09:12
opendevreviewBernard Cafarelli proposed openstack/neutron stable/train: Avoid writing segments to the DB repeatedly  https://review.opendev.org/c/openstack/neutron/+/82573509:13
ykarelack09:15
opendevreviewLajos Katona proposed openstack/neutron stable/pike: Remove neutron-lbaas from the CI jobs requirements  https://review.opendev.org/c/openstack/neutron/+/82329209:30
opendevreviewAnton Vazhnetsov proposed openstack/ovsdbapp master: vtep: add support for 'hardware_vtep' schema  https://review.opendev.org/c/openstack/ovsdbapp/+/82872009:32
opendevreviewMerged openstack/neutron stable/victoria: Make sure "dead vlan" ports cannot transmit packets  https://review.opendev.org/c/openstack/neutron/+/82849709:58
opendevreviewMerged openstack/ovn-octavia-provider stable/ussuri: Support creating members without a subnet ID  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/82704510:10
opendevreviewMerged openstack/ovn-octavia-provider stable/victoria: Support creating members without a subnet ID  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/82704010:10
opendevreviewZhouHeng proposed openstack/neutron-tempest-plugin master: Readd jobs for neutron-fwaas master branch  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/82844310:16
*** mdbooth3 is now known as mdbooth10:25
lajoskatonaralonsoh: Hi, an n-lib release is coming: https://review.opendev.org/c/openstack/releases/+/828452 , as I see the open patches in n-lib are not yet ready for merge, please check & vote if you would like to wait with release (anyway we can release again next week ;)10:39
ralonsohlajoskatona, well, actually I would prefer to have some patches merged10:40
ralonsohlajoskatona, https://review.opendev.org/c/openstack/neutron-lib/+/82753310:41
ralonsohI had a +2 from you10:41
ralonsohbut I needed to change the docs10:42
ralonsohI would like to have this one too10:42
lajoskatonaralonsoh: ok, that was in my mind, but was not sure where it is :-)10:42
ralonsohnow is rechecking10:42
lajoskatonaok10:43
opendevreviewFernando Royo proposed openstack/ovn-octavia-provider stable/ussuri: Reduce coverage threshold to allow old backports  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/82885611:01
afaranhaslaweq, hi, do you know what is needed to backport these patches to xena and wallaby? https://github.com/openstack/neutron/commit/391726bd4c0302ca3ce27f5de8e39ee4c6d91457 https://github.com/openstack/neutron/commit/ddf0568f39eea99ee6a415dc7592cb5681f060c4 it's a but confusing because there was a 'revert' https://review.opendev.org/c/openstack/neutron/+/82475011:21
opendevreviewMerged openstack/networking-ovn stable/train: Support creating members without a subnet ID  https://review.opendev.org/c/openstack/networking-ovn/+/82705111:34
ralonsohafaranha, I don't think we need to backport those patches. The devstack version supporting this new flag is the current one11:53
ralonsohin any case, you can ping slaweq next Tuesday11:53
afaranharalonsoh, the problem was with this patch for xena: https://review.opendev.org/c/openstack/neutron/+/82869711:54
afaranhaa comment from Bernard Cafarelli, The configure_for_func_testing.sh changes break functional/fullstack, another backport needed?11:54
afaranhaERROR 1045 (28000): Access denied for user 'root'@'localhost' (using password: YES)11:54
afaranhaI don't know if these are the correct patches that we need to backport, it was what I found that changed configure_for_func_testing on master and it's not present on Xena11:55
ralonsohafaranha, no no, what is failing is you change in the mysql user11:55
ralonsohnot the OVN compilation flag11:55
ralonsohI don't know if this version of devstack supports the change you introduced in configure_for_func_testing.sh11:56
afaranhaow, okay, so I need to figure out the correct user/password for this11:58
opendevreviewMerged openstack/neutron stable/victoria: [Stable Only] Enforce policy for qos_policy_id attribute  https://review.opendev.org/c/openstack/neutron/+/82701613:02
opendevreviewyangjianfeng proposed openstack/neutron-lib master: api-ref for ndp proxy  https://review.opendev.org/c/openstack/neutron-lib/+/80118113:21
opendevreviewyangjianfeng proposed openstack/neutron master: Add ndp proxy policy rules  https://review.opendev.org/c/openstack/neutron/+/82802213:33
opendevreviewyangjianfeng proposed openstack/neutron master: Add ndp proxy policy rules  https://review.opendev.org/c/openstack/neutron/+/82802213:36
opendevreviewyangjianfeng proposed openstack/neutron master: [Agent Side] L3 router support ndp proxy  https://review.opendev.org/c/openstack/neutron/+/74481513:39
*** dasm|off is now known as dasm14:21
ykarelralonsoh, when you get chance please check https://review.opendev.org/c/openstack/neutron/+/827017 and https://review.opendev.org/c/openstack/neutron/+/82742414:49
ykarelThanks in advance14:49
ralonsohok14:49
*** ykarel is now known as ykarel|away14:56
opendevreviewyangjianfeng proposed openstack/neutron-lib master: api-ref for ndp proxy  https://review.opendev.org/c/openstack/neutron-lib/+/80118115:09
ralonsohlajoskatona, https://review.opendev.org/c/openstack/neutron-lib/+/82753315:35
ralonsohcan you check it?15:36
ralonsohthanks in advance15:36
lajoskatonaralonsoh: sure15:36
ralonsohwe have the other n-lib patch almost merged15:36
ralonsohI think that's all in the n-lib backlog 15:36
lajoskatonaralonsoh: this one: https://review.opendev.org/c/openstack/neutron-lib/+/801181 ? Cool15:41
ralonsohyes, this one15:42
opendevreviewMerged openstack/neutron stable/ussuri: [Stable Only] Enforce policy for qos_policy_id attribute  https://review.opendev.org/c/openstack/neutron/+/82701716:24
opendevreviewMerged openstack/neutron master: Re enable test_network_v6.TestGettingAddress  https://review.opendev.org/c/openstack/neutron/+/82742416:25
opendevreviewMerged openstack/neutron stable/wallaby: [OVN] Check new added segments in OVN mech driver  https://review.opendev.org/c/openstack/neutron/+/82610316:40
mlavallelajoskatona: would you take a look at https://review.opendev.org/c/openstack/neutron/+/827580?16:43
opendevreviewMerged openstack/neutron-lib master: api-ref for ndp proxy  https://review.opendev.org/c/openstack/neutron-lib/+/80118117:11
*** marlinc is now known as Guest7317:16
stephenfinCan someone stick a +W on this doc patch? https://review.opendev.org/c/openstack/neutron/+/80177117:30
mlavallestephenfin: done17:31
stephenfinty :)17:31
lajoskatonamlavalle: nice series with the timeouts17:36
opendevreviewMerged openstack/neutron stable/wallaby: [Stable Only] Enforce policy for qos_policy_id attribute  https://review.opendev.org/c/openstack/neutron/+/82701517:41
ralonsohzzzeek, hi, do you have a couple of mins? this is about https://review.opendev.org/c/openstack/oslo.db/+/80477518:04
ralonsohI'm trying to replace the checks we have in neutron/neutron-lib of session.is_active18:05
ralonsohwhat we need to know is that the session has no pending command, that the transaction has nothing to commit18:06
ralonsohthe problem is that, when I disable autocommit, we can have a session with an active transaction18:06
ralonsohand I don't know how can I replace the old logic18:06
ralonsohwhere we knew a session was not in the middle of a transaction when !self._transaction.is_active (or wasn't transaction at all)18:07
*** dkehn_ is now known as dkehn18:11
opendevreviewBrian Haley proposed openstack/neutron master: DNM: Test IPv6 all (including tunnel) job definition  https://review.opendev.org/c/openstack/neutron/+/82863418:53
opendevreviewMerged openstack/neutron master: Wait longer before deleting DPDK vhu trunk bridges  https://review.opendev.org/c/openstack/neutron/+/82758019:57
opendevreviewMerged openstack/neutron master: doc: Remove references to dead VMWare NSX extensions  https://review.opendev.org/c/openstack/neutron/+/80177119:58
opendevreviewMerged openstack/neutron stable/train: Avoid writing segments to the DB repeatedly  https://review.opendev.org/c/openstack/neutron/+/82573519:58
*** dasm is now known as dasm|off21:49
opendevreviewJakub Libosvar proposed openstack/networking-ovn stable/train: [ovn] Prevent stale ports in the OVN database  https://review.opendev.org/c/openstack/networking-ovn/+/82880122:09
*** ubuntu is now known as Guest10822:28

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