| opendevreview | morice proposed openstack/neutron master: [ovn] AZs distribution in L3 port scheduler https://review.opendev.org/c/openstack/neutron/+/892604 | 07:04 |
|---|---|---|
| opendevreview | Lajos Katona proposed openstack/networking-bagpipe master: py311: add required projects to py311 job and add it to weekly https://review.opendev.org/c/openstack/networking-bagpipe/+/896490 | 08:20 |
| opendevreview | Lajos Katona proposed openstack/networking-bgpvpn master: py311: add required projects to py311 job and add it to weekly https://review.opendev.org/c/openstack/networking-bgpvpn/+/896491 | 08:23 |
| rammses_ | Hey guys, Again with the l2 segmented solution. How does neutron know that it has to announce a new server's fip ? There must be a event loop to trigger announcements right ? If so, is there a event for migrated vms ? | 08:27 |
| opendevreview | Merged openstack/neutron stable/2023.2: Use safer methods to get security groups on security group logging https://review.opendev.org/c/openstack/neutron/+/896335 | 09:42 |
| ralonsoh | bcafarel, hello! | 09:47 |
| ralonsoh | if you have 1 min: https://review.opendev.org/c/openstack/neutron/+/893447 | 09:47 |
| ralonsoh | thanks! | 09:47 |
| ralonsoh | sorry, a master core reviewer | 09:48 |
| ralonsoh | slaweq, ^^ | 09:48 |
| bcafarel | sure thing, I will check after meetings | 09:48 |
| bcafarel | ah I will wait for the backports reviews then :) | 09:49 |
| ralonsoh | yes, sorry | 09:49 |
| ralonsoh | first the master patch and then the backports with the correct hash | 09:49 |
| opendevreview | yatin proposed openstack/os-vif master: Revert "set default qos policy" https://review.opendev.org/c/openstack/os-vif/+/896440 | 10:16 |
| ralonsoh | ykarel, ^^ good catch. Please fix the merge conflicts | 10:17 |
| ykarel | ralonsoh, yes doing that | 10:18 |
| opendevreview | yatin proposed openstack/os-vif master: Revert "set default qos policy" https://review.opendev.org/c/openstack/os-vif/+/896440 | 10:23 |
| opendevreview | yatin proposed openstack/neutron master: [DNM] Test os-vif revert https://review.opendev.org/c/openstack/neutron/+/896504 | 10:25 |
| sondx25 | ralonsoh, - I have an OpenStack cluster includes Controllers of 20 Compute. (I use ML2/OVN) | 10:33 |
| sondx25 | - My Compute-Compute Bandwidth is 10 Gbps. The bandwidth of my VM-VM (mtu 1500) is 6 Gbps. | 10:33 |
| sondx25 | - I installed k8s on VMs (overlay over overlay), and my pod-to-pod bandwidth is only 1Gbps (80%-90% reduction). | 10:33 |
| sondx25 | - Meanwhile, in a similar lab on VMWare (NSX) environment, my pod-to-pod bandwidth is 5Gbps (10%-15% reduction). | 10:33 |
| sondx25 | -> Is it possible that the virtio driver interface has poor tunnel support? | 10:33 |
| sondx25 | please help me | 10:33 |
| ralonsoh | sondx25, if you have 6 gbps VM-VM, this is not a problem in the virtio driver, is a problem in the VM traffic. Check if you are fragmenting the packets inside the VM, try reducing the MTu of the upper overlay network | 10:38 |
| sondx25 | ralonsoh, I set the MTU of the pod is 1400, on VMware, I set the MTU of the pod is 1400. | 10:41 |
| ralonsoh | did you check if you have fragmentation? | 10:42 |
| sondx25 | ralonsoh, I will check this | 11:08 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: WIP == Add policy enforcer for "tags" service plugin https://review.opendev.org/c/openstack/neutron/+/896509 | 11:36 |
| opendevreview | yatin proposed openstack/neutron stable/2023.1: [PostgreSQL] Subnet entity with ServiceType grouped by both tables https://review.opendev.org/c/openstack/neutron/+/896521 | 11:45 |
| opendevreview | Merged openstack/networking-bagpipe master: py311: add required projects to py311 job and add it to weekly https://review.opendev.org/c/openstack/networking-bagpipe/+/896490 | 11:50 |
| opendevreview | yatin proposed openstack/neutron stable/zed: [PostgreSQL] Subnet entity with ServiceType grouped by both tables https://review.opendev.org/c/openstack/neutron/+/896511 | 11:53 |
| opendevreview | yatin proposed openstack/neutron stable/yoga: [PostgreSQL] Subnet entity with ServiceType grouped by both tables https://review.opendev.org/c/openstack/neutron/+/896522 | 11:54 |
| opendevreview | Arnaud Morin proposed openstack/neutron master: Add a new option to enable signals https://review.opendev.org/c/openstack/neutron/+/884711 | 12:49 |
| opendevreview | yatin proposed openstack/neutron master: [DNM] Test os-vif revert https://review.opendev.org/c/openstack/neutron/+/896504 | 13:02 |
| opendevreview | Arnaud Morin proposed openstack/neutron master: Set DVR qr-xyz interfaces DOWN on backup node https://review.opendev.org/c/openstack/neutron/+/869741 | 13:03 |
| ralonsoh | folks, I won't be able to attend to the Neutron meeting today | 13:08 |
| ralonsoh | but out new PTL, haleyb, will chair it | 13:08 |
| ralonsoh | our* | 13:08 |
| ralonsoh | thanks! | 13:08 |
| opendevreview | Felipe Figueroa Vergara proposed openstack/tap-as-a-service master: Enables external network mirroring https://review.opendev.org/c/openstack/tap-as-a-service/+/896515 | 13:08 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: WIP == Add policy enforcer for "tags" service plugin https://review.opendev.org/c/openstack/neutron/+/896509 | 13:28 |
| ralonsoh | ^^ slaweq am I going in the right direction? | 13:28 |
| ralonsoh | I've been manually testing with http request all the commands for network tags | 13:29 |
| ralonsoh | same for other resources | 13:29 |
| opendevreview | Merged openstack/ovn-octavia-provider stable/2023.2: Check multiple address of a LRP plugged to LS https://review.opendev.org/c/openstack/ovn-octavia-provider/+/896266 | 13:36 |
| slaweq | ralonsoh yes, I think it is correct | 13:43 |
| ralonsoh | thanks! | 13:43 |
| slaweq | but I didn't test it locally :) | 13:43 |
| haleyb | ping bcafarel, elvira, frickler, mlavalle, mtomaska, obondarev, slawek, tobias-urdin, ykarel, lajoskatona, jlibosva, averdagu, amotoki - meeting starting | 13:59 |
| mlavalle | o/ | 14:00 |
| jlibosva | o/ | 14:00 |
| haleyb | #startmeeting networking | 14:00 |
| opendevmeet | Meeting started Tue Sep 26 14:00:09 2023 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 |
| bcafarel | o/ | 14:00 |
| obondarev | hi | 14:00 |
| slaweq | o/ | 14:00 |
| mlavalle | haleyb: is this your first official act as new PTL? | 14:00 |
| mtomaska | o/ | 14:00 |
| haleyb | hello everyone, taking over for ralonsoh today | 14:00 |
| ralonsoh | thanks! | 14:00 |
| haleyb | i'm the future PTL, so i've still got training wheels on :) | 14:01 |
| mlavalle | :-) | 14:01 |
| haleyb | #topic announcements | 14:01 |
| haleyb | #link https://releases.openstack.org/bobcat/schedule.html | 14:01 |
| haleyb | basically one week left, so if there is anything critical for Bobcat it needs to be merged asap | 14:02 |
| haleyb | I did see this merged this morning - https://review.opendev.org/c/openstack/neutron/+/896335 | 14:02 |
| lajoskatona | o/ | 14:03 |
| aprats- | o/ | 14:03 |
| amorin | o/ | 14:03 |
| frickler | \o | 14:03 |
| haleyb | ralonsoh: since you're maybe here, do we need to push a new release tag for that ^^ | 14:03 |
| ralonsoh | code is working now and this should be backported along with the previous patch introducing it | 14:04 |
| ralonsoh | but is not needed | 14:04 |
| haleyb | ack, didn't seem critical | 14:04 |
| ralonsoh | nope | 14:05 |
| frickler | but better do an rc2 now than an .1 later? | 14:05 |
| lajoskatona | I think the release team will propse the new rc and the final for bobcat anyway if that was the question | 14:05 |
| ralonsoh | we have time for a rc2 | 14:05 |
| ralonsoh | lajoskatona, exactly | 14:05 |
| ykarel | o/ | 14:05 |
| frickler | right, you can wait until the end of this week I think | 14:05 |
| haleyb | lajoskatona: ah, ok, didn't know if we needed to manually propose this late | 14:05 |
| bcafarel | what about https://review.opendev.org/c/openstack/neutron/+/893447 revert (still in master)? affecting cold migration | 14:06 |
| lajoskatona | that should be included I suppose | 14:06 |
| ralonsoh | yes, that needs to be in bobcat | 14:06 |
| ralonsoh | so I'll propose the 2023.2 backport this week | 14:06 |
| ralonsoh | once the master branch patch is merged | 14:06 |
| haleyb | that was stuck due to previous gate failure | 14:06 |
| bcafarel | ack let's keep an eye on master and 2023.2 backport then | 14:07 |
| haleyb | if there are any other proposals for inclusion in bobcat please ping ralonsoh (or me) to prioritize | 14:08 |
| ralonsoh | +1 | 14:08 |
| haleyb | any other announcements? | 14:08 |
| haleyb | #topic bugs | 14:09 |
| haleyb | last week's report is from bcafarel | 14:09 |
| haleyb | #link https://lists.openstack.org/pipermail/openstack-discuss/2023-September/035216.html | 14:09 |
| bcafarel | and good progress on the unassigned since I posted it | 14:10 |
| haleyb | First one is (another) gate failure | 14:10 |
| haleyb | #link https://bugs.launchpad.net/neutron/+bug/2037239 | 14:10 |
| haleyb | neutron-tempest-plugin-openvswitch-* jobs are randomly failing in gate with an l3-ha issue | 14:11 |
| lajoskatona | ykarel suspected that an os-vif patch is behind it, but as I saw the logs I am not sure | 14:11 |
| haleyb | a couple of us have looked through recent reviews and logs bug no silver pullet yet | 14:12 |
| haleyb | lajoskatona: yes, i saw that review, https://review.opendev.org/c/openstack/neutron/+/896504 | 14:12 |
| lajoskatona | exactly | 14:12 |
| haleyb | first pass seems to have failed the same way | 14:12 |
| haleyb | tempest.lib.exceptions.TimeoutException: Request timed out | 14:13 |
| haleyb | Details: Router 6986c001-1030-4a25-96cf-8445da63f144 is not active on any of the L3 agents | 14:13 |
| ykarel | test patch with revert is still running https://zuul.openstack.org/status#896504 , results ther should confirm | 14:13 |
| ykarel | in 1st attempt os-vif patch was not used, i pushed a devstack patch to get it applied | 14:13 |
| haleyb | ykarel: ah, thanks, let's see if that helps | 14:13 |
| lajoskatona | +1 | 14:14 |
| *** ralonsoh is now known as ralonsoh_afk | 14:14 | |
| haleyb | if not i will take a look again at logs, or if anyone else has cycles... | 14:14 |
| haleyb | next unassigned is | 14:15 |
| haleyb | #link https://bugs.launchpad.net/neutron/+bug/2036603 | 14:15 |
| haleyb | ykarel: you think this is an advanced image issue based on last comment? | 14:17 |
| ykarel | haleyb, no not saying it's due to the images used, but other tests using those images are impacted | 14:17 |
| ykarel | as bug mentioned a test name, so just trying to point it's generic issue | 14:17 |
| haleyb | do we know if the image changed just recently? | 14:18 |
| haleyb | although i guess we didn't change it in our config | 14:19 |
| opendevreview | Terry Wilson proposed openstack/neutron master: Add has_lock_periodic decorator for OVN Maintenance https://review.opendev.org/c/openstack/neutron/+/896544 | 14:19 |
| haleyb | either way, is there anyone that can take a closer look at that bug? | 14:19 |
| ykarel | me haven't checked it if those are updated | 14:20 |
| ykarel | i can check it | 14:20 |
| haleyb | ykarel: ack, thanks! | 14:20 |
| haleyb | next unassigned | 14:21 |
| haleyb | #link https://bugs.launchpad.net/neutron/+bug/2036705 | 14:21 |
| haleyb | 'A port that is disabled and bound is still ACTIVE with ML2/OVN' | 14:21 |
| haleyb | was confirmed it's different from ML2/OVS on irc | 14:21 |
| opendevreview | Merged openstack/ovn-octavia-provider stable/2023.1: Check multiple address of a LRP plugged to LS https://review.opendev.org/c/openstack/ovn-octavia-provider/+/896267 | 14:22 |
| lajoskatona | I think this was on which I commented that I am not sure which one is the correct | 14:24 |
| haleyb | so i guess part of that is figuring out which driver is correct, octavia assumes OVS is | 14:24 |
| lajoskatona | yes, we can say that it was with OVs that is the "standard" and adopt OVN | 14:24 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: Introduce ovn_nb_global config section https://review.opendev.org/c/openstack/neutron/+/896545 | 14:25 |
| haleyb | I will add that comment, will need owner to work on it | 14:25 |
| lajoskatona | +1 | 14:26 |
| opendevreview | Terry Wilson proposed openstack/neutron master: Add has_lock_periodic decorator for OVN Maintenance https://review.opendev.org/c/openstack/neutron/+/896544 | 14:27 |
| haleyb | will move onto next | 14:27 |
| haleyb | #link https://bugs.launchpad.net/neutron/+bug/2036877 | 14:27 |
| haleyb | 'radvd seems to crash when ipv4 addresses are supplied as nameservers to ipv6 subnets' | 14:28 |
| haleyb | i just assigned this to myself, seems valid | 14:28 |
| haleyb | actually saw a similar bug with dhcp recently downstream | 14:29 |
| haleyb | moving onto next | 14:29 |
| haleyb | #link https://bugs.launchpad.net/neutron/+bug/2037107 | 14:29 |
| haleyb | 'slow port creation with a large amount of networkrbacs' | 14:30 |
| haleyb | ralonsoh took a look at this, could not reproduce on master but gave a lot of info on reproducing | 14:31 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: Introduce ovn_nb_global config section https://review.opendev.org/c/openstack/neutron/+/896545 | 14:31 |
| haleyb | there was just a reply this morning, i'll assume rodolfo will reply in time | 14:32 |
| haleyb | and last unassigned from last week | 14:32 |
| haleyb | #link https://bugs.launchpad.net/neutron/+bug/2036423 | 14:32 |
| haleyb | 'subnet's gateway ip can be unset while attached to router' | 14:32 |
| haleyb | i looked at this initially and noticed a couple of things | 14:33 |
| haleyb | #1 the api doc seems wrong, looks like a copy/paste from POST was done in PUT section | 14:33 |
| haleyb | # setting gateway_ip to None is valid, and setting it does not remove the router interface (maybe that wasn't assumed) | 14:34 |
| haleyb | i could not reproduce, but submitter just replied it might only happen on multinode | 14:35 |
| haleyb | anyone have a comment at least on the API behavior? | 14:35 |
| lajoskatona | +1 to fix the API ref | 14:36 |
| haleyb | lajoskatona: ack, assuming you also believe setting gateway_ip to None is valid :) | 14:37 |
| haleyb | https://docs.openstack.org/api-ref/network/v2/index.html?expanded=update-subnet-detail,create-subnet-detail#subnets | 14:37 |
| haleyb | actually the API does say it can be null... | 14:37 |
| haleyb | but if not specified on create one will be selected | 14:38 |
| lajoskatona | Is it really a valid usecase to set it to None? | 14:38 |
| amorin | yes | 14:39 |
| amorin | 'none': This subnet will not use a gateway | 14:39 |
| lajoskatona | thanks | 14:39 |
| amorin | openstack subnet set --help will show this to you | 14:39 |
| opendevreview | Terry Wilson proposed openstack/neutron master: Add has_lock_periodic decorator for OVN Maintenance https://review.opendev.org/c/openstack/neutron/+/896544 | 14:40 |
| lajoskatona | I assume if we set it to None at Create time it works as expected and the PUT is problematic | 14:40 |
| haleyb | i can update the api ref and will see if i can reproduce in multinode | 14:41 |
| haleyb | lajoskatona: that's a good question, my testing was mostly was changing it will PUT | 14:41 |
| haleyb | s/with | 14:41 |
| haleyb | lajoskatona: if you're interesting in learning more about it... :) j/k | 14:42 |
| haleyb | moving on, any other bugs to talk about? | 14:43 |
| lajoskatona | haleyb: perhaps 2nd part of this week I ll have some time for it | 14:43 |
| haleyb | ack | 14:43 |
| haleyb | So this week lajoskatona is the deputy, next week will be slaweq | 14:43 |
| slaweq | ok, good for me | 14:44 |
| lajoskatona | I am on it | 14:44 |
| haleyb | slaweq, lajoskatona: thanks | 14:45 |
| haleyb | #topic specs | 14:45 |
| haleyb | No real change from last week, just the OVN interconnect RFE, did have a comment so needs an update | 14:46 |
| haleyb | we can open the 2024.1 folder so just need to get https://review.opendev.org/c/openstack/neutron-specs/+/891342 merged | 14:46 |
| opendevreview | yatin proposed openstack/neutron-tempest-plugin master: [DNM] check os-vif https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/896549 | 14:47 |
| haleyb | moving on | 14:47 |
| opendevreview | yatin proposed openstack/neutron master: [DNM] Test os-vif revert https://review.opendev.org/c/openstack/neutron/+/896504 | 14:47 |
| haleyb | #topic community_goals | 14:48 |
| haleyb | I'll skip the first one, "Add support for the service role in neutron API policies" | 14:48 |
| haleyb | 2) neutron client deprecation | 14:48 |
| haleyb | lajoskatona: an update from last week? | 14:49 |
| lajoskatona | nothing to tell the truth, I was sucked into downstream work mostly | 14:49 |
| haleyb | lajoskatona: ack, and i did not click on all the "active patch" links in the wiki, will go through by next week | 14:50 |
| haleyb | ok, onto last section | 14:50 |
| haleyb | #topic on_demand | 14:50 |
| haleyb | since it's been pretty quiet i figure people have been waiting for a turn to speak? (haleyb is snarky this morning) | 14:51 |
| bcafarel | :) | 14:51 |
| aprats- | Hi! About https://review.opendev.org/c/openstack/neutron/+/890459 did someone had the time to take at look at this ? | 14:51 |
| amorin | ^ we are running this code in production since few weeks now, so we think it's ready for review | 14:53 |
| haleyb | aprats-: sorry, i did not have time to look yet, and i think ralonsoh had some thoughts based on last weeks meeting | 14:53 |
| haleyb | anything else? | 14:55 |
| haleyb | ykarel: there is a CI meeting today, right? | 14:55 |
| ykarel | yes | 14:55 |
| amorin | I have one here: https://review.opendev.org/c/openstack/neutron/+/884711 | 14:55 |
| ykarel | it's video today | 14:55 |
| haleyb | ykarel: and it's in :5 right? | 14:56 |
| bcafarel | ack | 14:56 |
| opendevreview | Arnaud Morin proposed openstack/neutron master: Register GMR with config https://review.opendev.org/c/openstack/neutron/+/884842 | 14:56 |
| ykarel | haleyb, yeah right | 14:56 |
| amorin | and this one ^ | 14:56 |
| lajoskatona | amorin, aprats-: I suppoe when the release push is over review will go back to normal again | 14:56 |
| amorin | make sense, thanks :) | 14:56 |
| aprats- | thx :) | 14:56 |
| haleyb | amorin: ack, will take a look this week | 14:57 |
| haleyb | thanks everyone for attending, need to end for next meeting | 14:58 |
| haleyb | #endmeeting | 14:58 |
| opendevmeet | Meeting ended Tue Sep 26 14:58:22 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:58 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/networking/2023/networking.2023-09-26-14.00.html | 14:58 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/networking/2023/networking.2023-09-26-14.00.txt | 14:58 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/networking/2023/networking.2023-09-26-14.00.log.html | 14:58 |
| mlavalle | o/ | 14:59 |
| amorin | thanks | 14:59 |
| lajoskatona | o/ | 15:00 |
| ykarel | #startmeeting neutron_ci | 15:00 |
| opendevmeet | Meeting started Tue Sep 26 15:00:45 2023 UTC and is due to finish in 60 minutes. The chair is ykarel. 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 |
| ykarel | This will be video meeting this time: https://meetpad.opendev.org/neutron-ci-meetings | 15:00 |
| ykarel | ping bcafarel, lajoskatona, mlavalle, mtomaska, ralonsoh, ykarel, jlibosva, elvira | 15:01 |
| mtomaska | o/ | 15:01 |
| lajoskatona | o/ | 15:02 |
| ykarel | #topic Actions from previous meetings | 15:03 |
| ykarel | ralonsoh to open bug for test_read_queue_change_state | 15:03 |
| ykarel | #link https://bugs.launchpad.net/neutron/+bug/2035382 | 15:03 |
| ykarel | #ralonsoh to check failure with ha functional test | 15:03 |
| ykarel | ykarel to switch jobs to CentOS 9-Stream in stable branches where applicable as per runtimes per release | 15:04 |
| ykarel | #link https://review.opendev.org/q/Ie4e807b1490d59390316ec20b499b7676acfe410 | 15:04 |
| ykarel | #topic Stable branches | 15:04 |
| ykarel | all good | 15:05 |
| ykarel | #topic Stadium projects | 15:05 |
| ykarel | #action lajoskatona to check consistency with bgpvpn related to https://zuul.openstack.org/build/8a624b4d29ea44589c9c83b0ec1da446 | 15:06 |
| ykarel | #topic Grafana | 15:06 |
| ykarel | https://grafana.opendev.org/d/f913631585/neutron-failure-rate | 15:06 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: Introduce ovn_nb_global config section https://review.opendev.org/c/openstack/neutron/+/896545 | 15:08 |
| ykarel | #topic Rechecks | 15:08 |
| lajoskatona | slaweq: this is the bug for the issue: https://bugs.launchpad.net/neutron/+bug/2037239 *most of issues at least) | 15:08 |
| haleyb | #link https://review.opendev.org/c/openstack/neutron/+/896504 (os-vif test patch) | 15:08 |
| ykarel | #topic fullstack/functional | 15:09 |
| ykarel | test_dvr_lifecycle_ha_with_snat_with_fips_with_cent_fips_no_gw and test_dvr_ha_router_interface_mtu_update | 15:09 |
| ykarel | #topic Tempest/Scenario | 15:10 |
| ykarel | tests with advanced image fails randomly, seen few occurances across branches and scenarios ovn/openvswitch/linuxbridge | 15:10 |
| ykarel | #link https://bugs.launchpad.net/neutron/+bug/2036603 | 15:10 |
| ykarel | #action ykarel to check https://bugs.launchpad.net/neutron/+bug/2036603 | 15:10 |
| ykarel | #link https://bugs.launchpad.net/neutron/+bug/2037239 | 15:11 |
| lajoskatona | for os-vif req bump: https://review.opendev.org/c/openstack/requirements/+/887962 | 15:14 |
| ykarel | #topic Periodic | 15:14 |
| ykarel | #link https://zuul.openstack.org/builds?job_name=neutron-tempest-plugin-linuxbridge&project=openstack/neutron | 15:14 |
| ykarel | #link https://bugs.launchpad.net/neutron/+bug/2037239 | 15:15 |
| ykarel | #link https://bugs.launchpad.net/neutron/+bug/2035578 | 15:15 |
| ykarel | #link https://bugs.launchpad.net/neutron/+bug/2028003 | 15:15 |
| ykarel | #link https://review.opendev.org/q/I18e65d79e56fe5995076eb9166da23fc14c92fc5 | 15:15 |
| ykarel | #topic On Demand | 15:16 |
| haleyb | #link https://review.opendev.org/c/openstack/neutron/+/895515 SLURP job enablement | 15:16 |
| ykarel | #endmeeting | 15:20 |
| opendevmeet | Meeting ended Tue Sep 26 15:20:35 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:20 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/neutron_ci/2023/neutron_ci.2023-09-26-15.00.html | 15:20 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/neutron_ci/2023/neutron_ci.2023-09-26-15.00.txt | 15:20 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/neutron_ci/2023/neutron_ci.2023-09-26-15.00.log.html | 15:20 |
| *** ralonsoh_afk is now known as ralonsoh | 15:24 | |
| opendevreview | Felipe Figueroa Vergara proposed openstack/tap-as-a-service master: Enables external network mirroring https://review.opendev.org/c/openstack/tap-as-a-service/+/896515 | 15:40 |
| opendevreview | Merged openstack/neutron stable/yoga: [PostgreSQL] Subnet entity with ServiceType grouped by both tables https://review.opendev.org/c/openstack/neutron/+/896522 | 16:00 |
| opendevreview | Merged openstack/neutron stable/2023.1: [PostgreSQL] Subnet entity with ServiceType grouped by both tables https://review.opendev.org/c/openstack/neutron/+/896521 | 16:15 |
| opendevreview | Merged openstack/ovn-octavia-provider stable/xena: Check multiple address of a LRP plugged to LS https://review.opendev.org/c/openstack/ovn-octavia-provider/+/896270 | 16:39 |
| opendevreview | Merged openstack/ovn-octavia-provider stable/wallaby: Check multiple address of a LRP plugged to LS https://review.opendev.org/c/openstack/ovn-octavia-provider/+/896271 | 16:39 |
| opendevreview | Merged openstack/ovn-octavia-provider stable/2023.1: Cover the use case of a member non existing https://review.opendev.org/c/openstack/ovn-octavia-provider/+/894925 | 16:40 |
| opendevreview | Merged openstack/networking-bgpvpn master: py311: add required projects to py311 job and add it to weekly https://review.opendev.org/c/openstack/networking-bgpvpn/+/896491 | 16:46 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: WIP == Add policy enforcer for "tags" service plugin https://review.opendev.org/c/openstack/neutron/+/896509 | 16:48 |
| *** ralonsoh is now known as ralonsoh_afk | 16:49 | |
| mnaser | Question .. what could be a reason for the revision number of a resource to increase without neutron logging anything ? | 16:59 |
| mnaser | Im troubleshooting a bug here and I see it go from 1…2….4 | 16:59 |
| mnaser | No log of it ever hitting 3 | 17:00 |
| jlibosva | can someone please have a look at https://review.opendev.org/c/openstack/neutron/+/890339 - it's had +2 for a while now, thank you :) | 17:14 |
| opendevreview | Merged openstack/neutron stable/2023.1: Check the device ID and host ID during virtual port binding https://review.opendev.org/c/openstack/neutron/+/895433 | 17:25 |
| opendevreview | Merged openstack/ovn-octavia-provider stable/xena: Cover the use case of a member non existing https://review.opendev.org/c/openstack/ovn-octavia-provider/+/894928 | 17:27 |
| opendevreview | Merged openstack/ovn-octavia-provider stable/wallaby: Cover the use case of a member non existing https://review.opendev.org/c/openstack/ovn-octavia-provider/+/894929 | 17:27 |
| opendevreview | Merged openstack/ovn-octavia-provider stable/yoga: Cover the use case of a member non existing https://review.opendev.org/c/openstack/ovn-octavia-provider/+/894927 | 17:27 |
| opendevreview | Merged openstack/ovn-octavia-provider stable/zed: Check multiple address of a LRP plugged to LS https://review.opendev.org/c/openstack/ovn-octavia-provider/+/896268 | 17:27 |
| opendevreview | Brian Haley proposed openstack/neutron master: Bump skip-level lower version to stable/2023.1 https://review.opendev.org/c/openstack/neutron/+/895515 | 17:43 |
| mnaser | I think I caught the root of https://bugs.launchpad.net/neutron/+bug/2024160 but I need a bit help of OVN brains. I think here what's happening is that OVN is giving two events when binding a subport: one is the PortBindingChassisEvent and the second one is "Matched DELETE: PortBindingUpdateVirtualPortsEvent" (seems like it clears out the old port binding?) | 17:45 |
| mnaser | and so because of this, 5 minutes later in the maintenance cycle, the port gets unbound from ovn | 17:45 |
| slaweq | jlibosva done | 18:00 |
| slaweq | mnaser I think that ralonsoh may be proper person to check it, I will ping him tomorrow morning about it if he will miss Your comment in the LP | 18:01 |
| mnaser | slaweq thank you, i plan on spending a bit more time jsut to see what i can get more but i think it's just a matter of finding the cleaneest solution now | 18:01 |
| jlibosva | slaweq: thanks! | 18:09 |
| haleyb | mnaser: yes, ralonsoh is the one that would know. just trying to follow all these changes/reverts, there's no possibility this is fixed with this revert - https://review.opendev.org/c/openstack/neutron/+/893447 ? | 18:15 |
| mnaser | haleyb: i tried, but it's related, because here in this case what happens is the follow-up event of "DELETE: PortBindingUpdateVirtualPortsEvent" wipes the port binding (in neutron), and then the next neutron maintenance cycle it unbinds the port from ovn | 18:17 |
| mnaser | mainly because ovn in 22.03.0 included https://github.com/ovn-org/ovn/commit/28d40c0a024227c9c845117087b7594723ff0ec7 which adds a delete/create event when virtual *and* container (aka subports) are changed | 18:17 |
| mnaser | sorry, 22.03.2 | 18:17 |
| mnaser | so really the CI doesn't notice it because if it's in a slow env the trunk will stay up <5 minutes till the maintenance cycle is alive | 18:18 |
| mnaser | and live migration fails because the vms are alive longer, enough for neutron to unbind them | 18:18 |
| mnaser | cuz we get these two events on trunk port binding => https://paste.opendev.org/raw/biItzGSdGRxOu78qWN5p/ | 18:19 |
| jlibosva | haleyb: would you mind sir? :) https://review.opendev.org/c/openstack/neutron/+/895849 | 18:46 |
| jlibosva | haleyb: this is in the context we talked about last week, what OVN version do we want to use for FT and n-t-p | 18:46 |
| haleyb | jlibosva: ack, that's one that can get through the gate :-/ | 18:58 |
| opendevreview | Mohammed Naser proposed openstack/neutron master: Skip trunk sub-ports in "update_virtual_port_host" https://review.opendev.org/c/openstack/neutron/+/896590 | 20:09 |
| opendevreview | Mohammed Naser proposed openstack/neutron master: Skip trunk sub-ports in "update_virtual_port_host" https://review.opendev.org/c/openstack/neutron/+/896590 | 20:10 |
| opendevreview | Merged openstack/neutron stable/zed: [PostgreSQL] Subnet entity with ServiceType grouped by both tables https://review.opendev.org/c/openstack/neutron/+/896511 | 20:19 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: Introduce ovn_nb_global config section https://review.opendev.org/c/openstack/neutron/+/896545 | 20:38 |
| opendevreview | Merged openstack/neutron master: Use compiled OVN version v23.09.0 https://review.opendev.org/c/openstack/neutron/+/895849 | 21:35 |
| opendevreview | Mohammed Naser proposed openstack/neutron master: Skip trunk sub-ports in "update_virtual_port_host" https://review.opendev.org/c/openstack/neutron/+/896590 | 22:50 |
| opendevreview | Miguel Lavalle proposed openstack/neutron master: [PoC][DNM] Router flavors and service type for OVN https://review.opendev.org/c/openstack/neutron/+/883988 | 23:27 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!