| opendevreview | Brian Haley proposed openstack/neutron-tempest-plugin master: Set GLOVAL_VENV=false to fix broken gate jobs https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/891550 | 00:15 |
|---|---|---|
| *** dmellado819181 is now known as dmellado81918 | 04:45 | |
| tore | haleyb: btw, I'm the submitter of https://bugs.launchpad.net/neutron/+bug/2031087 - just wanted to let you know I hang around here in CEST work-hours, so if you have any quick questions or suggestions or whatever that launchpad would be overkill for, do feel free to highlight me | 09:52 |
| ralonsoh | slaweq, https://review.opendev.org/c/openstack/neutron/+/889871 review done | 10:16 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Use HasStandardAttributes as parent class for Tags DB model https://review.opendev.org/c/openstack/neutron/+/891580 | 10:17 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-tempest-plugin master: DNM Just testing some sql queries https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/890419 | 10:18 |
| slaweq | ralonsoh thx I will address Your comments today | 10:20 |
| ralonsoh | frickler, hi! Did you see the error in neutron-dynamic-routing? | 11:37 |
| ralonsoh | Is in the last 4 releases | 11:38 |
| ralonsoh | https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/891550 | 11:38 |
| ralonsoh | e.g.: https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_7ab/891550/2/check/neutron-tempest-plugin-dynamic-routing/7ab85ff/testr_results.html | 11:38 |
| ralonsoh | frickler, https://bugs.launchpad.net/neutron/+bug/2031526 | 11:41 |
| ralonsoh | I'm going for lunch now, I'll check it later. We can't merge anything in n-t-p because of this | 11:42 |
| frickler | ralonsoh: yes, lajoskatona mentioned that last week, but I didn't get to look at it yet. seems some issue in docker container, help appreciated | 11:49 |
| lajoskatona | frickler, ralonsoh: I try to allocate some time this week or early next week to check it, though this docker stuff in n-d-r CI is new to me | 11:53 |
| frickler | lajoskatona: I'm holding a node now for debugging, seems easier than trying to reproduce locally. let me know your key if you want access, too | 12:12 |
| lajoskatona | frickler: thanks | 12:13 |
| lajoskatona | frickler: https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/891550 is fixing another issue for tempest with global venvs, and it is failing on the n-d-r job, do you think that temporary we can make the n-d-r jobs non-voting? | 12:14 |
| lajoskatona | frickler: if you have some idea of the issue | 12:15 |
| frickler | lajoskatona: commented on that patch, but yes, going n-v for n-d-r surely also is an option if there is no fast fix found | 12:20 |
| lajoskatona | frickler: thanks | 12:26 |
| opendevreview | Miro Tomaska proposed openstack/neutron master: Fix ovn-metadata agent sync of unused namespaces https://review.opendev.org/c/openstack/neutron/+/891232 | 12:28 |
| opendevreview | Lucas Alvares Gomes proposed openstack/neutron stable/xena: [OVN] ovn-db-sync check for router port differences https://review.opendev.org/c/openstack/neutron/+/891504 | 12:31 |
| opendevreview | Lucas Alvares Gomes proposed openstack/neutron stable/wallaby: [OVN] ovn-db-sync check for router port differences https://review.opendev.org/c/openstack/neutron/+/891506 | 12:35 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Use HasStandardAttributes as parent class for Tags DB model https://review.opendev.org/c/openstack/neutron/+/891580 | 12:38 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: [OVN] Make N-S traffic in the vlan tenant networks with PFs centralized https://review.opendev.org/c/openstack/neutron/+/889871 | 12:45 |
| opendevreview | Brian Haley proposed openstack/neutron-tempest-plugin master: Set GLOBAL_VENV=false to fix broken gate jobs https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/891550 | 12:58 |
| ralonsoh | lajoskatona, frickler thanks for checking and taking care of it | 13:06 |
| lajoskatona | ralonsoh: it's just frickler :P | 13:08 |
| ralonsoh | yeah, but you raised the issue last week | 13:08 |
| ralonsoh | thanks for that | 13:08 |
| opendevreview | Brian Haley proposed openstack/os-ken master: Remove superfluous sudo https://review.opendev.org/c/openstack/os-ken/+/891595 | 13:44 |
| haleyb | ralonsoh: so should I mark those n-d-r jobs as non-voting? Looking quickly at the logs it's not obvious what the failure is, only found a typo | 13:45 |
| haleyb | well, shouldn't say not obvious, the failure is "stderr: Error response from daemon: Container ce89787f1142ea66f8a865f56f971c4c4505f636ebc91739033f7799e14e44ae is not running" but i don't run docker enough to know why that happened | 13:46 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Use HasStandardAttributes as parent class for Tags DB model https://review.opendev.org/c/openstack/neutron/+/891580 | 14:00 |
| opendevreview | Brian Haley proposed openstack/neutron-tempest-plugin master: Make neutron-tempest-plugin-dynamic-routing-* non-voting https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/891600 | 14:05 |
| lajoskatona | haleyb: Hi, frickler is debugging it on one of the zuul hosts to have more info | 14:08 |
| haleyb | lajoskatona: ack, I pushed ^^^ to try and get the gate green, assuming i can rebase the VENV change on-top | 14:10 |
| lajoskatona | haleyb: yeah with frickler and ralonsoh we discussed it previously, and thanks for it, as we said let's wait a little with it | 14:24 |
| haleyb | ralonsoh, frickler: so it seems I can't just make those jobs non-voting, will have to combine with the VENV change if we want to fix the gate. I can do that unless there is a better plan | 15:02 |
| ralonsoh | the devstack patch has been reverted | 15:03 |
| ralonsoh | https://review.opendev.org/c/openstack/devstack/+/891280 | 15:03 |
| ralonsoh | so that should work now, if I'm not wrong | 15:03 |
| ralonsoh | sorry, the patch is approved but not merged yet | 15:04 |
| ralonsoh | so please wait for this revert | 15:04 |
| ralonsoh | haleyb, ^ | 15:04 |
| haleyb | ralonsoh: ack, even better | 15:14 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVS][QoS] Add QoS support for Trunk service, OVS driver https://review.opendev.org/c/openstack/neutron/+/839523 | 15:32 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Populate the "router.distributed" flag in ML2/OVN https://review.opendev.org/c/openstack/neutron/+/886992 | 15:33 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Replace "tenant_id" with "project_id" in IPAM engine https://review.opendev.org/c/openstack/neutron/+/877533 | 15:35 |
| frickler | ralonsoh: that is not the revert you are looking for | 15:48 |
| * haleyb will just twiddle his thumbs for now I guess, but yeah that patch wasn't a revert of the whole thing | 16:32 | |
| frickler | it seems the issue is within the quagga container that os-ken builds, but the only consumer and only test for that is n-d-r | 16:34 |
| frickler | haleyb: I don't think I'll be able to fix that today, feel free to proceed with the n-v path | 16:35 |
| haleyb | frickler: ack, i will have to combine my two patches together i think, but will push it out | 16:37 |
| frickler | I think it'll also be a good opportunity to bump the base container from 16.04 to something just a little bit more recent ;) | 16:41 |
| haleyb | oh yeah that is pretty old | 16:43 |
| opendevreview | Brian Haley proposed openstack/neutron-tempest-plugin master: Set GLOBAL_VENV=false to fix broken gate jobs https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/891550 | 16:43 |
| opendevreview | Miro Tomaska proposed openstack/neutron master: OVN Metadata handle process execeptions https://review.opendev.org/c/openstack/neutron/+/890986 | 17:37 |
| haleyb | lajoskatona: hey, if you're around can you look at a victoria and ussuri backport as you looked at the later ones. victoria one is https://review.opendev.org/c/openstack/neutron/+/890118 - thanks! | 19:10 |
| opendevreview | Miro Tomaska proposed openstack/neutron master: OVN Metadata handle process execeptions https://review.opendev.org/c/openstack/neutron/+/890986 | 19:50 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!