opendevreview | Merged openstack/nova master: Decorate only Flavor.get_* methods that execute queries https://review.opendev.org/c/openstack/nova/+/888513 | 04:13 |
---|---|---|
*** dmellado819181 is now known as dmellado81918 | 04:45 | |
opendevreview | Amit Uniyal proposed openstack/nova master: Added context manager for instance lock https://review.opendev.org/c/openstack/nova/+/873648 | 05:11 |
opendevreview | Amit Uniyal proposed openstack/nova master: Disconnecting volume from the compute host https://review.opendev.org/c/openstack/nova/+/877446 | 05:11 |
opendevreview | Amit Uniyal proposed openstack/nova master: Removed explicit call to delete attachment https://review.opendev.org/c/openstack/nova/+/891289 | 05:11 |
kevko | sean-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 |
opendevreview | melanie witt proposed openstack/nova master: Enable use of native threads in scatter_gather_cells https://review.opendev.org/c/openstack/nova/+/650172 | 07:29 |
gibi | dansmith: 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 days | 08:00 |
sean-k-mooney | kevko: 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 twice | 12:18 |
sean-k-mooney | i was not at the last pyshical summit. | 12:18 |
sean-k-mooney | effectivly we need a new api microversion to opt into dynamicaly selected AZ | 12:18 |
sean-k-mooney | we then need to ensure that when that is requested that we dont populate an az in the api | 12:19 |
sean-k-mooney | then 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 |
supamatt | As an operator using openstack, I do concur with kevko.. this is a much needed feature when using multiple Nova AZs | 12:20 |
sean-k-mooney | supamatt: you reasise we declard doing this to be out of scope of nova to do the last time it was proposed as a spec | 12:21 |
supamatt | I saw your comments to what Ultimum was proposing | 12:22 |
sean-k-mooney | under the "No more Orchestration" https://docs.openstack.org/nova/latest/contributor/project-scope.html#no-more-orchestration | 12:24 |
sean-k-mooney | princiapl. 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 scope | 12:24 |
sean-k-mooney | supamatt: 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 opipion | 12:26 |
sean-k-mooney | with 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 AZs | 12:27 |
supamatt | I think that's a good idea. | 12:27 |
sean-k-mooney | i 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-mooney | but in general we can try evolving the spec that is propsoed to make it something that will actully work | 12:30 |
sean-k-mooney | that 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 spec | 12:30 |
dansmith | gibi: ack, I hadn't seen it on any of mine in a long time | 13:30 |
*** blarnath is now known as d34dh0r53 | 13:31 | |
opendevreview | Dan Smith proposed openstack/nova master: Avoid lazy-loads in resize https://review.opendev.org/c/openstack/nova/+/891605 | 14:56 |
opendevreview | Merged openstack/nova master: Add upgrade check for compute-object-ids linkage https://review.opendev.org/c/openstack/nova/+/889740 | 20:40 |
kevko | sean-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 |
kevko | we know that bump microversion is needed | 20:44 |
kevko | sean-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 |
opendevreview | Merged openstack/nova master: Fix bindep for Debian bookworm https://review.opendev.org/c/openstack/nova/+/891256 | 21:16 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!