| opendevreview | Brian Haley proposed openstack/neutron stable/2023.1: OVN: Always try and create a metadata port on subnets https://review.opendev.org/c/openstack/neutron/+/881315 | 00:10 |
|---|---|---|
| opendevreview | Merged openstack/neutron stable/zed: Change flag check order in wait_until_address_ready() https://review.opendev.org/c/openstack/neutron/+/880956 | 01:54 |
| opendevreview | Merged openstack/neutron stable/yoga: Change flag check order in wait_until_address_ready() https://review.opendev.org/c/openstack/neutron/+/880959 | 01:54 |
| opendevreview | Merged openstack/neutron stable/xena: Change flag check order in wait_until_address_ready() https://review.opendev.org/c/openstack/neutron/+/880963 | 01:54 |
| opendevreview | Merged openstack/neutron stable/wallaby: Change flag check order in wait_until_address_ready() https://review.opendev.org/c/openstack/neutron/+/880966 | 01:54 |
| opendevreview | yatin proposed openstack/neutron master: Move to python3.9 as minimal python version https://review.opendev.org/c/openstack/neutron/+/881333 | 06:51 |
| opendevreview | Gregory Thiemonge proposed openstack/ovn-octavia-provider master: Replace python-neutronclient with openstacksdk https://review.opendev.org/c/openstack/ovn-octavia-provider/+/870514 | 07:18 |
| sahid | o/ | 07:40 |
| slaweq | ralonsoh ykarel lajoskatona hi, I'm off today but if You would have some time, please check https://review.opendev.org/c/openstack/neutron/+/879827 | 08:04 |
| slaweq | it should be now ready to review finally :) | 08:04 |
| slaweq | thx in advance | 08:04 |
| lajoskatona | slaweq: ack | 08:06 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: rpc/handlers/dhcp: fix get_network_info when no local subnets https://review.opendev.org/c/openstack/neutron/+/880131 | 08:10 |
| ralonsoh | suer, let me check | 08:10 |
| sahid | guys, regarding this ^, if you have a moment I'm wondering whether we need a rpc change | 08:11 |
| ykarel | slaweq, ack | 08:17 |
| lajoskatona | ykarel, ralonsoh: Hi, rubasov asked on infra about the failures, see the irclogs: https://meetings.opendev.org/irclogs/%23openstack-infra/%23openstack-infra.2023-04-24.log.html | 08:19 |
| ralonsoh | thanks for the info | 08:20 |
| ykarel | lajoskatona, yes https://review.opendev.org/c/openstack/neutron/+/881333 will fix it | 08:20 |
| ykarel | ralonsoh, please check when you get a chance ^ | 08:20 |
| lajoskatona | ykarel: yes, thanks | 08:21 |
| ralonsoh | we are no longer supporting 3.8? | 08:21 |
| ykarel | yes | 08:21 |
| ykarel | 2023.2+ only py3.9+ | 08:21 |
| ralonsoh | Oh my, I didn't read this TC decission | 08:21 |
| frickler | note that this affects more jobs. like neutron-tempest-plugin-linuxbridge | 08:21 |
| ralonsoh | ok, I'll check the current binaries for our projects | 08:22 |
| ykarel | mmm ^ bad then :( as those scenario jobs running on focal due to that jammy issue | 08:22 |
| ralonsoh | right, we can't move to 3.9 on focal | 08:23 |
| ralonsoh | ykarel, but we can manually install it | 08:23 |
| ralonsoh | ykarel, qq, why don't we move to 3.10? I'm talkin about your patch | 08:24 |
| ralonsoh | this is the default binary in 22.04 | 08:25 |
| ykarel | ralonsoh, i just followed governance change that says 3.9+ as some distro defaults to 3.9 | 08:25 |
| ralonsoh | ok | 08:25 |
| ykarel | wrt python3.9 manual install i think we can try that until jammy issue get's resolved | 08:26 |
| ralonsoh | we'll you check that? I can do it | 08:26 |
| ykarel | sure if you can do it it would be great, i can check post lunch | 08:27 |
| ralonsoh | perfect | 08:27 |
| ykarel | and also we can drop the job neutron-ovn-tempest-ovs-release-ubuntu-old | 08:31 |
| ralonsoh | right | 08:32 |
| opendevreview | Merged openstack/neutron master: Move to python3.9 as minimal python version https://review.opendev.org/c/openstack/neutron/+/881333 | 09:00 |
| ralonsoh | @folks: https://review.opendev.org/c/openstack/project-config/+/881340 | 09:00 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Remove "neutron-ovn-tempest-ovs-release-ubuntu-old" job https://review.opendev.org/c/openstack/neutron/+/881342 | 09:02 |
| opendevreview | Lajos Katona proposed openstack/networking-bagpipe master: Move to python3.9 as minimal python version https://review.opendev.org/c/openstack/networking-bagpipe/+/881343 | 09:24 |
| amorin | hey team, about CVE-2023-1668, you said earlier (last week), that neutron OVS Fw was affected, is there any way to reproduce that using regular security groups? | 09:44 |
| opendevreview | Lajos Katona proposed openstack/networking-bgpvpn master: Move to python3.9 as minimal python version https://review.opendev.org/c/openstack/networking-bgpvpn/+/881345 | 09:49 |
| ralonsoh | amorin, the OVS OF rules are affected but is that affecting the OVS FW? | 09:50 |
| amorin | the ovs openflow rules? | 09:51 |
| amorin | we are trying to understand how a security group can lead to create a of rule in ovs that is changing the ip headers | 09:53 |
| amorin | you were saying that maybe you have a way to reproduce the issue in ovs? | 09:54 |
| ralonsoh | just the opposite, I don't know how to reproduc | 09:54 |
| opendevreview | Lajos Katona proposed openstack/networking-odl master: Move to python3.9 as minimal python version https://review.opendev.org/c/openstack/networking-odl/+/881346 | 09:56 |
| opendevreview | Lajos Katona proposed openstack/networking-sfc master: Move to python3.9 as minimal python version https://review.opendev.org/c/openstack/networking-sfc/+/881348 | 10:00 |
| opendevreview | Lajos Katona proposed openstack/neutron-lib master: Move to python3.9 as minimal python version https://review.opendev.org/c/openstack/neutron-lib/+/881350 | 10:01 |
| opendevreview | Lajos Katona proposed openstack/neutron-fwaas master: Move to python3.9 as minimal python version https://review.opendev.org/c/openstack/neutron-fwaas/+/881351 | 10:03 |
| opendevreview | Lajos Katona proposed openstack/neutron-dynamic-routing master: Move to python3.9 as minimal python version https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/881352 | 10:05 |
| amorin | ah | 10:05 |
| opendevreview | Lajos Katona proposed openstack/neutron-vpnaas master: Move to python3.9 as minimal python version https://review.opendev.org/c/openstack/neutron-vpnaas/+/881353 | 10:07 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-tempest-plugin master: Move to python3.9 as minimal python version https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/881354 | 10:10 |
| lajoskatona | ykarel, rodolfo, all: as I see a nice group of changes for this min python version bump: https://review.opendev.org/q/topic:bug%252F2017478 :-) | 10:10 |
| ralonsoh | lajoskatona, you need to change the tempest jobs too | 10:11 |
| ralonsoh | I'm checking the fiirst one and "networking-bagpipe-tempest" needs to bump the python version | 10:12 |
| ralonsoh | --> https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/881354/1/zuul.d/master_jobs.yaml | 10:12 |
| ykarel | ralonsoh, wrt 881354 i think you will need to add task to install python3.9 too | 10:13 |
| ralonsoh | ykarel, there is a devstack method for this | 10:14 |
| ykarel | can use ensure-python role with var python_version: 3.9 | 10:14 |
| ralonsoh | this is why I used PYTHON3_VERSION | 10:14 |
| *** gthiemon1e is now known as gthiemonge | 10:14 | |
| ykarel | okk okk if that works then all is fine | 10:14 |
| ralonsoh | ok, I'll check the zuul output in any case | 10:14 |
| ralonsoh | if needed, I'll use this zuul role | 10:14 |
| ralonsoh | or task | 10:15 |
| ykarel | seems it will work as it is https://opendev.org/openstack/devstack/src/branch/master/inc/python#L462 | 10:17 |
| ralonsoh | perfect! | 10:17 |
| ralonsoh | lajoskatona, may I push updates for your patches? to fix the tempest jobs | 10:31 |
| jrosser | i'm seeing this filling the disks on a Zed deployment https://review.opendev.org/c/openstack/neutron/+/871274 - should the fix be backported? | 10:31 |
| ralonsoh | jrosser, no, just backport the patch | 10:36 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/zed: Honor debug mode in keepalived-state-change script logs https://review.opendev.org/c/openstack/neutron/+/881355 | 10:38 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/xena: Honor debug mode in keepalived-state-change script logs https://review.opendev.org/c/openstack/neutron/+/881356 | 10:38 |
| ralonsoh | jrosser, sorry, you said that, I read something different (to open a new bug) | 10:38 |
| ralonsoh | I was reading other mail at the same time | 10:38 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/wallaby: Honor debug mode in keepalived-state-change script logs https://review.opendev.org/c/openstack/neutron/+/881357 | 10:39 |
| jrosser | ralonsoh: thanks! | 10:49 |
| opendevreview | Lajos Katona proposed openstack/neutron-dynamic-routing master: Move to python3.9 as minimal python version https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/881352 | 11:04 |
| opendevreview | Lajos Katona proposed openstack/networking-bagpipe master: Move to python3.9 as minimal python version https://review.opendev.org/c/openstack/networking-bagpipe/+/881343 | 11:04 |
| lajoskatona | ralonsoh: sure, I just came back to it | 11:07 |
| ralonsoh | frickler, can you check https://review.opendev.org/c/openstack/devstack/+/881363? | 11:22 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-tempest-plugin master: Move to python3.9 as minimal python version https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/881354 | 11:24 |
| opendevreview | Merged openstack/neutron master: Fix dns_integration and ml2 plugin unit tests modules https://review.opendev.org/c/openstack/neutron/+/880855 | 11:24 |
| frickler | ralonsoh: did you test that locally? also, what is blocking the move to jammy? that would seem much more natural to me | 11:26 |
| ralonsoh | frickler, no virt support yet | 11:27 |
| ralonsoh | ykarel, is testing that now | 11:27 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/neutron stable/yoga: Honor debug mode in keepalived-state-change script logs https://review.opendev.org/c/openstack/neutron/+/881322 | 11:28 |
| ralonsoh | ykarel, I pushed a change to use the correct version of python in uwsgi | 11:42 |
| ralonsoh | but the focal version of uwsgi does not support python3.9 | 11:42 |
| ralonsoh | https://b80dd0342184b432be39-b6126bc810e26f5baaa04acfdead70c1.ssl.cf5.rackcdn.com/881354/2/check/neutron-tempest-plugin-openvswitch-enforce-scope-new-defaults/40facfa/controller/logs/screen-keystone.txt | 11:42 |
| ralonsoh | so we are in the middle of nowhere here | 11:42 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: rpc/handlers/dhcp: fix get_network_info when no local subnets https://review.opendev.org/c/openstack/neutron/+/880131 | 11:44 |
| *** iurygregory_ is now known as iurygregory | 11:55 | |
| ykarel | frickler, jammy switch blocked due to https://bugs.launchpad.net/neutron/+bug/1999249 | 11:57 |
| ykarel | few days back got an update from vexxhost folks that they will update remaining nodes in coming weeks to avoid that issue, so strict timeline though | 11:58 |
| ykarel | i am trying alternatives in https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/879036 but don't have better results | 11:59 |
| frickler | ykarel: ah, interesting, there was a similar discussion with octavia folks. my advice there also was to try to get git of nested virt as much as possible | 12:00 |
| frickler | will look at your patch later | 12:00 |
| ykarel | frickler, yes trying that but seems would need to split to multiple jobs to achieve that seeing the results in test patch, as with nested virt jobs working great we had clubbed api jobs too in past with scenario jobs | 12:02 |
| ralonsoh | ykarel, will nova apply this patch? | 12:08 |
| ykarel | ralonsoh, yes it can be accepted if adding suggested change in the patch | 12:12 |
| ralonsoh | perfect, then we'll move to Jammy in the versions you proposed | 12:13 |
| ralonsoh | using 3.9 in focal is not possible | 12:13 |
| ykarel | ralonsoh, your devstack patch didn't helped? | 12:13 |
| ralonsoh | yes and no: uwsgi in focal doesn't support 3.9 | 12:13 |
| ykarel | :( i think i read that somewhere | 12:14 |
| ralonsoh | the uwsgo config is correct but we don't have support | 12:14 |
| ralonsoh | in any case, the devstack patch is legit and could be merged | 12:14 |
| dmitriis | o/ ralonsoh, slaweq: if you have a minute for a PTL approval on https://review.opendev.org/c/openstack/releases/+/881231 - this is for a new lib release with the extra gateway API included. Just trying to get it sorted out for tests of the series itself to pass. I'll raise a change for a constraints bump after it lands as well. | 12:14 |
| ralonsoh | dmitriis, sorry, we released n-lib 10 days ago | 12:15 |
| ralonsoh | you can implement the code in Neutron with a TODO note | 12:15 |
| ralonsoh | I'm not going to approve this new version, is too soon and not needed | 12:16 |
| dmitriis | ralonsoh: ok, I'm going to copy-paste the definitions and exceptions into Neutron with a TODO note for now | 12:18 |
| dmitriis | ralonsoh: 1 more question while we are on the topic (if you have time). Frode asked a question about what to do with this review https://review.opendev.org/c/openstack/neutron/+/877831/comments/da4aad5d_94fb674e?tab=comments (his response is linked). Looks like what you asked about is fixed in the change that follows which already has 2+2 and WF+1. | 12:24 |
| dmitriis | Should we reorder the commits and drop the existing approvals or do you think the first change can be merged as-is? | 12:24 |
| dmitriis | I can do either, just asking for a preference to unblock it. | 12:25 |
| ralonsoh | I'll check it | 12:25 |
| dmitriis | ralonsoh: tyvm | 12:25 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Use a writer context for the online alembic migrations https://review.opendev.org/c/openstack/neutron/+/880867 | 12:33 |
| ralonsoh | lajoskatona, ykarel what should we do about the tempest jobs? options: 1) send a mail saying that we'll fix that in a couple of days or 2) remove them temporarily | 12:36 |
| lajoskatona | ralonsoh: you mean in this patch: https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/881354 ? | 12:57 |
| ralonsoh | lajoskatona, this patch is not working. These two | 12:57 |
| ralonsoh | https://review.opendev.org/c/openstack/nova/+/868419 | 12:57 |
| ralonsoh | https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/879036# | 12:57 |
| ralonsoh | the nova patch is what could fix the nested-virt problem | 12:58 |
| ralonsoh | my patch doesn't work because uwsgi in focal doesn't support 3.9 | 12:58 |
| lajoskatona | ahh, ok ,to move tempest to jammy | 12:59 |
| frickler | ykarel: so iiuc this is all only about the speed (and indirectly memory usage) for test? nothing that actually functionally depends on nested virt? | 13:02 |
| ykarel | frickler, yes right | 13:02 |
| ralonsoh | ykarel, can you join nova channel please? | 13:03 |
| ykarel | ralonsoh, ack | 13:03 |
| frickler | I wonder how much of those tests actually need ubuntu as advanced image or might run with updated cirros. I added a lot of the required tools there, like tcpdump | 13:04 |
| frickler | ah, same question over there | 13:05 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-tempest-plugin master: Move to python3.9 as minimal python version https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/881354 | 13:59 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-tempest-plugin master: Move to python3.9 as minimal python version https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/881354 | 14:10 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Admin procedure for duplicated or deleted OVN agents https://review.opendev.org/c/openstack/neutron/+/881204 | 14:17 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-tempest-plugin master: Move to python3.9 as minimal python version https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/881354 | 14:31 |
| *** dasm is now known as Guest12046 | 15:52 | |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-tempest-plugin master: Move to python3.9 as minimal python version https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/881354 | 16:05 |
| opendevreview | yatin proposed openstack/neutron-tempest-plugin master: [DNM] non nested virt jammy nodes with workaround v2 https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/881391 | 16:14 |
| opendevreview | yatin proposed openstack/neutron-tempest-plugin master: [DNM] non nested virt jammy nodes with workaround v2 https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/881391 | 16:33 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [DNM] check 880586 [2] https://review.opendev.org/c/openstack/neutron/+/881395 | 17:01 |
| *** Guest12046 is now known as dasm | 19:40 | |
| dansmith | anyone from neutron still around/ | 22:43 |
| dansmith | johnsom: maybe? | 22:43 |
| dansmith | okay, well, just FYI this series: https://review.opendev.org/q/topic:bug%252F2017478 | 22:44 |
| dansmith | has wedged the gate for nova, cinder and glance.. since ceph in devstack still requires focal (for reasons) all our ceph jobs still run there, and neutron now fails to install because of the py39 requirement | 22:45 |
| dansmith | I know it's a lot of work, but is it possible we could revert those until we can get something figured out? | 22:45 |
| opendevreview | Miguel Lavalle proposed openstack/neutron master: Add rate-limiting to metadata agents https://review.opendev.org/c/openstack/neutron/+/858879 | 23:51 |
| opendevreview | Miguel Lavalle proposed openstack/neutron master: Add rate-limiting to metadata agents https://review.opendev.org/c/openstack/neutron/+/858879 | 23:51 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!