| *** hjensas is now known as hjensas|afk | 00:04 | |
| opendevreview | Brian Haley proposed openstack/networking-ovn stable/train: Fix oslo.config duplicate option failure https://review.opendev.org/c/openstack/networking-ovn/+/796517 | 01:17 |
|---|---|---|
| opendevreview | Brian Haley proposed openstack/networking-ovn stable/train: Fix oslo.config duplicate option failure https://review.opendev.org/c/openstack/networking-ovn/+/796517 | 01:34 |
| opendevreview | liuyulong proposed openstack/neutron master: Bump neutron-lib to 2.12.0 https://review.opendev.org/c/openstack/neutron/+/796404 | 02:46 |
| opendevreview | liuyulong proposed openstack/neutron master: [QoS] Add rule type packet per second (pps) https://review.opendev.org/c/openstack/neutron/+/796363 | 02:46 |
| opendevreview | Yi Yang proposed openstack/os-ken master: Enhance os-ken to support Nicira PacketIn2 https://review.opendev.org/c/openstack/os-ken/+/795963 | 03:28 |
| opendevreview | Manu B proposed openstack/neutron-specs master: BGPaaS enhancements https://review.opendev.org/c/openstack/neutron-specs/+/783791 | 04:54 |
| opendevreview | Merged openstack/neutron master: Add support for OVN allow-stateless ACLs https://review.opendev.org/c/openstack/neutron/+/789974 | 05:17 |
| *** ramishra1 is now known as ramishra | 05:32 | |
| opendevreview | Manu B proposed openstack/neutron-specs master: BGPaaS enhancements https://review.opendev.org/c/openstack/neutron-specs/+/783791 | 05:41 |
| opendevreview | Manu B proposed openstack/neutron-specs master: BGPaaS enhancements https://review.opendev.org/c/openstack/neutron-specs/+/783791 | 05:42 |
| gibi | neutrinos: is there a good example how to backport neutron db schema migrations to stable branches with alembic? | 06:59 |
| slaweq | @gibi hi, we usually don't do that, at least not upstream | 06:59 |
| slaweq | but if we want to do it d/s we propose something like in https://review.opendev.org/c/openstack/neutron/+/601336/92/neutron/db/migration/alembic_migrations/versions/xena/expand/d863c3bdc0c5_add_active_allowed_address_pairs.py | 07:00 |
| slaweq | so we can backport it d/s and it will be able to handle later the case when table/column already exists | 07:00 |
| slaweq | is that something what You are asking for? | 07:00 |
| opendevreview | Yi Yang proposed openstack/os-ken master: Enhance os-ken to support Nicira PacketIn2 https://review.opendev.org/c/openstack/os-ken/+/795963 | 07:01 |
| gibi | slaweq: so basically you create a conditional migration on master to handle if the actual migration already happened in a previous release | 07:11 |
| gibi | due to downstream backporting of the migration | 07:11 |
| slaweq | gibi yes | 07:11 |
| slaweq | it won't hurt master at all, and helps us in d/s backport a lot | 07:11 |
| gibi | slaweq: thanks that is what I was looking for | 07:12 |
| slaweq | great :) | 07:12 |
| gibi | :) | 07:12 |
| *** rpittau|afk is now known as rpittau | 07:15 | |
| Zainub_Wahid | Hello, Yesterday, I asked about having an issue while deploying devstack. and you guys suggested me to set OVS as background instead of OVN/ML2. Do i need to change this in local.conf only? As, i tried setting Q_AGENT=Openvswitch | 07:41 |
| opendevreview | Yi Yang proposed openstack/os-ken master: Enhance os-ken to support Nicira PacketIn2 https://review.opendev.org/c/openstack/os-ken/+/795963 | 07:45 |
| ralonsoh | Zainub_Wahid, I think this is still valid | 07:53 |
| ralonsoh | https://gist.github.com/stephenfin/d1c000f7e52ff181c4673322b28eaf47#file-local-conf | 07:53 |
| Zainub_Wahid | Thank you | 07:54 |
| *** elodilles is now known as elodilles_afk | 08:10 | |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-tempest-plugin master: Update definition of the neutron-tempest-plugin-dvr-multinode-scenario https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/795492 | 08:11 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-tempest-plugin master: Enable tls-proxy in jobs where it was disabled https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/795929 | 08:12 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-tempest-plugin master: Add new scenario test for VIP address added as allowed addr pair https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/794841 | 08:12 |
| slaweq | ralonsoh lajoskatona hi, can You make another round of review of the patch https://review.opendev.org/c/openstack/neutron/+/601336 when You will have some time? | 08:14 |
| slaweq | thx in advance | 08:14 |
| ralonsoh | slaweq, sure | 08:17 |
| slaweq | thx a lot | 08:17 |
| *** gthiemon1e is now known as gthiemonge | 08:23 | |
| opendevreview | Yi Yang proposed openstack/os-ken master: Enhance os-ken to support Nicira PacketIn2 https://review.opendev.org/c/openstack/os-ken/+/795963 | 08:51 |
| opendevreview | Merged openstack/neutron-specs master: Remove Xena placeholder spec https://review.opendev.org/c/openstack/neutron-specs/+/781474 | 08:52 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-tempest-plugin master: Enforce new policy defaults and scopes in the API job https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/796612 | 09:19 |
| slaweq | ralonsoh lajoskatona can You check and approve https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/787718 ? thx in advance | 09:21 |
| ralonsoh | sure | 09:22 |
| opendevreview | Merged openstack/networking-ovn stable/stein: [EM releases] Move non-voting jobs to the experimental queue https://review.opendev.org/c/openstack/networking-ovn/+/796494 | 09:23 |
| opendevreview | Mamatisa Nurmatov proposed openstack/neutron master: use callback payloads for SECURITY_GROUP https://review.opendev.org/c/openstack/neutron/+/674044 | 09:33 |
| opendevreview | Hemanth N proposed openstack/neutron master: [OVN] neutron-ovn-metadat-agent add retry logic for sb_idl https://review.opendev.org/c/openstack/neutron/+/796613 | 09:33 |
| opendevreview | Slawek Kaplonski proposed openstack/networking-odl stable/stein: [EM releases] Move non-voting jobs to the experimental queue https://review.opendev.org/c/openstack/networking-odl/+/796496 | 09:48 |
| slaweq | lajoskatona ralonsoh can You also take a look at https://review.opendev.org/c/openstack/neutron/+/793141 and https://review.opendev.org/c/openstack/neutron/+/795979 ? thx | 09:55 |
| slaweq | ralonsoh and please also check https://review.opendev.org/c/openstack/neutron/+/778702 and https://review.opendev.org/c/openstack/neutron/+/778716 | 09:56 |
| slaweq | ralonsoh and https://review.opendev.org/c/openstack/neutron/+/777123 :) | 09:56 |
| slaweq | those last 3 are backports to stable releases, already reviewed by bcafarel :) | 09:58 |
| bcafarel | so, nothing for me to review then? :) | 10:05 |
| slaweq | bcafarel at least nothing from those patches :P | 10:06 |
| opendevreview | Merged openstack/neutron stable/wallaby: Provide the rpc_response_max_timeout parameter to sriov-agent https://review.opendev.org/c/openstack/neutron/+/796219 | 10:31 |
| opendevreview | Merged openstack/neutron stable/ussuri: Provide the rpc_response_max_timeout parameter to sriov-agent https://review.opendev.org/c/openstack/neutron/+/796221 | 10:31 |
| opendevreview | Kamil Sambor proposed openstack/neutron master: Add localport const and refactor https://review.opendev.org/c/openstack/neutron/+/796501 | 11:03 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: [Docs] Add info about vlan transparent networks' MTU https://review.opendev.org/c/openstack/neutron/+/796637 | 12:06 |
| opendevreview | Merged openstack/networking-ovn stable/train: Fix oslo.config duplicate option failure https://review.opendev.org/c/openstack/networking-ovn/+/796517 | 12:10 |
| opendevreview | Kamil Sambor proposed openstack/neutron master: Add localport const and refactor https://review.opendev.org/c/openstack/neutron/+/796501 | 12:51 |
| liuyulong | 1 | 12:59 |
| *** elodilles_afk is now known as elodilles | 13:04 | |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Remove ``ConfDriver`` code https://review.opendev.org/c/openstack/neutron/+/790999 | 13:18 |
| opendevreview | Bence Romsics proposed openstack/neutron-specs master: Allow multiple external gateways https://review.opendev.org/c/openstack/neutron-specs/+/779511 | 13:24 |
| liuyulong | hi there | 14:00 |
| liuyulong | #startmeeting neutron_l3 | 14:00 |
| opendevmeet | Meeting started Wed Jun 16 14:00:45 2021 UTC and is due to finish in 60 minutes. The chair is liuyulong. 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 'neutron_l3' | 14:00 |
| liuyulong | L3 meeting should be here? | 14:01 |
| haleyb | yes, and hi. i am in training so might be more lurking | 14:01 |
| lajoskatona | Hi | 14:02 |
| liuyulong | OK, cool, I think more Neutron developers should attend the meeting by default if they are in this channel. | 14:03 |
| liuyulong | #topic Announcements | 14:03 |
| liuyulong | The meeting boot does not response to the topic change? | 14:04 |
| liuyulong | OK, I see someone said that "#topic" not work now. | 14:05 |
| liuyulong | We change the meeting channel for L3 subteam, this is the only thing I want to mention in this section. | 14:06 |
| liuyulong | #link https://meetings.opendev.org/#Neutron_L3_Sub-team_Meeting | 14:06 |
| liuyulong | We have changed the IRC channels for all Neutron meetings to #openstack-neutron | 14:08 |
| lajoskatona | liuyulong: there are issues with chatbot, so perhaps this is just one of those | 14:08 |
| liuyulong | OK, no more announcements. Let's move on. | 14:08 |
| liuyulong | lajoskatona, I see the "topic issue" both in channel #openstack-meeting-alt and #openstack-nova. They just said the same thing there. | 14:10 |
| liuyulong | #topic Bugs | 14:10 |
| liuyulong | First one | 14:10 |
| liuyulong | #link https://bugs.launchpad.net/neutron/+bug/1926531 | 14:10 |
| liuyulong | I should say sorry, too many things bothered me that I forgot to submit the related fix. | 14:12 |
| liuyulong | This should be easy to fix, just check if there are floating IPs services by the DVR local router. If yes, do not run the veth pair device clean up action. | 14:13 |
| liuyulong | Next one | 14:15 |
| liuyulong | #link https://bugs.launchpad.net/neutron/+bug/1926045 | 14:15 |
| liuyulong | As I said before, such behaviour should be the feature. But the reporter re-open it, so let's wait what he/she will response. | 14:16 |
| liuyulong | Next one | 14:16 |
| liuyulong | #link https://bugs.launchpad.net/neutron/+bug/1930200 | 14:16 |
| liuyulong | "[RFE] Add support for Node-Local virtual IP" I guess, I should add the bug title here. | 14:17 |
| liuyulong | This RFE has been approved. I just left some comments there, the main concern from me is how the packet transmit on the datapath (flows). | 14:18 |
| liuyulong | if i understand correctly, when VMs are going to communit with a "Node-Local virtual IP". The destination IP address will be changed to another VM's fixed IP. | 14:22 |
| liuyulong | #link https://meetings.opendev.org/meetings/neutron_drivers/2021/neutron_drivers.2021-06-04-14.01.log.html#l-22 | 14:24 |
| liuyulong | ^^^ this is the discussion about this RFE. | 14:25 |
| liuyulong | They just mentioned "169.254.169.254", so my thougth is just based on that. | 14:26 |
| liuyulong | OK, last one | 14:27 |
| liuyulong | #link https://bugs.launchpad.net/neutron/+bug/1931953 | 14:27 |
| liuyulong | "[RFE] Openflow-based DVR L3" | 14:27 |
| liuyulong | Another flow based L3 request. | 14:27 |
| haleyb | wasn't there a proposal for this before that we decided to not complete? | 14:27 |
| liuyulong | We already have https://bugs.launchpad.net/neutron/+bug/1705536 and https://blueprints.launchpad.net/neutron/+spec/openflow-based-dvr | 14:28 |
| ralonsoh | this is an old RFE, there are some old patches | 14:28 |
| liuyulong | #link https://bugs.launchpad.net/neutron/+bug/1931953/comments/1 | 14:28 |
| haleyb | right, and we said let's use OVN instead | 14:28 |
| liuyulong | Yes, I mentioned to the reporter about all these information. | 14:28 |
| ralonsoh | IMO this should be discussed in the drivers meeting | 14:28 |
| lajoskatona | agree | 14:29 |
| ralonsoh | I'll add the topic | 14:29 |
| liuyulong | Actually I would like to see if python based neutron agents can achive a higher performance by using ovs flows (dpdk) for L3 functionalities. | 14:31 |
| ralonsoh | for sure it will | 14:31 |
| ralonsoh | but the problem is the complexity of the code, as I remember when I was helping David on this | 14:31 |
| ralonsoh | and all the corner cases | 14:32 |
| liuyulong | And, for python neutron agents, such solution should be easily to upgrade for existing cloud. | 14:32 |
| liuyulong | ralonsoh, yes, DVR is really complicated, too many cases should be covered. | 14:33 |
| liuyulong | From my personal experiences, Neutron L3 functionalities are now based on kernel network stack (datapath) which can not meet increasing performance requirements in real production cloud. | 14:35 |
| ralonsoh | yeah, this is a know problem. For sure offloading the processing to OF rules could speed up that | 14:36 |
| *** gthiemon1e is now known as gthiemonge | 14:37 | |
| ralonsoh | of course, the complexity of this change could be huge | 14:37 |
| ralonsoh | and I don't know, now having OVN, if that could be relevant | 14:37 |
| liuyulong | But current implementation has a very very advanced alternatives is to translate the iptables to eBPF which is in userspace with high performance. | 14:38 |
| ralonsoh | well, all kernel filtering has been merged in one single tool, nft | 14:39 |
| liuyulong | New kernel supports that, but it is very experimental. | 14:39 |
| ralonsoh | I don't know if going through this path has any benefit | 14:39 |
| ralonsoh | instead of spending time on nft | 14:39 |
| liuyulong | There are some works is to offload iptables rules to BPF program transparently. So maybe we just use the iptables to set the rules as we do now, then everything done by the kernel In an ideal state. :) | 14:42 |
| liuyulong | ralonsoh, can OVN use ovs-DPDK to accelerate packet trasimit rate? | 14:43 |
| ralonsoh | for sure and HW offload | 14:44 |
| liuyulong | #link https://docs.openstack.org/networking-ovn/latest/admin/dpdk.html | 14:46 |
| ralonsoh | we can discuss about this offline | 14:46 |
| liuyulong | Maybe we can write something about the performance increment in this doc. | 14:46 |
| ralonsoh | I think we have better documentation | 14:46 |
| ralonsoh | *updated | 14:46 |
| liuyulong | #topic On demand agenda | 14:47 |
| liuyulong | OK, we still have sometime. | 14:47 |
| liuyulong | Firstly, I'd like to request the team to review the code https://review.opendev.org/q/topic:%22bp%252Fdistributed-dhcp-for-ml2-ovs%22+(status:open%20OR%20status:merged) | 14:48 |
| ralonsoh | I'll add them to my pile | 14:48 |
| liuyulong | It almost done. Then we can say someday the DHCP agent can get retired. | 14:48 |
| lajoskatona | Good question to have job for this later in CI | 14:49 |
| liuyulong | So... I have another request about the metadata-agent, I want to make it distributed. | 14:49 |
| liuyulong | And someday make metadata-agent retired... | 14:49 |
| liuyulong | Since it is single function, just used during VM booting time mostly. | 14:51 |
| liuyulong | So, in the foreseeable future, neutron will only run ovs-agent and L3-agent. If we can get higher datapath performance by ovs-based (securigy group, L3 and so on). | 14:52 |
| liuyulong | Then another thing about Neutron we should take care should be the control plane performace. | 14:54 |
| liuyulong | Recently I was thinking a problem about can we create 100/200/300K VMs into one single network. Can Neutron L2 pop handle that? Can OVN handle that? | 14:57 |
| ralonsoh | not at all | 14:57 |
| liuyulong | OK, just throw out some ideas, and hope everyone can think about it. We still have a long way to go. :) | 14:59 |
| liuyulong | Alright, time is up. | 15:00 |
| liuyulong | Thank you guys. | 15:00 |
| ralonsoh | bye | 15:00 |
| liuyulong | #endmeeting | 15:00 |
| opendevmeet | Meeting ended Wed Jun 16 15:00:15 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:00 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/neutron_l3/2021/neutron_l3.2021-06-16-14.00.html | 15:00 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/neutron_l3/2021/neutron_l3.2021-06-16-14.00.txt | 15:00 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/neutron_l3/2021/neutron_l3.2021-06-16-14.00.log.html | 15:00 |
| lajoskatona | Bye | 15:00 |
| liuyulong | ok, this action is working. LOL | 15:00 |
| liuyulong | Bye | 15:00 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: New API test for subnet's service types https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/787718 | 15:05 |
| opendevreview | Merged openstack/neutron stable/victoria: Remove tempest job with neutron-lib master from CI https://review.opendev.org/c/openstack/neutron/+/778702 | 15:05 |
| opendevreview | Merged openstack/neutron stable/train: Disable not used services in the tempest and rally jobs https://review.opendev.org/c/openstack/neutron/+/778716 | 15:05 |
| opendevreview | Merged openstack/neutron stable/rocky: Don't configure dnsmasq entries for "network" ports https://review.opendev.org/c/openstack/neutron/+/777123 | 15:06 |
| opendevreview | Merged openstack/neutron stable/victoria: Added common config and SR-IOV agent config to sanity check https://review.opendev.org/c/openstack/neutron/+/796033 | 15:06 |
| *** rpittau is now known as rpittau|afk | 16:11 | |
| opendevreview | Merged openstack/neutron stable/victoria: Provide the rpc_response_max_timeout parameter to sriov-agent https://review.opendev.org/c/openstack/neutron/+/796220 | 16:17 |
| opendevreview | Merged openstack/neutron stable/train: Provide the rpc_response_max_timeout parameter to sriov-agent https://review.opendev.org/c/openstack/neutron/+/796222 | 16:42 |
| *** ricolin_ is now known as ricolin | 17:49 | |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Remove ``ConfDriver`` code https://review.opendev.org/c/openstack/neutron/+/790999 | 17:49 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Populate self.floating_ips_dict using "ip rule" information https://review.opendev.org/c/openstack/neutron/+/794604 | 18:12 |
| opendevreview | Brian Haley proposed openstack/networking-ovn stable/train: Revert "Make networking-ovn-octavia-v2-dsvm-scenario non-voting" https://review.opendev.org/c/openstack/networking-ovn/+/796660 | 18:41 |
| opendevreview | Brian Haley proposed openstack/networking-ovn stable/train: Revert "Make networking-ovn-octavia-v2-dsvm-scenario non-voting" https://review.opendev.org/c/openstack/networking-ovn/+/796660 | 18:48 |
| opendevreview | Brian Haley proposed openstack/networking-ovn stable/train: Add Health Monitor support https://review.opendev.org/c/openstack/networking-ovn/+/796063 | 19:05 |
| opendevreview | Merged openstack/neutron master: Add CONTEXT_READER decorator to "get_ha_router_port_bindings" https://review.opendev.org/c/openstack/neutron/+/794236 | 19:38 |
| opendevreview | Merged openstack/neutron master: Remove the devstack/lib/ovs in favour of "ovs_source" from DevStack https://review.opendev.org/c/openstack/neutron/+/793470 | 23:39 |
| opendevreview | Brian Haley proposed openstack/ovn-octavia-provider master: Support creating members without a subnet ID https://review.opendev.org/c/openstack/ovn-octavia-provider/+/795896 | 23:53 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!