| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-10-15 | |||
| 15:51:57 | cid | \o/ | |
| 15:53:03 | JayF | cid: feel free to attend or not, I thought I'd put both of us on the ping list :) | |
| 15:53:40 | cid | Certainly. I will be attending. | |
| 16:01:24 | Uggla | adamcarthur5 are you in the room ? | |
| 16:01:56 | JayF | yeah he's in | |
| 19:23:55 | opendevreview | Dan Smith proposed openstack/nova-specs master: Add cross-cell-live-migration spec https://review.opendev.org/c/openstack/nova-specs/+/1008081 | |
| #openstack-nova - 2026-10-16 | |||
| 05:24:24 | abhishekk | @dansmith answer for your cache related question for multipart download support, https://review.opendev.org/c/openstack/glance/+/434558 | |
| 05:24:43 | abhishekk | sorry wrong channel | |
| 07:03:44 | opendevreview | Thomas Goirand proposed openstack/placement master: Honor sys.argv when parsing the config in the WSGI entry point https://review.opendev.org/c/openstack/placement/+/1010823 | |
| 07:53:17 | opendevreview | Thomas Goirand proposed openstack/placement master: Honor sys.argv when parsing the config in the WSGI entry point https://review.opendev.org/c/openstack/placement/+/1010823 | |
| 09:50:23 | opendevreview | Thomas Goirand proposed openstack/nova master: Lock the info cache row in instance_info_cache_update https://review.opendev.org/c/openstack/nova/+/1010847 | |
| 13:29:58 | opendevreview | Takashi Kajinami proposed openstack/nova master: Revert "disable ovn based testing of test_live_migration_with_trunk" https://review.opendev.org/c/openstack/nova/+/997768 | |
| 13:42:23 | nathharp_ | hey all - I've not proposed any fixes for nova before. What's the etiquette on asking for reviews? | |
| 15:49:46 | opendevreview | fivetime proposed openstack/nova master: Decide osprofiler tracing after the config is parsed https://review.opendev.org/c/openstack/nova/+/1010918 | |
| 16:31:51 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reproduce trusted cert bugs https://review.opendev.org/c/openstack/nova/+/1010021 | |
| 16:32:15 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reproduce and document trusted cert bugs https://review.opendev.org/c/openstack/nova/+/1010021 | |
| 16:36:47 | dansmith | gibi: by the way, I asked on the ops session this morning if anyone cares about this feature | |
| 16:37:16 | dansmith | gibi: there was basically no response, but the little discussion we had was mostly about all the ways it provides no guarantees worth relying on | |
| 16:37:47 | gibi | :) | |
| 16:38:15 | gibi | thanks of asking ops about it | |
| 16:40:11 | gibi | then maybe I will stop at this repro and doc patch and does not try to fix the image cache part either | |
| 16:42:28 | dansmith | I don't think it's worth fixing, FWIW | |
| 16:43:02 | dansmith | even if you plugged that hole, there are issues with the overall thing being useful.. spending resources on CC is probably a much better use of time | |
| 16:43:14 | gibi | yeah make sense | |
| 16:44:16 | gibi | should we deprecate the field in the server create and rebuild API? and maybe couple of releases later return 409 if provided? (very much not microversion friendly but I don't know how to rip this out) | |
| 16:45:38 | dansmith | idk, I guess I care so little it feels like not even worth doing the deprecation work, but advertising a false security feature is also bad | |
| 17:07:31 | gmaan | you mean trusted cert one? | |
| 17:10:54 | gmaan | if this one, i agree that no benefit of deprecation until we remove it completely without microversion which is hard to. Documenting it as know issue is kind of warning/deprecation signal. | |
| 17:11:21 | dansmith | yup | |
| 20:18:19 | opendevreview | Thomas Goirand proposed openstack/placement master: Honor sys.argv when parsing the config in the WSGI entry point https://review.opendev.org/c/openstack/placement/+/1010823 | |
| 20:42:41 | opendevreview | Chris Breu proposed openstack/nova master: libvirt: reserve device slots for new Cinder data volumes https://review.opendev.org/c/openstack/nova/+/1011008 | |
| 20:43:05 | opendevreview | Chris Breu proposed openstack/nova-specs master: Add spec for reserving Cinder device slots on resize https://review.opendev.org/c/openstack/nova-specs/+/1011009 | |
| #openstack-nova - 2026-10-17 | |||
| 06:27:54 | opendevreview | fivetime proposed openstack/nova master: Do not share nested objects with the cross-cell clone https://review.opendev.org/c/openstack/nova/+/1011054 | |
| 14:56:46 | opendevreview | YushoYamaguchi proposed openstack/nova-specs master: Allow disabling host checksum offload for vhost-user https://review.opendev.org/c/openstack/nova-specs/+/1011067 | |
| 21:20:27 | opendevreview | Merged openstack/nova master: libvirt: update description for live_migration_completion_timeout https://review.opendev.org/c/openstack/nova/+/874083 | |
| #openstack-nova - 2026-10-19 | |||
| 02:36:57 | opendevreview | ChungWon Lee proposed openstack/nova master: Reduce node build fail value by time https://review.opendev.org/c/openstack/nova/+/939929 | |
| 03:55:40 | opendevreview | Seyeong Kim proposed openstack/nova master: Keep destination PCI claims during live migration checks https://review.opendev.org/c/openstack/nova/+/1011197 | |
| 14:10:02 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reproduce and document trusted cert bugs https://review.opendev.org/c/openstack/nova/+/1010021 | |
| 14:43:50 | opendevreview | Dan Smith proposed openstack/nova master: Fix comment about libvirt domain name usage https://review.opendev.org/c/openstack/nova/+/1011317 | |
| 14:44:04 | dansmith | trivial comment correctness review ^ | |
| 14:44:40 | dansmith | ah dammit | |
| 14:45:11 | opendevreview | Dan Smith proposed openstack/nova master: Fix comment about libvirt domain name usage https://review.opendev.org/c/openstack/nova/+/1011317 | |
| 16:11:00 | gmaan | done | |
| 18:13:13 | opendevreview | Merged openstack/nova master: trivial: Fix outstanding flake8 issues https://review.opendev.org/c/openstack/nova/+/999714 | |
| 18:31:07 | opendevreview | Dan Smith proposed openstack/nova master: Make libvirt update domain name during live migration https://review.opendev.org/c/openstack/nova/+/1011366 | |
| 18:31:07 | opendevreview | Dan Smith proposed openstack/nova master: Change the instance_name_template default to use UUID https://review.opendev.org/c/openstack/nova/+/1011367 | |
| 19:02:53 | opendevreview | Merged openstack/nova master: Fix comment about libvirt domain name usage https://review.opendev.org/c/openstack/nova/+/1011317 | |
| 19:28:43 | opendevreview | Merged openstack/nova master: Add ruff-check https://review.opendev.org/c/openstack/nova/+/974441 | |
| 19:34:40 | opendevreview | melanie witt proposed openstack/nova master: Enable G (flake8-logging-format) rules https://review.opendev.org/c/openstack/nova/+/998557 | |
| #openstack-nova - 2026-10-20 | |||
| 01:19:00 | opendevreview | Adam Harwell proposed openstack/nova master: Stop the service when its start fails https://review.opendev.org/c/openstack/nova/+/1009119 | |
| 07:42:06 | opendevreview | fivetime proposed openstack/nova master: Do not share nested objects with the cross-cell clone https://review.opendev.org/c/openstack/nova/+/1011054 | |
| 08:56:34 | opendevreview | Juan Larriba proposed openstack/nova master: Add OpenTelemetry tracing middleware for Nova https://review.opendev.org/c/openstack/nova/+/982871 | |
| 09:21:42 | opendevreview | Kamil Sambor proposed openstack/nova master: deprecate: concurrency_backend config option https://review.opendev.org/c/openstack/nova/+/1011450 | |
| 11:02:10 | opendevreview | Moritz Tanner proposed openstack/nova master: libvirt: Support extending local volumes https://review.opendev.org/c/openstack/nova/+/878763 | |
| 12:27:15 | opendevreview | Kamil Sambor proposed openstack/nova master: Fix pickling error that forces oslo.service to fall back to fork https://review.opendev.org/c/openstack/nova/+/1000956 | |
| 13:21:13 | opendevreview | Juan Larriba proposed openstack/nova master: Add OpenTelemetry tracing middleware for Nova https://review.opendev.org/c/openstack/nova/+/982871 | |
| 13:40:44 | opendevreview | Abhishek Kekane proposed openstack/nova master: Add Glance preferred stores support https://review.opendev.org/c/openstack/nova/+/1011470 | |
| 14:03:18 | sean-k-mooney | gibi: fyi i noticed a neutron fixture race in on of kamil's test under threading. i have a local fix and ill file a bug and push it up soon | |
| 14:03:38 | sean-k-mooney | tldr we can mutate ports in the nturon fixutre whiel we are iterating over them causign a runtime error | |
| 14:05:42 | gibi | sean-k-mooney: ohh good catch | |
| 14:11:19 | opendevreview | Dan Smith proposed openstack/nova master: Change the instance_name_template default to use UUID https://review.opendev.org/c/openstack/nova/+/1011367 | |
| 14:35:25 | sean-k-mooney | gibi: just waiting for the tests to finish locally but this is the bug report https://bugs.launchpad.net/nova/+bug/2171516 | |
| 14:35:33 | sean-k-mooney | im not sure how often that happens | |
| 14:35:43 | sean-k-mooney | its the first time i have seen it but who knows | |
| 15:11:50 | opendevreview | Kamil Sambor proposed openstack/nova master: deprecate: concurrency_backend config option https://review.opendev.org/c/openstack/nova/+/1011495 | |
| 15:16:36 | opendevreview | Kamil Sambor proposed openstack/nova master: deprecate: concurrency_backend config option https://review.opendev.org/c/openstack/nova/+/1011450 | |
| 15:26:18 | gibi | sambork: could you put the deprecation patch top of the default changing spec to make it more logical? | |
| 15:26:27 | gibi | s/spec/patch/ | |
| 15:26:53 | sambork | @gibi, sure, sorry forgot! | |
| 15:26:58 | gibi | no worries | |
| 15:27:24 | sean-k-mooney | shoudl it not be before? | |
| 15:27:24 | gibi | also, can we switch the default for the CLIs as well? | |
| 15:27:32 | sean-k-mooney | i gues either order is ok | |
| 15:27:39 | gibi | I would not deprecated the old option until we default to it | |
| 15:35:11 | opendevreview | sean mooney proposed openstack/nova master: Synchronize Neutron fixture port operations https://review.opendev.org/c/openstack/nova/+/1011497 | |
| 15:35:27 | sean-k-mooney | gibi: ^ pass locally but that the fix | |
| 15:36:35 | sean-k-mooney | @utils.synchronized('neutron-fixture') is an inmemory lock by default so it shoudl nto block parallel tox workers so i think that pretty simple | |
| 16:03:04 | opendevreview | Merged openstack/nova master: Stop hanging the test if threadpool executor is stuck in cleanup https://review.opendev.org/c/openstack/nova/+/1006351 | |
| 16:03:20 | opendevreview | Merged openstack/nova master: Reduce the unnecessary executors shutdown logs in test jobs https://review.opendev.org/c/openstack/nova/+/1004160 | |
| 16:26:47 | opendevreview | Kamil Sambor proposed openstack/nova master: deprecate: concurrency_backend config option https://review.opendev.org/c/openstack/nova/+/1011513 | |
| 17:19:53 | tkajinam | sambork, it seems you proposed duplicated changes to deprecate concurrency_backend ? | |
| 17:19:58 | tkajinam | 1011450: deprecate: concurrency_backend config option | https://review.opendev.org/c/openstack/nova/+/1011450 | |
| 17:20:05 | tkajinam | 1011513: deprecate: concurrency_backend config option | https://review.opendev.org/c/openstack/nova/+/1011513 | |
| 17:20:35 | tkajinam | I guess 1011513 is correct because it follows but we might need your confirmation | |
| 17:21:51 | gmaan | i think so | |
| 17:22:30 | gmaan | sambork: also one comment from gibi on CLI default switch too, if you can check and accordingly I will +w this https://review.opendev.org/c/openstack/nova/+/1009027 | |
| 17:23:28 | gmaan | oh, you are doing that in next change, nevermind +w the base one https://review.opendev.org/c/openstack/nova/+/1011513 | |
| 17:23:54 | sean-k-mooney | we have duplicates now https://review.opendev.org/c/openstack/nova/+/1011450 | |
| 17:24:08 | sean-k-mooney | and https://review.opendev.org/c/openstack/nova/+/1011513 | |
| 17:25:08 | sean-k-mooney | to me changing the default bahvior and deprection the config option are unrelated | |
| 17:25:19 | sean-k-mooney | we are chaing the meaing of auto | |
| 17:25:26 | sean-k-mooney | not the default of the config option | |
| 17:25:46 | sean-k-mooney | but as long as we abandon one fo the deprecation patches its fine | |
| 17:28:46 | gmaan | 1011513 is more than just deprecating the config but also changing default for CLI | |
| 17:28:59 | gmaan | anyways i commented there in gerrit | |
| 17:29:24 | gmaan | title of 1011513 and 1011450 are same but they are not duplicate | |
| 18:16:01 | abhishek_ | sean-k-mooney, dansmith: Circling back to rbd ec snapshot topic from PTG, I have tested locally with changes made to devstack-plugin-ceph https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/1011533 | |
| 18:16:48 | abhishek_ | the snapshot now gets stored in data (EC) pool and metadata stored in metadata pool, so we don't need any changes explicitly done in nova to expose data pool | |
| 18:25:42 | sean-k-mooney | abhishek_: i think that woudl also be true for maniala by the way | |
| 18:26:08 | sean-k-mooney | i saw there was a dicsion about addign some extra spec for the data pool | |