| opendevreview | Miguel Lavalle proposed openstack/neutron master: [PoC][DNM] Router flavors and service type for OVN https://review.opendev.org/c/openstack/neutron/+/883988 | 00:16 |
|---|---|---|
| sondx25 | ralonsoh, I set the QoS tap interface of the VM to qdisc fq_codel, but the network performance is still not improved. Please help me. | 03:17 |
| ykarel | Continuity, what ovn version you using? | 05:32 |
| ykarel | there were some fixes for fip + vlan in core ovn like https://bugzilla.redhat.com/show_bug.cgi?id=2007120 + multiple fixes in neutron like https://bugs.launchpad.net/neutron/+bug/2003455 | 05:34 |
| ykarel | may be you missing some of those? | 05:34 |
| opendevreview | Luis Tomas Bolivar proposed openstack/ovn-bgp-agent master: Make debug log less chatty https://review.opendev.org/c/openstack/ovn-bgp-agent/+/887550 | 05:57 |
| opendevreview | ZhouHeng proposed openstack/neutron master: Set result when lswitch port exist https://review.opendev.org/c/openstack/neutron/+/881771 | 06:15 |
| Continuity | ykarel: thanks ill take a look and see where we are in regards to those | 06:31 |
| sondx25 | ykarel, - I use OpenStack + ML2/OVN (OpenStack Yoga, OVN 22.03.0, Open vSwitch 2.17.0). | 06:51 |
| sondx25 | - Initially, the Compute node has only a few virtual machines, I see bandwidth between 2 VMs (in the same self-service network) in the same Compute node is 20 Gbps. Then the number of virtual machines at the Compute node increased to 100 VMs, (but the CPU used of node Compute only 30%), then I see bandwidth between 2 VMs (in the same self-service network) in this Compute node is 6 Gbps. | 06:51 |
| sondx25 | - Note: This does not happen with OpenStack Yoga, ML2/OpenvSwitch configuration: | 06:51 |
| sondx25 | [securitygroup] | 06:51 |
| sondx25 | firewall_driver = iptables_hybrid | 06:51 |
| sondx25 | ykarel, In case, my system uses OpenStack + ML2/OVN (OpenStack Yoga, OVN 22.03.0, Open vSwitch 2.17.0). | 06:55 |
| sondx25 | The number of virtual machines at the Compute node increased to 100 VMs, (but the CPU used of node Compute only 30%), then I see bandwidth between 2 VMs (in the same self-service network) in this Compute node is 6 Gbps. | 06:55 |
| sondx25 | At this point, I disable port security and security group all ports of that self-service network, then the bandwidth between 2 VMs reaches 20 Gbps again. | 06:55 |
| sondx25 | ralonsoh, - I use OpenStack + ML2/OVN (OpenStack Yoga, OVN 22.03.0, Open vSwitch 2.17.0). | 06:58 |
| sondx25 | - Initially, the Compute node has only a few virtual machines, I see bandwidth between 2 VMs (in the same self-service network) in the same Compute node is 20 Gbps. Then the number of virtual machines at the Compute node increased to 100 VMs, (but the CPU used of node Compute only 30%), then I see bandwidth between 2 VMs (in the same self-service network) in this Compute node is 6 Gbps. | 06:58 |
| sondx25 | At this point, I disable port security and security group all ports of that self-service network, then the bandwidth between 2 VMs reaches 20 Gbps again. | 06:58 |
| sondx25 | - Note: This does not happen with OpenStack Yoga, ML2/OpenvSwitch configuration: | 06:58 |
| sondx25 | [securitygroup] | 06:58 |
| sondx25 | firewall_driver = iptables_hybrid | 06:58 |
| sondx25 | - Please see detail in video (https://youtu.be/Xnsyo0DZZO8). | 06:58 |
| ralonsoh | sondx25, we talked yesterday about this. You can ask this question in the OVN mailing list. If this problem is due to the conntrack table, this could be treated as an improvement in core OVN | 07:17 |
| ralonsoh | you can also use DPDK in your environment | 07:17 |
| ralonsoh | and if the performance is that important, you can also use HW offload (that requires an additional cost) | 07:18 |
| ralonsoh | sondx25, do you have the same test video with ML2/OVS and hybrid firewall? | 07:21 |
| opendevreview | ZhouHeng proposed openstack/neutron master: Set result when lswitch port exist https://review.opendev.org/c/openstack/neutron/+/881771 | 07:25 |
| ykarel | sondx25, yeap would be better to ask ovn mail list as Rodolfo suggested. I am unsure what causing this, but increasing vm and performance drop is a good data point | 07:37 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Add a new extension "security-groups-rules-belongs-to-default-sg" https://review.opendev.org/c/openstack/neutron/+/883907 | 07:49 |
| opendevreview | Merged openstack/ovn-bgp-agent master: Make debug log less chatty https://review.opendev.org/c/openstack/ovn-bgp-agent/+/887550 | 08:13 |
| ykarel | ralonsoh, sondx25 to rule out conntrack involvement may be can try stateless security group? | 08:14 |
| ralonsoh | ykarel, sondx25 or use remote CIDR instead of remote group (that affects the OF rules, not directly the conntrack rules). But worth to try both options | 08:15 |
| ykarel | +1 | 08:15 |
| opendevreview | ZhouHeng proposed openstack/neutron master: Improve ACL comparison efficiency https://review.opendev.org/c/openstack/neutron/+/885449 | 08:19 |
| sondx25 | rolonsoh, I used remote CIDR and remote group, the results are the same. | 08:29 |
| sondx25 | ykarel, rolonsoh, I try to use stateless security group, great bandwidth result is 20 Gbps. | 08:40 |
| opendevreview | Lucas Alvares Gomes proposed openstack/neutron stable/2023.1: [OVN] Expose chassis hosting information in LSP https://review.opendev.org/c/openstack/neutron/+/887579 | 08:41 |
| ralonsoh | sondx25, so as you presumed yesterday, this is an "issue" in the conntrack module | 08:41 |
| opendevreview | Lucas Alvares Gomes proposed openstack/neutron stable/zed: [OVN] Expose chassis hosting information in LSP https://review.opendev.org/c/openstack/neutron/+/887580 | 08:41 |
| opendevreview | Lucas Alvares Gomes proposed openstack/neutron stable/yoga: [OVN] Expose chassis hosting information in LSP https://review.opendev.org/c/openstack/neutron/+/887581 | 08:41 |
| opendevreview | Lucas Alvares Gomes proposed openstack/neutron stable/xena: [OVN] Expose chassis hosting information in LSP https://review.opendev.org/c/openstack/neutron/+/887582 | 08:42 |
| opendevreview | Lucas Alvares Gomes proposed openstack/neutron stable/wallaby: [OVN] Expose chassis hosting information in LSP https://review.opendev.org/c/openstack/neutron/+/887583 | 08:42 |
| ykarel | sondx25, ralonsoh great, found https://developers.redhat.com/articles/2022/11/17/benchmarking-improved-conntrack-performance-ovs-300#performance_statistics significant improvement in ovs>=3.0.0 wrt this | 08:45 |
| sondx25 | ykarel, rolonsoh, Thank you so much for your help. | 09:04 |
| ykarel | Thanks sondx25 for checking, would be good if you also update the launchpad bug with all these findings https://bugs.launchpad.net/neutron/+bug/1996593 | 09:26 |
| sondx25 | ykarel, This bug (https://bugs.launchpad.net/neutron/+bug/1996593) was posted by myself. I will update it. | 09:33 |
| ykarel | yeap i noticed that :) | 09:33 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [FT] Move ``BaseOVSTestCase`` class to concurrency 1 executor https://review.opendev.org/c/openstack/neutron/+/887590 | 09:51 |
| sondx25 | ykarel, i see, thank you :) | 10:02 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-tempest-plugin master: Add "-d 1" option to the ncat client. https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/887591 | 10:15 |
| Continuity | ykarel: we are running OVN22.10, we are using kolla ansible to deploy | 10:35 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Use the new network HA parameter https://review.opendev.org/c/openstack/neutron/+/881742 | 10:58 |
| sondx25 | ykarel, https://developers.redhat.com/articles/2022/11/17/benchmarking-improved-conntrack-performance-ovs-300# . This article is about conntrack in userspace datapath. I am using kernel. | 11:00 |
| opendevreview | Bence Romsics proposed openstack/neutron master: ovs-agent: React to DB down just like to server down https://review.opendev.org/c/openstack/neutron/+/887257 | 11:02 |
| opendevreview | ZhouHeng proposed openstack/neutron master: Set result when lswitch port exist https://review.opendev.org/c/openstack/neutron/+/881771 | 11:24 |
| rubasov | ralonsoh, mlavalle: hi, I believe my deputy week would start 17th of July, however I have PTO planned for that week, would you maybe swap your deputy week with me? | 11:41 |
| ralonsoh | rubasov, let me check | 11:41 |
| ralonsoh | rubasov, yes (I need to update the dates) | 11:42 |
| ralonsoh | I can swap with you, no problem | 11:42 |
| rubasov | ralonsoh: cool, thanks | 11:42 |
| rubasov | ralonsoh: to be sure: 17th of July will be yours then and 24th of July mine | 11:43 |
| ralonsoh | rubasov, yes, actually I'm in PTO starting the 24th, so perfect | 11:43 |
| rubasov | all good then, thank you! | 11:44 |
| ykarel | sondx25, ack i didn't read that in detail | 11:51 |
| ykarel | Continuity, i didn't find 22.10 tag in ovn, may be you refering ubuntu kinetic 22.10? | 11:57 |
| ykarel | if yes then that doesn't seem to have that fix | 12:00 |
| Continuity | ykarel: sorry, so looking in the ovn container we are running 22.09.1 | 12:04 |
| Continuity | i think you are right though, we are seeing vlan fip ports ending up centralised, rather than being distributed. | 12:04 |
| Continuity | we are planning an upgrade for later this week/early next | 12:04 |
| ykarel | Continuity, ack 22.09.1 that fix is not there | 12:12 |
| Continuity | i think though that 22.09.1 is the latest available on ubuntu jammy | 12:14 |
| opendevreview | Lajos Katona proposed openstack/neutron stable/2023.1: Delete sg rule which remote is the deleted sg https://review.opendev.org/c/openstack/neutron/+/887464 | 12:23 |
| ykarel | ubuntu jammy i see is 22.03.2, so you seems to be on kinetic | 12:24 |
| ykarel | https://packages.ubuntu.com/kinetic-updates/ovn-central | 12:24 |
| opendevreview | Lajos Katona proposed openstack/neutron stable/zed: Delete sg rule which remote is the deleted sg https://review.opendev.org/c/openstack/neutron/+/887465 | 12:25 |
| opendevreview | Lajos Katona proposed openstack/neutron stable/yoga: Delete sg rule which remote is the deleted sg https://review.opendev.org/c/openstack/neutron/+/887466 | 12:25 |
| opendevreview | Lajos Katona proposed openstack/neutron stable/xena: Delete sg rule which remote is the deleted sg https://review.opendev.org/c/openstack/neutron/+/887467 | 12:26 |
| opendevreview | Lajos Katona proposed openstack/neutron stable/wallaby: Delete sg rule which remote is the deleted sg https://review.opendev.org/c/openstack/neutron/+/887468 | 12:26 |
| grami[m] | Hi all I wonder if someone could help me understand why a bug that was fixed in Ussuri and wallaby but somehow is no longer in the zed release https://bugs.launchpad.net/neutron/+bug/1920976 | 12:29 |
| grami[m] | I'm going to make a cherry-pick for it now as this should have been placed in zed | 12:31 |
| opendevreview | Graeme Moss proposed openstack/neutron stable/zed: [OVN] Migrate "reside-on-redirect-chassis" for distributed FIP https://review.opendev.org/c/openstack/neutron/+/887469 | 12:39 |
| opendevreview | Graeme Moss proposed openstack/neutron stable/zed: [OVN] Migrate "reside-on-redirect-chassis" for distributed FIP https://review.opendev.org/c/openstack/neutron/+/887469 | 12:40 |
| grami[m] | Could I ask if some one could review ^^ please please | 12:41 |
| opendevreview | Graeme Moss proposed openstack/neutron stable/zed: [OVN] Migrate "reside-on-redirect-chassis" for distributed FIP https://review.opendev.org/c/openstack/neutron/+/887469 | 12:43 |
| opendevreview | Graeme Moss proposed openstack/neutron stable/zed: [OVN] Migrate "reside-on-redirect-chassis" for distributed FIP https://review.opendev.org/c/openstack/neutron/+/887469 | 12:48 |
| ralonsoh | grami[m], you should first check https://review.opendev.org/c/openstack/neutron/+/878450 | 12:50 |
| ykarel | yeap ^ i was looking for, it's reimplemented differently now | 12:52 |
| ykarel | ralonsoh, i need to go out, so would be unavailable in today's meeting | 12:52 |
| ralonsoh | ykarel, no problem, thanks! | 12:52 |
| opendevreview | Merged openstack/neutron master: Don't allow deletion of the router ports without IP addresses https://review.opendev.org/c/openstack/neutron/+/886983 | 12:53 |
| opendevreview | yatin proposed openstack/neutron stable/2023.1: Disable pool recycle in tests https://review.opendev.org/c/openstack/neutron/+/887610 | 12:54 |
| grami[m] | Thanks :) yeah this is confusing. Because with a DVR and FIP I have to force the internal port from true to false. | 12:55 |
| opendevreview | yatin proposed openstack/neutron stable/zed: Disable pool recycle in tests https://review.opendev.org/c/openstack/neutron/+/887611 | 12:55 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2023.1: [OVN] Prevent Trunk creation/deletion with parent port bound https://review.opendev.org/c/openstack/neutron/+/887600 | 13:27 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/zed: [OVN] Prevent Trunk creation/deletion with parent port bound https://review.opendev.org/c/openstack/neutron/+/887601 | 13:29 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/yoga: [OVN] Prevent Trunk creation/deletion with parent port bound https://review.opendev.org/c/openstack/neutron/+/887603 | 13:32 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/xena: [OVN] Prevent Trunk creation/deletion with parent port bound https://review.opendev.org/c/openstack/neutron/+/887605 | 13:32 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/wallaby: [OVN] Prevent Trunk creation/deletion with parent port bound https://review.opendev.org/c/openstack/neutron/+/887606 | 13:50 |
| ralonsoh | Ping list: bcafarel, elvira, frickler, mlavalle, mtomaska, obondarev, slawek, tobias-urdin, ykarel, lajoskatona, jlibosva, averdagu, amotoki | 14:00 |
| ralonsoh | #startmeeting networking | 14:00 |
| opendevmeet | Meeting started Tue Jul 4 14:00:09 2023 UTC and is due to finish in 60 minutes. The chair is ralonsoh. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:00 |
| opendevmeet | The meeting name has been set to 'networking' | 14:00 |
| ralonsoh | hello all | 14:00 |
| obondarev | hi | 14:00 |
| elvira | hi o/ | 14:00 |
| ralonsoh | rubasov, slaweq, lajoskatona? | 14:01 |
| lajoskatona | o/ | 14:01 |
| bcafarel | o/ | 14:01 |
| rubasov | o/ | 14:01 |
| slaweq | o/ | 14:01 |
| slaweq | sorry for being late | 14:01 |
| ralonsoh | np | 14:01 |
| ralonsoh | ok, let's start (US folks are celebrating today) | 14:01 |
| ralonsoh | #topic annoucements | 14:02 |
| ralonsoh | #link https://releases.openstack.org/bobcat/schedule.html | 14:02 |
| ralonsoh | and finally we are in Bobcat-2 | 14:02 |
| ralonsoh | #link https://releases.openstack.org/bobcat/schedule.html#b-2 | 14:02 |
| ralonsoh | And we have the corresponding "releases" patches | 14:02 |
| ralonsoh | #link https://review.opendev.org/c/openstack/releases/+/887499 | 14:02 |
| ralonsoh | #link https://review.opendev.org/c/openstack/releases/+/887492 | 14:03 |
| ralonsoh | and today we have merged the requirements patch for n-lib 3.7.0 | 14:03 |
| lajoskatona | toooo much information :-) | 14:03 |
| ralonsoh | yeah hehehe (I have more...) | 14:03 |
| ralonsoh | related to these "releases" patches, there are other 2 that should have your attention | 14:04 |
| ralonsoh | the first ovn-bgp-agent release | 14:04 |
| ralonsoh | #link https://review.opendev.org/c/openstack/releases/+/887385 | 14:04 |
| lajoskatona | \o/ | 14:04 |
| ralonsoh | and the EOL of n-odl in Ussuri (because the CI is broken) | 14:04 |
| ralonsoh | #link https://review.opendev.org/c/openstack/releases/+/887152 | 14:04 |
| ralonsoh | so please, after this meeting check these ^^ patches and vote | 14:04 |
| ralonsoh | thanks! | 14:04 |
| ralonsoh | something else you want to add? | 14:05 |
| ralonsoh | ok, let's go for the bugs | 14:05 |
| ralonsoh | #topic bugs | 14:05 |
| ralonsoh | last week report is from haleyb | 14:06 |
| ralonsoh | #link https://lists.openstack.org/pipermail/openstack-discuss/2023-July/034314.html | 14:06 |
| ralonsoh | first of all, thanks for your collaboration opening and solving bugs in Neutron | 14:06 |
| ralonsoh | the report is full of them! but most of them already addressed | 14:06 |
| ralonsoh | so, again, thank you all! | 14:06 |
| ralonsoh | the only one that is not addressed is | 14:06 |
| ralonsoh | #link https://bugs.launchpad.net/neutron/+bug/2025410 | 14:07 |
| ralonsoh | haleyb, "reproduced" this issue but actually what we find in the qroute namespace is the stale arp entry | 14:08 |
| ralonsoh | and I think that's ok (??). Should we manually remove the arp entry? | 14:08 |
| ralonsoh | ok, next week I'll ping haleyb about this one, to have more feedback | 14:10 |
| slaweq | I'm not sure but I think we were adding manually some arp entries in dvr routers in the past | 14:10 |
| ralonsoh | he investigated this one and he could maybe provide a resolution | 14:10 |
| ralonsoh | let me check | 14:10 |
| ralonsoh | I only see that in DVR routers | 14:11 |
| obondarev | for FIPs, right? | 14:11 |
| ralonsoh | not really, these are the fixed_ips | 14:12 |
| slaweq | https://github.com/openstack/neutron/blob/master/neutron/agent/l3/dvr.py#L46 | 14:12 |
| ralonsoh | ^^ exactly | 14:12 |
| slaweq | it's like that even in master | 14:13 |
| obondarev | ah, I see | 14:13 |
| ralonsoh | what is the name of the dvr router namespace in the compute nodes? | 14:13 |
| slaweq | but maybe it's not really called in master anymore | 14:13 |
| slaweq | ralonsoh it's just qrouter-XXX | 14:13 |
| ralonsoh | ok yes, snat is for networker node | 14:14 |
| slaweq | yes | 14:14 |
| ralonsoh | ok, I'll check with Brian and I'll try to reproduce it too locally | 14:14 |
| slaweq | and for FIPs there is fip-<network_id> namespace also | 14:14 |
| slaweq | on computes | 14:14 |
| ralonsoh | anyway, we can continue the investigation offline | 14:15 |
| ralonsoh | do you have any other bug to bring to this section? | 14:16 |
| ralonsoh | this week amotoki is the deputy, next week will be mlavalle | 14:16 |
| ralonsoh | ok? | 14:16 |
| ralonsoh | I'll ping amotoki later today (or I'll send him a mail) | 14:17 |
| ralonsoh | next section | 14:17 |
| ralonsoh | #topic os-ken | 14:17 |
| ralonsoh | I think you already know we are moving away from storyboard | 14:17 |
| lajoskatona | good to hear | 14:18 |
| ralonsoh | my question is about the content of https://storyboard.openstack.org/#!/page/about | 14:18 |
| ralonsoh | and related to the second question: do we really need to open a launchpad site? | 14:18 |
| ralonsoh | or can we handle the os-ken bugs as n-t-p or n-lib ones, in the same Neutron site | 14:18 |
| ralonsoh | ? | 14:18 |
| lajoskatona | isn't neutron is good to collect these bugs? | 14:18 |
| lajoskatona | +1 | 14:19 |
| ralonsoh | I prefer this option ^ | 14:19 |
| slaweq | we can have os-ken tag | 14:19 |
| frickler | I'd say neutron is fine, just have a new tag | 14:19 |
| lajoskatona | sounds good with the tag | 14:19 |
| ralonsoh | exactly, same as [neutron-lib] | 14:19 |
| ralonsoh | so I think it is decided: I'll send a mail just to make this public | 14:20 |
| ralonsoh | thank you folks | 14:20 |
| lajoskatona | thanks, good plan | 14:20 |
| ralonsoh | along with that, I've realized that we have some (around 15 patches) pending from ryu to be backported... | 14:20 |
| ralonsoh | my bad... | 14:20 |
| ralonsoh | I'll start collecting them and opening the corresponding LP bugs | 14:20 |
| frickler | ralonsoh: what was your question about the about page? I didn't get that | 14:21 |
| ralonsoh | to open a https://bugs.launchpad.net/os-ken site | 14:21 |
| ralonsoh | but I prefer to have os-ken bugs in the Neutron bug list | 14:21 |
| frickler | ok, fine then | 14:22 |
| ralonsoh | ok, so I'll send the mail after this meeting | 14:22 |
| ralonsoh | next topic | 14:22 |
| ralonsoh | #topic specs | 14:22 |
| ralonsoh | all pending specs are merged now and we no longer accept new ones (although this is not strictly specified as in Nova project) | 14:23 |
| ralonsoh | I need to talk to TC folks to know how to create this milestone, same as Nova | 14:23 |
| ralonsoh | in any case, thank you all for your time reviewing | 14:23 |
| ralonsoh | next topic | 14:24 |
| ralonsoh | #topic community_goals | 14:24 |
| ralonsoh | 1) Consistent and Secure Default RBAC | 14:24 |
| ralonsoh | slaweq, please | 14:24 |
| slaweq | sure | 14:24 |
| slaweq | this week we merged https://review.opendev.org/c/openstack/neutron-lib/+/887191 | 14:24 |
| slaweq | which is needed to move on with service role patch in neutron | 14:25 |
| slaweq | but I just today realized that it probably breaks some UT https://bugs.launchpad.net/neutron/+bug/2025753 | 14:25 |
| slaweq | so I need to investigate it first before I will propose new neutron-lib release | 14:26 |
| slaweq | that's all from my side | 14:26 |
| ralonsoh | (+1 for this periodic job) | 14:26 |
| ralonsoh | thanks for the update! | 14:26 |
| ralonsoh | btw, is there a bug/etherpad/link? | 14:26 |
| slaweq | link for what? | 14:27 |
| ralonsoh | for documentation purposes only | 14:27 |
| ralonsoh | for the service role stuff | 14:27 |
| ralonsoh | maybe you can open a LP bug to track that | 14:27 |
| slaweq | sure | 14:28 |
| ralonsoh | perfect! and thanks | 14:28 |
| ralonsoh | the next section if | 14:28 |
| ralonsoh | is | 14:28 |
| ralonsoh | 2) Neutron client deprecation | 14:28 |
| ralonsoh | lajoskatona, sdk patches are merged | 14:28 |
| lajoskatona | The usual ehterpad: https://etherpad.opendev.org/p/python-neutronclient_deprecation | 14:28 |
| lajoskatona | yes one is still waiting for VPN: https://review.opendev.org/c/openstack/openstacksdk/+/886822 | 14:29 |
| ralonsoh | ah, I missed this one | 14:29 |
| lajoskatona | I will ping sdk folks with it | 14:29 |
| ralonsoh | is there a SDK patch in "releases"? | 14:29 |
| lajoskatona | other than that I started to work on sfc things | 14:30 |
| ralonsoh | just to know if they are going to release them soon | 14:30 |
| lajoskatona | I realized that api-ref is missing from n-lib: https://review.opendev.org/c/openstack/neutron-lib/+/887193 | 14:30 |
| ralonsoh | 1600 lines! | 14:31 |
| lajoskatona | and I pushed wip patches for sdk and neutronclient: https://review.opendev.org/c/openstack/openstacksdk/+/887387 & https://review.opendev.org/c/openstack/python-neutronclient/+/887388 | 14:31 |
| lajoskatona | api-ref is always a lot of chars :-) | 14:31 |
| lajoskatona | it was hidden in sfc repo, so mostly just copy-paste | 14:31 |
| ralonsoh | I'll add all these pending patches to the meeting agenda after this meeting | 14:32 |
| lajoskatona | ack | 14:32 |
| ralonsoh | lajoskatona, and again, thanks a lot for your work! | 14:32 |
| lajoskatona | From next week I will be on 2 weeks PTO, so no progress is expected :-) | 14:32 |
| ralonsoh | good to know! enjoy | 14:33 |
| ralonsoh | and the last topic for today | 14:33 |
| ralonsoh | #topic on_demand | 14:33 |
| ralonsoh | elvira, has one topic | 14:33 |
| elvira | yes :) | 14:33 |
| elvira | Right now there are parameters for certain operations that are not useful depending on the driver. I think it would be sensible to convey to the users that such parameters are not available for the driver they have when they try to use it. | 14:34 |
| elvira | for example: this arose with the network logging object | 14:34 |
| elvira | when you create it, there is a --target parameter that is not useful with the ML2/OVN driver, but it is useful with ML2/OVS | 14:34 |
| elvira | I think this might be the case for many other commands. Right now we just ignore parameters that are not expected. Should we take a more comprehensive approach somehow? | 14:35 |
| ralonsoh | how is the "target" parameter used in ML2/OVS? jsut asking | 14:36 |
| elvira | An approach I was suggested from our QE folks was to answer with an error through the API, explaining that the parameter is not supported | 14:36 |
| elvira | ralonsoh: in ml2/OVS you can target only 1 port, but in OVN you always log per security group | 14:37 |
| elvira | you cannot tighten the logging to just one port (well, maybe technically you could, by creating 1 security group only for that specific port) | 14:38 |
| ralonsoh | do you have a list of parameters with/without use per mech driver? | 14:38 |
| lajoskatona | this sounds like driver driven API :-) | 14:38 |
| ralonsoh | ^ right and "we don't do this in Neutron"... | 14:39 |
| elvira | lajoskatona: That is why I'm asking here, because maybe we could see if there is a way of doing this that is benefitial for the users of all drivers | 14:39 |
| lajoskatona | good idea, thans for coming with this topic here | 14:39 |
| ralonsoh | I think that will depend on the parameter. In this case where a parameter is useless for one mech driver, it could be documented | 14:39 |
| slaweq | one thing I'm affraid of is that we can expose to the regular user details about cloud internals, like backend used | 14:40 |
| slaweq | and I think we shouldn't do that for normal users | 14:40 |
| ralonsoh | ^ another reason too | 14:40 |
| slaweq | for admin only api this may be ok but not for regular user | 14:40 |
| elvira | ralonsoh: Another solution could be stating it in the CLI if a parameter is only useful for 1 driver | 14:41 |
| elvira | But I don't know how of a burden it is right now to change CLI messages | 14:41 |
| ralonsoh | exactly, that could be defined in the CLI help | 14:41 |
| ralonsoh | this is easy to be changed in the OSC repo | 14:41 |
| elvira | Isn't this on the deprecated Neutron CLI anymore? | 14:42 |
| ralonsoh | no no, not neutronclient | 14:42 |
| rubasov | I also don't see a problem with some drivers returning a "Not Implemented" error | 14:42 |
| ralonsoh | openstackclient | 14:42 |
| elvira | rubasov: Oh, that would keep the API driver-agnostic | 14:42 |
| ralonsoh | in this case that maybe (maybe) could be done by the logging manager, listing the loaded drivers | 14:43 |
| rubasov | yeah in that case nobody needs to know what driver is configured in neutron, just that this won't work here... | 14:43 |
| ralonsoh | elvira, do you have more parameters? or just this one? | 14:43 |
| slaweq | for qos we have admin api to list available rules and parameters supported by loaded drivers | 14:44 |
| elvira | ralonsoh: right now I only have this one, but talking with bcafarel he explained to me that this is the case for many parameters and the linuxbridge driver | 14:44 |
| slaweq | maybe we can do something similar for logging api, but for admin only | 14:44 |
| elvira | So that is mostly why I wanted to talk about it on a general perspective, just in case it was benefitial for more users | 14:45 |
| elvira | Anyway, I can also create a launchpad to continue this discussion | 14:45 |
| ralonsoh | elvira, I think we can't provide a general solution. That will depend on the parameter | 14:45 |
| elvira | if this sounds like a legit problem | 14:45 |
| ralonsoh | in this case you have several solution that could be applied in parallel: 1) the CLI help command, 2) the driver listing and 3) the logging manager rejecting the command knowing the loaded drivers (that are only OVS and OVN) | 14:46 |
| ralonsoh | so please, open a LP bug to track this and the solutions implemented | 14:46 |
| elvira | ralonsoh: ok! thanks a lot for the insights to everyone | 14:47 |
| elvira | I will open it :) | 14:47 |
| lajoskatona | +1 for the bug | 14:47 |
| ralonsoh | and that's all I have for today folks, anything else? | 14:48 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/2023.1: Don't allow deletion of the router ports without IP addresses https://review.opendev.org/c/openstack/neutron/+/887614 | 14:48 |
| ralonsoh | in about 10 mins we'll have the CI meeting in this channel | 14:48 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/zed: Don't allow deletion of the router ports without IP addresses https://review.opendev.org/c/openstack/neutron/+/887615 | 14:48 |
| ralonsoh | thank you all! | 14:48 |
| ralonsoh | #endmeeting | 14:48 |
| opendevmeet | Meeting ended Tue Jul 4 14:48:41 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:48 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/networking/2023/networking.2023-07-04-14.00.html | 14:48 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/networking/2023/networking.2023-07-04-14.00.txt | 14:48 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/networking/2023/networking.2023-07-04-14.00.log.html | 14:48 |
| lajoskatona | o/ | 14:48 |
| obondarev | o/ | 14:48 |
| rubasov | o/ | 14:48 |
| slaweq | o/ | 14:48 |
| elvira | o/ | 14:49 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/yoga: Don't allow deletion of the router ports without IP addresses https://review.opendev.org/c/openstack/neutron/+/887616 | 14:49 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/xena: Don't allow deletion of the router ports without IP addresses https://review.opendev.org/c/openstack/neutron/+/887617 | 14:49 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/wallaby: Don't allow deletion of the router ports without IP addresses https://review.opendev.org/c/openstack/neutron/+/887618 | 14:49 |
| lajoskatona | slaweq: do we have video meeting today? | 14:53 |
| slaweq | lajoskatona yes | 14:53 |
| lajoskatona | ack, thanks | 14:54 |
| slaweq | ping bcafarel, lajoskatona, mlavalle, mtomaska, ralonsoh, ykarel, jlibosva, elvira - CI meeting starting in 1 minute at https://meetpad.opendev.org/neutron-ci-meetings | 14:59 |
| slaweq | #startmeeting neutron_ci | 15:00 |
| opendevmeet | Meeting started Tue Jul 4 15:00:09 2023 UTC and is due to finish in 60 minutes. The chair is slaweq. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
| opendevmeet | The meeting name has been set to 'neutron_ci' | 15:00 |
| slaweq | #link https://meetpad.opendev.org/neutron-ci-meetings | 15:00 |
| slaweq | Grafana dashboard: https://grafana.opendev.org/d/f913631585/neutron-failure-rate?orgId=1 | 15:02 |
| slaweq | #topic Actions from previous meetings | 15:02 |
| slaweq | ralonsoh to remove networking-odl periodic jobs | 15:02 |
| slaweq | ralonsoh to check failed test_update_minimum_bandwidth_queue functional test | 15:03 |
| ralonsoh | #link https://review.opendev.org/c/openstack/neutron/+/887590 | 15:03 |
| slaweq | slaweq to move LB scenario job to periodic queue | 15:05 |
| slaweq | ykarel to send follow up patch for ipv6 job failure | 15:05 |
| slaweq | https://review.opendev.org/c/openstack/neutron/+/887142 | 15:06 |
| slaweq | #topic Stable branches | 15:06 |
| bcafarel | https://bugs.launchpad.net/neutron/+bug/2025486 | 15:06 |
| bcafarel | fix backports: https://review.opendev.org/q/Id1328d7cba418fa7c227ae9db4fe83c09fd06035 | 15:07 |
| ralonsoh | #link https://review.opendev.org/q/Id1328d7cba418fa7c227ae9db4fe83c09fd06035 | 15:07 |
| slaweq | #topic Stadium projects | 15:08 |
| lajoskatona | bagpipe fix: https://review.opendev.org/c/openstack/networking-bagpipe/+/887024 | 15:08 |
| slaweq | #topic Grafana | 15:09 |
| slaweq | https://grafana.opendev.org/d/f913631585/neutron-failure-rate?orgId=1 | 15:09 |
| slaweq | https://review.opendev.org/c/openstack/neutron/+/886961 | 15:14 |
| slaweq | https://review.opendev.org/c/openstack/neutron/+/880006 | 15:14 |
| slaweq | #action ralonsoh to try reduce concurency in functional job(s) | 15:19 |
| slaweq | #topic fullstack/functional | 15:20 |
| slaweq | https://ce11c0ee0a74ee8ebb7e-9c035ca54e1e355b36ad4d338836f375.ssl.cf2.rackcdn.com/884474/8/check/neutron-fullstack-with-uwsgi/ac36b87/testr_results.html | 15:20 |
| slaweq | #topic Tempest/Scenario | 15:21 |
| slaweq | https://9405e983985bd9b7eb91-5469ab4f5c2741453cb8b95135b2c449.ssl.cf1.rackcdn.com/886982/1/check/neutron-tempest-plugin-openvswitch-iptables_hybrid/2128985/testr_results.html | 15:21 |
| slaweq | https://ccc35cfa38f56032f297-95ab28bd06b01f2c7089eb38812248e0.ssl.cf2.rackcdn.com/860766/17/check/neutron-ovs-tempest-dvr-ha-multinode-full/8993e46/testr_results.html | 15:23 |
| slaweq | #topic Periodic | 15:25 |
| slaweq | openstack-tox-py310-with-neutron-lib-master | 15:25 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider master: Fix port for Load Balancer Health Check for FIP https://review.opendev.org/c/openstack/ovn-octavia-provider/+/886603 | 15:26 |
| slaweq | #endmeeting | 15:29 |
| opendevmeet | Meeting ended Tue Jul 4 15:29:33 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:29 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/neutron_ci/2023/neutron_ci.2023-07-04-15.00.html | 15:29 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/neutron_ci/2023/neutron_ci.2023-07-04-15.00.txt | 15:29 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/neutron_ci/2023/neutron_ci.2023-07-04-15.00.log.html | 15:29 |
| frickler | ralonsoh: regarding os-ken, I was thinking about a LP tag instead of a subject tag, since one can filter notification based on it, like e.g. l3-bgp and dns are currently used. not sure though if end users should care/know about that or if these tags are to be handled by the bug deputy? | 15:52 |
| ralonsoh | frickler, this tag will be added by the bug deputy | 15:55 |
| ralonsoh | but yes, we can add this tag too | 15:55 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2023.1: [FT] Move ``BaseOVSTestCase`` class to concurrency 1 executor https://review.opendev.org/c/openstack/neutron/+/887633 | 15:57 |
| opendevreview | Merged openstack/neutron master: [OVN] Prevent Trunk creation/deletion with parent port bound https://review.opendev.org/c/openstack/neutron/+/885154 | 16:01 |
| opendevreview | Merged openstack/neutron master: Ensure traffic is not centralized if DVR is enabled https://review.opendev.org/c/openstack/neutron/+/886626 | 16:01 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/yoga: [OVN] Prevent Trunk creation/deletion with parent port bound https://review.opendev.org/c/openstack/neutron/+/887603 | 16:20 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/xena: [OVN] Prevent Trunk creation/deletion with parent port bound https://review.opendev.org/c/openstack/neutron/+/887605 | 16:21 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/wallaby: [OVN] Prevent Trunk creation/deletion with parent port bound https://review.opendev.org/c/openstack/neutron/+/887606 | 16:25 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: WIP == Add a "port" child table "porthardwareoffloadtype" https://review.opendev.org/c/openstack/neutron/+/882832 | 16:40 |
| opendevreview | Merged openstack/networking-bagpipe master: [sqlalchemy-20]: remove subtransactions=True https://review.opendev.org/c/openstack/networking-bagpipe/+/887024 | 17:05 |
| opendevreview | Merged openstack/neutron master: [FT] Move ``BaseOVSTestCase`` class to concurrency 1 executor https://review.opendev.org/c/openstack/neutron/+/887590 | 19:34 |
| opendevreview | Merged openstack/neutron master: [ovn][ipv6] Add some more tests to skiplist https://review.opendev.org/c/openstack/neutron/+/887142 | 19:34 |
| opendevreview | Miguel Lavalle proposed openstack/neutron master: [PoC][DNM] Router flavors and service type for OVN https://review.opendev.org/c/openstack/neutron/+/883988 | 22:45 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!