| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-05-15 | |||
| 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. | |
| 10:02:15 | sean-k-mooney | i raced with gibi | |
| 10:02:18 | sean-k-mooney | its on its way now | |
| 10:20:51 | opendevreview | Merged openstack/nova master: Mention IRC topic update in PTL post-PTG checklist https://review.opendev.org/c/openstack/nova/+/949711 | |
| 10:36:00 | gibi | :) | |
| 10:46:59 | opendevreview | Balazs Gibizer proposed openstack/nova master: Document threading mode and tuneables https://review.opendev.org/c/openstack/nova/+/949364 | |
| 10:46:59 | opendevreview | Balazs Gibizer proposed openstack/nova master: WIP: allow service to start with threading https://review.opendev.org/c/openstack/nova/+/948311 | |
| 10:47:00 | 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 | |
| 12:04:02 | gibi | sean-k-mooney: dansmith: most of the eventlet removal series is ready for review. You can ignore the last two patches for now but the rest should be good for a review round | |
| 12:04:20 | gibi | series starts here https://review.opendev.org/c/openstack/nova/+/947966 | |
| 12:04:54 | sean-k-mooney | ack, whats the status fo the oslo threading backend | |
| 12:05:22 | sean-k-mooney | are we still waiting on an inital release with that supprot? | |
| 12:05:54 | sean-k-mooney | i know the inial part of the seriese is not depending on that but has there been much progress? i saw some maining list posts asking to merge it | |
| 12:08:29 | gibi | sean-k-mooney: yeah we waiting for the merge and release | |
| 12:09:07 | gibi | there is good progress on the patch and has one +2 | |
| 12:09:15 | gibi | so it is not far behind | |
| 12:10:05 | sean-k-mooney | ack. i see the previous run on the first patch was green how has the ci stablity been so far | |
| 12:11:42 | gibi | I see now suspicious tempest test failure on the last patch in the nova-next job. So I would say we are in good shape tempestvise. I also run a munch of negative scenarios with hanging or slow DB. | |
| 12:11:54 | gibi | bunch of :) | |
| 12:12:01 | gibi | and I ran them locally obviously | |
| 12:13:21 | gibi | as we don't have the oslo.service release yet we will still have time to run nova-next a couple of times before we land the switchover | |
| 12:13:24 | gibi | in nova-next | |
| 12:14:10 | gibi | The test result are also looking good on the eventlet based nova-scheduler side. I don't track any suspicious failures. I rechecked failed patches with explanation pointing out existing known instabilities | |
| 12:14:27 | gibi | I will keep monitoring the results | |
| 12:15:24 | gibi | I will do one more integration check locally with the latest oslo.service patch version and then I will also +1 the patch there | |
| 12:15:49 | gibi | the author is responsive, he fixed couple of issue I reported and did improvements for us | |
| 12:16:18 | gibi | he is planning for folluw up patches based on recent non blocking comments | |
| 12:17:05 | sean-k-mooney | that all sounds good | |
| 12:17:31 | gibi | sean-k-mooney: if you want to run the unit/functional tests in the last couple of patches (after the doc patch) then you need to manually install the oslo.service patch in your venv as there we start to actively depend on the backend initialization logic | |
| 12:17:53 | gibi | same is true for the doc gen target as well | |
| 12:18:19 | sean-k-mooney | well | |
| 12:18:32 | sean-k-mooney | the docs is only ture if we are auto generating the config docs | |
| 12:18:46 | sean-k-mooney | liek it shoudl work with the old version but not render the new options right | |
| 12:19:09 | sean-k-mooney | or perhaps not | |