| opendevreview | Merged openstack/neutron master: OVN TestNBDbResources wait for NB_Global table to be present https://review.opendev.org/c/openstack/neutron/+/825530 | 04:00 |
|---|---|---|
| opendevreview | Lajos Katona proposed openstack/neutron master: Remove OS_TEST_TIMEOUT=600 from dsvm-fullstack https://review.opendev.org/c/openstack/neutron/+/827488 | 07:35 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/xena: Make configure_for_func_testing compatible with e.g. Centos https://review.opendev.org/c/openstack/neutron/+/829526 | 08:20 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/xena: Add functional and fullstack jobs with FIPS enabled https://review.opendev.org/c/openstack/neutron/+/828697 | 08:20 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/xena: Add FIPS enabled scenario jobs https://review.opendev.org/c/openstack/neutron/+/828142 | 08:20 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/wallaby: Make configure_for_func_testing compatible with e.g. Centos https://review.opendev.org/c/openstack/neutron/+/829435 | 08:21 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/wallaby: Add functional and fullstack jobs with FIPS enabled https://review.opendev.org/c/openstack/neutron/+/829527 | 08:21 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/wallaby: Add FIPS enabled scenario jobs https://review.opendev.org/c/openstack/neutron/+/828143 | 08:21 |
| ralonsoh | slaweq, ^^ I'm reviewing those patches now | 08:24 |
| ralonsoh | can you check https://review.opendev.org/c/openstack/devstack/+/828877 | 08:24 |
| ralonsoh | ? | 08:24 |
| ralonsoh | thanks in advance! | 08:24 |
| opendevreview | Oleg Bondarev proposed openstack/neutron master: [fullstack] use noop fw driver for Local IP conntrack test https://review.opendev.org/c/openstack/neutron/+/829659 | 08:28 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/victoria: Fix race condition retrieving logical router rows https://review.opendev.org/c/openstack/ovn-octavia-provider/+/810489 | 08:35 |
| slaweq | ralonsoh sure, I will check it | 08:50 |
| slaweq | and with those my backports please wait a bit, I first want to run experimental queue to check if all is working fine there | 08:51 |
| slaweq | :) | 08:51 |
| ralonsoh | sure | 08:53 |
| opendevreview | Luis Tomas Bolivar proposed openstack/networking-ovn stable/train: Check gateway IP while looking for LR plugged to LS https://review.opendev.org/c/openstack/networking-ovn/+/829576 | 08:58 |
| ralonsoh | slaweq, sorry, last ping (at least this morning): https://bugs.launchpad.net/neutron/+bug/1961184 | 09:11 |
| ralonsoh | lucasagomes, ^^ | 09:12 |
| ralonsoh | the key point is the last question there | 09:12 |
| congnt1 | Hi @ralonsoh, can i ask a question about ovn? | 09:16 |
| congnt1 | I use network l3 ovn. | 09:17 |
| congnt1 | Instance --> Router --> External Network | 09:17 |
| congnt1 | Instance want Public IP can assign a floating IP from External Network. | 09:17 |
| congnt1 | But my External network run out of IP. | 09:17 |
| congnt1 | I need to add another CIDR, what is solution you use? | 09:17 |
| congnt1 | - Add subnet with new CIDR in External Network (Single large layer-2 networks become complex at scale and involve significant failure domains) | 09:17 |
| congnt1 | - Create another External network (Difficult for user) | 09:17 |
| ralonsoh | congnt1, you can add a second CIDR but should have access to the GW IP | 09:18 |
| lucasagomes | ralonsoh, checking | 09:19 |
| congnt1 | ralonsoh: i can't understand your solution, can you send me docs on openstack about your solution? | 09:20 |
| ralonsoh | congnt1, you can add a second subnet on the external network | 09:20 |
| ralonsoh | but be aware that from this subnet you need to have access to the GW IP | 09:21 |
| congnt1 | ralonsoh: My external network use vlan, if i am add multiple subnet in this network, it will very complex at scale and involve significant failure domains, right? | 09:22 |
| ralonsoh | congnt1, so then delete this subnet and add a new one with a bigger CIDR | 09:23 |
| congnt1 | @ralonsoh are you still using this solution for your external network? | 09:24 |
| ralonsoh | congnt1, I'm not using any particular solution, that depends on your needs | 09:25 |
| congnt1 | thank you @ralonsoh | 09:27 |
| lucasagomes | ralonsoh, re question there, it's something to think about. As you said, if we change it in ml/ovn we know will create a gap with ml2/ovs which usually not ideal | 09:29 |
| ralonsoh | lucasagomes, I'll keep that BZ on hold then | 09:30 |
| lucasagomes | ralonsoh, espcially in this case which is via API right, cause different monitoring systems will have different results based on the driver used | 09:30 |
| ralonsoh | right | 09:30 |
| lucasagomes | ralonsoh, maybe something to bring up at the PTG ? | 09:30 |
| ralonsoh | exactly | 09:30 |
| lucasagomes | I do agree that having it as ACTIVE sounds better overall | 09:31 |
| lucasagomes | but for the sake of standards it would be good to have it in the others drivers too | 09:31 |
| lajoskatona | ralonsoh, slaweq: Hi, could you please check this one if you have few minutes: https://review.opendev.org/c/openstack/neutron/+/827488 , it's for fullstack timeouts | 09:37 |
| ralonsoh | lajoskatona, sure | 09:37 |
| labedz | hello, slaweq: I put a response to your comment: https://review.opendev.org/c/openstack/neutron/+/826545 | 09:44 |
| slaweq | lajoskatona I will check it now | 09:50 |
| slaweq | labedz and Your's just after :) | 09:50 |
| slaweq | labedz I just replied | 10:12 |
| labedz | slaweq: good point, checking that | 10:13 |
| labedz | slaweq: but still when there is no snat ns then things goes as usuall | 10:14 |
| slaweq | labedz but I was wondering why You want to change it for DVR-ha and not for centralized ha routers? | 10:15 |
| slaweq | shouldn't it behave the same way in both cases? | 10:15 |
| slaweq | the only difference there is that "ha-" ports are in different namespaces | 10:16 |
| slaweq | but except that it works the same way | 10:16 |
| labedz | yeap, it should be the same for L3 with/wo SNAT; fix is simple... just check for qrouter and not snat as qrouter ns is valid in both cases | 10:18 |
| slaweq | labedz++ | 10:18 |
| slaweq | ralonsoh I replied to Your comment in https://bugs.launchpad.net/neutron/+bug/1961184 | 10:19 |
| ralonsoh | slaweq, thanks | 10:20 |
| labedz | BTW what you think about using ddt and contextlib for unit tests? | 10:22 |
| congnt1 | slaweq: can i ask a question about routed provider network? | 10:24 |
| congnt1 | I have a external network run out of IP, i want to add a subnet in this network, but use different VLAN. I research and find feature routed provider network. | 10:24 |
| congnt1 | It's use segment to represent multiple layer-2 networks, But as far as I know, each compute can only be map to 1 segment, right? | 10:24 |
| congnt1 | Is there any way to have a compute map to multiple segments? | 10:24 |
| congnt1 | Thank you | 10:24 |
| congnt1 | I am using ovn | 10:31 |
| zhouhenglc | ralonsoh: hi, do you have time check https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/828443 | 10:35 |
| ralonsoh | sure | 10:35 |
| zhouhenglc | thanks :) | 10:36 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/wallaby: Use a thread local variable to store the Nova Notifier enable flag https://review.opendev.org/c/openstack/neutron/+/828156 | 10:44 |
| ralonsoh | bcafarel, please, check ^^ | 10:44 |
| ralonsoh | I rebased it to be on top of https://review.opendev.org/c/openstack/neutron/+/829211 | 10:44 |
| slaweq | congnt1 I think it's the limitation which we currently have, there is e.g. bug https://bugs.launchpad.net/neutron/+bug/1764738 opened for the same thing IIUC | 10:50 |
| bcafarel | ralonsoh: sure, looking | 11:06 |
| bcafarel | ah it got the pep8 failures for pyroute, indeed should pass CI now | 11:07 |
| congnt1 | slaveweq: Oh, this bug is open since 2018, looks like it's not done yet. | 11:12 |
| congnt1 | Do you have any solution about add another Public IP to openstack? Because our old provider network run out of IP. | 11:12 |
| congnt1 | - If i add new subnet in old provider network, it will use same VLAN, and increase broadcase domain | 11:12 |
| congnt1 | - If i add new provider network. User can't assign floating IP from new provider network. User need to create another router, and set external gateway to new provider network, then the user can get the floating IP | 11:12 |
| congnt1 | Thank you | 11:12 |
| zhouhenglc | slaweq: hi I reply you in https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/828443 | 11:26 |
| opendevreview | Merged openstack/networking-ovn stable/train: Use Port_Binding up column to set Neutron port status https://review.opendev.org/c/openstack/networking-ovn/+/823279 | 11:36 |
| elvira | lajoskatona: Hi! Do you know if you will discuss RFEs on the drivers meeting tomorrow? (Just asking because I think many people are on PTO that day) | 11:49 |
| opendevreview | Merged openstack/neutron master: [OVN] Extend port binding parameter validation https://review.opendev.org/c/openstack/neutron/+/818420 | 11:52 |
| slaweq | lajoskatona ralonsoh can You check backports serie https://review.opendev.org/c/openstack/neutron/+/828142 ? | 11:53 |
| slaweq | it should be good to go now | 11:53 |
| slaweq | and it only adds jobs in the experimental queue for now | 11:53 |
| slaweq | thx in advance | 11:53 |
| ralonsoh | let me check | 11:54 |
| slaweq | also wallaby serie https://review.opendev.org/c/openstack/neutron/+/828143/ should be ready to review | 11:57 |
| slaweq | fullstack-fips job there is timing out but can investigate it later too | 11:57 |
| slaweq | all other fips jobs are running fine there woo | 11:57 |
| slaweq | *too | 11:57 |
| opendevreview | Merged openstack/ovn-octavia-provider stable/ussuri: Reduce coverage threshold to allow old backports https://review.opendev.org/c/openstack/ovn-octavia-provider/+/828856 | 11:59 |
| opendevreview | Mamatisa Nurmatov proposed openstack/neutron master: Add new class for Logging Driver methods https://review.opendev.org/c/openstack/neutron/+/829522 | 12:07 |
| opendevreview | Merged openstack/neutron stable/xena: Use Port_Binding up column to set Neutron port status https://review.opendev.org/c/openstack/neutron/+/823267 | 12:33 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/ussuri: Fix race condition retrieving logical router rows https://review.opendev.org/c/openstack/ovn-octavia-provider/+/827077 | 12:37 |
| opendevreview | Merged openstack/neutron stable/wallaby: Use Port_Binding up column to set Neutron port status https://review.opendev.org/c/openstack/neutron/+/823269 | 12:43 |
| opendevreview | Mamatisa Nurmatov proposed openstack/neutron master: Add new class for Logging API methods https://review.opendev.org/c/openstack/neutron/+/829522 | 13:04 |
| opendevreview | Merged openstack/neutron stable/victoria: Use Port_Binding up column to set Neutron port status https://review.opendev.org/c/openstack/neutron/+/823271 | 13:05 |
| opendevreview | Merged openstack/neutron stable/ussuri: Use Port_Binding up column to set Neutron port status https://review.opendev.org/c/openstack/neutron/+/823272 | 13:05 |
| congnt1 | slaweq: Oh, this bug is open since 2018, looks like it's not done yet. | 13:19 |
| congnt1 | Do you have any solution about add another Public IP to openstack? Because our old provider network run out of IP. | 13:19 |
| congnt1 | - If i add new subnet in old provider network, it will use same VLAN, and increase broadcase domain | 13:19 |
| congnt1 | - If i add new provider network. User can't assign floating IP from new provider network. User need to create another router, and set external gateway to new provider network, then the user can get the floating IP | 13:19 |
| congnt1 | Thank you | 13:19 |
| dmitriis | slaweq: o/ if you get a chance, fnordahl is looking for a clarification on https://review.opendev.org/c/openstack/neutron/+/808961/22/neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/ovn_client.py#251 https://review.opendev.org/c/openstack/neutron/+/808961/22/neutron/plugins/ml2/drivers/ovn/mech_driver/mech_driver.py#931 before doing another | 13:28 |
| dmitriis | upload. | 13:28 |
| dmitriis | as for the test failure, it will be gone as the new neutron-lib release is now out | 13:29 |
| lajoskatona | dmitriis: you have to update requirements.txt I think to use new neutron-lib | 13:42 |
| *** congnt1 is now known as congnt | 13:43 | |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Support filtering for QoS rule type list https://review.opendev.org/c/openstack/neutron/+/827683 | 13:53 |
| ralonsoh | mlavalle, https://review.opendev.org/c/openstack/neutron/+/828172 | 13:55 |
| ralonsoh | if you have 1 min | 13:55 |
| ralonsoh | thanks! | 13:55 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [OVN] Off-path SmartNIC DPU Port Binding with OVN https://review.opendev.org/c/openstack/neutron/+/808961 | 13:57 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [OVN] Off-path SmartNIC DPU Documentation https://review.opendev.org/c/openstack/neutron/+/828103 | 13:57 |
| ykarel | mnasiadka, hi u around? | 13:57 |
| ykarel | sorry was looking for Piotr Parczewski for https://bugs.launchpad.net/neutron/+bug/1928330 | 13:58 |
| opendevreview | Frode Nordahl proposed openstack/neutron-specs master: smartnic-dpu: Update implementation details https://review.opendev.org/c/openstack/neutron-specs/+/829715 | 14:04 |
| dmitriis | lajoskatona: yeah, will do that, just wanted to agree on the one review comment before reuploading with the updated requirements file | 14:04 |
| fnordahl | lajoskatona, dmitriis: it will be overridden by global upper constraints though. | 14:10 |
| fnordahl | lajoskatona, dmitriis: I made an attempt at adding that as a depends-on, but don't know if Zuul is clever enough. | 14:10 |
| fnordahl | A upper constraints review is in-flight here: https://review.opendev.org/c/openstack/requirements/+/829670 | 14:11 |
| dmitriis | fnordahl: yes, there's a recheck in-flight since there was a timeout. Hopefully it will land soon. | 14:12 |
| fnordahl | slaweq: responded to your question about passing in the port_context, there is a reason for it, and happy to do it in a different way if you still have comments on the approach. | 14:14 |
| opendevreview | Merged openstack/neutron stable/wallaby: Use a thread local variable to store the Nova Notifier enable flag https://review.opendev.org/c/openstack/neutron/+/828156 | 14:23 |
| opendevreview | Merged openstack/neutron stable/xena: Remove run-devstack role from run_functional_job playbook https://review.opendev.org/c/openstack/neutron/+/829436 | 14:23 |
| mnasiadka | ykarel: he's my colleague - parallax alive? | 14:26 |
| mnasiadka | ykarel: I think he got it to working, so I don't know if that bug is valid. | 14:26 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [OVN] Off-path SmartNIC DPU Port Binding with OVN https://review.opendev.org/c/openstack/neutron/+/808961 | 14:27 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [OVN] Off-path SmartNIC DPU Documentation https://review.opendev.org/c/openstack/neutron/+/828103 | 14:27 |
| parallax | hi, alive | 14:27 |
| ykarel | mnasiadka, Thanks will wait for his confirmation | 14:27 |
| ykarel | hi parallax, may be u can check the comment in https://bugs.launchpad.net/neutron/+bug/1928330 and confirm | 14:28 |
| parallax | I'll see if I can re-test this on Wallaby; I think it was originally reported against Victoria, I don't think I'll be able to check master though | 14:33 |
| ykarel | ack Thanks parallax | 14:39 |
| *** congnt1 is now known as congnt | 15:27 | |
| parallax | np | 15:42 |
| opendevreview | Mamatisa Nurmatov proposed openstack/neutron master: Add new class for Logging API methods https://review.opendev.org/c/openstack/neutron/+/829522 | 16:03 |
| opendevreview | Brian Haley proposed openstack/neutron master: DNM: Test IPv6 all (including tunnel) job definition https://review.opendev.org/c/openstack/neutron/+/828634 | 16:08 |
| mlavalle | ralonsoh: I did have 1 min :-) | 16:18 |
| ralonsoh | mlavalle, thanks | 16:18 |
| opendevreview | Maor Blaustein proposed openstack/neutron-tempest-plugin master: Fix dependencies and PEP 8 new comments. https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/827292 | 16:32 |
| opendevreview | Maor Blaustein proposed openstack/neutron-tempest-plugin master: Fix dependencies and PEP 8 new comments. https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/827292 | 16:35 |
| opendevreview | Luis Tomas Bolivar proposed openstack/networking-ovn stable/train: Check gateway IP while looking for LR plugged to LS https://review.opendev.org/c/openstack/networking-ovn/+/829576 | 16:47 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [WIP] [OVN][Placement] Drive binding by placement allocation https://review.opendev.org/c/openstack/neutron/+/786478 | 17:26 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN][Placement] Read the initial config in the maintenance worker https://review.opendev.org/c/openstack/neutron/+/821865 | 17:41 |
| ralonsoh | lajoskatona, do you know in what job are we executing "MinBwAllocationPlacementTest"? | 17:58 |
| ralonsoh | this is in tempest.scenario.test_network_qos_placement | 17:58 |
| gibi | ralonsoh: https://review.opendev.org/c/openstack/neutron/+/806170 | 18:08 |
| gibi | neutron-ovs-tempest-multinode-full | 18:08 |
| ralonsoh | gibi, thanks a lot | 18:08 |
| gibi | ralonsoh: nova runs it in nova-next | 18:08 |
| ralonsoh | gibi, I'm going to try adding it to OVN | 18:09 |
| gibi | sounds interesting :) | 18:09 |
| opendevreview | Merged openstack/neutron stable/xena: Make configure_for_func_testing compatible with e.g. Centos https://review.opendev.org/c/openstack/neutron/+/829526 | 18:10 |
| opendevreview | Merged openstack/neutron master: [OVN] Sync QoS policies https://review.opendev.org/c/openstack/neutron/+/813052 | 18:10 |
| opendevreview | Merged openstack/neutron stable/xena: Create a PeriodicWorker for DbQuotaNoLockDriver clean up https://review.opendev.org/c/openstack/neutron/+/828172 | 18:10 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN][Placement] Read the initial config in the maintenance worker https://review.opendev.org/c/openstack/neutron/+/821865 | 18:15 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [WIP] [OVN][Placement] Drive binding by placement allocation https://review.opendev.org/c/openstack/neutron/+/786478 | 18:15 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-lib master: [DNM] Refactor session "is_active" handling for sqlalchemy-20 https://review.opendev.org/c/openstack/neutron-lib/+/828738 | 18:22 |
| opendevreview | Lajos Katona proposed openstack/tap-as-a-service master: Modify taas extensions to use neutron-lib API def https://review.opendev.org/c/openstack/tap-as-a-service/+/824927 | 18:33 |
| opendevreview | Merged openstack/neutron master: Add table for pps limitaion https://review.opendev.org/c/openstack/neutron/+/804213 | 18:49 |
| *** kopecmartin_ is now known as kopecmartin | 19:11 | |
| jlibosva | mlavalle: hi, do you know if I can somehow re-open https://review.opendev.org/c/openstack/neutron/+/805768 ? I'd like to push a change but gerrit won't let me | 21:53 |
| mlavalle | jlibosva: look now and please tell me if that's what you wanted | 21:55 |
| jlibosva | mlavalle: yes, thanks! how did you do it? | 21:55 |
| opendevreview | Jakub Libosvar proposed openstack/neutron stable/wallaby: Add wait for the post-fork event to nb/sb objects https://review.opendev.org/c/openstack/neutron/+/805768 | 21:55 |
| mlavalle | jlibosva: just pressed the restore button | 21:56 |
| mlavalle | in Gerrit's GUI | 21:56 |
| jlibosva | hmm, I couldn't find it | 21:56 |
| jlibosva | mlavalle: thanks! | 21:56 |
| mlavalle | jlibosva: you are welcome | 21:56 |
| mlavalle | jlibosva: the button is in the upper right hand side, just below your name. Now you should see the rebase / abandon / edit buttons there | 21:57 |
| jlibosva | mlavalle: maybe I don't have permissions - I have an old not refreshed window and I don't see any button there | 21:59 |
| jlibosva | mlavalle: I see rebase and edit now | 21:59 |
| mlavalle | jlibosva: you don't see abandon? | 22:00 |
| jlibosva | mlavalle: no | 22:01 |
| jlibosva | and I don't see Restore here either https://review.opendev.org/c/openstack/neutron/+/805769 | 22:01 |
| opendevreview | Brian Haley proposed openstack/neutron master: DNM: Test IPv6 all (including tunnel) job definition https://review.opendev.org/c/openstack/neutron/+/828634 | 22:04 |
| *** haleyb_ is now known as haleyb | 22:04 | |
| *** dasm is now known as dasm|off | 23:07 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!