| *** marlinc is now known as Guest1409 | 00:13 | |
| opendevreview | Merged openstack/neutron master: Exclude neutron-dynamic-routing from abandon script https://review.opendev.org/c/openstack/neutron/+/865462 | 00:59 |
|---|---|---|
| opendevreview | Merged openstack/neutron master: Amend documentation for fullstack tests https://review.opendev.org/c/openstack/neutron/+/866854 | 00:59 |
| *** marlinc is now known as Guest1415 | 01:51 | |
| opendevreview | liuxie proposed openstack/neutron-fwaas master: Support standard_attrs for fwaas resources https://review.opendev.org/c/openstack/neutron-fwaas/+/866988 | 03:02 |
| opendevreview | liuxie proposed openstack/neutron-fwaas master: Support standard_attrs for fwaas resources https://review.opendev.org/c/openstack/neutron-fwaas/+/866988 | 03:04 |
| *** marlinc is now known as Guest1425 | 04:07 | |
| opendevreview | liuxie proposed openstack/neutron-fwaas master: Support standard_attrs for fwaas resources https://review.opendev.org/c/openstack/neutron-fwaas/+/866988 | 04:50 |
| *** marlinc is now known as Guest1433 | 05:25 | |
| opendevreview | yatin proposed openstack/neutron master: Use same ovs/ovn versions in local test setups https://review.opendev.org/c/openstack/neutron/+/867075 | 05:44 |
| *** marlinc is now known as Guest1440 | 06:27 | |
| opendevreview | yatin proposed openstack/neutron master: Use same ovs/ovn versions in local test setups https://review.opendev.org/c/openstack/neutron/+/867075 | 07:30 |
| *** marlinc is now known as Guest1445 | 07:30 | |
| opendevreview | Lajos Katona proposed openstack/tap-as-a-service master: Fix tox.ini for tox4 https://review.opendev.org/c/openstack/tap-as-a-service/+/866998 | 08:30 |
| opendevreview | yatin proposed openstack/neutron master: Use same ovs/ovn versions in local test setups https://review.opendev.org/c/openstack/neutron/+/867075 | 08:37 |
| *** marlinc is now known as Guest1454 | 08:53 | |
| *** marlinc is now known as Guest1461 | 09:56 | |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Fix homedir permissions https://review.opendev.org/c/openstack/neutron/+/867008 | 10:04 |
| seba | I'm having a problem with the CI on https://review.opendev.org/c/openstack/neutron/+/866489 TIMEOUTS and a few FAILURES, but nothing points to my patch. Is there a known problem with the CI or should I just recheck it another time? | 10:56 |
| lajoskatona | seba: some of the problems are perhaps related to the tox 4 issue, but as this https://review.opendev.org/c/zuul/zuul-jobs/+/866935 is merged that has now a workaround | 11:00 |
| seba | does that require a rebase to apply or can I just do a recheck? | 11:02 |
| frickler | seba: the CI is still broken, please wait with rechecking | 11:02 |
| frickler | also no rebase is needed unless there is a merge conflict | 11:02 |
| seba | okay && noted, tnx | 11:03 |
| frickler | seba: lajoskatona: https://review.opendev.org/c/openstack/devstack/+/867065 is what will need to be merged | 11:03 |
| frickler | slaweq: ^^ but also note the timeouts that looks similar to what we were trying to chase yesterday | 11:03 |
| lajoskatona | frickler: thanks, I missed this patch this morning, keep an eye on it | 11:08 |
| *** marlinc is now known as Guest1472 | 11:11 | |
| slaweq | frickler I thought yesterday that those timeouts are related to my changes in devstack but maybe they're not | 11:12 |
| slaweq | and yes, those looks very similar | 11:13 |
| ykarel | slaweq, frickler looks like those TIMEOUTs started with https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/857031 | 11:49 |
| ykarel | Update nested-virt testing for the 2023.1 cycle | 11:50 |
| slaweq | ykarel it can be like that as what I saw was that all vms basically were booting very slowly | 11:50 |
| ralonsoh | slaweq, did you check the difference in time between both versions? | 11:51 |
| slaweq | ralonsoh what difference You mean? | 11:52 |
| ralonsoh | between before and after this patch | 11:52 |
| slaweq | no, I didn't | 11:52 |
| ralonsoh | in anycase, please report a bug for this | 11:53 |
| slaweq | sure, I will | 11:53 |
| ralonsoh | thanks a lot | 11:53 |
| ykarel | slaweq, you mean it was just slow and then once booted ssh connections were fine? | 11:57 |
| slaweq | ykarel no, it was slow (or something else happened there) so that in console log it wasn't never booted properly | 11:57 |
| slaweq | maybe it wasn't really slow but due to some issue it hanged | 11:58 |
| slaweq | it's even more likely probably | 11:58 |
| ykarel | slaweq, ohkk in above patch i saw some console logs with login prompt but ssh timeout was trying to confirm that | 11:58 |
| ykarel | i meant in https://860bd4f2227d1daddb95-ca0266261d8d95f33f1974de7a62fd54.ssl.cf5.rackcdn.com/866489/5/gate/neutron-tempest-plugin-linuxbridge/8decd6e/tmp74ty0f_b | 11:59 |
| ykarel | but yes in some case console log was without login prompt | 12:00 |
| slaweq | ralonsoh ykarel bug reported https://bugs.launchpad.net/neutron/+bug/1999249 | 12:02 |
| ralonsoh | slaweq, thanks! | 12:02 |
| ykarel | Thanks slaweq | 12:02 |
| ykarel | frickler, on that patch you mentioned: Cirros 0.5.1 seems to have issues booting on Jammy hosts, do you recall what was that issue? may be similar issue in 0.5.2 too? | 12:07 |
| ykarel | iirc 0.5.2 only had 1 extra patch required by some nova-next feature | 12:08 |
| ykarel | https://github.com/cirros-dev/cirros/compare/0.5.1..0.5.2 | 12:11 |
| lajoskatona | ykarel: perhaps better to go back before 0.5.1? | 12:14 |
| ralonsoh | but then we can use Jammy hosts | 12:20 |
| ralonsoh | we can't* | 12:20 |
| lajoskatona | ahh, ok | 12:20 |
| ykarel | lajoskatona, ralonsoh issue seems to be jammy hosts itself as wrt test vms i see SSH issues with advanced images too | 12:30 |
| ralonsoh | ykarel, but in those cases, did you see if the VM booted correctly? | 12:31 |
| ykarel | ralonsoh, for those i see: No correct output in console of server 32b2add2-061a-4748-83b8-d735e0239466 found. Guest operating system status can't be checked. | 12:31 |
| ralonsoh | well, if we have this is because the VM didn't start | 12:32 |
| *** marlinc is now known as Guest1480 | 12:33 | |
| ykarel | it's not always same, for some cases console logs are there | 12:35 |
| frickler | ykarel: I think the issue was related to ahci, but I don't know the details. somehow cirros kernel hanging during vm boot | 12:52 |
| ykarel | frickler, ack | 12:59 |
| *** marlinc is now known as Guest1488 | 13:37 | |
| opendevreview | yatin proposed openstack/neutron master: Use same ovs/ovn versions in local test setups https://review.opendev.org/c/openstack/neutron/+/867075 | 13:38 |
| *** dasm|off is now known as dasm | 14:03 | |
| haleyb | So is anyone else seeing failures in neutron-ovn-tempest-ovs-release-ubuntu-old ? i see a failure to install tempest on devstack setup | 14:13 |
| haleyb | venv: commands[0]> tempest verify-config -uro /tmp/tmp.aGZ4j15UUB | 14:13 |
| haleyb | venv: failed with tempest is not allowed, use allowlist_externals to allow it | 14:13 |
| haleyb | i was going to look into changing tox.ini in tempest but... | 14:14 |
| ralonsoh | haleyb, yes, this is already reported. There is a mail thread about this | 14:18 |
| ralonsoh | there are some devstack patches pushed for master and stable branches | 14:19 |
| ralonsoh | https://review.opendev.org/c/openstack/devstack/+/867065 | 14:19 |
| haleyb | ralonsoh: i guess i hadn't seen that thread, just the one on tox4, this is supposedly pinned since it's a neutron change | 14:19 |
| haleyb | ah | 14:19 |
| ralonsoh | yes, this is the problem | 14:19 |
| ralonsoh | neutron change? | 14:19 |
| haleyb | tox4 is just a mess | 14:19 |
| haleyb | ralonsoh: my change was neutron but failing in devstack, the change you linked seems to fix it | 14:20 |
| haleyb | ralonsoh: since you're around, i had updated your patch yesterday night, https://review.opendev.org/c/openstack/neutron/+/866969 - but if you look at the test patch i created there's still some work to do | 14:22 |
| haleyb | didn't know if we wanted to fix it all in one patch ? | 14:22 |
| ralonsoh | I think so | 14:22 |
| haleyb | ralonsoh: ok, sounds good to me. since you're maybe more caught-up on tox4 this morning, have you seen mention of pylint failures with "ModuleNotFoundError: No module named 'neutron'" ? | 14:27 |
| haleyb | https://zuul.opendev.org/t/openstack/build/9b1474ebf20b49409e37bea3d032311b | 14:27 |
| haleyb | i've been googling tox 3->4 migration but nothing stands out yet | 14:28 |
| ralonsoh | I would wait for the devstack patches first | 14:28 |
| haleyb | ack, it's almost merged | 14:28 |
| *** marlinc is now known as Guest1498 | 15:16 | |
| haleyb | frickler: can i update https://review.opendev.org/c/openstack/python-openstackclient/+/866811 to fix the unit test failures? | 15:39 |
| frickler | haleyb: sure, I didn't get to revisit it yet, feel free to take over | 15:43 |
| haleyb | frickler: thanks, didn't know if your note meant to fix more than the unit tests but figured i'd start there | 15:43 |
| frickler | seems adding the argument to the hash needs also to be made optional | 15:44 |
| frickler | and of course adding a functional test would be nice, as the unit test simply confirmed the broken parameter | 15:45 |
| haleyb | frickler: oh, in the None case, hmm | 15:45 |
| * haleyb was quick to pull the review trigger | 15:45 | |
| frickler | haleyb: yes, similar to the "if parsed_args.tag:" case | 15:45 |
| opendevreview | Merged openstack/neutron master: Fix homedir permissions https://review.opendev.org/c/openstack/neutron/+/867008 | 16:03 |
| lajoskatona | gmann: sure, I started to collect on etherpad and we can discuss one-by-one to do not lose coverage and such | 16:28 |
| *** marlinc is now known as Guest1503 | 16:35 | |
| opendevreview | Merged openstack/neutron stable/zed: Dont raise RouterInterfaceNotFound on overlap check router ports https://review.opendev.org/c/openstack/neutron/+/866835 | 17:53 |
| *** marlinc is now known as Guest1506 | 17:54 | |
| opendevreview | Merged openstack/neutron stable/yoga: Dont raise RouterInterfaceNotFound on overlap check router ports https://review.opendev.org/c/openstack/neutron/+/866836 | 18:02 |
| opendevreview | Merged openstack/neutron stable/xena: Dont raise RouterInterfaceNotFound on overlap check router ports https://review.opendev.org/c/openstack/neutron/+/866837 | 18:02 |
| opendevreview | Merged openstack/neutron stable/wallaby: Dont raise RouterInterfaceNotFound on overlap check router ports https://review.opendev.org/c/openstack/neutron/+/866838 | 18:05 |
| *** marlinc is now known as Guest1511 | 19:00 | |
| *** marlinc is now known as Guest1518 | 20:18 | |
| *** dasm is now known as dasm|off | 21:13 | |
| *** marlinc is now known as Guest1523 | 21:36 | |
| *** marlinc is now known as Guest1528 | 23:00 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!