| ykarel | Thanks for confirming and fixing the unstack issue | 04:34 |
|---|---|---|
| ykarel | and hopefully you also didn't reproduced your original issue on fresh vm and stack.sh, right? | 04:34 |
| opendevreview | yatin proposed openstack/neutron master: [DNM] Check migration ovn https://review.opendev.org/c/openstack/neutron/+/972820 | 05:10 |
| ralonsoh | hello folks, please check https://review.opendev.org/c/openstack/neutron/+/972669 | 08:22 |
| ralonsoh | this is for 2025.1 only | 08:22 |
| ralonsoh | it is tested in https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/972266 | 08:22 |
| ralonsoh | slaweq, lajoskatona ^^ thanks!! | 08:22 |
| daidv_1 | haleyb, lajoskatona Happy new year ^^ I updated patch for DNS forwarder extendsion, hope you guys can spent sometime to review it for me, thanks alot: https://review.opendev.org/c/openstack/neutron/+/951390 | 08:28 |
| lajoskatona | daidv_1, ralonsoh: Hi, on my list | 08:30 |
| daidv_1 | thank you in advance :D | 08:31 |
| opendevreview | Brian Haley proposed openstack/networking-sfc master: Change the vxlan vni_range to be 1:1000 https://review.opendev.org/c/openstack/networking-sfc/+/967783 | 08:58 |
| *** srelf_ is now known as ContinuitySR | 09:25 | |
| *** ContinuitySR is now known as Continuity | 09:25 | |
| opendevreview | Eduardo Olivares proposed x/whitebox-neutron-tempest-plugin master: [multicast] Only use vlan_transparent option when really required https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/972899 | 10:08 |
| opendevreview | Elvira García Ruiz proposed openstack/tap-as-a-service master: Add neutron-tempest-plugin ovn tests to zuul https://review.opendev.org/c/openstack/tap-as-a-service/+/973057 | 10:19 |
| opendevreview | Brian Haley proposed openstack/networking-bgpvpn master: Remove py312 jobs from check and gate queue https://review.opendev.org/c/openstack/networking-bgpvpn/+/972786 | 10:33 |
| *** dmellado4 is now known as dmellado | 11:41 | |
| opendevreview | James Bhattarai proposed openstack/neutron master: Docs Changes in CLI FQDN of Neutron BGP Speaker Create https://review.opendev.org/c/openstack/neutron/+/973073 | 12:04 |
| opendevreview | Lajos Katona proposed openstack/neutron-fwaas master: CI: Add required_projects to unit test jobs https://review.opendev.org/c/openstack/neutron-fwaas/+/973079 | 12:33 |
| opendevreview | Eduardo Olivares proposed x/whitebox-neutron-tempest-plugin master: Remove try/except from VlanTransparencyWhiteboxBaseTest resource_setup https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/973080 | 12:34 |
| opendevreview | Merged openstack/networking-sfc master: Change the vxlan vni_range to be 1:1000 https://review.opendev.org/c/openstack/networking-sfc/+/967783 | 12:55 |
| jlibosva | ralonsoh: hi, I tried to play a bit with the functional tests setup/teardown resources and if we do not build a new DB for every test but only per test worker - the speedup is in the order of magnitude - from 2 hours down to 18 minutes. But there are still some issues there. I think there is another bug in the OVN base class. It's only OVN tests that have problems with the | 13:33 |
| jlibosva | DB | 13:33 |
| jlibosva | ralonsoh: speaking about https://review.opendev.org/c/openstack/neutron/+/972985 | 13:33 |
| ralonsoh | jlibosva, let me check (let me finish one quick thing) | 13:42 |
| opendevreview | Eduardo Olivares proposed x/whitebox-neutron-tempest-plugin master: [multicast] Only use vlan_transparent option when really required https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/972899 | 14:21 |
| opendevreview | Eduardo Olivares proposed x/whitebox-neutron-tempest-plugin master: Remove try/except from VlanTransparencyWhiteboxBaseTest resource_setup https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/973080 | 14:21 |
| ralonsoh | jlibosva, I don't think this is strictly related to the OVN tests | 14:26 |
| ralonsoh | some workers had initially problems with the DB: | 14:27 |
| ralonsoh | sqlalchemy.exc.OperationalError: (pymysql.err.OperationalError) (1050, "Table 'agents' already exists") | 14:27 |
| ralonsoh | then the following tests using MySQL failed too | 14:27 |
| jlibosva | the DB migrations tests make sense since they assume empty DB | 14:28 |
| ralonsoh | jlibosva, so you can maybe check without `test_walk_versions`, `test_db_upgrade_is_idempotent` and `TestModelsMigrations` | 14:29 |
| ralonsoh | jlibosva, what about Terry's suggestion about creating the DDL files and load them at the beginning of every MySQL test? | 14:30 |
| jlibosva | ralonsoh:here https://review.opendev.org/c/openstack/neutron/+/972961 | 14:30 |
| ralonsoh | cool! let me check it | 14:31 |
| jlibosva | ralonsoh: didn't help much though in terms of overall execution time | 14:31 |
| jlibosva | ralonsoh: that's why I started digging deeper and came up with not creating the schema per every test but reuse since even the DDL takes some time | 14:31 |
| ralonsoh | so still 2 hours | 14:31 |
| jlibosva | ralonsoh: I measured it manually and the DDL was significantly faster than create_all() | 14:32 |
| jlibosva | ralonsoh: https://bugs.launchpad.net/neutron/+bug/2136827/comments/4 | 14:32 |
| ralonsoh | jlibosva, but if https://review.opendev.org/c/openstack/neutron/+/972961 is using the DDL files, why does it take 2 hours? | 14:34 |
| jlibosva | ralonsoh: I don't have a good answer to that yet | 14:37 |
| jlibosva | but | 14:37 |
| jlibosva | Ran: 1147 tests in 1651.9493 sec. <- this is a significant improvement | 14:37 |
| jlibosva | ah no, it's not | 14:37 |
| ralonsoh | let me execute your patch locally | 14:38 |
| ykarel | #startmeeting neutron_ci | 15:01 |
| opendevmeet | Meeting started Mon Jan 12 15:01:14 2026 UTC and is due to finish in 60 minutes. The chair is ykarel. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:01 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:01 |
| opendevmeet | The meeting name has been set to 'neutron_ci' | 15:01 |
| ykarel | Ping list: bcafarel, lajoskatona, slawek, mlavalle, mtomaska, ralonsoh, ykarel, jlibosva, elvira | 15:01 |
| ykarel | Hi everyone, it's on IRC today | 15:01 |
| ralonsoh | hello | 15:01 |
| mtomaska | o/ | 15:01 |
| elvira | hi! | 15:01 |
| lajoskatona | o/ | 15:02 |
| slaweq | o/ | 15:02 |
| haleyb | o/ | 15:03 |
| bcafarel | o/ | 15:03 |
| ykarel | let's start with the topics | 15:03 |
| ykarel | lajoskatona to check vpnaas and bgpvpn UT failures | 15:03 |
| ykarel | #link https://bugs.launchpad.net/neutron/+bug/2137489 | 15:03 |
| ykarel | i see both green now, thx lajoskatona | 15:04 |
| ykarel | ralonsoh to check sfc failures | 15:04 |
| ralonsoh | fixed and merged | 15:04 |
| ykarel | #link https://bugs.launchpad.net/neutron/+bug/2137633 | 15:04 |
| ykarel | thx ralonsoh | 15:04 |
| ykarel | ralonsoh to check https://bugs.launchpad.net/neutron/+bug/2137467 this with jakub | 15:04 |
| opendevreview | Lajos Katona proposed openstack/neutron-fwaas master: CI: Add required_projects to unit test jobs https://review.opendev.org/c/openstack/neutron-fwaas/+/973079 | 15:04 |
| ralonsoh | still in progress | 15:05 |
| ralonsoh | but the issue has been identified | 15:05 |
| ykarel | thx jlibosva ralonsoh | 15:05 |
| ykarel | #link slawek to check fwaas functional job failure | 15:05 |
| ykarel | #link https://bugs.launchpad.net/neutron/+bug/2137480 | 15:05 |
| ykarel | this too fixed, but have a new issue but can see next | 15:06 |
| ykarel | thx slaweq | 15:06 |
| ykarel | #topic Stable branches | 15:06 |
| ykarel | all good as per periodic and not much activity with reviews | 15:06 |
| ykarel | bcafarel, anything else to add? | 15:06 |
| bcafarel | yes, it was mostly quiet (and OK on the few ones that went in) | 15:06 |
| ykarel | s/reviews/backports | 15:06 |
| ykarel | ok thx | 15:07 |
| ykarel | #topic Stadium projects | 15:07 |
| ykarel | dynamic-routing still broken https://zuul.openstack.org/buildset/15ea550c13584730a51dc3576568702a | 15:07 |
| lajoskatona | quite good compared to last weeks | 15:07 |
| ykarel | Known one https://bugs.launchpad.net/neutron/+bug/2130631 | 15:07 |
| lajoskatona | yes that one is is red | 15:07 |
| lajoskatona | for fwaas I think it is only the required-projects missing, see my patch: https://review.opendev.org/c/openstack/neutron-fwaas/+/973079 | 15:07 |
| ykarel | yes better this week with all those fixes merged | 15:07 |
| ykarel | neutron-fwaas tox jobs https://zuul.openstack.org/buildset/068ba76c76da4d9185ac8635a80154a4 | 15:07 |
| ykarel | broken with https://review.opendev.org/c/openstack/neutron-fwaas/+/972195 | 15:07 |
| lajoskatona | I hope now I have the yaml correct | 15:07 |
| ykarel | either need neutron release or job to use unreleased neutron | 15:08 |
| ralonsoh | we have a patch to release beta 28.0.0.b1 | 15:08 |
| ralonsoh | https://review.opendev.org/c/openstack/releases/+/972457 | 15:08 |
| lajoskatona | yeah, that can be ok also, or my patch with required-projects | 15:08 |
| ralonsoh | ^^ this one has the project_id changes and should be used | 15:08 |
| ykarel | yes some issue there | 15:08 |
| lajoskatona | ralonsoh: thanks | 15:08 |
| ykarel | but i couldn't get how the job was working on check but not on periodic | 15:09 |
| ykarel | in periodic neutron was not included as required-projects while in check it does but couldn't spot where thats included | 15:09 |
| lajoskatona | I didn't check but my theory is that the template works differently | 15:09 |
| ralonsoh | ykarel, that should be the reason, if included in required-projects, should use master by defaultr | 15:10 |
| lajoskatona | I mean using openstack-python3-jobs-neutron in template list, or just listing jobs like openstack-tox-py312-with-oslo-master | 15:10 |
| ykarel | ralonsoh, but i couldn't find required-projects: neutron at all in check jobs running against neutron-fwaas | 15:11 |
| ralonsoh | that's weird, I'll check it later | 15:11 |
| ralonsoh | in any case, we should bump the tag of neutron in the stadium projects | 15:11 |
| ykarel | +1 | 15:12 |
| ykarel | ok let's move | 15:12 |
| ykarel | #topic Rechecks | 15:12 |
| ykarel | not much activity, and recheck wise not bad | 15:12 |
| ykarel | we had zuul outage last week and that resulted in couple of rechecks along with some intermittent known issues | 15:12 |
| ykarel | bare recheck wise 1/28 rechecks, so that good too | 15:13 |
| ykarel | #topic fullstack/functional | 15:13 |
| ykarel | for this we seeing timeouts, but i see ralonsoh jlibosva looking into optimizing the tests | 15:13 |
| ykarel | so hopefully that will help | 15:14 |
| ykarel | #topic Periodic | 15:15 |
| ykarel | broken https://zuul.openstack.org/builds?job_name=propose-translation-update&project=openstack/neutron | 15:15 |
| ykarel | Recent zuul upgrade/secret rotate broke this, https://review.opendev.org/c/openstack/project-config/+/973029 should probably fix, tomorrows run will confirm it | 15:15 |
| ykarel | #topic Grafana | 15:15 |
| ykarel | https://grafana.opendev.org/d/f913631585/neutron-failure-rate | 15:15 |
| ykarel | let's have quick look here too | 15:16 |
| ykarel | the spikes on 8/9 are related to the zuul/secret issue | 15:18 |
| ykarel | which were unblocked by https://review.opendev.org/c/openstack/project-config/+/972828 | 15:18 |
| ykarel | apart from that looks fine, some other spikes likely patch specific | 15:19 |
| ykarel | anyting to add? | 15:19 |
| ralonsoh | no thanks | 15:19 |
| lajoskatona | nothing from me | 15:19 |
| mtomaska | . | 15:19 |
| bcafarel | all good :) | 15:19 |
| ykarel | #topic On Demand | 15:20 |
| ykarel | anything else you would like to raise? | 15:20 |
| ralonsoh | nope | 15:20 |
| haleyb | i had a qustion about a patch | 15:20 |
| haleyb | #link https://review.opendev.org/c/openstack/neutron/+/972734 | 15:20 |
| haleyb | i didn't quite understand your comment | 15:20 |
| ykarel | sure looking | 15:20 |
| haleyb | or feel free to look after meeting, the series was all blocked with the zuul issues last week | 15:21 |
| ykarel | haleyb, for periodic jobs irrelevant-files are irrelevant | 15:21 |
| ykarel | ie. jobs always runs | 15:22 |
| haleyb | and for experimental? | 15:22 |
| ykarel | for experimental they are relevant, but considering experimental as on-demand so we should be fine even if irrelevant-files not set | 15:23 |
| ykarel | but can continue over patch itself | 15:23 |
| ykarel | thx everyone for joining | 15:24 |
| haleyb | sure | 15:24 |
| ykarel | let's close early and have everyone almost 35 minutes back | 15:24 |
| ykarel | #endmeeting | 15:24 |
| opendevmeet | Meeting ended Mon Jan 12 15:24:25 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:24 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/neutron_ci/2026/neutron_ci.2026-01-12-15.01.html | 15:24 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/neutron_ci/2026/neutron_ci.2026-01-12-15.01.txt | 15:24 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/neutron_ci/2026/neutron_ci.2026-01-12-15.01.log.html | 15:24 |
| mtomaska | o/ | 15:24 |
| ralonsoh | bye! | 15:24 |
| lajoskatona | o/ | 15:24 |
| elvira | I realized tap as a service did not have it's ovn tests added on the .zuul yaml so I uploaded a tiny patch fixing that in case anyone wants to check :) https://review.opendev.org/c/openstack/tap-as-a-service/+/973057 | 15:24 |
| ralonsoh | +1 | 15:25 |
| slaweq | o/ | 15:25 |
| slaweq | thx for the patch elvira | 15:25 |
| elvira | np! | 15:25 |
| ralonsoh | elvira, btw, if you want, you can also remove the wsgi job | 15:26 |
| slaweq | I added comment there | 15:26 |
| ralonsoh | now we only have wsgi job, so one of them is duplicated (neutron-tempest-plugin-tap-as-a-service, neutron-tempest-plugin-tap-as-a-service-wsgi) | 15:26 |
| slaweq | maybe we should add that job to gate queue also, wdyt? | 15:26 |
| ykarel | +1 | 15:27 |
| ykarel | to add to gate | 15:27 |
| elvira | oh sure, I will propose a new patchset then | 15:28 |
| elvira | Although I just saw gate jobs are starting because of the +2 | 15:28 |
| ykarel | i removed that already after reading slawek comment | 15:28 |
| opendevreview | Lajos Katona proposed openstack/neutron-fwaas master: CI: Add required_projects to unit test jobs https://review.opendev.org/c/openstack/neutron-fwaas/+/973079 | 15:28 |
| ralonsoh | elvira, don't worry, push a new one | 15:29 |
| elvira | oh ok awesome, I will make the change right now | 15:29 |
| elvira | thanks everyone!! | 15:29 |
| ralonsoh | pffff and we also have p312 jobs still running there with master branch (openstack-tox-py312-with-oslo-master) | 15:29 |
| ralonsoh | haleyb, ^ we would need a document not to loose any project when migrating the jobs | 15:30 |
| haleyb | i have a patch for neutron at least for that, maybe others i;ll have to look | 15:30 |
| ralonsoh | for example, when changing the python versions | 15:30 |
| haleyb | ralonsoh: yes, agreed | 15:30 |
| ralonsoh | I'll open a LP, just to have a reference | 15:30 |
| jlibosva | ralonsoh: I forgot how the functional tests are supposed to work :) What is the openstack_citest database for? | 15:31 |
| ralonsoh | jlibosva, just for FTs, it is created only for that | 15:31 |
| ralonsoh | jlibosva, if you have a devstack env, you can change the mysql user and database | 15:32 |
| jlibosva | ralonsoh: right but then we create a new database per each test | 15:32 |
| ralonsoh | yes, that's also the user | 15:32 |
| jlibosva | not user, database | 15:32 |
| ralonsoh | ahh, just for checking | 15:32 |
| ralonsoh | if mysql openstack_citest > /dev/null 2>&1 < /dev/null; then | 15:33 |
| ralonsoh | echo_summary "DB config appears to be complete, skipping." | 15:33 |
| ralonsoh | return 0 | 15:33 |
| ralonsoh | fi | 15:33 |
| jlibosva | so the DB itself is not used at alL? | 15:34 |
| ralonsoh | nope | 15:34 |
| jlibosva | and it's been always like that or it's changed? | 15:34 |
| opendevreview | Elvira García Ruiz proposed openstack/tap-as-a-service master: Update zuul jobs and gates definitions https://review.opendev.org/c/openstack/tap-as-a-service/+/973057 | 15:34 |
| ralonsoh | jlibosva, I think that was the only goal of this DB | 15:35 |
| ralonsoh | as you mentioned, each test creates its own DB | 15:35 |
| ralonsoh | jlibosva, the point is that the loop https://review.opendev.org/c/openstack/neutron/+/972961/1/neutron/tests/unit/testlib_api.py#169 is taking around 8 seconds in my testing VM | 15:46 |
| ralonsoh | same as the `create_all` call | 15:46 |
| ralonsoh | (at least in my env) | 15:46 |
| jlibosva | ralonsoh: yeah, that's why I thought the DDL file might be a dead end | 15:46 |
| ralonsoh | jlibosva, I've tested also using shell commands (mysqldump, mysql). The time is similar. The issue is the engine loading the DDL statements, regardless of the method (mysql cli, sqlalchemy) | 16:16 |
| ralonsoh | jlibosva, I need to leave now but the fastest alternative seems to backup directly the file | 16:16 |
| ralonsoh | that would be the fastest way | 16:16 |
| opendevreview | Michel Nederlof proposed openstack/neutron master: Keep virtual ports unbound when binding https://review.opendev.org/c/openstack/neutron/+/973115 | 16:24 |
| opendevreview | Merged x/whitebox-neutron-tempest-plugin master: Configurable setup hosts information https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/972033 | 16:56 |
| opendevreview | Merged openstack/neutron master: Add ``ModuleNotFoundError`` to ``get_api_worker_id`` https://review.opendev.org/c/openstack/neutron/+/972687 | 17:18 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: func: Do not teardown DB if not dirty https://review.opendev.org/c/openstack/neutron/+/972985 | 17:43 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: DNM: Trigger functional tests https://review.opendev.org/c/openstack/neutron/+/972986 | 17:43 |
| opendevreview | sean mooney proposed openstack/os-vif master: Add TAP device pre-creation support for OVS/OVN https://review.opendev.org/c/openstack/os-vif/+/971231 | 18:06 |
| sean-k-mooney | ralonsoh: https://review.opendev.org/c/openstack/nova/+/973149 works for booting a vm but i still need to test live migration with this chagne | 18:31 |
| sean-k-mooney | ill add a secodn patch that depens on the neutron change and updates one of the jobs tomorrow after is add a second compute and test this locally. | 18:32 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: func: Do not teardown DB if not dirty https://review.opendev.org/c/openstack/neutron/+/972985 | 18:40 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: DNM: Trigger functional tests https://review.opendev.org/c/openstack/neutron/+/972986 | 18:40 |
| opendevreview | Merged openstack/neutron master: bgp: Introduce BGP chassis commands https://review.opendev.org/c/openstack/neutron/+/960422 | 19:13 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: func: Do not teardown DB if not dirty https://review.opendev.org/c/openstack/neutron/+/972985 | 19:50 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: DNM: Trigger functional tests https://review.opendev.org/c/openstack/neutron/+/972986 | 19:50 |
| opendevreview | Brian Haley proposed openstack/neutron master: Update jobs to use the periodic zuul job templates https://review.opendev.org/c/openstack/neutron/+/972734 | 21:30 |
| opendevreview | Brian Haley proposed openstack/neutron master: Update jobs based on testing runtime for 2026.1 https://review.opendev.org/c/openstack/neutron/+/972734 | 21:38 |
| *** jlibosva is now known as Guest35716 | 21:50 | |
| opendevreview | Merged openstack/neutron stable/2025.1: [stable-only 2025.1] Initialize segment_network_range in non-WSGI server https://review.opendev.org/c/openstack/neutron/+/972669 | 22:45 |
| opendevreview | Brian Haley proposed openstack/neutron master: Use project_id key in port plugin code https://review.opendev.org/c/openstack/neutron/+/972982 | 23:25 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!