| haleyb | slaweq: i know you're not here, but think your grenade patch broke the gate, https://bugs.launchpad.net/neutron/+bug/1999705 - the best I could do is trigger a revert since it's EOD here, but it's probably fixable some other way | 02:25 |
|---|---|---|
| opendevreview | Brian Haley proposed openstack/neutron master: DNM: Test grenade revert https://review.opendev.org/c/openstack/neutron/+/867755 | 02:42 |
| opendevreview | yatin proposed openstack/neutron-tempest-plugin master: [DNM] Test stability of workaround https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/867650 | 07:06 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Enable ML2/OVS backend in the -ovs- grenade jobs explicitly https://review.opendev.org/c/openstack/neutron/+/867764 | 08:09 |
| slaweq | haleyb thx for reporting this, I just proposed fix https://review.opendev.org/c/openstack/neutron/+/867764 | 08:10 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: WIP Remove skipdist (that will call the project installation via setup.py) https://review.opendev.org/c/openstack/neutron/+/867554 | 08:51 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Testing https://review.opendev.org/c/openstack/neutron/+/867554 https://review.opendev.org/c/openstack/neutron/+/867616 | 08:52 |
| slaweq | ralonsoh hi, please check https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/867222 when You will have few minutes | 09:34 |
| slaweq | but no rush, it's not urgent at all | 09:34 |
| slaweq | ykarel hi, and also the same for You ^^ :) | 09:34 |
| ralonsoh | slaweq, sure (and I'm checking too the issue with grenade job definitions) | 09:34 |
| slaweq | ralonsoh for grenade jobs the problem was that all ovs-grenade jobs in our repo didn't really had explicitly set ovs as backend | 09:35 |
| ralonsoh | yeah | 09:35 |
| slaweq | so currently all those jobs are running with ovn backend | 09:35 |
| ralonsoh | btw, the pathc is passing | 09:35 |
| ralonsoh | folks, please review https://review.opendev.org/c/openstack/neutron/+/867764 | 09:35 |
| slaweq | I proposed patch to fix it alread | 09:35 |
| opendevreview | Merged openstack/neutron-fwaas-dashboard master: Imported Translations from Zanata https://review.opendev.org/c/openstack/neutron-fwaas-dashboard/+/866098 | 09:37 |
| ralonsoh | slaweq, qq | 09:39 |
| ralonsoh | in https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/867222/3/neutron_tempest_plugin/api/test_security_groups.py#70, is it possible to change the "create_security_group" method to read the "stateless_sg" config option? | 09:39 |
| ralonsoh | na, never mind | 09:39 |
| ralonsoh | let's make it explicit as is now | 09:39 |
| opendevreview | Vasyl Saienko proposed openstack/neutron master: Set ip_nonlocal_bind to 0 for HA routers https://review.opendev.org/c/openstack/neutron/+/867769 | 09:42 |
| slaweq | ralonsoh statateful/stateless SG can be created through API, there's no config option for that | 10:02 |
| ralonsoh | slaweq, no I mean the test config value | 10:19 |
| ralonsoh | but it's ok to make this explicit | 10:19 |
| slaweq | ahh, but then You would need to have separate job for stateful and stateless to have different config option in tempest set | 10:20 |
| slaweq | now we can test both in the same job(s) | 10:20 |
| ralonsoh | yeah, I prefer what you have implemented on a second thought | 10:22 |
| opendevreview | Anton Kurbatov proposed openstack/neutron master: Do not update static routes in snat-ns for dvr router with ha https://review.opendev.org/c/openstack/neutron/+/867678 | 10:24 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Add ``devlink.get_port`` method for devlink ports https://review.opendev.org/c/openstack/neutron/+/867526 | 10:31 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Add vnic_type and binding profile capabilities to LSP info https://review.opendev.org/c/openstack/neutron/+/867359 | 10:31 |
| slaweq | ralonsoh thx :) | 10:33 |
| slaweq | ykarel lajoskatona hi, can one of You check https://review.opendev.org/c/openstack/neutron/+/867764 ? | 11:49 |
| slaweq | it's needed to unblock our gate | 11:50 |
| ykarel_ | slaweq, ack | 11:58 |
| opendevreview | Vasyl Saienko proposed openstack/neutron master: Set ip_nonlocal_bind to 0 for HA routers https://review.opendev.org/c/openstack/neutron/+/867769 | 12:02 |
| opendevreview | Lajos Katona proposed openstack/tap-as-a-service master: OSC: Remove calls to neutronclient https://review.opendev.org/c/openstack/tap-as-a-service/+/867786 | 12:23 |
| lajoskatona | ralonsoh: Hi, during my playing with SDK & neutronclient & Heat, I started to fix stadiums also which use neutronclient | 12:24 |
| lajoskatona | ralonsoh: for taas https://review.opendev.org/c/openstack/tap-as-a-service/+/867786 will be the patch ( I have to fix some more unit tests) | 12:24 |
| lajoskatona | ralonsoh: do you think one "global" bug is enough for tracking or open one bug for each stadium? | 12:25 |
| ralonsoh | lajoskatona, I'll those patches to the Neutron meetings too | 12:25 |
| ralonsoh | lajoskatona, I think one will be enough | 12:25 |
| ralonsoh | we can have all patches with one topic | 12:25 |
| lajoskatona | ralonsoh: ack | 12:26 |
| frickler | lajoskatona: are you also planning to work on neutronclient not using neutronclient? for all bgp commands like "openstack bgp speaker list" there are currently deprecation warnings, which make no sense at all to me | 12:33 |
| opendevreview | Lajos Katona proposed openstack/tap-as-a-service master: OSC: Remove calls to neutronclient https://review.opendev.org/c/openstack/tap-as-a-service/+/867786 | 12:33 |
| lajoskatona | ralonsoh: https://bugs.launchpad.net/neutron/+bug/1999774 | 12:34 |
| ralonsoh | lajoskatona, perfect, I'll add it to the Neutron meeting agenda too | 12:34 |
| lajoskatona | frickler: yeah we should reach a singularity in worst case :P | 12:34 |
| lajoskatona | frickler: some stadiums has the client code in the repo itself (bgpvpn, sfc....) but for BGP commands.... We create OSC plugin in neutron-dynamic-routing or move those commands to python-openstackclient and SDK, whcih needs some discussion with the owners of those repos | 12:39 |
| *** gthiemon1e is now known as gthiemonge | 12:41 | |
| opendevreview | Vasyl Saienko proposed openstack/neutron master: Set ip_nonlocal_bind to 1 for HA routers and DVR snat https://review.opendev.org/c/openstack/neutron/+/867769 | 13:06 |
| opendevreview | yatin proposed openstack/neutron-tempest-plugin master: [DNM] Test stability of workaround https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/867650 | 13:16 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-tempest-plugin master: Run security group scenario tests against stateless security groups too https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/867802 | 13:17 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-tempest-plugin master: Run security group api tests against stateless security groups too https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/867222 | 13:31 |
| opendevreview | Vasyl Saienko proposed openstack/neutron stable/zed: Set ip_nonlocal_bind to 1 for HA routers and DVR snat https://review.opendev.org/c/openstack/neutron/+/867640 | 13:38 |
| haleyb | slaweq: thanks for the fix! | 14:21 |
| slaweq | haleyb yw :) and sorry that I broke our gate | 14:21 |
| sahid | slaweq: yep ++ only people that do nothing do not broke thing ;) | 14:23 |
| sahid | lajoskatona: o/ any chance that if you have a moment to look at https://review.opendev.org/c/openstack/neutron/+/841838 :-) | 14:23 |
| haleyb | it happens, will be green soon enough | 14:23 |
| lajoskatona | sahid: I will | 14:24 |
| *** dasm|off is now known as dasm | 14:30 | |
| opendevreview | Lajos Katona proposed openstack/tap-as-a-service master: OSC: Remove calls to neutronclient https://review.opendev.org/c/openstack/tap-as-a-service/+/867786 | 14:38 |
| opendevreview | Arnaud Morin proposed openstack/neutron master: Allow restoration of tun_ofports on agent restart https://review.opendev.org/c/openstack/neutron/+/860270 | 15:00 |
| amorin | hey ralonsoh, haleyb, I think I answered to most of the comments in the above patchset ^ | 15:02 |
| amorin | the only piece which is missing is about the tempfile | 15:02 |
| amorin | I cant figure out how to achieve that with fixture | 15:02 |
| haleyb | ralonsoh: i had created https://review.opendev.org/c/openstack/neutron-lib/+/867571 for tox4 as well, haven't looked at neutron-tempest-plugin, and being near EOY shutdown here probably won't get done by me | 15:02 |
| ralonsoh | amorin, I'll check it today | 15:03 |
| amorin | thanks, no hurry | 15:03 |
| ralonsoh | haleyb, ok, I'll check it now | 15:03 |
| haleyb | or we just pin everything to <4 and do it later, i'm starting to see lots of projects get broken by this just outside the opendev "tent" :( | 15:05 |
| opendevreview | Merged openstack/neutron master: Enable ML2/OVS backend in the -ovs- grenade jobs explicitly https://review.opendev.org/c/openstack/neutron/+/867764 | 15:06 |
| ralonsoh | ok, I'll check today, in a meeting now | 15:06 |
| ralonsoh | haleyb, btw, https://review.opendev.org/c/openstack/neutron/+/867554 | 15:13 |
| ralonsoh | but I dont' know why the privsep daemon is not correctly condifured | 15:14 |
| ralonsoh | this is why functional and fullstack are failing | 15:14 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [WIP] [OVN] OVN monitor agent https://review.opendev.org/c/openstack/neutron/+/866480 | 15:31 |
| haleyb | ralonsoh: that is a good question about privsep I have not seen it | 15:39 |
| lajoskatona | ralonsoh, haleyb: Hi, could you check this one for networking-odl functional timeouts: https://review.opendev.org/c/openstack/networking-odl/+/866849 (I forgot about it the last week) | 15:41 |
| ralonsoh | sure | 15:41 |
| haleyb | lajoskatona: ack | 15:43 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: segment: enable multisegments support for host https://review.opendev.org/c/openstack/neutron/+/841838 | 15:56 |
| opendevreview | Merged openstack/networking-odl master: CI: Increase the timeout of functional jobs https://review.opendev.org/c/openstack/networking-odl/+/866849 | 16:23 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: dhcp: fix issue when network is already removed https://review.opendev.org/c/openstack/neutron/+/865823 | 16:26 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [WIP] [OVN] OVN monitor agent https://review.opendev.org/c/openstack/neutron/+/866480 | 17:53 |
| ralonsoh | stephenfin, hi! | 17:54 |
| ralonsoh | I have a problem with tox4 and privsep | 17:54 |
| ralonsoh | this is the patch I'm pushing to Neutron | 17:54 |
| ralonsoh | that fixes all problems we had because tox was not installing neutron (editable) | 17:55 |
| ralonsoh | but I still have problems with privsep | 17:55 |
| ralonsoh | https://review.opendev.org/c/openstack/neutron/+/867554 | 17:55 |
| stephenfin | tox4 is badly broken atm. I've been seeing serious issues with pbr-based projects on it. I opened an issue against it last night https://github.com/tox-dev/tox/issues/2712 | 17:58 |
| stephenfin | ralonsoh: https://github.com/tox-dev/tox/issues/2730 looks relevant btw | 17:58 |
| ralonsoh | stephenfin, checking now | 17:59 |
| ralonsoh | stephenfin, and I think we are matching that (skipsdist = False,usedevelop = True), that least with my patch | 18:01 |
| ralonsoh | but still no rootwrap filters installed | 18:02 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: WIP Remove skipdist (that will call the project installation via setup.py) https://review.opendev.org/c/openstack/neutron/+/867554 | 18:03 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Testing https://review.opendev.org/c/openstack/neutron/+/867554 https://review.opendev.org/c/openstack/neutron/+/867616 | 18:03 |
| stephenfin | There are a lot of packaging errors in that log. I think this is the same pbr+tox4 incompatibility I've been seeing | 18:04 |
| opendevreview | Brian Haley proposed openstack/neutron master: WIP Remove skipdist (that will call the project installation via setup.py) https://review.opendev.org/c/openstack/neutron/+/867554 | 21:22 |
| *** sfinucan is now known as stephenfin | 22:55 | |
| *** dasm is now known as dasm|off | 23:59 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!