| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-10-20 | |||
| 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 | |
| 18:26:24 | abhishek_ | right | |
| 18:28:06 | sean-k-mooney | i have never actuly use manilla with ceph but as i noted on the ptg i used to deploy kolla ansibel with all my storage on ec pools in ceph a decade ago | |
| 18:28:14 | sean-k-mooney | so im glad that didnt break along the way | |
| 18:29:09 | gouthamr | nope, not going to break if you don't set an optional flag.. this is an opt-in feature.. | |
| 18:29:14 | abhishek_ | yeah, thanks for the pointers | |
| 18:29:36 | sean-k-mooney | gouthamr: what i was sayign is you probely dont need to add the feature at all | |
| 18:30:16 | gouthamr | sean-k-mooney: yes, you don't - today you can do it transparently to manila.. but, its not operator-friendly.. i.e., they may want to direct provisioning to an EC pool | |
| 18:30:58 | sean-k-mooney | gouthamr: well for a stroage project have ing more advanced policy over this might make sense | |
| 18:31:28 | sean-k-mooney | gouthamr: just noting that the ceph native cofnigution has been supproted in galnce, nova and cidner for a very long time | |
| 18:32:00 | sean-k-mooney | but as i also noted in the etherpad i dont knwo that the service projects have ever docudmented how to do it | |
| 18:32:16 | sean-k-mooney | but the installer and ceph comunity have in the past | |
| 18:32:30 | sean-k-mooney | so its one of those thing that if you dont knwo to ask you wont find out about | |
| 18:55:15 | opendevreview | Abhishek Kekane proposed openstack/nova master: Add Glance preferred stores support https://review.opendev.org/c/openstack/nova/+/1011470 | |
| 18:56:58 | sean-k-mooney | abhishek_: for ^ it would be nice if you enabeld that in the multi store job we have if it does not conflict with what that is already testing | |
| 18:57:10 | sean-k-mooney | or if we coudl enabel it in one of the other jobs if it does | |
| 18:58:21 | abhishek_ | ack, I will check and act accordingly (might need to add specific tempest test as well for this in that case) | |
| 19:00:12 | sean-k-mooney | if the current multi store config would configlict i woudl proably enabel this in the nova-alt-config job instead | |
| 19:00:23 | sean-k-mooney | i dont think that is currently testing any special storage config | |
| 19:00:29 | sean-k-mooney | so its less likely to conflict | |
| 19:01:43 | sean-k-mooney | that job existis to test non default configuration of nova withou any suggestion that it will become the new default unlike nova-next. | |
| 19:06:32 | abhishek_ | ack | |
| 19:50:15 | opendevreview | Abhishek Kekane proposed openstack/nova master: Add Glance preferred stores support https://review.opendev.org/c/openstack/nova/+/1011470 | |
| 19:53:57 | opendevreview | Merged openstack/nova master: Default native threading for console proxies https://review.opendev.org/c/openstack/nova/+/1009027 | |
| 23:23:02 | opendevreview | Dan Smith proposed openstack/nova master: Add old_cell_id to InstanceMapping DB model https://review.opendev.org/c/openstack/nova/+/1008834 | |
| 23:23:03 | opendevreview | Dan Smith proposed openstack/nova master: InstanceMapping object changes for old_cell_id https://review.opendev.org/c/openstack/nova/+/1008835 | |
| 23:23:03 | opendevreview | Dan Smith proposed openstack/nova master: Make external_events send to source and destination https://review.opendev.org/c/openstack/nova/+/1008836 | |
| 23:23:04 | opendevreview | Dan Smith proposed openstack/nova master: Instance lookup support for mid-cell-migration https://review.opendev.org/c/openstack/nova/+/1008837 | |
| 23:23:04 | opendevreview | Dan Smith proposed openstack/nova master: Initial conductor plumbing for conductor-orchestrated live migration https://review.opendev.org/c/openstack/nova/+/1008838 | |
| 23:23:05 | opendevreview | Dan Smith proposed openstack/nova master: REST API changes for conductor-orchestrated live migration https://review.opendev.org/c/openstack/nova/+/1008839 | |
| 23:23:05 | opendevreview | Dan Smith proposed openstack/nova master: Compute RPC refactors for re-use later https://review.opendev.org/c/openstack/nova/+/1008840 | |
| 23:23:06 | opendevreview | Dan Smith proposed openstack/nova master: Compute RPC methods for live migration v2 https://review.opendev.org/c/openstack/nova/+/1008841 | |
| 23:23:06 | opendevreview | Dan Smith proposed openstack/nova master: Add conductor-orchestrated live migration https://review.opendev.org/c/openstack/nova/+/1008842 | |
| 23:23:08 | opendevreview | Dan Smith proposed openstack/nova master: Functional tests for cross-cell live migrate https://review.opendev.org/c/openstack/nova/+/1008843 | |
| 23:23:08 | opendevreview | Dan Smith proposed openstack/nova master: Doc changes for cross-cell live migrate https://review.opendev.org/c/openstack/nova/+/1008844 | |
| 23:23:10 | opendevreview | Dan Smith proposed openstack/nova master: WIP: Extra instance check for migration_context https://review.opendev.org/c/openstack/nova/+/1008845 | |
| #openstack-nova - 2026-10-21 | |||
| 02:54:15 | opendevreview | YushoYamaguchi proposed openstack/nova-specs master: Allow disabling host checksum offload for vhost-user https://review.opendev.org/c/openstack/nova-specs/+/1011067 | |
| 03:37:12 | opendevreview | Abhishek Kekane proposed openstack/nova master: Add Glance preferred stores support https://review.opendev.org/c/openstack/nova/+/1011470 | |
| 04:50:04 | tkajinam | hmm it seems https://review.opendev.org/c/openstack/nova/+/1006786 broke some tests (at least a few founds in cinder-tempest-plugin) repeating volume attachments ... | |
| 04:50:32 | tkajinam | it actually breaks it correctly but I guess we have to update some test scenarios to make sure that they wait until attachment actually completes... | |
| 04:50:56 | tkajinam | context: https://zuul.opendev.org/t/openstack/build/7557579c282b406e95138b928e8e91e7 | |
| 07:16:43 | opendevreview | Kamil Sambor proposed openstack/nova master: deprecate: concurrency_backend config option https://review.opendev.org/c/openstack/nova/+/1011450 | |
| 07:16:44 | opendevreview | Kamil Sambor proposed openstack/nova master: Default native threading for CLI entry points https://review.opendev.org/c/openstack/nova/+/1011606 | |
| 07:44:46 | opendevreview | Juan Larriba proposed openstack/nova master: Add OpenTelemetry tracing middleware for Nova https://review.opendev.org/c/openstack/nova/+/982871 | |
| 11:25:33 | opendevreview | dalekseev proposed openstack/nova master: Restrict machine type check to QEMU instances https://review.opendev.org/c/openstack/nova/+/991137 | |
| 12:19:20 | gibi | tkajinam: good catch. could you please file a bug. I agree, based on the logs, that the test is not waiting for the attachment to finish | |
| 12:46:41 | tkajinam | gibi, will do | |
| 12:51:18 | gibi | thanks | |
| 13:23:09 | 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:26:06 | tkajinam | reported https://bugs.launchpad.net/nova/+bug/2171624 | |
| 13:27:16 | sean-k-mooney | attching multiple volume sconcurently should be porhibitied as we can only have one task in flight at once | |
| 13:27:24 | tkajinam | bauzas0 gibi , I guess ^^^ is the expected behavior after we merged 1006786 and we should drop that test case because such concurrent volume attachments to a single server should be avoided, but it'd be nice if you could double check my thought | |
| 13:27:29 | sean-k-mooney | previsoulsy it woucl have been serialised at the compute | |
| 13:27:46 | tkajinam | yeah that's what I thought | |
| 13:28:22 | sean-k-mooney | we have a chice i guess we can do it at the compute serially using the instnace lock | |
| 13:28:37 | sean-k-mooney | or we can block it at the api based on task_Sate!=none | |
| 13:28:42 | sean-k-mooney | although that can race | |
| 13:29:01 | sean-k-mooney | if your talking to 2 apis at the same time so we always need the instnace lock on teh compute regardless | |
| 13:30:31 | sean-k-mooney | the instance lock on the compute agent is also one of the things that guards agaisn starting a cinder volume migrate or retype durign say an instance live migrate | |
| 13:30:58 | tkajinam | I personally prefer api level lock and reject concurrent actions explicitly so that revert action can be simplified | |
| 13:31:01 | sean-k-mooney | the task state shoudl also help but its the final backstop to prevent 2 operations form trying to muteate the domain at the same time | |
| 13:31:12 | tkajinam | yeah | |
| 13:31:40 | sean-k-mooney | tkajinam: same but that does rely on use updating the task state before doing the rpc to the compute or ealy in the conductor flow | |