| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-05-15 | |||
| 18:10:12 | cardoe | I don't disagree. | |
| 18:10:24 | cardoe | One of the many specs on my queue to write. | |
| 18:11:29 | sean-k-mooney | cardoe: without deep diving into the spec we probably can supprot addign the subport info into the metadtata api or config drive | |
| 18:11:37 | sean-k-mooney | for any virt dirver | |
| 18:11:40 | sean-k-mooney | even ironic | |
| 18:11:58 | sean-k-mooney | but only if the trunk port is pass by uuid as part of the boot request | |
| 18:12:23 | sean-k-mooney | if ironic makes changes to the prots out of band however | |
| 18:12:45 | sean-k-mooney | i.e. maping that 1 neutorn trunk port to to a port group | |
| 18:12:57 | sean-k-mooney | your kind of out of luck | |
| 18:13:50 | sean-k-mooney | so like you could not really combin doign bondign via port groups and this | |
| 18:14:54 | JayF | Yeah, that's been the root pain the whole time. To model this all in network metadata on the nova side requires it to be modeled all the way through, which I think has to start with neutron not with nova | |
| 18:15:14 | sean-k-mooney | cardoe: or rather if you did try and combind them nova can only provide the info it had ebfore we call ironci to provision in the config drive | |
| 18:15:50 | sean-k-mooney | JayF:right we can add any info that is aviabel from neutron at the point we generate the config drive | |
| 18:16:07 | sean-k-mooney | but after that point either ironic need to update it itsself | |
| 18:16:16 | sean-k-mooney | or we would need a way to trigger a refresh | |
| 18:16:24 | JayF | (patches are close to merge that have ironic regen it) | |
| 18:16:54 | sean-k-mooney | JayF: is that to adresss the bonding bug mnaser reported | |
| 18:17:08 | JayF | I think we addressed that in a backportable way | |
| 18:17:26 | cardoe | which is the bug mnaser reported? | |
| 18:18:00 | JayF | https://launchpad.net/bugs/2106073 | |
| 18:18:06 | sean-k-mooney | https://bugs.launchpad.net/ironic/+bug/2106073 | |
| 18:18:13 | sean-k-mooney | :)_ | |
| 18:18:33 | sean-k-mooney | cardoe: its a similar problam statement | |
| 18:19:04 | sean-k-mooney | the config drive is needed to be abel to bootstrap the networkign to be able to hit the metadta when you are usign static ips or bonds | |
| 18:19:37 | sean-k-mooney | specificly any bond that require both the swtich and host to be configured in a given bonding mode for traffic to flow | |
| 18:20:50 | sean-k-mooney | cardoe: your usecase is related but seperate and i think valid | |
| 18:21:10 | sean-k-mooney | cardoe: ironic could enrich the metadat with the turnk port info | |
| 18:21:18 | sean-k-mooney | btu nova can do that too | |
| 18:21:33 | sean-k-mooney | and nova could do it for libvirt or vmware in a generic way | |
| 18:34:24 | cardoe | I would love if neutron modeled this all. Much like I’m trying to get L2 VNI in there. | |
| 18:35:06 | sean-k-mooney | cardoe: when we dicussed trunkports intially we also disucssed bond ports | |
| 18:35:23 | cardoe | Which is something already patched in nova | |
| 18:35:28 | sean-k-mooney | the latter just didnt get doen btu i dont see why you cant have the same bond port sub port relationship | |
| 18:36:06 | sean-k-mooney | port groups in ironci are a littel differnt however then just a bond or turnk port | |
| 18:36:10 | cardoe | Trunk boards are already modeled inside of neutron | |
| 18:36:16 | cardoe | Ports | |
| 18:36:41 | cardoe | It’s nova that doesn’t handle it | |
| 18:37:13 | cardoe | So it’s not really related to that bug. | |
| 18:41:40 | sean-k-mooney | cardoe: so i did a very quick review and asked a few questions but +1 for the over all idea | |
| 18:43:23 | sean-k-mooney | cardoe: im not sure if we currently test turnkprots in the nova ci jobs (maybe the ovs hybrid plug one?) but i know the neutron do and have a tempest plugin to do some more advanced testing with it | |
| 18:43:58 | sean-k-mooney | so whiel im not sure if the trunk connectivty test would be accepted in core tempest it definlly could be done in one of neutron tempest plugins | |
| 18:49:44 | sean-k-mooney | melwitt: speaking of specs are you plannign to work on the vtpm spec and implemetion this cycle. i think that was the plan aroudn the ptg but just touching base to make sure that has not changed? | |
| 18:50:31 | melwitt | argh | |
| 18:50:45 | melwitt | sean-k-mooney: yes I am -- still the plan. I have re-proposed the spec and noted the changes from the last approval in the commit message | |
| 18:51:15 | sean-k-mooney | cool. i will proably try and put aside some time to look at the open spec next week | |
| 18:51:48 | melwitt | sure, thanks | |
| 18:56:03 | melwitt | seen this a few times now, nova-ceph-multistore failures due to "nova.exception.ImageDeleteConflict: Conflict deleting image. Reason: HTTP 409 Conflict: Image a34561ed-6ef7-457c-8845-677f83af10de could not be deleted because it is in use: The image cannot be deleted because it is in use through the backend store outside of Glance.." | |
| 18:56:54 | sean-k-mooney | ya | |
| 18:56:57 | sean-k-mooney | i noticed that too | |
| 18:57:22 | melwitt | I wonder if anything's changed or why it's happening often recently. I haven't spent time looking into it yet | |
| 19:10:55 | cardoe | sean-k-mooney: https://review.opendev.org/c/openstack/nova/+/941227 is the implementation. Linked you’ll find the addition of tests and tempest patches as well to test it all. | |
| 19:32:13 | sean-k-mooney | cardoe: can you put the implemation and the spec on a gerrit topic called bp/<sepc/blueprint name> | |
| 19:33:18 | sean-k-mooney | convetionally the topic shoudl eb "bp/expose-vlan-trunking" in thei case but "spec/expose-vlan-trunking" is also perfectly fine | |
| 19:33:28 | cardoe | Will do. | |
| 19:33:59 | sean-k-mooney | cardoe: we try to keep teh filename/topic/spec link all the same. | |
| 19:36:52 | sean-k-mooney | cardoe: im goint to call it a day now but did you sway there are also existing tempest tests | |
| 19:37:37 | sean-k-mooney | cardoe: ah tehy were abandoned https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/941355 | |
| 19:55:27 | cardoe | sorry was replying from mobile. I'll resurrect things. | |
| #openstack-nova - 2025-05-16 | |||
| 04:49:26 | opendevreview | Merged openstack/nova master: live migration: Avoid volume rollback mismatches https://review.opendev.org/c/openstack/nova/+/946600 | |
| 06:48:03 | ishanwar[m] | dansmith: sean-k-mooney melwitt Can you please review | |
| 06:48:03 | ishanwar[m] | https://review.opendev.org/c/openstack/nova/+/945204 ? | |
| 07:51:33 | Uggla | melwitt, I agree I have seen the glance "issue" with the bug that was holding the gate this week. https://review.opendev.org/c/openstack/nova/+/949622, I rechecked and it passes. | |
| 07:53:18 | Uggla | melwitt, this makes the job quite flaky. | |
| 07:54:36 | sean-k-mooney | Uggla: my guess is we are not waitign for the image to be in an actiobal state or soemthign liek that | |
| 07:54:47 | sean-k-mooney | i.e. it might not yet be active when we try to delete it | |
| 07:55:22 | sean-k-mooney | the fix may be to make the test wait for glance to finsih doeing whatever its doing before finsihing | |
| 07:57:34 | Uggla | sean-k-mooney, I have not stat made stats to know if it happens a lot not. But I guess if I noticed it, it should be quite frequent. Maybe something we could quickly discuss in our next nova meeting. | |
| 09:42:29 | opendevreview | Masanori Kuroha proposed openstack/nova-specs master: Support for tracking traits removed from provider.yaml https://review.opendev.org/c/openstack/nova-specs/+/937587 | |
| 10:19:42 | sean-k-mooney | ishanwar[m]: i have left comments in gerrit | |
| 11:13:33 | opendevreview | Ivan Anfimov proposed openstack/placement master: Remove installation guide for openSUSE/SLES https://review.opendev.org/c/openstack/placement/+/949864 | |
| 14:14:53 | opendevreview | Stanislav Dmitriev proposed openstack/nova master: Retry on deadlocks while updating instance extra https://review.opendev.org/c/openstack/nova/+/950186 | |
| 16:52:34 | ishanwar[m] | <sean-k-mooney> "ishanwar: i have left comments..." <- Thanks sean-k-mooney will have a look | |
| 17:09:13 | opendevreview | Dan Smith proposed openstack/placement master: Fix placement allocate while over-capacity https://review.opendev.org/c/openstack/placement/+/945465 | |
| 17:30:55 | opendevreview | melanie witt proposed openstack/nova stable/2025.1: Amend functional reproducer for bug 1899835 https://review.opendev.org/c/openstack/nova/+/950213 | |
| 17:30:56 | opendevreview | melanie witt proposed openstack/nova stable/2025.1: live migration: Avoid volume rollback mismatches https://review.opendev.org/c/openstack/nova/+/950214 | |
| 17:31:51 | opendevreview | melanie witt proposed openstack/nova stable/2024.2: Amend functional reproducer for bug 1899835 https://review.opendev.org/c/openstack/nova/+/950215 | |
| 17:31:52 | opendevreview | melanie witt proposed openstack/nova stable/2024.2: live migration: Avoid volume rollback mismatches https://review.opendev.org/c/openstack/nova/+/950216 | |
| 17:32:38 | opendevreview | melanie witt proposed openstack/nova stable/2024.1: Amend functional reproducer for bug 1899835 https://review.opendev.org/c/openstack/nova/+/950217 | |
| 17:32:39 | opendevreview | melanie witt proposed openstack/nova stable/2024.1: live migration: Avoid volume rollback mismatches https://review.opendev.org/c/openstack/nova/+/950218 | |
| #openstack-nova - 2025-05-18 | |||
| 20:51:54 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add extra spec for sound device. https://review.opendev.org/c/openstack/nova/+/926126 | |
| 20:51:55 | opendevreview | Michael Still proposed openstack/nova master: Protect older compute managers from sound model requests. https://review.opendev.org/c/openstack/nova/+/940770 | |
| 20:51:55 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add extra specs for USB redirection. https://review.opendev.org/c/openstack/nova/+/927354 | |
| #openstack-nova - 2025-05-19 | |||
| 09:27:20 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Use common naming convention for ephemeral disk labels https://review.opendev.org/c/openstack/nova/+/947541 | |
| 09:50:40 | opendevreview | Balazs Gibizer proposed openstack/nova master: Translate scatter-gather to futurist https://review.opendev.org/c/openstack/nova/+/947966 | |
| 09:50:41 | opendevreview | Balazs Gibizer proposed openstack/nova master: Use futurist for _get_default_green_pool() https://review.opendev.org/c/openstack/nova/+/948072 | |
| 09:50:41 | opendevreview | Balazs Gibizer proposed openstack/nova master: Replace utils.spawn_n with spawn https://review.opendev.org/c/openstack/nova/+/948076 | |
| 09:50:42 | opendevreview | Balazs Gibizer proposed openstack/nova master: Add spawn_on https://review.opendev.org/c/openstack/nova/+/948079 | |
| 09:50:42 | opendevreview | Balazs Gibizer proposed openstack/nova master: Move ComputeManager to use spawn_on https://review.opendev.org/c/openstack/nova/+/948186 | |
| 09:50:43 | opendevreview | Balazs Gibizer proposed openstack/nova master: Move ConductorManager to use spawn_on https://review.opendev.org/c/openstack/nova/+/948187 | |
| 09:50:43 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make nova.utils.pass_context private https://review.opendev.org/c/openstack/nova/+/948188 | |
| 09:50:44 | opendevreview | Balazs Gibizer proposed openstack/nova master: Rename DEFAULT_GREEN_POOL to DEFAULT_EXECUTOR https://review.opendev.org/c/openstack/nova/+/948086 | |
| 09:50:45 | opendevreview | Balazs Gibizer proposed openstack/nova master: Print ThreadPool statistics https://review.opendev.org/c/openstack/nova/+/948340 | |
| 09:50:45 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make the default executor configurable https://review.opendev.org/c/openstack/nova/+/948087 | |
| 09:50:47 | opendevreview | Balazs Gibizer proposed openstack/nova master: Document threading mode and tuneables https://review.opendev.org/c/openstack/nova/+/949364 | |
| 09:50:47 | opendevreview | Balazs Gibizer proposed openstack/nova master: WIP: allow service to start with threading https://review.opendev.org/c/openstack/nova/+/948311 | |
| 09:50:49 | opendevreview | Balazs Gibizer proposed openstack/nova master: DNM:Run nova-next with n-sch in threading mode https://review.opendev.org/c/openstack/nova/+/948450 | |
| 09:52:25 | Uggla | bauzas, sean-k-mooney if you have 5mn can tou please review 949711: Mention IRC topic update in PTL post-PTG checklist | https://review.opendev.org/c/openstack/nova/+/949711 | |
| 09:55:19 | sean-k-mooney | Uggla: sure im aboot to kick off a vm os install so ill finsih that and then take a look | |
| 09:56:22 | Uggla | sean-k-mooney, thx no hurries at all. | |
| 09:56:39 | Uggla | This is more to not forget about it. | |