| opendevreview | Helen Chen proposed openstack/neutron master: Fix bug/2100294 to update IPv6 gateway in OVN https://review.opendev.org/c/openstack/neutron/+/962710 | 01:43 |
|---|---|---|
| open10k8s | hi folks, can i get feedback on this please? https://bugs.launchpad.net/neutron/+bug/2125500 | 04:18 |
| para_ | hi. i have a simple dns question: does designate+neutron dns integration supports private resolvers in private networks without internet access? i cant find the doc about this, neither in designate nor in neutron, so im wondering if the feature is available. thanks a lot | 08:48 |
| ralonsoh | para_, I don't know if we have the right people reading this now. Maybe you can try sending a mail to opendiscuss | 09:19 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron stable/2025.2: [OVS] Avoid duplicate logging of network packets https://review.opendev.org/c/openstack/neutron/+/964521 | 09:57 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron stable/2025.2: [OVS] Avoid duplicate logging of network packets https://review.opendev.org/c/openstack/neutron/+/964521 | 09:59 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron stable/2025.1: [OVS] Avoid duplicate logging of network packets https://review.opendev.org/c/openstack/neutron/+/964522 | 09:59 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron stable/2024.2: [OVS] Avoid duplicate logging of network packets https://review.opendev.org/c/openstack/neutron/+/964523 | 10:00 |
| opendevreview | Dai Dang Van proposed openstack/neutron master: Add dns forwarder l2 extension https://review.opendev.org/c/openstack/neutron/+/951390 | 10:42 |
| ralonsoh | open10k8s, I think I found the issue | 10:56 |
| ralonsoh | the problem seems to be in the elevated context used to update the port | 10:56 |
| ralonsoh | I'm trying a new approach, let me push a pathc | 10:56 |
| *** iurygregory_ is now known as iurygregory | 10:57 | |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Fix a race condition in OVN metadata port update https://review.opendev.org/c/openstack/neutron/+/962463 | 11:27 |
| opendevreview | Merged openstack/neutron master: [OVS] Avoid duplicate logging of network packets https://review.opendev.org/c/openstack/neutron/+/964477 | 11:52 |
| opendevreview | Merged openstack/neutron master: [UT] Allow any call order in ``test_send_ipv4_addr_adv_notif`` https://review.opendev.org/c/openstack/neutron/+/963921 | 11:52 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Fix a race condition in OVN metadata port update https://review.opendev.org/c/openstack/neutron/+/962463 | 13:31 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Fix a race condition in OVN metadata port update https://review.opendev.org/c/openstack/neutron/+/962463 | 13:32 |
| Uggla | Hi haleyb, do you think about a neutron/nova timeslot ? | 14:21 |
| haleyb | Uggla: hi, was just thinking about this as well. from what i can see in the proposals on the neutron etherpad there were only two items that overlap - https://bugs.launchpad.net/neutron/+bug/2120732 (metadata agent caching) and deprecating pass-through APIs | 14:23 |
| haleyb | i think we talked about the second there | 14:23 |
| haleyb | if you think it would be useful to meet for the bug that would be good, or maybe there are items on the nova etherpad to discuss? | 14:24 |
| Uggla | It seems I have this one too from lajoskatona https://bugs.launchpad.net/nova/+bug/2051685 | 14:25 |
| haleyb | ack, and that might just be a bug according to notes, so maybe a short x-project would work | 14:28 |
| lajoskatona | Uggla, haleyb: Hi, that bug is not Neutron realted, but I added the neutronclient migration topic | 14:28 |
| lajoskatona | not sure if we need anxproject for that | 14:28 |
| lajoskatona | I meant this bug: https://bugs.launchpad.net/nova/+bug/2051685 (After repeat of incomplete migration nova applies wrong (status=error) migration context in update_available_resource periodic job ) | 14:29 |
| haleyb | ok, so there's no neutron port issue there | 14:29 |
| lajoskatona | no that is related to migration only and no Neutron involment suprisingly :-) | 14:31 |
| Uggla | ok so we only have the client stuff topic | 14:33 |
| Uggla | in common | 14:33 |
| Uggla | lajoskatona, does 30mn ok for that ? | 14:33 |
| lajoskatona | I hope it will be too much :-) That will be perfect, thanks | 14:39 |
| haleyb | client and metadata | 14:39 |
| lajoskatona | ahh, ok I forgot the metadata | 14:39 |
| haleyb | sorry i'm in a meeting so not paying attention | 14:39 |
| Uggla | lajoskatona, and I'll find another slot for the migration bug. Is that ok ? | 14:40 |
| Uggla | lajoskatona, expected duration for the migration bug ? | 14:40 |
| lajoskatona | Uggla: I hope in 30minutes we can have that as well, Sunday afternoon I had a chat with sean-k-mooney, and based on that for me at least it is cleaner what should be a solution for that issue | 14:43 |
| Uggla | lajoskatona ok cool, that finally easier to cover all the points during the Neutron/Nova session. 👍 | 14:44 |
| Uggla | haleyb, just need to find a common 30mn timeslot now, ping me when you are free. | 14:45 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] The external networks GW chassis must the same as the GW LRP https://review.opendev.org/c/openstack/neutron/+/962155 | 14:55 |
| sean-k-mooney | lajoskatona: i just triaged this as high https://bugs.launchpad.net/nova/+bug/2051685 | 15:20 |
| sean-k-mooney | lajoskatona: in a least some case nova can move the migration form error to failed | 15:21 |
| sean-k-mooney | but that is potitlally unsafe to do if the admin has moved the vm to the same host again | 15:21 |
| sean-k-mooney | becasue https://opendev.org/openstack/nova/src/commit/a2fa7b1fa197b91aacac3aed9ba138905018465d/nova/compute/manager.py#L11940-L11993 | 15:21 |
| sean-k-mooney | will try and delete the instance files as part fo the clean up | 15:22 |
| sean-k-mooney | the commit that intoduced this was https://opendev.org/openstack/nova/commit/255b3f2f918843ca5dd9b99e109ecd2189b6b749 | 15:22 |
| sean-k-mooney | and it made some assumtion that intoduce a race condtion | 15:22 |
| sean-k-mooney | namely the assumed that the admin woudl not migrate the vm back to the same host before the cleanup ran on the dest | 15:23 |
| sean-k-mooney | in many cases that might eb true but nova is not guarding agaisnt that case and this is not valid to do in all cause where the migration errored | 15:23 |
| lajoskatona | sean-k-mooney: thanks for your time and the good discussion :-) | 15:25 |
| sean-k-mooney | talking atbout this at the sumit helped too | 15:27 |
| sean-k-mooney | i was not really aware we did https://opendev.org/openstack/nova/commit/255b3f2f918843ca5dd9b99e109ecd2189b6b749 | 15:27 |
| sean-k-mooney | i.e. maed error a transiatn sate instead of terminal with a perodic | 15:27 |
| sean-k-mooney | is this causing problem on the neutron side or si it purly on the nova side | 15:28 |
| sean-k-mooney | i see the current logic as having 2 undesireable sideffect | 15:29 |
| sean-k-mooney | first a vm can be accounted for twice once form the errord migration and once from the second migration that succeeded | 15:30 |
| sean-k-mooney | that woudl cause the duplicate cpu pinning issue that was orgianlly reported | 15:30 |
| sean-k-mooney | but also the _cleanup_incomplete_migrations function could alos lead to data lose in that case. | 15:31 |
| lajoskatona | sean-k-mooney: there are open patches on top of https://review.opendev.org/c/openstack/nova/+/745281 , perhaps related to this logic ? | 15:31 |
| sean-k-mooney | oh the other in the relation chang | 15:32 |
| sean-k-mooney | looking at them quickly no | 15:33 |
| sean-k-mooney | they are for other edgecases | 15:33 |
| sean-k-mooney | the dont appare to fix the bug intoduced by the first patch | 15:34 |
| sean-k-mooney | i think to "fix" this we need to do two things, 1 make sure we dont double count in update_aviable_resouces by checkign if the instnace is on the current host now. | 15:35 |
| sean-k-mooney | and 2 do not do clean up if the insnatce is assigend to the current host, just set the migration to failed | 15:35 |
| sean-k-mooney | let me put that in the bug report | 15:36 |
| sean-k-mooney | lajoskatona: i captured my toughts in https://bugs.launchpad.net/nova/+bug/2051685/comments/7 https://review.opendev.org/c/openstack/nova/+/747899/7 bight be a good impovement in general but its not a fix for the bug | 15:44 |
| lajoskatona | sean-k-mooney: cool, thanks, just thought to save it somewhere because I will forget for next week :-) | 15:46 |
| lajoskatona | sean-k-mooney: I try to check https://review.opendev.org/c/openstack/nova/+/747899/7 and understand what it is trying to fix, thanks for pointing | 15:47 |
| sean-k-mooney | its kind of simple | 15:47 |
| sean-k-mooney | its trying to change how nova treat inprogress live migration after an agent restart | 15:48 |
| sean-k-mooney | instead of setting them to error it trys to revert the migraiton and set it to faild | 15:48 |
| sean-k-mooney | im not sure that a good idea but that what its tryign to do | 15:48 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: bgp: Add thread to discover BGP connections for bridge https://review.opendev.org/c/openstack/neutron/+/964507 | 17:10 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: bgp: Add thread to discover BGP connections for bridge https://review.opendev.org/c/openstack/neutron/+/964507 | 17:53 |
| haleyb | Uggla: sorry for the slow response. we have times in our ptg schedule wednesday afternoon after 15:30, or any time thursday | 18:08 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: bgp: Keep bridge mappings consistent https://review.opendev.org/c/openstack/neutron/+/964429 | 18:15 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: bgp: Introduce Main router and chassis router commands https://review.opendev.org/c/openstack/neutron/+/959890 | 20:32 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: bgp: Introduce BGP chassis commands https://review.opendev.org/c/openstack/neutron/+/960422 | 20:32 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: bgp: OVN agent BGP extension https://review.opendev.org/c/openstack/neutron/+/960895 | 20:33 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: bgp: Detect BGP bridge creation https://review.opendev.org/c/openstack/neutron/+/964425 | 20:34 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: bgp: Keep bridge mappings consistent https://review.opendev.org/c/openstack/neutron/+/964429 | 20:34 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: bgp: Add thread to discover BGP connections for bridge https://review.opendev.org/c/openstack/neutron/+/964507 | 20:34 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: bgp: Add event to create BGP chassis resources https://review.opendev.org/c/openstack/neutron/+/961854 | 21:36 |
| opendevreview | Terry Wilson proposed openstack/neutron master: Handle virtual port parent lookup via Neutron DB https://review.opendev.org/c/openstack/neutron/+/957172 | 23:25 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!