| *** benj_88 is now known as benj_8 | 00:39 | |
| opendevreview | Adam Harwell proposed openstack/nova master: Stop the service when its start fails https://review.opendev.org/c/openstack/nova/+/1009119 | 01:19 |
|---|---|---|
| *** mikal3 is now known as mikal | 07:08 | |
| opendevreview | fivetime proposed openstack/nova master: Do not share nested objects with the cross-cell clone https://review.opendev.org/c/openstack/nova/+/1011054 | 07:42 |
| opendevreview | Juan Larriba proposed openstack/nova master: Add OpenTelemetry tracing middleware for Nova https://review.opendev.org/c/openstack/nova/+/982871 | 08:56 |
| opendevreview | Kamil Sambor proposed openstack/nova master: deprecate: concurrency_backend config option https://review.opendev.org/c/openstack/nova/+/1011450 | 09:21 |
| opendevreview | Moritz Tanner proposed openstack/nova master: libvirt: Support extending local volumes https://review.opendev.org/c/openstack/nova/+/878763 | 11:02 |
| 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 | 12:27 |
| opendevreview | Juan Larriba proposed openstack/nova master: Add OpenTelemetry tracing middleware for Nova https://review.opendev.org/c/openstack/nova/+/982871 | 13:21 |
| opendevreview | Abhishek Kekane proposed openstack/nova master: Add Glance preferred stores support https://review.opendev.org/c/openstack/nova/+/1011470 | 13:40 |
| 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 |
| sean-k-mooney | tldr we can mutate ports in the nturon fixutre whiel we are iterating over them causign a runtime error | 14:03 |
| gibi | sean-k-mooney: ohh good catch | 14:05 |
| 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:11 |
| 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 |
| sean-k-mooney | im not sure how often that happens | 14:35 |
| sean-k-mooney | its the first time i have seen it but who knows | 14:35 |
| opendevreview | Kamil Sambor proposed openstack/nova master: deprecate: concurrency_backend config option https://review.opendev.org/c/openstack/nova/+/1011495 | 15:11 |
| opendevreview | Kamil Sambor proposed openstack/nova master: deprecate: concurrency_backend config option https://review.opendev.org/c/openstack/nova/+/1011450 | 15:16 |
| gibi | sambork: could you put the deprecation patch top of the default changing spec to make it more logical? | 15:26 |
| gibi | s/spec/patch/ | 15:26 |
| sambork | @gibi, sure, sorry forgot! | 15:26 |
| gibi | no worries | 15:26 |
| sean-k-mooney | shoudl it not be before? | 15:27 |
| gibi | also, can we switch the default for the CLIs as well? | 15:27 |
| sean-k-mooney | i gues either order is ok | 15:27 |
| gibi | I would not deprecated the old option until we default to it | 15:27 |
| opendevreview | sean mooney proposed openstack/nova master: Synchronize Neutron fixture port operations https://review.opendev.org/c/openstack/nova/+/1011497 | 15:35 |
| sean-k-mooney | gibi: ^ pass locally but that the fix | 15: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 | 15:36 |
| 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 |
| opendevreview | Merged openstack/nova master: Reduce the unnecessary executors shutdown logs in test jobs https://review.opendev.org/c/openstack/nova/+/1004160 | 16:03 |
| opendevreview | Kamil Sambor proposed openstack/nova master: deprecate: concurrency_backend config option https://review.opendev.org/c/openstack/nova/+/1011513 | 16:26 |
| tkajinam | sambork, it seems you proposed duplicated changes to deprecate concurrency_backend ? | 17:19 |
| tkajinam | 1011450: deprecate: concurrency_backend config option | https://review.opendev.org/c/openstack/nova/+/1011450 | 17:19 |
| tkajinam | 1011513: deprecate: concurrency_backend config option | https://review.opendev.org/c/openstack/nova/+/1011513 | 17:20 |
| tkajinam | I guess 1011513 is correct because it follows but we might need your confirmation | 17:20 |
| gmaan | i think so | 17:21 |
| 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:22 |
| gmaan | oh, you are doing that in next change, nevermind +w the base one https://review.opendev.org/c/openstack/nova/+/1011513 | 17:23 |
| sean-k-mooney | we have duplicates now https://review.opendev.org/c/openstack/nova/+/1011450 | 17:23 |
| sean-k-mooney | and https://review.opendev.org/c/openstack/nova/+/1011513 | 17:24 |
| sean-k-mooney | to me changing the default bahvior and deprection the config option are unrelated | 17:25 |
| sean-k-mooney | we are chaing the meaing of auto | 17:25 |
| sean-k-mooney | not the default of the config option | 17:25 |
| sean-k-mooney | but as long as we abandon one fo the deprecation patches its fine | 17:25 |
| gmaan | 1011513 is more than just deprecating the config but also changing default for CLI | 17:28 |
| gmaan | anyways i commented there in gerrit | 17:28 |
| gmaan | title of 1011513 and 1011450 are same but they are not duplicate | 17:29 |
| 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 |
| 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:16 |
| sean-k-mooney | abhishek_: i think that woudl also be true for maniala by the way | 18:25 |
| sean-k-mooney | i saw there was a dicsion about addign some extra spec for the data pool | 18:26 |
| abhishek_ | right | 18:26 |
| 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 |
| sean-k-mooney | so im glad that didnt break along the way | 18:28 |
| gouthamr | nope, not going to break if you don't set an optional flag.. this is an opt-in feature.. | 18:29 |
| abhishek_ | yeah, thanks for the pointers | 18:29 |
| sean-k-mooney | gouthamr: what i was sayign is you probely dont need to add the feature at all | 18:29 |
| 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 |
| sean-k-mooney | gouthamr: well for a stroage project have ing more advanced policy over this might make sense | 18:30 |
| 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:31 |
| 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 |
| sean-k-mooney | but the installer and ceph comunity have in the past | 18:32 |
| sean-k-mooney | so its one of those thing that if you dont knwo to ask you wont find out about | 18:32 |
| opendevreview | Abhishek Kekane proposed openstack/nova master: Add Glance preferred stores support https://review.opendev.org/c/openstack/nova/+/1011470 | 18:55 |
| 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:56 |
| sean-k-mooney | or if we coudl enabel it in one of the other jobs if it does | 18:57 |
| abhishek_ | ack, I will check and act accordingly (might need to add specific tempest test as well for this in that case) | 18:58 |
| 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 |
| sean-k-mooney | i dont think that is currently testing any special storage config | 19:00 |
| sean-k-mooney | so its less likely to conflict | 19:00 |
| 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:01 |
| abhishek_ | ack | 19:06 |
| opendevreview | Abhishek Kekane proposed openstack/nova master: Add Glance preferred stores support https://review.opendev.org/c/openstack/nova/+/1011470 | 19:50 |
| opendevreview | Merged openstack/nova master: Default native threading for console proxies https://review.opendev.org/c/openstack/nova/+/1009027 | 19:53 |
| 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 |
| opendevreview | Dan Smith proposed openstack/nova master: InstanceMapping object changes for old_cell_id https://review.opendev.org/c/openstack/nova/+/1008835 | 23:23 |
| 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 |
| opendevreview | Dan Smith proposed openstack/nova master: Instance lookup support for mid-cell-migration https://review.opendev.org/c/openstack/nova/+/1008837 | 23:23 |
| 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 |
| 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 |
| opendevreview | Dan Smith proposed openstack/nova master: Compute RPC refactors for re-use later https://review.opendev.org/c/openstack/nova/+/1008840 | 23:23 |
| opendevreview | Dan Smith proposed openstack/nova master: Compute RPC methods for live migration v2 https://review.opendev.org/c/openstack/nova/+/1008841 | 23:23 |
| opendevreview | Dan Smith proposed openstack/nova master: Add conductor-orchestrated live migration https://review.opendev.org/c/openstack/nova/+/1008842 | 23:23 |
| opendevreview | Dan Smith proposed openstack/nova master: Functional tests for cross-cell live migrate https://review.opendev.org/c/openstack/nova/+/1008843 | 23:23 |
| opendevreview | Dan Smith proposed openstack/nova master: Doc changes for cross-cell live migrate https://review.opendev.org/c/openstack/nova/+/1008844 | 23:23 |
| opendevreview | Dan Smith proposed openstack/nova master: WIP: Extra instance check for migration_context https://review.opendev.org/c/openstack/nova/+/1008845 | 23:23 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!