| *** Jeff is now known as Guest31677 | 01:30 | |
| *** liuxie is now known as liushy | 06:10 | |
| ykarel | ralonsoh, ovsdbapp latest release also impacted functional tests | 06:54 |
|---|---|---|
| ykarel | https://0a89c7fd197f008c5875-18acae3550a14afbd46e45014db8ab53.ssl.cf5.rackcdn.com/openstack/e69ff7d8ffa648a59cce5f358556a99e/testr_results.html | 06:54 |
| * ykarel creating bug | 06:54 | |
| ralonsoh | ykarel, let me check | 06:55 |
| ralonsoh | ahhh yes, I was reading it, from the patch submitted by Brian | 06:55 |
| ralonsoh | I think this is the string method introduced by Terry | 06:55 |
| ralonsoh | sorry, not this patch | 06:56 |
| ralonsoh | ykarel, related to the SFC issues, do you think the DHCP patch is affecting it? | 06:57 |
| ykarel | ralonsoh, yes | 06:57 |
| ralonsoh | ok, I'll check it later, after the ovdsbapp issue | 06:58 |
| ykarel | reported https://bugs.launchpad.net/neutron/+bug/2131994 for functional | 07:03 |
| opendevreview | Rodolfo Alonso proposed openstack/ovsdbapp master: Set the virtual env classes default config directory to '' https://review.opendev.org/c/openstack/ovsdbapp/+/967804 | 08:01 |
| ralonsoh | ^^ I'll push a neutron patch testing the FT job using ^^ this patch | 08:01 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: DNM == Testing patch 967804, FT + ovsdbapp patch https://review.opendev.org/c/openstack/neutron/+/967805 | 08:03 |
| opendevreview | Rodolfo Alonso proposed openstack/ovsdbapp master: Set the virtual env classes default config directory to '' https://review.opendev.org/c/openstack/ovsdbapp/+/967804 | 08:10 |
| opendevreview | yatin proposed openstack/neutron unmaintained/2024.1: [DNM] Check tobiko job fix https://review.opendev.org/c/openstack/neutron/+/967807 | 08:12 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Revert "[eventlet-removal] Handle stop DHCP agent" https://review.opendev.org/c/openstack/neutron/+/967808 | 08:22 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-tempest-plugin master: DNM == Test DHCP agent patch revert in Neutron https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/967809 | 08:23 |
| opendevreview | Harald Jensås proposed openstack/neutron master: [WIP] Filter ML2 drivers by network type in vlan checks https://review.opendev.org/c/openstack/neutron/+/967810 | 08:36 |
| opendevreview | Harald Jensås proposed openstack/neutron master: [WIP] Filter ML2 drivers by network type in vlan checks https://review.opendev.org/c/openstack/neutron/+/967810 | 08:42 |
| opendevreview | Lajos Katona proposed openstack/neutron master: Do not stop dnsmasq and other external processes https://review.opendev.org/c/openstack/neutron/+/967811 | 09:29 |
| lajoskatona | eolivare, ralonsoh, ykarel: Hi, ------^ for dhcp-agent dataplane issue | 09:32 |
| ralonsoh | lajoskatona, there could be an issue with the DHCP patch and SFC | 09:32 |
| ralonsoh | I've pushed some testing patches | 09:33 |
| lajoskatona | it is now a partial revert of https://review.opendev.org/c/openstack/neutron/+/963386, so we can consider a full revert of that also but this can be used to test with tobiko I suppose | 09:33 |
| lajoskatona | ralonsoh: in networking-sfc repo? | 09:33 |
| ralonsoh | yes, one sec | 09:34 |
| ralonsoh | bug: https://bugs.launchpad.net/neutron/+bug/2131941 | 09:34 |
| ralonsoh | testing patches: | 09:34 |
| ralonsoh | * https://review.opendev.org/c/openstack/neutron/+/967808 | 09:34 |
| ralonsoh | * https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/967809 | 09:34 |
| ralonsoh | and it seems to work ^^ | 09:34 |
| ykarel | thx lajoskatona , should also clear sfc, vpnaas | 09:35 |
| ralonsoh | lajoskatona, qq: this patch (https://review.opendev.org/c/openstack/neutron/+/967811) does not stop the processes, but is it possible to stop the DHCP agent? | 09:35 |
| ralonsoh | let me use your last patch with the SFC testing patch | 09:36 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-tempest-plugin master: DNM == Test DHCP agent patch revert in Neutron https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/967809 | 09:36 |
| eolivare | lajoskatona, I will run tobiko with your patch too, thanks | 09:38 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Workaround for the OVN/OVS directories in functional job https://review.opendev.org/c/openstack/neutron/+/967813 | 09:52 |
| ralonsoh | slaweq, hello! please check https://review.opendev.org/c/openstack/ovsdbapp/+/967804. I've also pushed a patch for Neutron. The FT job is passing (still running). The broken jobs are now working: https://zuul.opendev.org/t/openstack/stream/0ad05889182f4623bbf3a4aae0a92ce6?logfile=console.log | 10:02 |
| ralonsoh | lajoskatona, your last patch (963386) is still breaking the SFC CI | 10:05 |
| lajoskatona | ralonsoh: I check it | 10:07 |
| ralonsoh | thanks a lot | 10:07 |
| lajoskatona | eolivare: thanks, I just started the experimental suit and in that we have devstack-tobiko-neutron, that job can be fine also I think | 10:08 |
| slaweq | ralonsoh: done | 10:09 |
| ralonsoh | thanks | 10:09 |
| eolivare | lajoskatona, devstack-tobiko-neutron uses ml2/ovn and you need ml2/ovs to test your patch - you can use this path: https://review.opendev.org/c/x/devstack-plugin-tobiko/+/967812 | 10:10 |
| lajoskatona | eolivare: ok, thanks | 10:13 |
| *** sfinucan is now known as stephenfin | 10:16 | |
| opendevreview | Lajos Katona proposed openstack/neutron-tempest-plugin master: DNM == Test DHCP agent patch revert in Neutron https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/967809 | 10:17 |
| opendevreview | Bence Romsics proposed openstack/neutron master: [docs] Add OVS/OVN min requirement table https://review.opendev.org/c/openstack/neutron/+/967816 | 10:20 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Workaround for the OVN/OVS directories in functional job https://review.opendev.org/c/openstack/neutron/+/967813 | 10:54 |
| eolivare | lajoskatona, the tests passed with your patch https://zuul.opendev.org/t/openstack/build/7f2b71de9d0840659c0428039bf9a22c | 11:07 |
| ralonsoh | same for SFC | 11:09 |
| lajoskatona | eolivare, ralonsoh: cool | 12:06 |
| opendevreview | Daniel Failing proposed openstack/neutron master: Improve OVNGatewayLeastLoadedScheduler runtime https://review.opendev.org/c/openstack/neutron/+/967827 | 12:08 |
| opendevreview | Daniel Failing proposed openstack/neutron master: Improve OVNGatewayLeastLoadedScheduler runtime https://review.opendev.org/c/openstack/neutron/+/967827 | 12:10 |
| opendevreview | Bence Romsics proposed openstack/neutron master: [docs] Add OVS/OVN min requirement table https://review.opendev.org/c/openstack/neutron/+/967816 | 13:20 |
| ralonsoh | slaweq, please check this WA for neutron: https://review.opendev.org/c/openstack/neutron/+/967813 | 13:30 |
| ralonsoh | ykarel_, I have no idea about what is happening in the ovsdbapp FT job, in particular in the proposed patch | 13:30 |
| *** ykarel_ is now known as ykarel | 13:30 | |
| ralonsoh | I'm going to send a testing patch | 13:30 |
| ralonsoh | that's weird | 13:30 |
| ralonsoh | https://zuul.opendev.org/t/openstack/builds?job_name=openstack-tox-functional&project=openstack/ovsdbapp | 13:31 |
| slaweq | ralonsoh: approved | 13:31 |
| ralonsoh | thanks | 13:31 |
| ykarel | ralonsoh, hmm strange, good to also see if these failures happen even with concurrency=1 | 13:32 |
| ralonsoh | checking now | 13:33 |
| opendevreview | Rodolfo Alonso proposed openstack/ovsdbapp master: DNM == Test functional job https://review.opendev.org/c/openstack/ovsdbapp/+/967842 | 13:36 |
| opendevreview | Rodolfo Alonso proposed openstack/ovsdbapp master: DNM == Test functional job on top of patch 967804 https://review.opendev.org/c/openstack/ovsdbapp/+/967843 | 13:37 |
| ralonsoh | I've opened a ovsdbapp bug: https://bugs.launchpad.net/ovsdbapp/+bug/2132018 | 13:48 |
| ralonsoh | the FT job is unstable (that wasn't happening before the last release) | 13:49 |
| opendevreview | Rodolfo Alonso proposed openstack/ovsdbapp master: Revert "Rework venv to support arbitrary schemas" https://review.opendev.org/c/openstack/ovsdbapp/+/967844 | 13:50 |
| opendevreview | Rodolfo Alonso proposed openstack/ovsdbapp master: DNM == Test functional job on top of patch 967844 https://review.opendev.org/c/openstack/ovsdbapp/+/967843 | 13:51 |
| opendevreview | Rodolfo Alonso proposed openstack/ovsdbapp master: DNM == Test functional job, concurrency=1 https://review.opendev.org/c/openstack/ovsdbapp/+/967842 | 13:58 |
| opendevreview | Rodolfo Alonso proposed openstack/ovsdbapp master: DNM == Test functional job, concurrency=1 https://review.opendev.org/c/openstack/ovsdbapp/+/967842 | 14:10 |
| opendevreview | Rodolfo Alonso proposed openstack/ovsdbapp master: DNM == Test functional job, concurrency=1 https://review.opendev.org/c/openstack/ovsdbapp/+/967842 | 14:19 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Wait for the "Chassis" creation in the HCG tests https://review.opendev.org/c/openstack/neutron/+/966789 | 14:20 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] The external networks GW chassis must the same as the GW LRP https://review.opendev.org/c/openstack/neutron/+/962155 | 14:21 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Sync the LRP Gateway_Chassis with the network HCG https://review.opendev.org/c/openstack/neutron/+/964381 | 14:21 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Initialize OVN agent in ``start`` method https://review.opendev.org/c/openstack/neutron/+/967219 | 14:22 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2025.2: [OVN] Initialize OVN agent in ``start`` method https://review.opendev.org/c/openstack/neutron/+/967300 | 14:24 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2025.1: [OVN] Initialize OVN agent in ``start`` method https://review.opendev.org/c/openstack/neutron/+/967302 | 14:26 |
| opendevreview | Rodolfo Alonso proposed openstack/ovsdbapp master: DNM == Test functional job, concurrency=1 https://review.opendev.org/c/openstack/ovsdbapp/+/967842 | 14:36 |
| haleyb | ralonsoh: so it looks like we've found the issue(s), is the plan to 1) fix functional job; 2) revert the dhcp change; 3) fix ovsdbapp; etc ? | 14:40 |
| haleyb | i see the patches and approvals | 14:40 |
| haleyb | and i guess releasing a new ovsdbapp is on that list | 14:41 |
| ralonsoh | haleyb, for ovsdbapp: we have identified the issue | 14:41 |
| ralonsoh | I've also pushed a WA that is in the gate now (for neutron) | 14:41 |
| ralonsoh | new problem: the ovsdbapp FT ci is unstable | 14:42 |
| ralonsoh | and not related to any patch merged recently (at least as far as I've tested) | 14:42 |
| haleyb | https://review.opendev.org/c/openstack/neutron/+/967813 and https://review.opendev.org/c/openstack/ovsdbapp/+/967804 | 14:42 |
| ralonsoh | yes | 14:42 |
| haleyb | yeah i thought i found something yesterday but wasn't it | 14:42 |
| ralonsoh | the second is not passing because of this issue in the ovsdbapp FT job | 14:42 |
| ralonsoh | I've created a bug in this project | 14:42 |
| ralonsoh | I'm checking something here: | 14:43 |
| ralonsoh | --> https://review.opendev.org/c/openstack/ovsdbapp/+/967842 | 14:43 |
| ralonsoh | about SFC: lajos pushed a patch that is fixing (without reverting) the DHCP agent problem | 14:43 |
| ralonsoh | also fixing tobiko (affected by the DHCP agent patch) | 14:43 |
| ralonsoh | link (one sec) | 14:43 |
| haleyb | https://review.opendev.org/c/openstack/neutron/+/967811 | 14:44 |
| ralonsoh | right | 14:44 |
| ralonsoh | it is still not passing the Neutron CI, but SFC and tobiko work with this patch | 14:44 |
| ralonsoh | SFC: https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/967809 | 14:45 |
| ralonsoh | so we have several fronts now | 14:45 |
| haleyb | glad things are going in the right direction, i'll take a look at the patches | 14:46 |
| opendevreview | Rodolfo Alonso proposed openstack/ovsdbapp master: DNM == Test functional job, concurrency=1 https://review.opendev.org/c/openstack/ovsdbapp/+/967842 | 14:50 |
| ralonsoh | haleyb, ykarel the ovsdbapp issue: it could be an ovn problem. I'm manually testing different OVN commits | 15:04 |
| ralonsoh | I've manually reproduced it locally | 15:04 |
| opendevreview | Merged openstack/neutron master: Workaround for the OVN/OVS directories in functional job https://review.opendev.org/c/openstack/neutron/+/967813 | 15:32 |
| opendevreview | Rodolfo Alonso proposed openstack/ovsdbapp master: DNM == Test functional job, concurrency=1 https://review.opendev.org/c/openstack/ovsdbapp/+/967842 | 16:39 |
| opendevreview | Rodolfo Alonso proposed openstack/ovsdbapp master: DNM == Test functional job, concurrency=1 https://review.opendev.org/c/openstack/ovsdbapp/+/967842 | 16:40 |
| opendevreview | Bodo Petermann proposed openstack/neutron master: Allow plugins to add periodics to maintenance worker https://review.opendev.org/c/openstack/neutron/+/939817 | 16:45 |
| opendevreview | Merged openstack/neutron master: [OVN] Wait for the "Chassis" creation in the HCG tests https://review.opendev.org/c/openstack/neutron/+/966789 | 17:15 |
| opendevreview | Daniel Failing proposed openstack/neutron master: Improve OVNGatewayLeastLoadedScheduler runtime https://review.opendev.org/c/openstack/neutron/+/967827 | 17:49 |
| opendevreview | Sergey Kraynev proposed openstack/neutron master: Replace response body in after hook for 404 error https://review.opendev.org/c/openstack/neutron/+/966498 | 17:52 |
| opendevreview | Lajos Katona proposed openstack/neutron master: [eventlet-removal] Handle systemctl stop for DHCP agent https://review.opendev.org/c/openstack/neutron/+/967902 | 18:35 |
| opendevreview | Jakub Libosvar proposed openstack/ovsdbapp master: venv: Fix ovsschema path searching https://review.opendev.org/c/openstack/ovsdbapp/+/967939 | 23:16 |
| opendevreview | Brian Haley proposed openstack/neutron master: Fix incorrect log message in ovn_client https://review.opendev.org/c/openstack/neutron/+/967940 | 23:21 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!