opendevreview | ZhouHeng proposed openstack/neutron-fwaas master: Revert "Retire neutron-fwaas project" https://review.opendev.org/c/openstack/neutron-fwaas/+/828149 | 00:47 |
---|---|---|
opendevreview | Merged openstack/neutron stable/victoria: [OVN] Add the VIF details "connectivity" parameter https://review.opendev.org/c/openstack/neutron/+/826440 | 01:03 |
*** Guest2 is now known as prometheanfire | 01:28 | |
opendevreview | liuyulong proposed openstack/neutron master: [Doc] packet rete limit user workflow and notes https://review.opendev.org/c/openstack/neutron/+/828673 | 02:51 |
opendevreview | liuyulong proposed openstack/neutron master: Fullstack tests of packet rate limit for ovs qos driver https://review.opendev.org/c/openstack/neutron/+/819418 | 02:51 |
opendevreview | Merged openstack/neutron stable/ussuri: [OVN] Add the VIF details "connectivity" parameter https://review.opendev.org/c/openstack/neutron/+/826441 | 05:24 |
opendevreview | ZhouHeng proposed openstack/neutron-fwaas master: Revert "Retire neutron-fwaas project" https://review.opendev.org/c/openstack/neutron-fwaas/+/828149 | 08:15 |
opendevreview | ZhouHeng proposed openstack/neutron-fwaas master: Revert "Retire neutron-fwaas project" https://review.opendev.org/c/openstack/neutron-fwaas/+/828149 | 08:23 |
bcafarel | sigh, grenade jobs failing on train again https://zuul.opendev.org/t/openstack/build/e850cb27f72a42aaaf18b555a31f2eb5 | 08:57 |
bcafarel | ERROR: This script does not work on Python 3.6 The minimum supported Python version is 3.7 hmm | 09:02 |
ykarel | bcafarel, seems need to backport https://review.opendev.org/c/openstack/devstack/+/827421 to stein | 09:06 |
bcafarel | ykarel++ sounds like it indeed! | 09:08 |
bcafarel | ok let's see how https://review.opendev.org/c/openstack/devstack/+/828769 goes | 09:12 |
opendevreview | Bernard Cafarelli proposed openstack/neutron stable/train: Avoid writing segments to the DB repeatedly https://review.opendev.org/c/openstack/neutron/+/825735 | 09:13 |
ykarel | ack | 09:15 |
opendevreview | Lajos Katona proposed openstack/neutron stable/pike: Remove neutron-lbaas from the CI jobs requirements https://review.opendev.org/c/openstack/neutron/+/823292 | 09:30 |
opendevreview | Anton Vazhnetsov proposed openstack/ovsdbapp master: vtep: add support for 'hardware_vtep' schema https://review.opendev.org/c/openstack/ovsdbapp/+/828720 | 09:32 |
opendevreview | Merged openstack/neutron stable/victoria: Make sure "dead vlan" ports cannot transmit packets https://review.opendev.org/c/openstack/neutron/+/828497 | 09:58 |
opendevreview | Merged openstack/ovn-octavia-provider stable/ussuri: Support creating members without a subnet ID https://review.opendev.org/c/openstack/ovn-octavia-provider/+/827045 | 10:10 |
opendevreview | Merged openstack/ovn-octavia-provider stable/victoria: Support creating members without a subnet ID https://review.opendev.org/c/openstack/ovn-octavia-provider/+/827040 | 10:10 |
opendevreview | ZhouHeng proposed openstack/neutron-tempest-plugin master: Readd jobs for neutron-fwaas master branch https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/828443 | 10:16 |
*** mdbooth3 is now known as mdbooth | 10:25 | |
lajoskatona | ralonsoh: 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 |
ralonsoh | lajoskatona, well, actually I would prefer to have some patches merged | 10:40 |
ralonsoh | lajoskatona, https://review.opendev.org/c/openstack/neutron-lib/+/827533 | 10:41 |
ralonsoh | I had a +2 from you | 10:41 |
ralonsoh | but I needed to change the docs | 10:42 |
ralonsoh | I would like to have this one too | 10:42 |
lajoskatona | ralonsoh: ok, that was in my mind, but was not sure where it is :-) | 10:42 |
ralonsoh | now is rechecking | 10:42 |
lajoskatona | ok | 10:43 |
opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/ussuri: Reduce coverage threshold to allow old backports https://review.opendev.org/c/openstack/ovn-octavia-provider/+/828856 | 11:01 |
afaranha | slaweq, 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/+/824750 | 11:21 |
opendevreview | Merged openstack/networking-ovn stable/train: Support creating members without a subnet ID https://review.opendev.org/c/openstack/networking-ovn/+/827051 | 11:34 |
ralonsoh | afaranha, I don't think we need to backport those patches. The devstack version supporting this new flag is the current one | 11:53 |
ralonsoh | in any case, you can ping slaweq next Tuesday | 11:53 |
afaranha | ralonsoh, the problem was with this patch for xena: https://review.opendev.org/c/openstack/neutron/+/828697 | 11:54 |
afaranha | a comment from Bernard Cafarelli, The configure_for_func_testing.sh changes break functional/fullstack, another backport needed? | 11:54 |
afaranha | ERROR 1045 (28000): Access denied for user 'root'@'localhost' (using password: YES) | 11:54 |
afaranha | I 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 Xena | 11:55 |
ralonsoh | afaranha, no no, what is failing is you change in the mysql user | 11:55 |
ralonsoh | not the OVN compilation flag | 11:55 |
ralonsoh | I don't know if this version of devstack supports the change you introduced in configure_for_func_testing.sh | 11:56 |
afaranha | ow, okay, so I need to figure out the correct user/password for this | 11:58 |
opendevreview | Merged openstack/neutron stable/victoria: [Stable Only] Enforce policy for qos_policy_id attribute https://review.opendev.org/c/openstack/neutron/+/827016 | 13:02 |
opendevreview | yangjianfeng proposed openstack/neutron-lib master: api-ref for ndp proxy https://review.opendev.org/c/openstack/neutron-lib/+/801181 | 13:21 |
opendevreview | yangjianfeng proposed openstack/neutron master: Add ndp proxy policy rules https://review.opendev.org/c/openstack/neutron/+/828022 | 13:33 |
opendevreview | yangjianfeng proposed openstack/neutron master: Add ndp proxy policy rules https://review.opendev.org/c/openstack/neutron/+/828022 | 13:36 |
opendevreview | yangjianfeng proposed openstack/neutron master: [Agent Side] L3 router support ndp proxy https://review.opendev.org/c/openstack/neutron/+/744815 | 13:39 |
*** dasm|off is now known as dasm | 14:21 | |
ykarel | ralonsoh, when you get chance please check https://review.opendev.org/c/openstack/neutron/+/827017 and https://review.opendev.org/c/openstack/neutron/+/827424 | 14:49 |
ykarel | Thanks in advance | 14:49 |
ralonsoh | ok | 14:49 |
*** ykarel is now known as ykarel|away | 14:56 | |
opendevreview | yangjianfeng proposed openstack/neutron-lib master: api-ref for ndp proxy https://review.opendev.org/c/openstack/neutron-lib/+/801181 | 15:09 |
ralonsoh | lajoskatona, https://review.opendev.org/c/openstack/neutron-lib/+/827533 | 15:35 |
ralonsoh | can you check it? | 15:36 |
ralonsoh | thanks in advance | 15:36 |
lajoskatona | ralonsoh: sure | 15:36 |
ralonsoh | we have the other n-lib patch almost merged | 15:36 |
ralonsoh | I think that's all in the n-lib backlog | 15:36 |
lajoskatona | ralonsoh: this one: https://review.opendev.org/c/openstack/neutron-lib/+/801181 ? Cool | 15:41 |
ralonsoh | yes, this one | 15:42 |
opendevreview | Merged openstack/neutron stable/ussuri: [Stable Only] Enforce policy for qos_policy_id attribute https://review.opendev.org/c/openstack/neutron/+/827017 | 16:24 |
opendevreview | Merged openstack/neutron master: Re enable test_network_v6.TestGettingAddress https://review.opendev.org/c/openstack/neutron/+/827424 | 16:25 |
opendevreview | Merged openstack/neutron stable/wallaby: [OVN] Check new added segments in OVN mech driver https://review.opendev.org/c/openstack/neutron/+/826103 | 16:40 |
mlavalle | lajoskatona: would you take a look at https://review.opendev.org/c/openstack/neutron/+/827580? | 16:43 |
opendevreview | Merged openstack/neutron-lib master: api-ref for ndp proxy https://review.opendev.org/c/openstack/neutron-lib/+/801181 | 17:11 |
*** marlinc is now known as Guest73 | 17:16 | |
stephenfin | Can someone stick a +W on this doc patch? https://review.opendev.org/c/openstack/neutron/+/801771 | 17:30 |
mlavalle | stephenfin: done | 17:31 |
stephenfin | ty :) | 17:31 |
lajoskatona | mlavalle: nice series with the timeouts | 17:36 |
opendevreview | Merged openstack/neutron stable/wallaby: [Stable Only] Enforce policy for qos_policy_id attribute https://review.opendev.org/c/openstack/neutron/+/827015 | 17:41 |
ralonsoh | zzzeek, hi, do you have a couple of mins? this is about https://review.opendev.org/c/openstack/oslo.db/+/804775 | 18:04 |
ralonsoh | I'm trying to replace the checks we have in neutron/neutron-lib of session.is_active | 18:05 |
ralonsoh | what we need to know is that the session has no pending command, that the transaction has nothing to commit | 18:06 |
ralonsoh | the problem is that, when I disable autocommit, we can have a session with an active transaction | 18:06 |
ralonsoh | and I don't know how can I replace the old logic | 18:06 |
ralonsoh | where 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 dkehn | 18:11 | |
opendevreview | Brian Haley proposed openstack/neutron master: DNM: Test IPv6 all (including tunnel) job definition https://review.opendev.org/c/openstack/neutron/+/828634 | 18:53 |
opendevreview | Merged openstack/neutron master: Wait longer before deleting DPDK vhu trunk bridges https://review.opendev.org/c/openstack/neutron/+/827580 | 19:57 |
opendevreview | Merged openstack/neutron master: doc: Remove references to dead VMWare NSX extensions https://review.opendev.org/c/openstack/neutron/+/801771 | 19:58 |
opendevreview | Merged openstack/neutron stable/train: Avoid writing segments to the DB repeatedly https://review.opendev.org/c/openstack/neutron/+/825735 | 19:58 |
*** dasm is now known as dasm|off | 21:49 | |
opendevreview | Jakub Libosvar proposed openstack/networking-ovn stable/train: [ovn] Prevent stale ports in the OVN database https://review.opendev.org/c/openstack/networking-ovn/+/828801 | 22:09 |
*** ubuntu is now known as Guest108 | 22:28 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!