Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-10-30
07:39:47 openstackgerrit Arthur Dayne proposed openstack/nova master: libvirt:driver:Disallow AIO=native when 'O_DIRECT' is not available https://review.opendev.org/682772
08:50:25 alex_xu gibi: good morning
08:50:58 gibi alex_xu: good morning
08:51:11 alex_xu gibi: just want to check the plan of project update
08:51:31 gibi alex_xu: for me the slides looks OK
08:51:39 alex_xu yes, I think it is ok also
08:51:57 alex_xu do we need to expand the U release, I guess there are more specs
08:52:23 alex_xu but can't sure which is for sure can be accepted
08:52:38 alex_xu gibi: so who responsible for which pages?
08:53:01 gibi alex_xu: let me check what are the freshly approved spec...
08:53:19 gibi alex_xu: I think we should only mention already approved specs
08:53:31 alex_xu cool
08:55:45 gibi alex_xu: regarding spliting the work. I'm OK if you or stephenfin talks about the generic part of the beginning. I can talk about couple of features on slides 5 - 9. I think we can note who talk about what in the speaker notes
08:56:36 alex_xu gibi: ok, cool, I guess all we familar with different features
08:56:48 gibi alex_xu: yes definitly
08:57:29 alex_xu gibi: looks like stephenfin is on vacation already, probably we need to catch him at Shanghai :)
08:58:03 gibi alex_xu: I've just checked we can less specs approved than what is already mentioned on the Ussuri slides so I think we dont need to extend that
08:59:19 gibi alex_xu: yes stephenfin is already on vacation. I will travell on Friday. Lets try to sit together at some point during Monday next week
08:59:47 alex_xu gibi: ok, no problem. the features in slides looks like a feature begin from previous release, and looks like more ensure to be accepted
08:59:50 gibi alex_xu: I will add some speaker notes now about what I can talk about
09:00:04 alex_xu gibi: cool
09:34:05 alex_xu gibi: it is time to assign everything else to Stephen :)
09:34:26 gibi alex_xu: yeah. I guess that is the default behavior :)
09:34:32 alex_xu hah
09:35:19 gibi alex_xu: I will drop a mail to you and stephen to summarize what we talked here now, and ask stephenfin about a sitdown on Monday
09:35:32 alex_xu gibi: cool, thanks
11:19:31 openstackgerrit Silvan Kaiser proposed openstack/nova master: Move Nova Quobyte driver to LibvirtMountedFileSystemVolumeDriver https://review.opendev.org/687066
12:27:04 AJaeger nova stable cores, please review https://review.opendev.org/692032 to update train opensuse job so that Infra can retire openSUSE 15.0
13:10:13 openstackgerrit Brin Zhang proposed openstack/nova-specs master: Support re-configure deleted_on_termination in server https://review.opendev.org/580336
13:46:10 mriedem smcginnis: queens-em tag patch is up https://review.opendev.org/#/c/692142/
13:46:16 mriedem elod: ^
13:50:25 smcginnis Thanks mriedem
13:50:39 smcginnis Did that last one make it, or still stuck in recheck hell?
13:56:28 mriedem merged late last night
13:58:59 smcginnis Nice!
14:06:07 elod mriedem: thanks, i'm a bit sad that gibi's patch sets couldn't make it, but of course, those are not crucial to be part of the final release o:)
14:06:51 elod so, the final and em patch looks good, thanks!
14:26:31 openstackgerrit Merged openstack/nova master: Log some stats for image pre-cache https://review.opendev.org/688173
14:39:30 mriedem someone want to take a pass at mel's host_status UNKNOWN policy change patch? i'm +2 on it. https://review.opendev.org/#/c/679181/
14:39:57 mriedem same with gibi's evacuate support for qos ports series https://review.opendev.org/#/q/topic:bp/support-move-ops-with-qos-ports-ussuri+status:open
14:40:49 mriedem dansmith: when you get a chance it'd be cool if you could take a pass at my re-worked fix for the cross_az_attach=False bug fix https://review.opendev.org/#/c/469675/
14:41:12 mriedem should be 35% less gross than before
14:41:13 dansmith mriedem: enqueued
14:41:28 dansmith 35.0%? pretty precise grossness measurement
14:41:39 mriedem i was in the lab all last night
14:41:57 dansmith heh
14:42:23 efried mriedem: I was working through that
14:42:47 efried I have some questions, so dansmith if you wouldn't mind holding off approving it...
14:43:01 mriedem efried: which? the az one?
14:43:08 efried sorry, the host_status-unknown one
14:43:13 mriedem oh
14:43:27 mriedem dan and i are talking about the az one
14:43:36 efried ack
14:55:58 sean-k-mooney dansmith: thanks for reviewing the image metadta prefilter series. ill try and adress your comments this week
14:56:18 dansmith sean-k-mooney: np
14:56:45 mriedem easy multi-cell functional test-only change that's been sitting a long time https://review.opendev.org/#/c/452006/
14:56:56 mriedem 2.5 years...
14:57:44 dansmith let's not rush that one
15:00:55 efried mriedem, melwitt: I'm +2 on the host_status UNKNOWN policy patch, but would like some help understanding a thing before +Wing. (No hurry)
15:03:41 efried mriedem: how come those other release patches are stacked on the queens-em one?
15:03:58 dansmith mriedem: for N volumes your patch also avoids making N volume.get() calls instead opting for the bulk query once yeah?
15:03:58 mriedem no particular reason, laziness
15:04:04 efried ight
15:04:17 mriedem dansmith: yeah - which is where i mentioned i could split that part out if it helps
15:04:42 dansmith oh wait
15:05:01 dansmith it actually just does volume.get() on each one in _get_volumes_for_bdms()
15:05:23 dansmith I was thinking this was an improvement, but it just moves where you make all the individual calls
15:07:59 mriedem efried: i replied in https://review.opendev.org/#/c/679181/
15:08:35 mriedem dansmith: yeah, i mentioned using GET /volumes with a list but as commented i don't think that would work and i should remove that comment
15:08:52 dansmith yeah, sorry, just pulling all this context back in
15:09:04 mriedem it does mean we'd avoid multiple GETs on the same volume, but you can't really boot from volume with multiple servers and the same pre-existing volume anyway
15:09:37 mriedem unless it's a multiattach volume anyway
15:10:41 melwitt efried: let me fix it, I think I just spaced bc gmann added those two in a patch below mine
15:10:52 mriedem melwitt: do it in a follow up
15:11:00 mriedem it's hard enough getting things through the gate right now
15:11:00 melwitt ok
15:11:14 efried I just want to understand why the tests work
15:11:21 efried still reading mriedem's reply...
15:11:25 mriedem efried: the policy docs are just an omission
15:11:39 mriedem note that those are just docs...not code
15:11:40 efried oh, that's just a doc thing? (/me doesn't really understand policy)
15:11:41 efried got it.
15:11:57 mriedem right, the policy rule is real, the API method / route stuff is docs
15:12:38 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add functional test for two-cell scheduler behaviors https://review.opendev.org/452006
15:13:13 mriedem ah gdi, fake_nodes
15:13:33 efried melwitt, mriedem: +W. If you want to hit any of that other stuff in the fup too, feel free, but none of it was super important.
15:14:08 melwitt ok, thanks
15:14:20 gibi mriedem: fix is on the way, and then I will +2 it
15:14:27 efried melwitt: also, wouldn't hurt to update the bp text for at least these two things:
15:14:27 efried - the name of the policy rule
15:14:27 efried - the fact that the host_status field is omitted, not included as ""
15:15:01 efried (there turned out to be no spec for this, right?)
15:15:03 melwitt ah, right, will do
15:15:09 efried thx
15:15:10 melwitt yeah no spec
15:15:25 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add functional test for two-cell scheduler behaviors https://review.opendev.org/452006
15:18:32 dansmith mriedem: question for you in there, and figure I'm standing by for you to remove that comment and split the patches if you're going to
15:19:27 mriedem i'll split the patches if needed but not keen to if i can help it, at least for the volume GET stuff since that touches a lot of unit tests. the config option docs and such are easy to split out though, but also minor.
15:19:48 dansmith mriedem: yep I figured, hence my comment about it :)
15:20:05 dansmith it's easy enough to see what test changes are related to the refactor so I'm not too concerned

Earlier   Later