| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-03-08 | |||
| 15:56:27 | mriedem | so i don't see why we're even attempting a resize | |
| 15:56:27 | efried | sean-k-mooney Sounds reasonable to me. Launchpad bug? Propose a fix? | |
| 15:56:29 | sean-k-mooney | mriedem: the connect to ovs may not be apparent but there makefile enabled flake8 if its installed which it is tanks to devstack and the test defined by flake8-import-order seam to be enabled by default | |
| 15:56:39 | dansmith | johnthetubaguy: even quotas are better than TC stuff :) | |
| 15:56:55 | kashyap | mriedem: Heh, that's a good place to step back and see, if it's even needed. | |
| 15:56:58 | mriedem | kashyap: because resize_to should be 0 since this would be false https://review.openstack.org/#/c/433793/3/nova/compute/manager.py@5006 | |
| 15:57:01 | sean-k-mooney | efried: am yes i have asked our ci team to open one and submit a patch to make it optional | |
| 15:58:21 | mriedem | kashyap: multipath is also enabled on these things, which i'm just going to wildly make a guess could ruin everything | |
| 15:58:29 | sean-k-mooney | efried: if we are gong to run the unit tests in the ci we will just use tox and it will install the test-requriments in the virtual env itself so i dont know why devstack is installing them by default for every thing that is installed via pip | |
| 15:58:41 | mriedem | b/c we sure as hell don't test swap volume with multipath backends | |
| 15:58:57 | openstackgerrit | Gábor Antal proposed openstack/nova-specs master: Show persisted scheduler hints in server details https://review.openstack.org/440580 | |
| 15:59:28 | kashyap | mriedem: Yeah, 'multipath' -- never in my life I could grok that goddamend thing. Whenever I set out to focus on it to learn, I get distracted. | |
| 16:01:00 | openstack | Launchpad bug 1671144 in devstack "test-requirements are wrongly installed system-wide" [Undecided,New] | |
| 16:01:00 | sean-k-mooney | efried: mriedem looks like they have already opened the bug https://bugs.launchpad.net/devstack/+bug/1671144 | |
| 16:01:21 | openstackgerrit | John Garbutt proposed openstack/nova master: POC: add context.check_scope https://review.openstack.org/435485 | |
| 16:03:05 | lyarwood | mriedem: we don't appear to test this resize path during our swap_volume tempest tests btw, I'll follow up with that. | |
| 16:07:19 | mriedem | kashyap: ah there are 2 A9K volumes | |
| 16:07:22 | mriedem | 2 swap operations | |
| 16:07:34 | mriedem | 10GB A9K -> 10GB 3PAR, | |
| 16:07:42 | kashyap | And... | |
| 16:07:48 | mriedem | then 10GB 3PAR -> 15GB A9K | |
| 16:07:52 | mriedem | so that's the resize | |
| 16:08:03 | kashyap | Ah, okay, then we're talking about the the second one. | |
| 16:12:17 | openstackgerrit | Maciej Szankin proposed openstack/nova master: conf: fix netconf, my_ip and host are unclear https://review.openstack.org/441210 | |
| 16:15:04 | cburgess | I know at least some of you are Lenovo ThinkPad users, what is the goto model these days? | |
| 16:16:25 | mriedem | cburgess: dansmith and melwitt have T460 i think | |
| 16:16:31 | mriedem | cburgess: i'm getting a P50 | |
| 16:16:39 | dansmith | mriedem: I have a T460s | |
| 16:16:54 | dansmith | mriedem: but my choice is MBP 13" remember :) | |
| 16:16:55 | mriedem | lyarwood: for https://review.openstack.org/#/c/431540/ is that swap volume from A->B and B->A, or also A->B, B->C? | |
| 16:17:06 | mriedem | dansmith: yeah i know | |
| 16:18:07 | lyarwood | mriedem: both | |
| 16:18:33 | cburgess | dansmith mriedem Yeah I use MBP personally but asking for a friend (read parents). Thanks for the feedback | |
| 16:18:54 | dansmith | heh | |
| 16:19:42 | mriedem | cburgess: get them a chromebook then | |
| 16:19:43 | openstackgerrit | Sarafraj Singh proposed openstack/nova-specs master: Spec to add per instance timeout for LM https://review.openstack.org/442809 | |
| 16:19:56 | mriedem | cheap and simple | |
| 16:20:10 | med_ | mfisch just bought an HP Spectre 360 15" -- nice gear cburgess | |
| 16:20:43 | med_ | though probably overkill for anyone's parents | |
| 16:20:45 | cburgess | mriedem For reasons I won't even begin to get into it needs to be windows with execl, and word, and outlook, and IE. | |
| 16:20:57 | med_ | unless they are Linus's kids | |
| 16:22:52 | mriedem | idk | |
| 16:22:52 | mriedem | cburgess: probably just so they can move their stuff and re-use the same tools, rather than upload existing docs to google drive or something | |
| 16:23:07 | mriedem | old people, including myself, do not like change | |
| 16:23:31 | mriedem | kashyap: yeah so i think the confirmation from danpb nails it, the first swap which is no resize is ok, but the 2nd which is the resize fails | |
| 16:23:53 | mriedem | med_: i can't remember the last time i've been carded since i don't drink | |
| 16:23:57 | mriedem | but maybe to get in | |
| 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 | |