| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-05-15 | |||
| 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] | https://review.opendev.org/c/openstack/nova/+/945204 ? | |
| 06:48:03 | ishanwar[m] | dansmith: sean-k-mooney melwitt Can you please review | |
| 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: libvirt: Add extra specs for USB redirection. https://review.opendev.org/c/openstack/nova/+/927354 | |
| 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 | |
| #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: Replace utils.spawn_n with spawn https://review.opendev.org/c/openstack/nova/+/948076 | |
| 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:42 | opendevreview | Balazs Gibizer proposed openstack/nova master: Move ComputeManager to use spawn_on https://review.opendev.org/c/openstack/nova/+/948186 | |
| 09:50:42 | opendevreview | Balazs Gibizer proposed openstack/nova master: Add spawn_on https://review.opendev.org/c/openstack/nova/+/948079 | |
| 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:43 | opendevreview | Balazs Gibizer proposed openstack/nova master: Move ConductorManager to use spawn_on https://review.opendev.org/c/openstack/nova/+/948187 | |
| 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: Make the default executor configurable https://review.opendev.org/c/openstack/nova/+/948087 | |
| 09:50:45 | opendevreview | Balazs Gibizer proposed openstack/nova master: Print ThreadPool statistics https://review.opendev.org/c/openstack/nova/+/948340 | |
| 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:47 | opendevreview | Balazs Gibizer proposed openstack/nova master: Document threading mode and tuneables https://review.opendev.org/c/openstack/nova/+/949364 | |
| 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: WIP: allow service to start with threading https://review.opendev.org/c/openstack/nova/+/948311 | |
| 10:46:59 | opendevreview | Balazs Gibizer proposed openstack/nova master: Document threading mode and tuneables https://review.opendev.org/c/openstack/nova/+/949364 | |
| 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 | |
| 12:19:23 | gibi | the docs target imports code, the import trigger backend initialization and in the current oslo.service code mergerd we cannot call init multiple times with the same backend. In the open oslo.service patch this is imporoved. | |
| 12:19:27 | sean-k-mooney | it kind of depend on what code paths we are hitting | |
| 12:19:41 | sean-k-mooney | ah ok | |
| 12:20:15 | sean-k-mooney | ya so that not the first time that has casued isseus. we have the eventlet monkey patch env arg because of that too | |
| 12:20:31 | sean-k-mooney | i.e. we disable monkeypatching in the docs env due to the multiple imports | |
| 12:20:54 | sean-k-mooney | that and well we dont need to monkey patch sphinx build | |
| 12:21:07 | sean-k-mooney | /need/want/ | |
| 12:21:12 | gibi | yeah it is similar | |
| 12:21:29 | gibi | but it will work OK as soon as the open oslo.service patch released | |
| 12:21:55 | gibi | as it will allow initing the backend mutliple times if we init it with the same type and it will do noop after the first init | |
| 12:23:49 | sean-k-mooney | if that got held up for any reason we could just wrap our call to init in the run_once decorator anyway to work around it | |
| 12:24:03 | sean-k-mooney | but cool none of that sounds concerning to me | |
| 12:24:59 | gibi | sean-k-mooney: btw, could you check https://review.opendev.org/c/openstack/devstack/+/948436 as well? | |
| 12:26:03 | gibi | that is the small devstack patch to make it possible to pass env variables from zuul.yaml to systemd | |
| 12:26:10 | sean-k-mooney | i can proably check that first :) im workign on seting up docs/reno for a new project currently so i dont have time to deep dive into the eventlet serise right away btu the devstack patch should be fine | |
| 12:27:32 | sean-k-mooney | gibi: i looked at that when you first pushed it so im +1 on it still | |
| 12:27:40 | gibi | thanks | |
| 12:28:04 | sean-k-mooney | gibi: since we are talking about devstack https://review.opendev.org/c/openstack/devstack/+/939825 | |