| opendevreview | yangzhe proposed openstack/neutron master: Cleanup py27 support https://review.opendev.org/c/openstack/neutron/+/891313 | 06:07 |
|---|---|---|
| opendevreview | Dr. Jens Harbott proposed openstack/os-ken master: Fix quagga container https://review.opendev.org/c/openstack/os-ken/+/891658 | 07:21 |
| frickler | haleyb: ^^ this works for me locally, I think it will still fail in CI though, since os-ken isn't used from git in the n-d-r job's tempest env | 07:22 |
| frickler | ah, forget to link the bug | 07:30 |
| opendevreview | Lajos Katona proposed openstack/neutron-dynamic-routing master: DNM: Test patch only https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/890806 | 07:34 |
| lajoskatona | frickler: don't know if it works, but I added your patch as depends-on to https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/890806, let's see if tempest will be happy | 07:35 |
| frickler | lajoskatona: the job is already running for os-ken. but it will likely fail since the wrong version is getting used in tempest | 07:46 |
| lajoskatona | frickler: ok, thanks | 07:50 |
| opendevreview | Lucas Alvares Gomes proposed openstack/neutron master: [OVN] Disable the mcast_flood_reports option for LSPs https://review.opendev.org/c/openstack/neutron/+/888127 | 08:36 |
| frickler | lajoskatona: you can see ubuntu:16.04 still being used in https://a270c540ac71407d2085-c0b6ca9dff9e9ad2315a3f9688c1a88d.ssl.cf2.rackcdn.com/891658/1/check/neutron-tempest-plugin-dynamic-routing/05878b4/controller/logs/tempest_log.txt | 08:44 |
| frickler | not sure how to fix that, I'd propose to make the job n-v in os-ken for now and then make a new release with the fix | 08:45 |
| lajoskatona | frickler: haleyb's patch do that: https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/891550, | 08:50 |
| lajoskatona | frickler: ohhh, but just for check queue.... | 08:50 |
| frickler | lajoskatona: also need to do the same for os-ken to merge the fix there | 08:50 |
| opendevreview | Lajos Katona 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 | 08:51 |
| frickler | I'll wrap my os-ken fix in a similar patch | 08:51 |
| frickler | but even the neutron install doesn't use os-ken from git, you can see 2.6.0 being installed everywhere here https://a270c540ac71407d2085-c0b6ca9dff9e9ad2315a3f9688c1a88d.ssl.cf2.rackcdn.com/891658/1/check/neutron-tempest-plugin-dynamic-routing/05878b4/controller/logs/devstacklog.txt | 08:52 |
| frickler | so the deployment testing on os-ken is completely bogus | 08:52 |
| *** dasm is now known as Guest9148 | 09:01 | |
| opendevreview | Dr. Jens Harbott proposed openstack/os-ken master: Fix quagga container https://review.opendev.org/c/openstack/os-ken/+/891658 | 09:10 |
| opendevreview | Dr. Jens Harbott proposed openstack/os-ken master: Make neutron-dynamic-routing job non-voting https://review.opendev.org/c/openstack/os-ken/+/891665 | 09:10 |
| frickler | ralonsoh: lajoskatona: haleyb: ^^ I would propose to merge this and make a new os-ken release. then we have time for someone to make the neutron jobs actually use os-ken from git | 09:12 |
| frickler | also moving from quagga to frr might be nice, but that will need some further code reworking | 09:13 |
| frickler | tobias-urdin: ^^ fyi | 09:13 |
| tobias-urdin | frickler: ack | 09:24 |
| tobias-urdin | agree that should probably be rebuilt to use frr image from quay.io for example | 09:31 |
| ralonsoh | frickler, approved. qq about https://review.opendev.org/c/openstack/os-ken/+/891658/2. Changing the default baseimage='ubuntu:20.04' won't affect other users? | 09:41 |
| ralonsoh | Although I'm not aware of this | 09:41 |
| ralonsoh | I mean, other users using this class | 09:41 |
| frickler | ralonsoh: I don't know about any other users. the only testing we seem to have for this is in n-d-r | 09:42 |
| frickler | ralonsoh: we can (and likely should) add a reno | 09:42 |
| ralonsoh | yeah, that's my point | 09:42 |
| ralonsoh | yes, a reno will be helpful | 09:42 |
| frickler | ralonsoh: are you o.k. if I do that in a fup? avoids rerunning the n-d-r job (or at least I hope so) | 09:44 |
| ralonsoh | ok | 09:44 |
| ralonsoh | once we have this new patch, I'll propose a new os-ken release asap | 09:46 |
| opendevreview | Merged openstack/neutron stable/victoria: Fix ACL sync when default sg group is created https://review.opendev.org/c/openstack/neutron/+/890118 | 09:51 |
| opendevreview | Dr. Jens Harbott proposed openstack/os-ken master: Add releasenote for quagga fix https://review.opendev.org/c/openstack/os-ken/+/891673 | 09:56 |
| opendevreview | Dr. Jens Harbott proposed openstack/os-ken master: Cleanup py27 support https://review.opendev.org/c/openstack/os-ken/+/890735 | 09:58 |
| opendevreview | Dr. Jens Harbott proposed openstack/os-ken master: Remove superfluous sudo https://review.opendev.org/c/openstack/os-ken/+/891595 | 09:58 |
| frickler | rebased those other two patches so that we can get them all merged before doing a release | 09:59 |
| frickler | actually I should do another one to add py310 support to setup.cfg | 10:00 |
| opendevreview | Dr. Jens Harbott proposed openstack/os-ken master: Add Python 3.10 support to setup.cfg https://review.opendev.org/c/openstack/os-ken/+/891675 | 10:04 |
| opendevreview | Tobias Urdin proposed openstack/os-ken master: Use FRR instead of Quagga when testing https://review.opendev.org/c/openstack/os-ken/+/891677 | 10:07 |
| * tobias-urdin *holds thumbs* | 10:08 | |
| frickler | tobias-urdin: as I wrote earlier, that change won't really be tested in CI, as os-ken isn't installed from git, do you want to look into that, too? otherwise I'd try to test locally, but likely not this week | 10:15 |
| tobias-urdin | hm ok that's little bit annoying, I can test it locally but not sure how to just run that integrated test stuff | 10:16 |
| tobias-urdin | or perhaps better to look at running from git | 10:16 |
| ralonsoh | we can release the fix for quagga for now | 10:27 |
| ralonsoh | once release, we can try using FRR and release a new os-ken version | 10:28 |
| ralonsoh | once released* | 10:28 |
| opendevreview | Merged openstack/neutron stable/ussuri: Fix ACL sync when default sg group is created https://review.opendev.org/c/openstack/neutron/+/888651 | 10:32 |
| stephenfin | ralonsoh: just fyi, still seeing a sqlalchemy 2.x error with neutron. Not sure why it's not being seen locally in your tips job https://zuul.opendev.org/t/openstack/build/8cc136d3ff004125a2ef2ba425bce969 | 10:53 |
| stephenfin | (patch is https://review.opendev.org/c/openstack/requirements/+/879743) | 10:54 |
| ralonsoh | stephenfin, this test is part of a very recent added test that I don't think is being correctly executed | 10:55 |
| ralonsoh | stephenfin, https://review.opendev.org/c/openstack/neutron/+/887257 | 10:55 |
| ralonsoh | stephenfin, https://review.opendev.org/c/openstack/neutron/+/887257/4/neutron/db/agents_db.py#503 | 10:55 |
| ralonsoh | I don't think this is correct, right? | 10:55 |
| stephenfin | that's what looks to be incorrect, yes | 10:56 |
| ralonsoh | should we use | 10:56 |
| stephenfin | that should be wrapped in sqlalchemy.text iirc | 10:56 |
| ralonsoh | connection.scalar(select([1])) | 10:56 |
| ralonsoh | ? | 10:56 |
| stephenfin | or that | 10:56 |
| stephenfin | that would be cleaner | 10:56 |
| ralonsoh | perfect, I'll push a patch today | 10:56 |
| stephenfin | ah, openstack-tox-py310-with-sqlalchemy-master is periodic | 10:57 |
| stephenfin | which is why you didn't see that | 10:57 |
| stephenfin | would it be worth making it non-periodic for any changes to neutron.db | 10:57 |
| stephenfin | ? | 10:57 |
| stephenfin | *making it gating | 10:57 |
| ralonsoh | hmmm I don't think so but I'll ask this in the next neutron meeting | 10:58 |
| stephenfin | (y) | 10:59 |
| ralonsoh | added to https://wiki.openstack.org/wiki/Network/Meetings#On_Demand_Agenda | 10:59 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [DNM] Use SQLAlchemy expression "select" https://review.opendev.org/c/openstack/neutron/+/891691 | 11:06 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Use SQLAlchemy expression "select" https://review.opendev.org/c/openstack/neutron/+/891691 | 11:44 |
| ralonsoh | lajoskatona, slaweq ^ please check this to unblock the sqlalchemy2.0 requirements patch | 11:45 |
| lajoskatona | ralonsoh: watching/following it | 11:58 |
| ralonsoh | lajoskatona, btw, the CI won't check that (at least with UT and SQL2.0) | 11:58 |
| ralonsoh | but I've tested this manually | 11:58 |
| ralonsoh | I'll propose a UT CI with SQL 2.0 | 11:58 |
| ralonsoh | that is missing | 11:58 |
| lajoskatona | ralonsoh: ahh, ok | 11:59 |
| frickler | lajoskatona: do you also want to review to os-ken stack? or who would be better to ping about it? | 12:03 |
| frickler | s/to/the/ | 12:03 |
| frickler | also agree with ralonsoh that moving to frr would be worth a dedicated release, likely a major one, even if it is "only" testing | 12:04 |
| frickler | so maybe a bit tight with upcoming lib freeze, too | 12:05 |
| ralonsoh | I wouldn't make this change right now | 12:05 |
| ralonsoh | only the fix, as frickler said | 12:05 |
| ralonsoh | (before the bobcat release) | 12:05 |
| lajoskatona | ralonsoh, frickler: agree, fix it for the release now, and go for FRR when we have no release pressure | 12:13 |
| opendevreview | Merged openstack/os-ken master: Make neutron-dynamic-routing job non-voting https://review.opendev.org/c/openstack/os-ken/+/891665 | 12:33 |
| *** EugenMayer44 is now known as EugenMayer4 | 13:04 | |
| opendevreview | Merged openstack/os-ken master: Fix quagga container https://review.opendev.org/c/openstack/os-ken/+/891658 | 13:34 |
| opendevreview | Merged openstack/os-ken master: Add releasenote for quagga fix https://review.opendev.org/c/openstack/os-ken/+/891673 | 13:34 |
| opendevreview | Merged openstack/os-ken master: Add Python 3.10 support to setup.cfg https://review.opendev.org/c/openstack/os-ken/+/891675 | 13:34 |
| gthiemonge | Hey Folks, did you have a chance to look at https://bugs.launchpad.net/neutron/+bug/2028651 ? it causes a regression in Octavia with IPv6 load balancers. | 13:35 |
| slaweq | ralonsoh lajoskatona mlavalle haleyb hi, can You check https://review.opendev.org/c/openstack/neutron/+/889871 when You will have a minute, thx in advance | 13:38 |
| ralonsoh | sure | 13:38 |
| mlavalle | ralonsoh, slaweq, lajoskatona, haleyb, obondarev_: can I get some eyes on this patch, when you have time: https://review.opendev.org/c/openstack/python-openstackclient/+/891557. it should be an easy one | 13:39 |
| ralonsoh | gthiemonge, I'll check it today | 13:39 |
| gthiemonge | ralonsoh: thanks | 13:40 |
| lajoskatona | slaweq, mlavalle: checking | 13:43 |
| opendevreview | Elod Illes proposed openstack/os-vif master: DNM: dummy change to test gate health https://review.opendev.org/c/openstack/os-vif/+/891812 | 15:03 |
| opendevreview | Merged openstack/neutron master: Force DB migration script to be run before some fullstack tests https://review.opendev.org/c/openstack/neutron/+/891040 | 15:06 |
| opendevreview | Elod Illes proposed openstack/python-neutronclient master: DNM: dummy change to test gate health https://review.opendev.org/c/openstack/python-neutronclient/+/891830 | 15:09 |
| *** Guest9148 is now known as dasm | 15:11 | |
| mlavalle | obondarev_, haleyb: thanks for the comments. Responded to them :-) | 15:12 |
| opendevreview | Miro Tomaska proposed openstack/neutron master: Fix ovn-metadata agent sync of unused namespaces https://review.opendev.org/c/openstack/neutron/+/891232 | 15:16 |
| opendevreview | Brian Haley proposed openstack/neutron master: Update OVN db sync code log messages https://review.opendev.org/c/openstack/neutron/+/890832 | 15:22 |
| frickler | ralonsoh: what about https://review.opendev.org/c/openstack/os-ken/+/890735 ? IMO that's also a good cleanup and together with haleyb's patch we would then be ready for a release? | 15:31 |
| frickler | also, would it make sense to add n-d-r core to os-ken core? | 15:32 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: Set GLOBAL_VENV=false to fix broken gate jobs https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/891550 | 18:19 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: Update Cirros image to 0.6.2 https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/890177 | 20:08 |
| opendevreview | Merged openstack/neutron stable/wallaby: Adding placement auth options to oslo.config entry_points https://review.opendev.org/c/openstack/neutron/+/887523 | 21:17 |
| opendevreview | Merged openstack/neutron stable/2023.1: hash-ring: Retry all DB operations if inactive https://review.opendev.org/c/openstack/neutron/+/891273 | 21:49 |
| opendevreview | Merged openstack/neutron stable/zed: hash-ring: Retry all DB operations if inactive https://review.opendev.org/c/openstack/neutron/+/891274 | 21:49 |
| opendevreview | Merged openstack/neutron stable/yoga: hash-ring: Retry all DB operations if inactive https://review.opendev.org/c/openstack/neutron/+/891275 | 21:49 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!