| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-03-08 | |||
| 16:24:02 | med_ | good point. | |
| 16:24:31 | mriedem | lyarwood: yeah i think i thought about that test wrinkle at one point when talking with scottda about testing swap volume | |
| 16:24:48 | mriedem | it would be pretty simple, creating 2 volumes, where the 2nd is larger and we swap from small to big | |
| 16:25:06 | kashyap | mriedem: Ah, cool that you tied up that loose end with that analysis | |
| 16:25:33 | mriedem | edleafe: so i think the thing is all sorted out :L) | |
| 16:25:34 | mriedem | :) | |
| 16:25:59 | edleafe | mriedem: See? Bring me in, problem gets solved! | |
| 16:26:43 | mriedem | i do enjoy that the L3 guys from japan call me Matt-san | |
| 16:27:10 | edleafe | On IRC we'll call you mriedem-san from now on | |
| 16:27:55 | mriedem | in high school i got people to call me dave for a few weeks just because i liked the name | |
| 16:28:03 | mriedem | so call me dave | |
| 16:28:23 | edleafe | s/Matt/Dave/g | |
| 16:28:54 | dansmith | mriedem: weren't you a little old for such shenanigans in high school? | |
| 16:28:55 | dansmith | that's middle school stuff | |
| 16:30:08 | artom | mriedem, so https://en.wikipedia.org/wiki/David_Readman is you then? | |
| 16:31:53 | mriedem | dansmith: i think it was exactly the right time | |
| 16:32:35 | dansmith | heh | |
| 16:32:41 | mriedem | artom: ha, no i'm this guy http://www.swipco.org/contact-us/ | |
| 16:32:47 | mriedem | Dave Riedemann, Housing Programs Administrator | |
| 16:33:15 | mriedem | under armor for a professional photo, classy | |
| 16:33:19 | mriedem | that's dave riedemann classy | |
| 16:33:31 | mriedem | that's mfing SW iowa classy | |
| 16:33:39 | artom | And lip gloss as well | |
| 16:34:03 | mriedem | no gloss required | |
| 16:34:13 | artom | You're just naturally fabulous | |
| 16:34:16 | mriedem | that's probably leftover bacon grease since it's iowa | |
| 16:34:33 | mriedem | pork and corn for every meal boys | |
| 16:34:53 | artom | Pork and Corn is the name of my Korn tribute band. | |
| 16:37:00 | johnthetubaguy | dansmith: melwitt: thinking about quota and resize | |
| 16:37:21 | johnthetubaguy | dansmith: melwitt: what if we only do a check when you request the resize, and actually not care about races for resize? | |
| 16:37:40 | dansmith | johnthetubaguy: well, that's what I meant by accept the leakiness | |
| 16:38:04 | johnthetubaguy | dansmith: melwitt: longer term I am tempted we just do it based on allocations in placement, so you end up needing double quota during the resize, as thats technically more accurate on the resource usage | |
| 16:38:05 | dansmith | johnthetubaguy: just depends on how concerned we are about people gaming that by the resize-all situation I mentioned | |
| 16:38:11 | dansmith | johnthetubaguy: yep | |
| 16:38:31 | johnthetubaguy | I was thinking about the folks that only had out enough quota for the given space they have | |
| 16:39:18 | johnthetubaguy | dansmith: crazy, crazy, idea, limit the number of resizes you are allowed to do at once | |
| 16:39:35 | dansmith | johnthetubaguy: with a quota or a static value? | |
| 16:39:48 | dansmith | johnthetubaguy: resizes are heavy so that's probably not a bad idea regardless | |
| 16:40:49 | johnthetubaguy | dansmith: I could go either way, a static conf, with the option of -1 might be enough (to save any API change for now) | |
| 16:41:22 | dansmith | johnthetubaguy: yeah, it's pretty much exactly like the "max parallel builds" and "max parallel migrations" confs we have now, just tenant-scoped | |
| 16:41:27 | dansmith | johnthetubaguy: but yeah, I like it | |
| 16:41:51 | johnthetubaguy | dansmith: yeah, per project rather than per host | |
| 16:41:55 | dansmith | yeah | |
| 16:42:00 | dansmith | sounds good to me | |
| 16:42:22 | johnthetubaguy | yeah, so check just on the way in, but limit the number you are allowed in conf | |
| 16:42:26 | johnthetubaguy | thats quite simple really | |
| 16:42:46 | johnthetubaguy | I quite like the double allocation eventually, once we have the placement thing working like that | |
| 16:43:21 | sfinucan | vladikr: little comment on https://review.openstack.org/#/c/430026. Happy to approve after | |
| 16:43:32 | vladikr | sfinucan, thanks | |
| 16:43:51 | dansmith | johnthetubaguy: yeah placement makes all of this easier when it gains knowledge of the project | |
| 16:43:59 | dansmith | johnthetubaguy: so wanna dump the above into a comment on there? | |
| 16:44:07 | johnthetubaguy | yup yup | |
| 16:44:12 | dansmith | rock on | |
| 16:45:41 | sfinucan | mriedem: Could I get the -2 lifted from https://review.openstack.org/#/c/345396/, please? | |
| 16:47:00 | openstackgerrit | Stephen Finucane proposed openstack/nova master: console: introduce basic framework for security proxying https://review.openstack.org/345396 | |
| 16:47:01 | openstackgerrit | Stephen Finucane proposed openstack/nova master: console: introduce the VeNCrypt RFB authentication scheme https://review.openstack.org/345398 | |
| 16:47:01 | openstackgerrit | Stephen Finucane proposed openstack/nova master: console: introduce framework for RFB authentication https://review.openstack.org/345397 | |
| 16:47:02 | openstackgerrit | Stephen Finucane proposed openstack/nova master: console: provide an RFB security proxy implementation https://review.openstack.org/345399 | |
| 16:47:15 | johnthetubaguy | dansmith: added a comment, double check that makes sense | |
| 16:49:04 | dansmith | johnthetubaguy: yep thanks | |
| 16:49:19 | openstackgerrit | Matthew Booth proposed openstack/nova master: fdatasync() downloaded images before use https://review.openstack.org/441246 | |
| 16:49:20 | openstackgerrit | Matthew Booth proposed openstack/nova master: Ensure image conversion flushes output data to disk https://review.openstack.org/443230 | |
| 16:50:40 | mdbooth | sdague: You were +2 on https://review.openstack.org/#/c/441246/. I just changed the commit message to reflect that there's a second part to the puzzle. | |
| 16:51:30 | mdbooth | (Also, there's a second part to the puzzle ;)) | |
| 16:59:02 | mriedem | sfinucan: nope | |
| 16:59:07 | mriedem | sfinucan: the blueprint isn't approved | |
| 16:59:11 | mriedem | and i don't see a re-proposed spec | |
| 16:59:53 | sfinucan | mriedem: Ah, I only thought -2s only applied after spec approval deadline. I'll resub the spec now | |
| 17:03:59 | mriedem | dansmith: something else i thought about yesterday with your simple tenant usage change | |
| 17:04:09 | mriedem | but in get_all in the compute API we have a check for cells v1 | |
| 17:04:32 | dansmith | mriedem: on a call, but let me think about that when I'm done | |
| 17:05:01 | dansmith | not sure it's as important here. the cells1 check is purely to avoid overwriting data in the replication, but here we're looking at history, so probably fine | |
| 17:05:03 | dansmith | anyway, biab | |
| 17:05:21 | openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: Websockify security proxy framework https://review.openstack.org/443234 | |
| 17:09:58 | openstackgerrit | Sujitha proposed openstack/nova master: Reduce calls to load_tags() to 0 https://review.openstack.org/435146 | |
| 17:29:32 | jroll | so openstackclient defaults to the latest microversion, do y'all pin that or send 'latest'? | |
| 17:30:25 | openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: spawn/delete #1: no-ops https://review.openstack.org/438119 | |
| 17:30:37 | openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: spawn/destroy #2: functional https://review.openstack.org/438598 | |
| 17:31:10 | openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: power_on/off and reboot https://review.openstack.org/427380 | |
| 17:32:18 | openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: spawn/destroy #3: TaskFlow https://review.openstack.org/438729 | |
| 17:32:31 | openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: spawn/destroy #4: full flavor https://review.openstack.org/391288 | |
| 17:32:48 | openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: console https://review.openstack.org/409402 | |
| 17:34:54 | cfriesen | at the PTG a suggestion was made to add an option to the live migration request to specify "force live migration after X seconds". Was anyone planning on running with that? | |
| 17:35:54 | jroll | never mind my question, I found it, the answer is that it's pinned | |
| 17:36:03 | mriedem | jaypipes: ComputeNode.stats is where all of the virt driver capabilities dict garbage lives right? | |
| 17:37:50 | raj_singh | cfriesen: yes | |
| 17:38:12 | raj_singh | cfriesen: I have a spec https://review.openstack.org/#/c/442809/ | |
| 17:38:22 | mriedem | claudiub: jaypipes: can you both take a look at this? https://review.openstack.org/#/c/364468/ | |
| 17:38:29 | mriedem | claudiub: guestions in there for you wrt the hyperv driver | |
| 17:38:40 | mriedem | jaypipes: questions in there for you about nested resource providers and placement | |
| 17:38:45 | mriedem | sfinucan: ^ questions/comments inline | |
| 17:39:34 | mriedem | seems like the numa/pci information is something that would be fed into nested resource provider via the new get_inventory method on the virt driver | |
| 17:39:39 | raj_singh | cfriesen: and this https://review.openstack.org/#/c/438467/ | |
| 17:39:49 | mriedem | so the libvirt driver could say what it's inventory of available pci devices is | |
| 17:40:21 | mriedem | and then the scheduler you query placement for that info and place an instance that requires pci onto one of those hosts, and for instances that don't require pci, it weighs those away from those providers | |
| 17:42:27 | sean-k-mooney | mriedem: looking at stpehens spec i dont know if you need driver changes so much as compute agent changes | |
| 17:42:52 | mriedem | compute agent? | |
| 17:42:59 | mriedem | we don't have a compute agent | |
| 17:43:19 | sean-k-mooney | n-cpu? | |
| 17:43:46 | cfriesen | compute server | |