| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-09-10 | |||
| 13:47:13 | bauzas | gibi: any notes you'd like to see in the prelude ? | |
| 13:47:46 | gibi | bauzas: I have nothing specific in my mind right now | |
| 13:48:04 | gibi | but I will do a scan of merged renos to get a view | |
| 13:48:07 | stephenfin | bauzas: have you noted that XenAPI is gone? | |
| 13:48:13 | stephenfin | that's merging as we spak | |
| 13:48:15 | stephenfin | *speak | |
| 13:48:23 | bauzas | gibi: I'll use your own cycle highlits as a base | |
| 13:48:30 | bauzas | stephenfin: ah, no, gtk | |
| 13:48:48 | bauzas | stephenfin: in general, I take a look at the main reno master page | |
| 13:48:58 | bauzas | so I'll see it, I guess ;) | |
| 13:49:00 | gibi | bauzas: cool | |
| 13:49:15 | sean-k-mooney | stephenfin: i started reviewin the xenapi removeal then stop as i realed it mainly just code deletion and lyarwood and gibi? had alredy reviewed most of it | |
| 13:49:49 | sean-k-mooney | im glad that is going to make it :) | |
| 13:50:33 | stephenfin | sean-k-mooney: you might be interested in https://review.opendev.org/#/c/748533/ | |
| 13:51:08 | sean-k-mooney | didnt i reveiew an previous version | |
| 13:51:16 | sean-k-mooney | i do need to take another look | |
| 13:51:22 | stephenfin | oh, so you did | |
| 13:51:24 | stephenfin | my bad | |
| 13:51:36 | sean-k-mooney | no its good to remind me to look again | |
| 13:52:54 | stephenfin | https://review.opendev.org/745605 still needs to be respun too :) | |
| 13:53:15 | sean-k-mooney | oh the filters | |
| 13:53:21 | sean-k-mooney | actully ill go do that now | |
| 13:53:31 | lyarwood | sean-k-mooney: some additional review on these would also be good btw, slightly out of my area tbh https://review.opendev.org/#/c/746949/ | |
| 13:53:40 | lyarwood | artom: ^ also if you have bandwidth | |
| 13:54:06 | sean-k-mooney | oh the fixture change | |
| 13:54:24 | lyarwood | and the series on top adding the actual tests | |
| 13:54:32 | sean-k-mooney | am yes are they mergeragbe after today/feature freeze | |
| 13:54:56 | sean-k-mooney | if so ill do the filter deprecation first then ill take a look at those | |
| 13:55:43 | lyarwood | ah yeah of course sorry | |
| 13:56:04 | sean-k-mooney | no its fine i really should have reviewd that series properly by now | |
| 13:56:10 | sean-k-mooney | i have been putting it off | |
| 13:56:51 | sean-k-mooney | mainly for lack of brain power and opertunity to sit down and do it properly | |
| 13:57:28 | artom | lyarwood, ah, yeah, I'm slowly working my way up that series | |
| 14:05:30 | lyarwood | artom: cool thanks, I was fine with the house keeping stuff up until that point FWIW, I just don't have the context to know if we are asserting the right things in the actual LM tests | |
| 14:07:33 | artom | lyarwood, ack | |
| 14:39:05 | sean-k-mooney | smcginnis: oh you create the non clinet lib stable branches already | |
| 14:39:09 | sean-k-mooney | that is good to know. | |
| 14:40:03 | sean-k-mooney | we have 2 patches for os-vif to backprot to stable/victoria but they are not merged on master yet and i was going to wait a little while to ensure we dont have any issue before mergeing them anyway | |
| 14:40:32 | sean-k-mooney | but i guess this means i dont strictly have to wait to rc1 for master to be open for wallaby in os-vif | |
| 14:41:09 | sean-k-mooney | sill going to hold off for a bit however | |
| 14:48:48 | smcginnis | sean-k-mooney: Yep, non-client library freeze was last week. So those are already done for victoria. | |
| 14:49:04 | smcginnis | Sneaks up quick. | |
| 14:53:50 | sean-k-mooney | oh ya i know i am just used to waiting till the main project reach rc1 for the master branch to reopen and stable brances to be created | |
| 14:54:14 | sean-k-mooney | smcginnis: but creating the branch early also works | |
| 14:54:45 | sean-k-mooney | smcginnis: also its nice to not have to manually do this in gerrit anymore although i guess i have not done that in a few releases at this point | |
| 14:55:24 | smcginnis | Unofficial projects would need to, but anything official should all be going through the releases repo. | |
| 14:55:42 | sean-k-mooney | yep | |
| 15:24:38 | bauzas | lyarwood: gibi: we don't have better docs for glance image cloning but a spec ? https://specs.openstack.org/openstack/nova-specs/specs/victoria/approved/nova-image-download-via-rbd.html | |
| 15:26:39 | gibi | bauzas: there is an open doc patch https://review.opendev.org/#/c/731577/ | |
| 15:27:20 | bauzas | uh ok | |
| 15:28:08 | bauzas | gibi: so we have 2 different things related to RBD | |
| 15:28:14 | bauzas | gibi: one is the glance multistore | |
| 15:28:16 | gibi | yes | |
| 15:28:24 | bauzas | the other being the precaching | |
| 15:28:29 | gibi | yes | |
| 15:28:36 | bauzas | honestly, I'd then write two different bullets | |
| 15:28:56 | bauzas | because most ops probably don't give a clue about what a glance multistore is | |
| 15:29:06 | bauzas | and both are independent | |
| 15:29:27 | bauzas | ie. you can precache with or without multistore, and the other way too | |
| 15:29:35 | bauzas | dansmith: amirite ? | |
| 15:30:02 | dansmith | precache in nova has nothing to do with glance multistore | |
| 15:30:32 | bauzas | that's what I understood | |
| 15:30:44 | bauzas | the rbd dowload, that's the precache, right? | |
| 15:30:55 | dansmith | no | |
| 15:31:01 | bauzas | holy shit then | |
| 15:31:04 | bauzas | I'm confused | |
| 15:31:22 | dansmith | the rbd download thing is just letting nova download the image direct from rbd instead of streaming it through glance, if it has access to the rbd that backs glance | |
| 15:31:49 | bauzas | ok, that's what I understood, but then what's precaching ? | |
| 15:32:04 | bauzas | oh | |
| 15:32:11 | bauzas | pulling it to libvirt cache? | |
| 15:32:30 | bauzas | whether you pull it thru Glance or from the RBD directly | |
| 15:32:34 | bauzas | correct ? | |
| 15:32:52 | dansmith | yes, just triggering the image cache to download the image, separately from an actual instance boot | |
| 15:32:58 | bauzas | ok | |
| 15:33:06 | bauzas | so, what was merged in V ? | |
| 15:33:14 | bauzas | from the notes, I see precaching | |
| 15:33:20 | bauzas | ah na | |
| 15:33:22 | bauzas | nvm | |
| 15:33:25 | dansmith | all of that is separate from the glance multistore support in nova :) | |
| 15:33:30 | bauzas | it's RBD download | |
| 15:33:37 | bauzas | precaching was a U thing | |
| 15:33:45 | bauzas | correct ? | |
| 15:33:57 | bauzas | U or T, I don't remind exactly | |
| 15:34:32 | dansmith | I think precaching was T yeah | |
| 15:34:33 | dansmith | sorry, U | |
| 15:35:14 | sean-k-mooney | so the two new things in V are the rbd direct download and seperate the multi store support | |
| 15:35:41 | dansmith | yes, but I'm not sure the rbd download thing is worth a special callout in the release notes, if that's what we're discussing here | |
| 15:35:57 | sean-k-mooney | in the prelude ya | |
| 15:36:14 | bauzas | dansmith: kk, now I understand, thanks | |
| 15:36:22 | bauzas | yeah, for the prelude, we'll see | |
| 15:36:34 | bauzas | we don't have a shit ton of new stuff, even a single microversion to provide | |
| 15:36:51 | bauzas | but I agree, this sounds just a small improvement | |
| 15:37:01 | bauzas | not worth a prelude | |
| 15:37:12 | bauzas | (even if that went somehow to the cycle highlights :p ) | |
| 15:40:39 | openstackgerrit | Stephen Finucane proposed openstack/nova master: tox: Enable parallel docs build https://review.opendev.org/751034 | |
| 15:42:03 | sean-k-mooney | stephenfin: cool i basically only did the tox.ini part of that locally and didnt really know what to do for the extentions | |
| 15:42:30 | stephenfin | sean-k-mooney: it won't work though. oslo.config is borked | |
| 15:42:56 | stephenfin | I've pushed a patch for it but it could be a while before it lands. It's a bug but also not massively high priority | |
| 15:42:57 | sean-k-mooney | oh so they actully are not parralel safe | |
| 15:43:07 | stephenfin | afraid not | |