| *** pvxe0 is now known as pvxe | 00:32 | |
| *** pvxe8 is now known as pvxe | 01:32 | |
| opendevreview | Merged openstack/neutron master: Return 409 Conflict to tenant user deleting port attached to FIP https://review.opendev.org/c/openstack/neutron/+/881827 | 01:35 |
|---|---|---|
| opendevreview | Michal Nasiadka proposed openstack/neutron master: haproxy: Add support for configuring syslog https://review.opendev.org/c/openstack/neutron/+/884407 | 07:02 |
| opendevreview | Lajos Katona proposed openstack/networking-odl stable/yoga: Fix tox4 issues and define project upper constraints https://review.opendev.org/c/openstack/networking-odl/+/884359 | 08:03 |
| dvo-plv_ | ralonsoh: Hello, thank you for the review. Unfortinatelly you passed by my yesterday question | 08:06 |
| dvo-plv_ | I would like to ask you again | 08:06 |
| dvo-plv_ | I would like to clarify this comment https://review.opendev.org/c/openstack/nova-specs/+/859290/11..15/specs/2023.2/approved/support-napatech-linkvirtualization-smartnic.rst#b60 | 08:06 |
| dvo-plv_ | You left two questions. Lets start from the second, I have added our hardware, which will suport this feature https://review.opendev.org/c/openstack/nova-specs/+/859290/15/specs/2023.2/approved/support-napatech-linkvirtualization-smartnic.rst#74 | 08:06 |
| dvo-plv_ | Also all other smartnics, which support this representor port type can use this solution. https://review.opendev.org/c/openstack/nova-specs/+/859290/15/specs/2023.2/approved/support-napatech-linkvirtualization-smartnic.rst#28 | 08:06 |
| dvo-plv_ | But I do not understand what you mean under the first question. Does it mean what benefit Nova and Neutron will get with our presence ? | 08:08 |
| ralonsoh | In L74 you are referring to numbers without any reference, please add links to specific websites | 08:19 |
| ralonsoh | the first one is not about the benefit | 08:20 |
| ralonsoh | is about the transparency of your solution | 08:20 |
| ralonsoh | if just with a short change in os-vif when creating the port representor will be enough | 08:21 |
| ralonsoh | to be honest, I've been "playing" with other HW solutions, and there are many small differences and gaps between the kernel and HWOL solution | 08:21 |
| ralonsoh | this is why I'm asking for a CI | 08:21 |
| ralonsoh | but I never questioned the importance of your spec and your solution | 08:22 |
| dvo-plv_ | as was agreed at meetings with Neutron and Nova folks, we will provide ci verifications. List of the tests is here https://review.opendev.org/c/openstack/nova-specs/+/859290/11..15/specs/2023.2/approved/support-napatech-linkvirtualization-smartnic.rst#b158. Should i add this list to the spec file ? | 08:26 |
| ralonsoh | IMO no, this is not part of the spec. But again, I feel this spec is too short and doesn't cover any parity gap between kernel and HWOL | 08:27 |
| ralonsoh | and I'm pretty sure there are gaps | 08:28 |
| *** Continuity_ is now known as Continuity | 08:28 | |
| dvo-plv_ | With ci we are on the finish line with Vanilla deployment. Integration with our hardware is in the progress. I believe it will be ready in 1-2 weeks, according to our ci team's load | 08:29 |
| ralonsoh | that will be perfect | 08:29 |
| dvo-plv_ | Okay, I will add additional information regarding hwol. | 08:32 |
| dvo-plv_ | Regarding OpenStack layer. Does there enough information about changes, or I should add some more information? | 08:33 |
| ralonsoh | this is a Nova spec, you should ask Nova folks | 08:34 |
| ralonsoh | from neutron point of view, this change is "small" | 08:34 |
| ralonsoh | I will +1 the spec once you make these changes | 08:34 |
| opendevreview | Lajos Katona proposed openstack/networking-bagpipe master: Add os_ken_app to EVPN OVSDataplaneDriver https://review.opendev.org/c/openstack/networking-bagpipe/+/884695 | 08:35 |
| opendevreview | Arnaud Morin proposed openstack/neutron master: Add a new option to enable signals https://review.opendev.org/c/openstack/neutron/+/884711 | 08:55 |
| opendevreview | Arnaud Morin proposed openstack/neutron master: Register GMR with config https://review.opendev.org/c/openstack/neutron/+/884842 | 08:55 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-lib master: [API REF] Add api-ref documentation for default SG rules API https://review.opendev.org/c/openstack/neutron-lib/+/884578 | 09:30 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-tempest-plugin master: Define proper common API extensions for 2023.1 and Zed jobs https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/884476 | 09:34 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-tempest-plugin master: New basic API tests for the default SG rules templates CRUDs https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/883553 | 09:35 |
| dvo-plv_ | ralonsoh: I wouldl ike to ask you, about your thinks regarding this Sean's comment https://review.opendev.org/c/openstack/nova-specs/+/859290/8/specs/2023.2/approved/napatech-linkvirtualization-smartnic-support.rst#166 | 09:50 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-tempest-plugin master: New basic API tests for the default SG rules templates CRUDs https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/883553 | 09:50 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-tempest-plugin master: [Default SG rules] Test to check if SG rules are created from template https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/884475 | 09:50 |
| dvo-plv_ | Sean would like that we just add some link to Napatech documentation reagarding our solution like Mellanox | 09:51 |
| dvo-plv_ | Also we have to update Prerequisites item for our PMD driver | 09:52 |
| ralonsoh | dvo-plv_, and what is the question? | 09:52 |
| dvo-plv_ | Does these tow items which I described is all, or you would like some other changes in this doc | 09:53 |
| ralonsoh | dvo-plv_, but why don't you ask him? | 09:53 |
| dvo-plv_ | I asked him and agreed changes. but this is Neiutron doc, so I would like to ask Neutron folks regadring that | 09:55 |
| ralonsoh | but that is part of the feature development | 09:56 |
| ralonsoh | anything required to configure the deployment should be described | 09:56 |
| ralonsoh | so yes, you should document that | 09:56 |
| ralonsoh | don't you need to perform any previous step, before deploying OpenStack? | 09:56 |
| dvo-plv_ | We need to prepare Compute nodes, install our drivers to be able operate nics and PMD driver | 09:58 |
| ralonsoh | this is exactly what Sean is asking for | 09:59 |
| dvo-plv | yes, I understand this, but Could we add some steps directly to the doc, or it should be added only via link to the external doc ? | 10:30 |
| ralonsoh | dvo-plv, what you should do is to create a new document in the Neutron repository providing, step by step, like in https://docs.openstack.org/neutron/latest/admin/config-ovs-offload.html, how to configure the host, how to configure Nova and Neutron and how to create a port and a VM | 10:35 |
| dvo-plv | greate, we will create separate doc, thanks | 10:38 |
| opendevreview | Lajos Katona proposed openstack/networking-bagpipe master: Fix issues due to rcent RBAC changes and removal of neutron.debug https://review.opendev.org/c/openstack/networking-bagpipe/+/883064 | 10:43 |
| opendevreview | Lajos Katona proposed openstack/networking-bagpipe master: Fix issues due to rcent RBAC changes and removal of neutron.debug https://review.opendev.org/c/openstack/networking-bagpipe/+/883064 | 10:55 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Default SG rules - use new rules templates to create rules for SGs https://review.opendev.org/c/openstack/neutron/+/884474 | 10:56 |
| opendevreview | Lajos Katona proposed openstack/networking-bgpvpn master: Fix issues due to rcent RBAC changes https://review.opendev.org/c/openstack/networking-bgpvpn/+/883057 | 11:03 |
| opendevreview | Lajos Katona proposed openstack/neutron-tempest-plugin master: DNM: test master bgpvpn job https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/884515 | 11:05 |
| stephenfin | A question on allowed-address-pairs: is there any reason this only supports individual IP addresses as opposed to CIDRs/IP address ranges? Does it scale with, say ~4000 individual IP addresses (/20)? | 11:08 |
| tore | stephenfin: specifying CIDR ranges WFM | 11:19 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: WIP == Change RBAC relationship loading method to "joined" https://review.opendev.org/c/openstack/neutron/+/884877 | 11:22 |
| ralonsoh | tore, this is not supported by some backends | 11:22 |
| ralonsoh | stephenfin, that should be tested, I can't provide a performance value | 11:23 |
| tore | ralonsoh: I see. well, appears to work with linuxbridge at least | 11:25 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-lib master: Add a "GROUP BY" clause on queries with RBAC entries https://review.opendev.org/c/openstack/neutron-lib/+/884878 | 11:30 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-tempest-plugin master: Don't use "Zero-I/O mode" in nc client in the TCP mode https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/884706 | 12:00 |
| opendevreview | sean mooney proposed openstack/neutron master: Send ovn heatbeat more often. https://review.opendev.org/c/openstack/neutron/+/883687 | 12:23 |
| opendevreview | Merged openstack/neutron master: Use ``TextClause`` to define the DB model "server_default" https://review.opendev.org/c/openstack/neutron/+/883421 | 12:39 |
| opendevreview | Merged openstack/neutron master: [sqlalchemy-20] Open a connection to execute a command https://review.opendev.org/c/openstack/neutron/+/883521 | 12:40 |
| sahid | o/ hey ralonsoh regarding your patches related to RBAC entries, have you been able to mesure/notice a great performance improvement? we also have done similar change but I think we are missing that piece with the group by | 12:46 |
| sahid | we may try to do some tests internally and share result if that can help | 12:49 |
| stephenfin | ralonsoh, tore: I didn't know it would work anywhere. I'll give it a go | 12:51 |
| lajoskatona | ralonsoh: Hi, good news, bagpipe-bgpvpn tempest passed, I think I have to make a dependency chain between patches that is not circular :-) | 13:17 |
| lajoskatona | ralonsoh: I hope at least that will help... | 13:18 |
| ralonsoh | lajoskatona, that's fantastic! good job | 13:19 |
| lajoskatona | ralonsoh: https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/884515 | 13:19 |
| ralonsoh | stephenfin, no no, it won't work anywhere. For example Ml2/OVN does not support CIDRs | 13:19 |
| ralonsoh | slaweq, ^^ please check this patch too | 13:19 |
| lajoskatona | ralonsoh: now 4 patches are depending on each other, so I have to find which one should be the first :-) | 13:19 |
| ralonsoh | sahid, from 1 minute to 2 seconds | 13:20 |
| ralonsoh | with 2500 RBACs, 10 nets and 10 projects | 13:20 |
| ralonsoh | lajoskatona, but this one https://review.opendev.org/c/openstack/networking-bagpipe/+/883064 should be first, right? | 13:21 |
| ralonsoh | ah ah ah it depends on other one | 13:22 |
| sahid | ralonsoh: have you try using postgresql, I think it's why we have not tried to share it upstream and keept it internally | 13:22 |
| ralonsoh | sahid, no, mysql | 13:22 |
| ralonsoh | lajoskatona, why don't you merge https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/883838 first | 13:22 |
| ralonsoh | and then the other dependant ones? | 13:22 |
| ralonsoh | regardless of the failing tests | 13:23 |
| slaweq | ralonsoh do You mean https://review.opendev.org/c/openstack/networking-bagpipe/+/883064 ? | 13:23 |
| ralonsoh | slaweq, yes, but there are 4 pending patches | 13:24 |
| slaweq | I just approved it | 13:24 |
| ralonsoh | the list is long hehehehe | 13:24 |
| ralonsoh | thanks!! | 13:24 |
| lajoskatona | slaweq, ralonsoh: exactly :-) I have to be sure because now I am not sure if the tempest change is really necessary. | 13:24 |
| ralonsoh | lajoskatona, then please update https://review.opendev.org/c/openstack/networking-bgpvpn/+/883057 removing the depends-on | 13:25 |
| ralonsoh | if that is the case | 13:25 |
| slaweq | ok but what is "tempest change" related to this? | 13:28 |
| ralonsoh | https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/883838 | 13:28 |
| slaweq | I see some neutron-tempest-plugin patch but nothing in tempest | 13:28 |
| slaweq | ok | 13:30 |
| slaweq | thx | 13:30 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Change RBAC relationship loading method to "joined" https://review.opendev.org/c/openstack/neutron/+/884877 | 14:12 |
| ralonsoh | lajoskatona, so are you going to abandon https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/883838? | 14:13 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Change RBAC relationship loading method to "joined" https://review.opendev.org/c/openstack/neutron/+/884877 | 15:04 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Remove backwards compatibility with OVN < v20.09 https://review.opendev.org/c/openstack/neutron/+/884898 | 15:14 |
| *** Continuity__ is now known as Continuity | 15:15 | |
| ralonsoh | lajoskatona, qq, if you know, do you know what is happening to FT jobs in ODL? | 15:30 |
| opendevreview | Rodolfo Alonso proposed openstack/networking-odl stable/yoga: Fix tox4 issues and define project upper constraints https://review.opendev.org/c/openstack/networking-odl/+/884359 | 15:31 |
| opendevreview | Rodolfo Alonso proposed openstack/networking-odl stable/yoga: Fix tox4 issues and define project upper constraints https://review.opendev.org/c/openstack/networking-odl/+/884359 | 15:45 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Implement ``get_port_type_virtual_and_parents`` method https://review.opendev.org/c/openstack/neutron/+/882557 | 16:04 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Move ``determine_bind_host`` to ``ovn.utils`` https://review.opendev.org/c/openstack/neutron/+/882562 | 16:07 |
| opendevreview | Rodolfo Alonso proposed openstack/networking-odl stable/xena: Fix tox4 issues and define project upper constraints https://review.opendev.org/c/openstack/networking-odl/+/884905 | 16:19 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Prevent binding a virtual type port https://review.opendev.org/c/openstack/neutron/+/882588 | 16:30 |
| opendevreview | Rodolfo Alonso proposed openstack/networking-odl stable/xena: Fix tox4 issues and define project upper constraints https://review.opendev.org/c/openstack/networking-odl/+/884905 | 16:32 |
| opendevreview | Rodolfo Alonso proposed openstack/networking-odl stable/yoga: Fix tox4 issues and define project upper constraints https://review.opendev.org/c/openstack/networking-odl/+/884359 | 17:07 |
| opendevreview | Lajos Katona proposed openstack/neutron-lib master: api-ref: add api-ref for bindings https://review.opendev.org/c/openstack/neutron-lib/+/884916 | 17:47 |
| opendevreview | Merged openstack/neutron master: Revert "[OVN] Remove backwards compatibility with OVN < v20.09" https://review.opendev.org/c/openstack/neutron/+/884733 | 18:25 |
| opendevreview | Miro Tomaska proposed openstack/neutron-tempest-plugin master: Add sleep to two server port forwarding test https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/884933 | 19:36 |
| opendevreview | Merged openstack/neutron master: Add description field to the security_group_default_rules resource https://review.opendev.org/c/openstack/neutron/+/883386 | 20:29 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: Define proper common API extensions for 2023.1 and Zed jobs https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/884476 | 20:29 |
| opendevreview | Merged openstack/neutron master: [S-RBAC] Add API policies for get and activate port bindings https://review.opendev.org/c/openstack/neutron/+/884613 | 20:30 |
| opendevreview | Brian Haley proposed openstack/neutron master: Fix 'consider-using-with' warning https://review.opendev.org/c/openstack/neutron/+/884947 | 20:44 |
| opendevreview | Brian Haley proposed openstack/neutron master: Update OVN client _get_port_options() code and utils https://review.opendev.org/c/openstack/neutron/+/884950 | 21:44 |
| opendevreview | Merged openstack/networking-odl stable/zed: Tox4: add allowlist_externals where necessary https://review.opendev.org/c/openstack/networking-odl/+/884325 | 22:35 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!