Wednesday, 2023-08-16

opendevreviewMerged openstack/nova master: Decorate only Flavor.get_* methods that execute queries  https://review.opendev.org/c/openstack/nova/+/88851304:13
*** dmellado819181 is now known as dmellado8191804:45
opendevreviewAmit Uniyal proposed openstack/nova master: Added context manager for instance lock  https://review.opendev.org/c/openstack/nova/+/87364805:11
opendevreviewAmit Uniyal proposed openstack/nova master: Disconnecting volume from the compute host  https://review.opendev.org/c/openstack/nova/+/87744605:11
opendevreviewAmit Uniyal proposed openstack/nova master: Removed explicit call to delete attachment  https://review.opendev.org/c/openstack/nova/+/89128905:11
kevkosean-k-mooney thank you, my colleguage viktor will reply today ..btw, several customers wants this feature (az-afiinity/anti-az-afinnity ) from us ...i was also discussing this with someone on summit ..i think julia kreger ..07:13
opendevreviewmelanie witt proposed openstack/nova master: Enable use of native threads in scatter_gather_cells  https://review.opendev.org/c/openstack/nova/+/65017207:29
gibidansmith: re: Cursor needed to be reset because of commit/rollback and can no longer be fetched from.: That seems fairly frequent in nova. At least there was 15 hint in the last 30 days08:00
sean-k-mooneykevko: yep we have discussed it at the summit before. it could be done but not the way you suggested it requires addtional change to the api. just for context this has been disccused at at least 3 PTGs and has been rejected in the form ye preposed in the spec at least twice12:18
sean-k-mooneyi was not at the last pyshical summit.12:18
sean-k-mooneyeffectivly we need a new api microversion to opt into dynamicaly selected AZ12:18
sean-k-mooneywe then need to ensure that when that is requested that we dont populate an az in the api12:19
sean-k-mooneythen we need a something likely prior to the placement query to determin what azs can be used althogh that could happen after a a filter/weigher 12:20
supamattAs an operator using openstack, I do concur with kevko.. this is a much needed feature when using multiple Nova AZs12:20
sean-k-mooneysupamatt: you reasise we declard doing this to be out of scope of nova to do the last time it was proposed as a spec12:21
supamattI saw your comments to what Ultimum was proposing12:22
sean-k-mooneyunder the "No more Orchestration"  https://docs.openstack.org/nova/latest/contributor/project-scope.html#no-more-orchestration12:24
sean-k-mooneyprinciapl. i.e. we do not want to implement advanced orchestation supprot in nova itselsf. with that said there are less invaisve ways to enable this that might be in scope12:24
sean-k-mooneysupamatt: i understand why peope want this but for technial and practical reasons this is not something that could be enbled without using a new api microverion to have in my opipion12:26
sean-k-mooneywith the new api microversion however we could enable this feature. that microversion si opting out of the current defaulting that prevents the schduler for selecting AZs12:27
supamattI think that's a good idea. 12:27
sean-k-mooneyi need to go to the pharmacy and pick up my blood presseur persciption so ill be afk for 30 mins or so. 12:29
sean-k-mooneybut in general we can try evolving the spec that is propsoed to make it something that will actully work12:30
sean-k-mooneythat is why i did not -2 it. im not agaisnt the feature in genral jsut wanted to let kevko know that the acutal cahnge will need to be more complicated then sugested in the spec12:30
dansmithgibi: ack, I hadn't seen it on any of mine in a long time13:30
*** blarnath is now known as d34dh0r5313:31
opendevreviewDan Smith proposed openstack/nova master: Avoid lazy-loads in resize  https://review.opendev.org/c/openstack/nova/+/89160514:56
opendevreviewMerged openstack/nova master: Add upgrade check for compute-object-ids linkage  https://review.opendev.org/c/openstack/nova/+/88974020:40
kevkosean-k-mooney: my colleguage has working prototype with bumped microversion as you mentioned above ...and working .. wouldn't be best to share somehow this temporary work ? 20:43
kevkowe know that bump microversion is needed20:44
kevkosean-k-mooney thank you very much for discussion, I appreciate it very much,  we (of course) know that the spec will be probably reworked and we are open to amend the spec and implement as we agree ... after all, there is a gerrit review for that so that we can get to the final patchset :)20:47
opendevreviewMerged openstack/nova master: Fix bindep for Debian bookworm  https://review.opendev.org/c/openstack/nova/+/89125621:16

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!