| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [eventlet-removal] Remove the usage of eventlet in the Neutron API (2) https://review.opendev.org/c/openstack/neutron/+/941202 | 06:33 |
|---|---|---|
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: DNM - Test "neutron-ovn-tempest-ipv6-only-ovs*" with WSGI https://review.opendev.org/c/openstack/neutron/+/939347 | 06:34 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [eventlet-deprecation] Add a warning message after monkey_patched https://review.opendev.org/c/openstack/neutron/+/941207 | 07:17 |
| sahid | lajoskatona: o/ if you have a moment can you try to repick all the patches related to ovs and see if you have still the issue? | 07:30 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [eventlet-removal] Remove "logger" mechanism from ML2/OVN CI jobs https://review.opendev.org/c/openstack/neutron/+/941211 | 08:09 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-tempest-plugin master: [eventlet-removal] Remove "logger" mechanism from ML2/OVN CI jobs https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/941212 | 08:10 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [eventlet-removal] Remove the usage of eventlet in the Neutron API (2) https://review.opendev.org/c/openstack/neutron/+/941202 | 08:10 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: DNM - Test "neutron-ovn-tempest-ipv6-only-ovs*" with WSGI https://review.opendev.org/c/openstack/neutron/+/939347 | 08:10 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [eventlet-removal] Remove "logger" mechanism from ML2/OVN CI jobs https://review.opendev.org/c/openstack/neutron/+/941211 | 08:33 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [eventlet-removal] Remove the usage of eventlet in the Neutron API (2) https://review.opendev.org/c/openstack/neutron/+/941202 | 08:33 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: DNM - Test "neutron-ovn-tempest-ipv6-only-ovs*" with WSGI https://review.opendev.org/c/openstack/neutron/+/939347 | 08:33 |
| ralonsoh | ykarel, haleyb please check these 2 patches | 08:35 |
| ralonsoh | * https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/941212 | 08:35 |
| ralonsoh | * https://review.opendev.org/c/openstack/neutron/+/941211 | 08:35 |
| ralonsoh | thanks! | 08:35 |
| ralonsoh | ^ logger, that is a test mech driver, is still calling monkey_patch | 08:35 |
| ralonsoh | and is breaking everything... | 08:35 |
| ralonsoh | otherwiseguy, tested yesterday just commenting the monkey_patch method and everything worked fine | 08:36 |
| ralonsoh | so I think I've lost 1 month trying to re-implement the hash ring manager without any need | 08:36 |
| lajoskatona | sahid: looks ok, let me check in detail your patches, really promising in devstack | 08:39 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: DNM - Test "neutron-ovn-tempest-ipv6-only-ovs*" with WSGI https://review.opendev.org/c/openstack/neutron/+/939347 | 08:57 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Change ``PeriodicWorker`` executor to ``ThreadPoolExecutor`` https://review.opendev.org/c/openstack/neutron/+/940707 | 08:57 |
| sahid | lajoskatona: thanks ! | 09:04 |
| ykarel | ralonsoh, ack | 09:19 |
| ykarel | so that logger not needed? or that has to be also fixed? | 09:20 |
| ralonsoh | ykarel, it will be fixed once eventlet is removed from the tests | 09:20 |
| ralonsoh | but makes no sense to use it with ovn | 09:20 |
| ralonsoh | this is not used with ovs, for example | 09:20 |
| opendevreview | Lajos Katona proposed openstack/neutron-tempest-plugin master: Bagpipe: make neutron-tempest-plugin-bgpvpn-bagpipe non-voting https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/941218 | 09:22 |
| sahid | lajoskatona: i have rebuilt my devstack and suffering the same issue than you, agent is running but not registered | 09:24 |
| lajoskatona | I rebuilt the devstack with ./unstack.sh; ./stack.sh, with your latest patches, strange.... | 09:26 |
| sahid | I have noticed this error into the log, you may have the same: Feb 11 10:22:46 devstack01 neutron-openvswitch-agent[1570367]: ERROR os_ken.controller.controller [-] Error in the datapath 00002607f5684142 from ('127.0.0.1', 59416) | 09:27 |
| ykarel | ralonsoh, ack not sure why not used with ovs, is there any reason for that it was used only with ovn? | 09:27 |
| ykarel | looks added very long back | 09:27 |
| ralonsoh | ykarel, no idea, that's the problem | 09:27 |
| ralonsoh | but why using it? | 09:27 |
| ykarel | that's what /me trying to understand before removing it | 09:30 |
| ykarel | somehow i recall this catched some issues in past | 09:31 |
| ralonsoh_ | but right now, we can't use logger because if calling eventlet | 09:32 |
| ykarel | yes that's fine to remove but just we should add context and have it's coverage back later | 09:33 |
| ykarel | i mean fine to remove to fix CI | 09:33 |
| ralonsoh_ | ykarel, I'll update the patches to add this comment | 09:34 |
| opendevreview | Vladimir Prokofev proposed openstack/neutron-specs master: Add spec for random-fully per-FIP feature RFE. https://review.opendev.org/c/openstack/neutron-specs/+/932650 | 09:34 |
| ykarel | k thx will check post lunch | 09:35 |
| opendevreview | Takashi Kajinami proposed openstack/neutron-fwaas master: Use common constants from neutron-lib for agent_mode https://review.opendev.org/c/openstack/neutron-fwaas/+/941220 | 09:44 |
| opendevreview | Takashi Kajinami proposed openstack/tap-as-a-service master: Prohibit installation in too old python versions https://review.opendev.org/c/openstack/tap-as-a-service/+/941221 | 09:46 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Create a deterministic hash ring node UUID generator https://review.opendev.org/c/openstack/neutron/+/941008 | 10:07 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-tempest-plugin master: [eventlet-removal] Remove "logger" mechanism from ML2/OVN CI jobs https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/941212 | 10:13 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [eventlet-removal] Remove "logger" mechanism from ML2/OVN CI jobs https://review.opendev.org/c/openstack/neutron/+/941211 | 10:13 |
| opendevreview | Rodolfo Alonso proposed x/whitebox-neutron-tempest-plugin master: [eventlet-removal] Remove "logger" mechanism from ML2/OVN CI jobs https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/941224 | 10:14 |
| opendevreview | Rodolfo Alonso proposed openstack/ovn-octavia-provider master: [eventlet-removal] Remove "logger" mechanism from ML2/OVN CI jobs https://review.opendev.org/c/openstack/ovn-octavia-provider/+/941225 | 10:17 |
| opendevreview | Rodolfo Alonso proposed openstack/ovn-octavia-provider master: Ignore reno artefacts (RELEASENOTES.rst and reno.cache) https://review.opendev.org/c/openstack/ovn-octavia-provider/+/941226 | 10:18 |
| ralonsoh_ | ykarel, check this: https://review.opendev.org/c/openstack/neutron/+/939347 (this testing patch is manually removing the logger mech driver from this job) | 10:21 |
| ralonsoh_ | 17 jobs finished, all OK (3 still running) | 10:21 |
| *** ralonsoh_ is now known as ralonsoh | 10:21 | |
| opendevreview | Merged openstack/neutron stable/2024.2: Catch when the process does not exist when killing it https://review.opendev.org/c/openstack/neutron/+/941105 | 10:25 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/os-ken master: hub: implement threads related functions https://review.opendev.org/c/openstack/os-ken/+/938673 | 10:30 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/os-ken master: hub: add equivalents for eventlet functions using native https://review.opendev.org/c/openstack/os-ken/+/938875 | 10:30 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/os-ken master: hub: implement listen as native https://review.opendev.org/c/openstack/os-ken/+/938876 | 10:30 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/os-ken master: hub: use native as default implementation https://review.opendev.org/c/openstack/os-ken/+/938337 | 10:30 |
| bbezak | Hi, small update on bug I mentioned yesterday - for ml2/ovs users - adding back no_track keepalived flag fixes the L3 HA failover on FIP detach - https://bugs.launchpad.net/neutron/+bug/2097770 | 10:55 |
| ykarel | ralonsoh, ok good | 10:56 |
| opendevreview | yatin proposed openstack/neutron-tempest-plugin master: [DNM] Check n-d-r workaround with logger mech_driver removed https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/941230 | 11:08 |
| ykarel | ralonsoh, also triggered ^ to confirm all good with dynamic routing job with logger removed | 11:08 |
| ralonsoh | checking that | 11:09 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/os-ken master: hub: implement threads related functions https://review.opendev.org/c/openstack/os-ken/+/938673 | 11:16 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/os-ken master: hub: add equivalents for eventlet functions using native https://review.opendev.org/c/openstack/os-ken/+/938875 | 11:16 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/os-ken master: hub: implement listen as native https://review.opendev.org/c/openstack/os-ken/+/938876 | 11:16 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/os-ken master: hub: use native as default implementation https://review.opendev.org/c/openstack/os-ken/+/938337 | 11:16 |
| ralonsoh | bbezak, sorry, what does it mean? The default value is true | 11:18 |
| ralonsoh | what version of keepalived are you using? | 11:19 |
| ralonsoh | bbezak, can you post in the LP bug the differences between in the keepalived config? | 11:25 |
| bbezak | this is keepalived v2.2.8 built with Rocky Linux 9.5 kolla caracal. also confirmed on antelope RL 9.4 | 11:30 |
| bbezak | sure will do | 11:30 |
| bbezak | ralonsoh: https://bugs.launchpad.net/neutron/+bug/2097770/comments/4 | 12:07 |
| ralonsoh | bbezak, I don't think this is a matter of reverting both patches | 12:13 |
| ralonsoh | but this: https://review.opendev.org/c/openstack/neutron/+/869013/5/neutron/agent/linux/keepalived.py#227 | 12:13 |
| ralonsoh | in any case, I need to recheck the code again | 12:14 |
| bbezak | yes, this removes no_track effectively | 12:14 |
| ralonsoh | I think this change in L227 is not necessary (actually is inverting the logic) | 12:14 |
| bbezak | thank you | 12:14 |
| bbezak | however by the commit message of that change - no track flag removal was on purpose | 12:20 |
| bbezak | maybe assumption was wrong | 12:20 |
| opendevreview | Merged openstack/neutron master: Use centralized constants for [ovn] neutron_sync_mode https://review.opendev.org/c/openstack/neutron/+/941018 | 13:20 |
| ralonsoh | bbezak, this is to track the interfaces but not the VIPs | 13:35 |
| ykarel | ralonsoh, n-d-r one failed even removing logger , 5/10 failed https://zuul.opendev.org/t/openstack/buildset/43a33fd35a6a472da8e903ecd494a06a so there some another issue | 13:35 |
| ralonsoh | ykarel, yes, there is indeed another issue | 13:36 |
| ralonsoh | something related to the router interface removal | 13:36 |
| ralonsoh | but this is not happening in non n-d-r jobs | 13:36 |
| ralonsoh | I'll continue investigating that | 13:36 |
| ralonsoh | but all the patches related to the removal of eventlet in the API are legit | 13:37 |
| ralonsoh | I' | 13:37 |
| ralonsoh | I'll comment that today in the team meeting | 13:37 |
| ykarel | ack, so those hash rings ones also still valid? i think you mentioned above those not needed with logger removal | 13:37 |
| ralonsoh | I'll send today the list of patches | 13:38 |
| ralonsoh | one sec | 13:38 |
| ralonsoh | https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/941224 | 13:38 |
| ralonsoh | https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/941212 | 13:38 |
| ralonsoh | https://review.opendev.org/c/openstack/neutron/+/941211 | 13:38 |
| ralonsoh | https://review.opendev.org/c/openstack/neutron/+/941202 | 13:38 |
| ralonsoh | https://review.opendev.org/c/openstack/neutron/+/940707 | 13:38 |
| ralonsoh | I had it prepared for today's meeting | 13:38 |
| ralonsoh | I've -W the ones related to the hash ring change | 13:38 |
| ykarel | +1 thx | 13:39 |
| ykarel | ralonsoh, in 941202 i see some failures in ovn jobs | 13:40 |
| ykarel | Error while executing command: HttpException: 500, : 500 Internal Server Error: The server has either erred or is incapable of performing the requested operation. | 13:40 |
| ralonsoh | yes, I saw it, I'm still investigating it | 13:41 |
| bbezak | ralonsoh: right. so FIPs shouldn't be tracked as well - cause when are they detached failover occurs | 13:41 |
| ykarel | ack i recall we had a lp for it long back | 13:41 |
| bbezak | * ralonsoh: right. so FIPs shouldn't be tracked as well - cause when they are detached failover occurs | 13:41 |
| ralonsoh | bbezak, FIPs are the IP addresses in "virtual_ipaddress_excluded" section | 13:44 |
| bbezak | yeah | 13:44 |
| bbezak | still, no_track added there helps | 13:46 |
| bbezak | https://github.com/acassen/keepalived/blob/c73d1bb109a382ac6cf4df82ca4ec2fef67398e3/doc/man/man5/keepalived.conf.5.in#L1992-L2000 | 13:49 |
| bbezak | according to that we need no_track surely. as FIP got removed, keepalived by design will do the transition | 13:50 |
| haleyb | #startmeeting networking | 14:00 |
| opendevmeet | Meeting started Tue Feb 11 14:00:38 2025 UTC and is due to finish in 60 minutes. The chair is haleyb. 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 |
| haleyb | Ping list: bcafarel, elvira, frickler, mlavalle, mtomaska, obondarev, slaweq, tobias-urdin, ykarel, lajoskatona, jlibosva, averdagu, haleyb, ralonsoh | 14:00 |
| mlavalle | \o | 14:00 |
| obondarev | o/ | 14:00 |
| ralonsoh_ | hello | 14:00 |
| ykarel | o/ | 14:00 |
| mblue_ | o/ | 14:00 |
| *** ralonsoh_ is now known as ralonsoh | 14:00 | |
| bcafarel | o/ | 14:01 |
| sahid | o/ | 14:01 |
| elvira | o/ | 14:01 |
| haleyb | #announcements | 14:01 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [L3] Do not track the fixed IP / FIPs in the keepalived configuration https://review.opendev.org/c/openstack/neutron/+/941247 | 14:01 |
| haleyb | We are in week R-7 of Epoxy | 14:01 |
| frickler | o/ | 14:01 |
| haleyb | E-3 milestone at week R-5 (Feb 24) | 14:01 |
| lajoskatona | o/ | 14:02 |
| haleyb | #link https://releases.openstack.org/epoxy/schedule.html | 14:02 |
| haleyb | So about 2 weeks until feature freeze | 14:02 |
| haleyb | Reminder: If you have a topic for the drivers meeting on Friday, please add it to the wiki @ https://wiki.openstack.org/wiki/Meetings/NeutronDrivers | 14:03 |
| haleyb | And of course a reminder to use the priorities dashboard for patches in the "ready to merge" state | 14:03 |
| haleyb | Please put reviews there or ping people here | 14:03 |
| haleyb | We previously merged change to remove linuxbridge driver, i've been trying to merge the dibbler removal patch this week but so far the gate has said 'no', will try again after meeting | 14:04 |
| haleyb | was there any progress fixing things that broke due to linuxbridge? | 14:05 |
| haleyb | i also proposed n-t-p and devstack changes yesterday | 14:05 |
| lajoskatona | som with bagpipe | 14:06 |
| haleyb | #link https://review.opendev.org/q/topic:%22remove-linuxbridge-agent%22 | 14:06 |
| lajoskatona | https://review.opendev.org/c/openstack/networking-bagpipe/+/940765 | 14:06 |
| lajoskatona | I will change the topic to have all under the same | 14:06 |
| haleyb | lajoskatona: thanks, i see that is needed for mine | 14:06 |
| lajoskatona | I pushed another one to make the job non-voting temporary | 14:07 |
| haleyb | +1 | 14:07 |
| lajoskatona | https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/941218 | 14:08 |
| haleyb | last announcement is it's PTL/TC election season for 2025.2 cycle, please read the ML email for further information | 14:08 |
| haleyb | i have no other announcements, anyone else? | 14:09 |
| lajoskatona | today will be a meeting with TC and foundation I suppose | 14:10 |
| lajoskatona | https://lists.openinfra.org/archives/list/foundation@lists.openinfra.org/thread/3B7OWPRXB4KD2DVX7SYYSHYYRNCKVV46/ | 14:10 |
| lajoskatona | https://board.openinfra.org/strategic-consideration | 14:10 |
| lajoskatona | about merging to/with Linuxfoundation | 14:10 |
| lajoskatona | there will be another one this week for other timezones, today is 1600UTC | 14:11 |
| haleyb | oh, yes, right after this meeting, so i will make sure to end on-time | 14:11 |
| haleyb | hopefully everyone saw that email and responses | 14:12 |
| haleyb | moving on | 14:12 |
| haleyb | #topic bugs | 14:12 |
| haleyb | bcafarel was the deputy last week, his report is at | 14:13 |
| haleyb | #link https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/ST54KVP3RCC5XOZSWBHJ65M3Y3EZFME4/ | 14:13 |
| haleyb | the two critical have patches proposed/merged, thanks for that | 14:13 |
| haleyb | as do the two high ones | 14:14 |
| haleyb | and 2/3 of medium have patches as well | 14:14 |
| cardoe | I was hoping I could get a +1 for https://review.opendev.org/c/openstack/python-openstackclient/+/940595 which is a fix for routers. | 14:15 |
| haleyb | and ralonsoh picked-up the eventlet-removal one, thanks | 14:15 |
| haleyb | #link https://bugs.launchpad.net/neutron/+bug/2097257 | 14:15 |
| haleyb | [eventlet-removal] Use ``ThreadPoolExecutor`` for ``periodics.PeriodicWorker`` | 14:15 |
| ralonsoh | ^ proposed by otherwiseguy | 14:15 |
| lajoskatona | cardoe: will check it, thanks | 14:16 |
| otherwiseguy | ralonsoh: on that one, I don't know if we need it or not. things seemed to work without it after not monkey patching | 14:16 |
| haleyb | ack, it didn't have a patch linked in it but i do remember one now that you mention it | 14:16 |
| ralonsoh | otherwiseguy, we'll see that in the CI, checking the periodicity of the method call | 14:17 |
| ralonsoh | as you did before | 14:17 |
| otherwiseguy | not sure why the patch changed the frequency of periodic calls while eventlet was still monkeypatching--but only when the api was active. was weird. | 14:18 |
| otherwiseguy | probably a red herring | 14:18 |
| haleyb | ok, i will let you two decide on the fate of any change there after testing | 14:19 |
| haleyb | last one was on an older release | 14:20 |
| haleyb | #link https://bugs.launchpad.net/neutron/+bug/2097610 | 14:20 |
| haleyb | i will add a note about reproducing on somethine 2023.2 or later | 14:20 |
| haleyb | any other bugs to discuss? | 14:22 |
| haleyb | deputy this week is elvira, next week is slaweq - that good with both of you? | 14:22 |
| elvira | sure! | 14:22 |
| ralonsoh | slaweq, is on PTO | 14:22 |
| ralonsoh | I'll ping him next day | 14:22 |
| haleyb | ack, thanks | 14:23 |
| haleyb | #topic community goals | 14:24 |
| haleyb | lajoskatona: neutronclient update update | 14:24 |
| haleyb | #link https://review.opendev.org/q/topic:%22bug/1999774%22+status:open | 14:24 |
| lajoskatona | no news this week | 14:24 |
| lajoskatona | the patches from last week under review, but I had no time to work on this | 14:25 |
| haleyb | ack, they all at least seem to be green and have reviews | 14:25 |
| lajoskatona | yes quite good attention from Horizon team | 14:26 |
| haleyb | and last is eventlet | 14:26 |
| ralonsoh | yes I have a list of patches | 14:26 |
| ralonsoh | https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/941224 | 14:26 |
| ralonsoh | https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/941212 | 14:26 |
| ralonsoh | https://review.opendev.org/c/openstack/neutron/+/941211 | 14:26 |
| ralonsoh | https://review.opendev.org/c/openstack/neutron/+/941202 | 14:26 |
| ralonsoh | https://review.opendev.org/c/openstack/neutron/+/940707 | 14:26 |
| ralonsoh | that should be enough to remove eventlet from the API | 14:27 |
| ralonsoh | NOTE: | 14:27 |
| lajoskatona | sahid also works on ovs-agent patches: https://review.opendev.org/q/topic:%22bug/2087939%22 | 14:27 |
| ralonsoh | I'm removing logger from the ML2/OVN jobs because (1) this is calling monkey_patch(), (2) this is a testing mech driver class, (3) this is not used in the jobs and (4) no user will call it in a production env | 14:28 |
| lajoskatona | I try to follow these and help where I can | 14:28 |
| ralonsoh | yes, OVS patches are the second part | 14:28 |
| ralonsoh | but is there a CI job running without eventlet? | 14:28 |
| sahid | ralonsoh: yes | 14:29 |
| sahid | the last patch | 14:29 |
| ralonsoh | https://review.opendev.org/c/openstack/neutron/+/937765 | 14:29 |
| sahid | https://review.opendev.org/c/openstack/neutron/+/937765 | 14:29 |
| sahid | yes | 14:29 |
| ralonsoh | the CI is not passing | 14:29 |
| sahid | still working on it | 14:29 |
| sahid | all the previous are passing | 14:29 |
| ralonsoh | ok, maybe we can merge the os-ken ones | 14:29 |
| ralonsoh | and release a new version | 14:30 |
| sahid | i think that will help to make progress | 14:30 |
| ralonsoh | ok, let's focus on them. If there are further errors, we can release minor versions | 14:30 |
| haleyb | ok, so start at this one | 14:31 |
| haleyb | #link https://review.opendev.org/c/openstack/os-ken/+/938326 | 14:31 |
| lajoskatona | +1 | 14:31 |
| sahid | thank you | 14:31 |
| haleyb | there is a relation change there | 14:31 |
| haleyb | sahid: when those are done can you propose os-ken release? or ping me if you need any help | 14:31 |
| sahid | sure I will | 14:32 |
| sahid | and if needed will ping you aswell | 14:32 |
| sahid | thank you | 14:32 |
| haleyb | ack, thanks for all your work (and rodolfo and everyone else) | 14:33 |
| haleyb | #topic on-demand | 14:33 |
| haleyb | any other topics? | 14:33 |
| haleyb | nothing on the wiki | 14:34 |
| lajoskatona | nothing from me | 14:34 |
| haleyb | ok, if nothing else have a good week, lets get some of these eventlet changes merged :) | 14:35 |
| mlavalle | not from me either | 14:35 |
| MengyangZhang[m] | I just a quick question, would the neutron community be interested in supporting applying QoS changes live to VMs | 14:35 |
| ralonsoh | this is already supported | 14:35 |
| ralonsoh | you can change the QoS settings of a port that is attached | 14:35 |
| cardoe | I did have some questions. | 14:36 |
| MengyangZhang[m] | okk, is there a doc for this, want to learn more about it | 14:36 |
| cardoe | I'm an operator looking to use Neutron for managing our bare metal infra. | 14:36 |
| cardoe | And specifically networking. | 14:37 |
| ralonsoh | MengyangZhang[m], the Neutron API, let me find that | 14:37 |
| ralonsoh | MengyangZhang[m], https://docs.openstack.org/api-ref/network/v2/index.html#quality-of-service | 14:37 |
| cardoe | You guys have https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/5SX35AQZ65G6HWQ7WARLJTJK3ASAOWNU/ on your wiki which is what I'm looking to follow up on. | 14:37 |
| cardoe | I'm specifically using L2VNI. There's been some out of tree tweaks around that by Juniper and others. There's also https://bugs.launchpad.net/ovn-bgp-agent/+bug/2017890 | 14:39 |
| cardoe | I'm just wondering how to best engage on my use case. I'd like to contribute it upstream as best as possible. | 14:39 |
| ralonsoh | To be honest, I think Julia has a better perspective of this topic. I think you should first sync with Julia about what is the Ironic's plan about "decoupling" certain areas from Neutron | 14:41 |
| haleyb | yes, i would agree was just trying to read that and look at wiki | 14:41 |
| ralonsoh | the ovn-bgp-agent is for OVN only, I don't think that suits your requirements | 14:41 |
| cardoe | I've been speaking with Julia about it. | 14:42 |
| cardoe | I'm encouraging less of a decoupling and more of a lean in harder. :D | 14:42 |
| haleyb | are there specific neutron changes for this effort? | 14:43 |
| cardoe | Well just usage behaviors and maybe I'm understanding the features wrong. | 14:44 |
| cardoe | So I've got a VXLAN fabric with leafs and edges. But some of my servers can sit on two separate fabrics. | 14:44 |
| cardoe | Previously when we used VLAN, I could use physical_network as the fabric / separation. But VXLAN explicitly disallows physical_network. | 14:45 |
| cardoe | With L2VNI, we'll have a VLAN mapped to that VXLAN on that switch pair serving the physical server for example as well. It would seem like the segments feature would work to create VLAN segment on a VXLAN network. But things feel more L3VNI. I'm not sure if it'll be a blocker. | 14:47 |
| haleyb | so if there is a change required in neutron, it needs to be proposed via our RFE process, I have not seen any request in this area yet | 14:47 |
| cardoe | I'm interested in using OVN with this as well and have it either join the fabric with something like... https://github.com/toreanderson/evpn_agent/ | 14:47 |
| cardoe | I'll happily write them up. I just wanted to first ask if this use case was something that would be acceptable. | 14:48 |
| haleyb | https://wiki.openstack.org/wiki/Meetings/NeutronDrivers | 14:48 |
| haleyb | i am just not super familiar with this effort | 14:49 |
| cardoe | In some cases there would just be neutron and nothing else. e.g. the BMC / iLO / iDRAC (whatever ya wanna call it) network. | 14:50 |
| cardoe | I'll do a better job of writing this up and explaining it in the future. | 14:51 |
| cardoe | I just was more curious around the acceptability of neutron as a data center network manager. | 14:51 |
| haleyb | hmm, i guess i would not try and stretch neutron that far | 14:52 |
| haleyb | but would have to see a proposal | 14:52 |
| cardoe | okay. So basically Ironic runs its own DHCP server for certain cases still. I've been asking why not just use a neutron network with neutron supplied DHCP for the BMC network. | 14:53 |
| cardoe | As one example. | 14:53 |
| cardoe | I'll refine it better before I bring it up again. Sorry. | 14:54 |
| cardoe | TheJulia: btw ^ | 14:54 |
| haleyb | we only have a few minutes left, want to give people time if they wish to attend TC meeting | 14:55 |
| frickler | 1600 UTC is 1h away fwiw | 14:55 |
| haleyb | oh, i saw 10am ET | 14:56 |
| haleyb | i have a hard stop either way | 14:56 |
| haleyb | cardoe: thanks for bringing it up, please loop-back once you have more info from ironic team | 14:57 |
| spatel | Folks, I have question related OVN chassis priority. I want to change chassis priority so my external port for SRIOV get schedule on specified chassis - https://paste.opendev.org/show/by6LCZQ8jSCAwlDeWv4K/ | 14:57 |
| ralonsoh | spatel, we can talk after the meeting | 14:58 |
| spatel | I have vlan tenant based networking I am not running any L3 vRouter | 14:58 |
| haleyb | #endmeeting | 14:58 |
| opendevmeet | Meeting ended Tue Feb 11 14:58:10 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:58 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/networking/2025/networking.2025-02-11-14.00.html | 14:58 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/networking/2025/networking.2025-02-11-14.00.txt | 14:58 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/networking/2025/networking.2025-02-11-14.00.log.html | 14:58 |
| spatel | ralonsoh +1 | 14:58 |
| lajoskatona | Bye | 14:58 |
| ralonsoh | bye | 14:58 |
| mlavalle | \o | 14:58 |
| haleyb | have a good week everyone | 14:58 |
| obondarev | o/ | 14:58 |
| ralonsoh | spatel, so you have several GW chassis, I guess | 14:58 |
| spatel | ralonsoh I was on time :) look like meeting just end | 14:58 |
| ralonsoh | external ports use HA_Chassis_Group to host the port | 14:59 |
| spatel | I have 3 controller not but I want to add 2 more nodes which answer my DHCP request for sriov ports | 14:59 |
| ralonsoh | but the priority is not selectable | 14:59 |
| ralonsoh | you can change it manually | 14:59 |
| spatel | How? | 14:59 |
| spatel | I am not able to find command to change priority in HA_Chassis_Group | 14:59 |
| spatel | All doc is discussing about lrp port priority which is related to L3 router (In my case I don't have one) | 15:00 |
| ralonsoh | spatel, you need to write it manually on the ha_chassis registers | 15:01 |
| ralonsoh | each ha_chassis_group is a set of ha_chassis registers | 15:02 |
| spatel | Do you have any command example ? | 15:02 |
| ralonsoh | each one has a decreasing priority | 15:02 |
| ralonsoh | one sec | 15:02 |
| ralonsoh | spatel, you need to remember that you'll have more than 1 ha_chassis per ha_chassis_group | 15:05 |
| ralonsoh | and the priority must be different | 15:05 |
| ralonsoh | to change that | 15:05 |
| spatel | Sure! | 15:05 |
| ralonsoh | ovn-nbctl set ha_chassis e0b19db8-1fa7-448f-865e-bfb1704f26fb priority=2000 | 15:05 |
| ralonsoh | for example | 15:05 |
| spatel | Let me try! | 15:05 |
| spatel | That works :) | 15:07 |
| spatel | Let me see my DHCP request getting answer by that node | 15:07 |
| spatel | ralonsoh hmm! I have changed but it getting revert in few minutes | 15:13 |
| spatel | look like its re-updating tables | 15:14 |
| ralonsoh | Neutron binds the port once and doesn't change it | 15:14 |
| ralonsoh | I don't know what is reverting this change | 15:15 |
| spatel | See here - https://paste.opendev.org/show/bVjxjnjUol5jXnb8rWFw/ | 15:15 |
| opendevreview | Elvira García Ruiz proposed x/whitebox-neutron-tempest-plugin master: Fix SGL test multicompute and cleanup https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/941108 | 15:19 |
| spatel | ralonsoh How many chassis I can be have with external_ids:ovn-cms-options="enable-chassis-as-gw" ? | 15:26 |
| spatel | I have 4 in place and trying to add 5th one | 15:26 |
| spatel | but not accepting | 15:27 |
| spatel | Sorry, look like it took some time | 15:27 |
| ralonsoh | spatel, as many as you want. The hardcoded limit for ha_chassis_group is 5 | 15:28 |
| opendevreview | Elvira García Ruiz proposed x/whitebox-neutron-tempest-plugin master: Fix SGL test multicompute and cleanup https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/941108 | 15:28 |
| spatel | ralonsoh Thank you! | 15:30 |
| spatel | is this look OK to you? I have so many duplicate chassis after changing priority - https://paste.opendev.org/show/b5QxoTMmaLAQIX2HO6b6/ | 15:30 |
| ralonsoh | spatel, these ha_chassis register belong to different ha_chassis_group | 15:37 |
| spatel | hmm! | 15:38 |
| spatel | Help me understand because my knowledge of OVN is very new - https://paste.opendev.org/show/bnzgOuKn2ZCw2ENpstbD/ | 15:40 |
| spatel | each SRIOV port has own chassis group? | 15:40 |
| spatel | I see.. I can see each sriov port has own chassis group and related nodes owning that ports | 15:42 |
| ralonsoh | spatel, no, an external port will be scheduled to a ha_chassis_group, but this is created per network | 15:44 |
| ralonsoh | https://github.com/openstack/neutron/blob/master/neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/ovn_client.py#L619 | 15:44 |
| spatel | ralonsoh this confusing me here what is neutron-extport-57978c72-9db9-4755-ab07-e1c356be586f ? | 15:46 |
| spatel | Look into this output - https://paste.opendev.org/show/bqkVsbeTn7M5JSX7N7Wn/ | 15:47 |
| spatel | Its matching with one of my sriov port ID | 15:47 |
| sahid | just two tests are failing https://fd523a58635e77859e01-780831070818942681cb8e09b67e92b7.ssl.cf2.rackcdn.com/937765/57/check/neutron-ovs-tempest-with-os-ken-master/87e4ca9/testr_results.html | 16:20 |
| sahid | hopefully we will be able to make progress soon | 16:21 |
| *** elodilles_pto is now known as elodilles | 16:21 | |
| ralonsoh | eolivare, hi! If you have 1 min: https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/941224 | 16:56 |
| ralonsoh | thanks! | 16:56 |
| ralonsoh | haleyb, hello, if you have 1 min, please check | 16:56 |
| ralonsoh | https://review.opendev.org/c/openstack/neutron/+/941211 | 16:57 |
| ralonsoh | https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/941212 | 16:57 |
| ralonsoh | thanks! | 16:57 |
| haleyb | ralonsoh: done and done, thanks | 16:58 |
| ralonsoh | thanks | 17:00 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: WIP == [OVN] Router GW port deletion should be done once only https://review.opendev.org/c/openstack/neutron/+/941262 | 17:26 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [eventlet-removal] Remove the usage of eventlet in the Neutron API (2) https://review.opendev.org/c/openstack/neutron/+/941202 | 17:48 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Revert "Track all interfaces in Keepalived" https://review.opendev.org/c/openstack/neutron/+/941283 | 17:52 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Create a deterministic hash ring node UUID generator https://review.opendev.org/c/openstack/neutron/+/941008 | 17:57 |
| opendevreview | Renjing Xiao proposed openstack/neutron-tempest-plugin master: Cover both enabled and disabled cases in nested snat validation test https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/940906 | 18:11 |
| opendevreview | Renjing Xiao proposed openstack/neutron-tempest-plugin master: Cover both enabled and disabled cases in nested snat validation test https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/940906 | 18:21 |
| opendevreview | Renjing Xiao proposed x/whitebox-neutron-tempest-plugin master: Add nested snat tesin whitebox https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/940908 | 18:52 |
| opendevreview | Merged x/whitebox-neutron-tempest-plugin master: [eventlet-removal] Remove "logger" mechanism from ML2/OVN CI jobs https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/941224 | 18:52 |
| opendevreview | Renjing Xiao proposed x/whitebox-neutron-tempest-plugin master: Add nested snat test in whitebox https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/940908 | 18:54 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: [eventlet-removal] Remove "logger" mechanism from ML2/OVN CI jobs https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/941212 | 19:21 |
| opendevreview | Merged openstack/neutron master: [eventlet-removal] Remove "logger" mechanism from ML2/OVN CI jobs https://review.opendev.org/c/openstack/neutron/+/941211 | 19:21 |
| opendevreview | Merged openstack/neutron master: Remove dibbler code from l3 agent and elsewhere https://review.opendev.org/c/openstack/neutron/+/934283 | 21:48 |
| opendevreview | Merged openstack/neutron master: Use shared constants for [ovn] ovn_l3_scheduler https://review.opendev.org/c/openstack/neutron/+/941019 | 22:08 |
| opendevreview | Renjing Xiao proposed x/whitebox-neutron-tempest-plugin master: Add nested snat tesin whitebox https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/940908 | 23:08 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!