| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-03-08 | |||
| 15:50:22 | sean-k-mooney | mriedem: when devstack does a pip install of the nova source it installs both the requirements.txt and test-requiremnts.txt gloablly | |
| 15:50:26 | lyarwood | mriedem: yeah that allows you to swap multiple times now | |
| 15:51:08 | kashyap | mriedem: Okay, danpb seems to have nailed it, with his QEMU POV: | |
| 15:52:13 | kashyap | Ok, so QEMU will be using a block device and you can't resize those. More specifically the 'block_resize' command when used against a block device, will just refresh qemu's view of the block device size, and if the requested resize size is larger than the block device size, it'l give EINVAL. IOW, you can use 'block_resize' on a SCSI device *if* you arrange for a resize to be done in the remote | |
| 15:52:19 | kashyap | iSCSI server first, and if you then tell the client host to rescan LUNs, and only then tell QEMU to apply the new size. | |
| 15:52:29 | sean-k-mooney | mriedem: that means that anyting else that uses pep8 is now runnning the addtional ordering tests | |
| 15:52:52 | sean-k-mooney | mriedem: as a result we cant build ovs-dpdk from sources as they dont enforcre the import ordering. | |
| 15:53:23 | sean-k-mooney | mriedem: devstack should not do https://github.com/openstack-dev/devstack/blob/6523d6e09789076d0a430bf7b608511fcb1c406a/inc/python#L336-L348 by default | |
| 15:55:43 | dansmith | johnthetubaguy: see the questions pending on this quotas thing? https://review.openstack.org/#/c/416521 | |
| 15:55:52 | dansmith | johnthetubaguy: it doesn't call you out specifically, but it might as well :P | |
| 15:56:19 | johnthetubaguy | dansmith: on the TC call, those should keep me amused while I am listening | |
| 15:56:21 | mriedem | kashyap: so i got the GET volume response info from the compute logs which has the size for each volume, and they both say 10 | |
| 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 | |