| opendevreview | Terry Wilson proposed openstack/neutron master: DNM: Test PortBindingChassisEvent only when chassis modified https://review.opendev.org/c/openstack/neutron/+/949955 | 03:53 |
|---|---|---|
| *** ykarel__ is now known as ykarel | 04:06 | |
| opendevreview | Rico Lin proposed openstack/neutron master: Fix ovn db sync with log resources https://review.opendev.org/c/openstack/neutron/+/948053 | 05:22 |
| opendevreview | Rico Lin proposed openstack/neutron master: Fix ovn db sync with log resources https://review.opendev.org/c/openstack/neutron/+/948053 | 05:35 |
| *** ralonsoh_ is now known as ralonsoh | 05:43 | |
| ralonsoh | mlavalle, please check my comment in https://review.opendev.org/c/openstack/neutron/+/949816 | 05:43 |
| ralonsoh | I think we need to backport this fix up to 2024.1 | 05:44 |
| ralonsoh | just a heads-up | 05:44 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Make ``tunnel_ranges`` and ``network_vlan_ranges`` private members https://review.opendev.org/c/openstack/neutron/+/948313 | 05:50 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Make ``tunnel_ranges`` and ``network_vlan_ranges`` private members https://review.opendev.org/c/openstack/neutron/+/948313 | 05:51 |
| opendevreview | Rico Lin proposed openstack/neutron master: Fix ovn db sync with log resources https://review.opendev.org/c/openstack/neutron/+/948053 | 08:17 |
| ralonsoh | slaweq, lajoskatona hello folks | 08:31 |
| ralonsoh | I don't remember if today we have the drivers meeting | 08:32 |
| ralonsoh | or it was cancelled | 08:32 |
| ralonsoh | (I think the second) | 08:32 |
| slaweq | hi ralonsoh I don't remember neighter | 08:35 |
| slaweq | according to team meeting's logs we canceled last week's drivers meeting | 08:37 |
| slaweq | but this week we didn't talk about it I think | 08:37 |
| slaweq | I don't think we have anything new in the agendat to discuss today | 08:38 |
| opendevreview | Brian Haley proposed openstack/neutron stable/2024.2: AddressGroup API collection should be the resource name in plural https://review.opendev.org/c/openstack/neutron/+/949971 | 08:44 |
| opendevreview | Brian Haley proposed openstack/neutron stable/2024.2: [OVN] Support address group for ovn driver https://review.opendev.org/c/openstack/neutron/+/949852 | 08:44 |
| opendevreview | Brian Haley proposed openstack/neutron stable/2024.1: AddressGroup API collection should be the resource name in plural https://review.opendev.org/c/openstack/neutron/+/949972 | 08:45 |
| opendevreview | Brian Haley proposed openstack/neutron stable/2024.1: [OVN] Support address group for ovn driver https://review.opendev.org/c/openstack/neutron/+/949854 | 08:45 |
| haleyb | i will be on a plane during that time, no drivers for me | 08:46 |
| ralonsoh | haleyb, slaweq ok, I've reviewed the agenda and there is nothing new | 08:55 |
| ralonsoh | I'll send a mail cancelling today's meeting | 08:55 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-tempest-plugin master: List networks with limit and filter by provider network attrs https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/949975 | 09:10 |
| Continuity | Morning Neutron, | 09:11 |
| Continuity | Has anyone come across an issue where when upgrading (using kolla ansible), when OVN/Neutron is upgraded some router HA CHASSIS nodes come up with the same priority as another node | 09:11 |
| Continuity | Not on every network but on some | 09:11 |
| Continuity | This is on 2023.2 | 09:11 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-tempest-plugin master: List networks with limit and filter by provider network attrs https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/949975 | 09:11 |
| ralonsoh | Continuity, do these HA_Chassis registers belong to the same HA_Chassis_Group? | 09:12 |
| Continuity | yes | 09:13 |
| Continuity | So three nodes, two end up with the same prio | 09:13 |
| Continuity | We had to manually adjust | 09:13 |
| Continuity | To restore traffic flow | 09:13 |
| ralonsoh | Continuity, what was the previous version? | 09:13 |
| ralonsoh | did any GW chassis change the ID? Check in the logs if there was any ``ChassisEvent`` | 09:16 |
| *** ykarel_ is now known as ykarel | 09:21 | |
| haleyb | ralonsoh: hey, i just answered your question - it does seem to be testing on OVN. and sorry i keep getting disconnected terrible wifi today | 09:23 |
| ralonsoh | haleyb, no problem at all, we can talk on Monday | 09:27 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Add provider network resources filter for "network" https://review.opendev.org/c/openstack/neutron/+/949771 | 09:28 |
| haleyb | ralonsoh: ack, and i seemed to have figured out the test issue your plurality change fixed it, thanks for that. that fix has become a hot topic for me downstream, i was surprised when we moved a cloud to OVN and remote address groups didn't work | 09:31 |
| ralonsoh | haleyb, in any case, I have some concerns about this feature | 09:31 |
| ralonsoh | how can I manually test it? | 09:32 |
| ralonsoh | because following the procedure in the description | 09:32 |
| ralonsoh | (one sec, I'll give you the link) | 09:32 |
| ralonsoh | https://docs.openstack.org/neutron/latest/admin/config-address-scopes.html | 09:33 |
| haleyb | ralonsoh: i can add some notes to the original bug, i just can't access my cloud at the moment | 09:33 |
| *** elodille1 is now known as elodilles | 09:33 | |
| ralonsoh | haleyb, no rush, let's talk on monday | 09:33 |
| haleyb | ralonsoh: that link was address scopes, that change is for remote address groups | 09:33 |
| haleyb | https://bugs.launchpad.net/neutron/+bug/1982287 | 09:34 |
| ralonsoh | haleyb, ohh sorry | 09:34 |
| ralonsoh | yes, yes, I really need to sleep a bit | 09:34 |
| ralonsoh | haleyb, we can have problems with release team | 09:35 |
| haleyb | ralonsoh: we can talk monday then, as i mentioned the n-t-p patch seems to test it | 09:35 |
| ralonsoh | because we are backporting a feature | 09:35 |
| ralonsoh | https://review.opendev.org/c/openstack/neutron/+/949852/3/releasenotes/notes/ovn-driver-support-address-group-fac3d96af5093a17.yaml | 09:35 |
| ralonsoh | I know this is supported in OVN for a long time | 09:35 |
| ralonsoh | but backporting a feature is not correct | 09:36 |
| haleyb | ralonsoh: right, the feature itself merged in wallaby i think, this was more a bug fix | 09:36 |
| haleyb | i.e. no one tested that OVN worked or not | 09:36 |
| Continuity | ralonsoh: Still trying to get teh info about new and old versions. Nothing in the logs for ChassisEvent | 09:36 |
| ralonsoh | haleyb, the OVN feature was merged time ago, this patch 949852 is a new Neutron feature | 09:37 |
| ralonsoh | is also mentioned in the reno | 09:37 |
| ralonsoh | if we backport it, we'll have a warning from the release team for sure | 09:37 |
| ralonsoh | if you need it in 2024.2, I think you should backport it in D/S | 09:37 |
| ralonsoh | Continuity, once the L3 scheduler creates the HA_Chassis_Group, the only place where we can modify it is in the ChassisEvent | 09:39 |
| ralonsoh | if we detect that a GW chassis is deleted, we recalculate the HA_Chassis_Group and the related HA_Chassis | 09:40 |
| ralonsoh | but we clean any leftover | 09:40 |
| ralonsoh | and we don't create HA_Chassis registers with duplicated priority | 09:41 |
| ralonsoh | one sec | 09:41 |
| ralonsoh | Continuity, hold on, we still don't use HA_Chassis_Group for LRPs | 09:42 |
| ralonsoh | where did you find this duplicated register? what resource was used? an external Logical_Switch_Port? | 09:42 |
| ralonsoh | (I think I assumed GW LRP by default, but that is wrong) | 09:43 |
| Continuity | ralonsoh: ovn-nbctl ha-chassis-group-list | 09:49 |
| Continuity | 0618dbf9-914d-41de-accc-e710a51cb1ca (occ00003) priority 32766 | 09:50 |
| Continuity | 54fc840c-1192-497f-bbfa-51321f872877 (occ00001) priority 32766 | 09:50 |
| Continuity | 733f7b00-d9b0-40d7-ba05-3a15db0356be (occ00002) priority 32767 | 09:50 |
| Continuity | We had to adjust the top one, after aligning it with ovn-nbctl list Gateway_Chassis | 09:50 |
| Continuity | For awareness we are running baremetal nodes | 09:52 |
| haleyb | ralonsoh: let me find the the original series and follow-on fix, i'll add that comment in the 2024.2 change (just did) and will add a topic to weekly meeting | 09:54 |
| haleyb | ralonsoh: and hopefully you get some sleep | 09:54 |
| ralonsoh | Continuity, did any of these chassis change somehow? | 10:14 |
| ralonsoh | did you change the AZs? | 10:15 |
| ralonsoh | are all of them active now? | 10:15 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Require alembic db migration scripts to be idempotent https://review.opendev.org/c/openstack/neutron/+/950139 | 10:15 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Require alembic db migration scripts to be idempotent https://review.opendev.org/c/openstack/neutron/+/950139 | 10:42 |
| opendevreview | Ivan Anfimov proposed openstack/neutron master: Remove installation guide for openSUSE/SLES https://review.opendev.org/c/openstack/neutron/+/949868 | 11:13 |
| jkulik | Hi folks, when using flavors in Neutron to be able to create different types of routers, is there quota support for the different types/flavors of routers or just for the overall number of routers in general? | 12:11 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-fwaas master: [S-RBAC] New default API policies added https://review.opendev.org/c/openstack/neutron-fwaas/+/948060 | 12:20 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-fwaas stable/2025.1: Switch to neutron-tempest-plugin-fwaas job for 2025.1 branch https://review.opendev.org/c/openstack/neutron-fwaas/+/950156 | 12:25 |
| ralonsoh | jkulik, you can print the network quotas with $ openstack quota show --network. There are quotas for number of routers, that's all | 12:43 |
| jkulik | ok. thanks | 12:44 |
| slaweq | ykarel ralonsoh lajoskatona hi, do you know if the issue like in https://bugs.launchpad.net/neutron/+bug/2079048 still happens? I was trying to find some fresh logs but I couldn't | 12:58 |
| ralonsoh | slaweq, I don't remember, to be honest | 13:00 |
| ykarel | slaweq, don't recall seeing this from quite some time | 13:01 |
| slaweq | thx, I will close this LP then for now | 13:01 |
| ykarel | ok | 13:01 |
| slaweq | we can reopen if issue will happen again | 13:01 |
| ykarel | ++ | 13:01 |
| slaweq | ralonsoh lajoskatona haleyb please check https://review.opendev.org/c/openstack/neutron-fwaas/+/948060 when You will have few minutes | 14:47 |
| ralonsoh | sure | 14:47 |
| slaweq | and also https://review.opendev.org/c/openstack/neutron-fwaas/+/950156 which is small CI change only | 14:47 |
| lajoskatona | slaweq: sure. by the way thanks for checking my similar patches with parameterized :-) | 14:55 |
| slaweq | lajoskatona yw :) | 15:00 |
| opendevreview | Rico Lin proposed openstack/neutron master: Fix ovn db sync with log resources https://review.opendev.org/c/openstack/neutron/+/948053 | 15:15 |
| Continuity | ralonsoh: not that we can see. Ill get the lads to have a look through the logs | 16:19 |
| Continuity | And they are all active and working correctly now | 16:20 |
| opendevreview | Sebastian Lohff proposed openstack/neutron-fwaas master: Set service type when registering provider config https://review.opendev.org/c/openstack/neutron-fwaas/+/950202 | 16:26 |
| opendevreview | Sebastian Lohff proposed openstack/networking-bgpvpn master: Set service type when registering provider config https://review.opendev.org/c/openstack/networking-bgpvpn/+/950212 | 17:19 |
| opendevreview | Merged openstack/neutron master: Fix ovn_db_sync for user defined flavor routers https://review.opendev.org/c/openstack/neutron/+/949816 | 20:10 |
| opendevreview | Miguel Lavalle proposed openstack/neutron stable/2025.1: Fix ovn_db_sync for user defined flavor routers https://review.opendev.org/c/openstack/neutron/+/950239 | 21:18 |
| opendevreview | Miguel Lavalle proposed openstack/neutron stable/2024.2: Fix ovn_db_sync for user defined flavor routers https://review.opendev.org/c/openstack/neutron/+/950240 | 21:18 |
| opendevreview | Miguel Lavalle proposed openstack/neutron stable/2024.1: Fix ovn_db_sync for user defined flavor routers https://review.opendev.org/c/openstack/neutron/+/950241 | 21:19 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!