| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-04-04 | |||
| 10:28:09 | johnthetubaguy | sfinucan: the use I like is specifying things that image requires in order to work correctly | |
| 10:29:59 | johnthetubaguy | sfinucan: but less is more when it comes to APIs, so if its something only the admin wants to control, I prefer not adding image metadata. For people who care we force them to add flavor extra specs now, rather than there being no way to change the defaults, possibly for zero gain. | |
| 10:30:27 | sfinucan | johnthetubaguy: Right. I think something like CPU thread or PCI policies falls into that category (the image would boot, but the combined performance impact could mean the application is not fit for purpose and therefore "broken") | |
| 10:30:36 | sfinucan | Hmm, yeah - that's the other perspective | |
| 10:31:30 | sfinucan | Long story short, that spec was applying the above idea to emulator thread policies. I don't care enough about it to pursue it though, heh | |
| 10:39:00 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Rewrap 'code-review' at ¬79 characters https://review.openstack.org/384960 | |
| 11:07:54 | kashyap | sfinucan: Why rewrap? | |
| 11:08:00 | kashyap | It's at 72 now? | |
| 11:08:39 | sfinucan | kashyap: I think I had a change after it at one point? | |
| 11:08:47 | kashyap | Something like that. FWIW, it looks okay to me | |
| 11:08:52 | kashyap | I think I vaguely recall some list discussion | |
| 11:08:56 | sfinucan | It was in merge conflict so I just fixed it :) | |
| 11:09:59 | kashyap | Ah, okay :-) | |
| 11:48:38 | openstackgerrit | Lee Yarwood proposed openstack/nova-specs master: Libvirt: Enable QEMU native LUKs handling https://review.openstack.org/437070 | |
| 11:49:10 | lyarwood | mdbooth, kashyap ^ if you have spec review time today, I'm not 100% sure that we will have the required versions of QEMU and libvirt in the gate for Pike tbh | |
| 11:49:29 | kashyap | lyarwood: I did notice it, actually | |
| 11:49:41 | kashyap | lyarwood: Yes, we don't have the reqiured versions | |
| 11:50:20 | lyarwood | kashyap: right, but 17.04 will, I just don't think it's going to be used in the gate as it isn't LTS | |
| 11:50:22 | kashyap | lyarwood: We should really get that in-progress (over a year) DevStack plugin that lets us use QEMU / libvirt from Git (and newest releases / RCs) with Nova | |
| 11:50:42 | lyarwood | kashyap: link? I don't think I've seen that | |
| 11:51:17 | kashyap | lyarwood: https://review.openstack.org/#/c/108714/ | |
| 11:51:35 | lyarwood | kashyap: thanks! | |
| 11:51:37 | kashyap | lyarwood: Err, scratch that -- that's the OLD one that was subimtted directly as part of DevStack | |
| 11:51:52 | kashyap | But the agreed decision was to make it a plugin | |
| 11:52:24 | lyarwood | kk | |
| 11:55:52 | openstackgerrit | Lee Yarwood proposed openstack/nova-specs master: Libvirt: Enable QEMU native LUKs handling https://review.openstack.org/437070 | |
| 11:56:17 | mdbooth | lyarwood: Will try to look today | |
| 11:56:44 | lyarwood | mdbooth: thanks | |
| 12:05:34 | lyarwood | kashyap: thanks for the link in the review btw | |
| 12:05:41 | lyarwood | kashyap: I'll check that out after lunch | |
| 12:06:17 | kashyap | lyarwood: Np. I'll be looking at that plugin again today, been meaning to but couldn't get around | |
| 12:07:01 | jaypipes | snikitin_: done | |
| 12:08:55 | snikitin_ | jaypipes, thanks! | |
| 12:12:31 | openstackgerrit | Silvan Kaiser proposed openstack/nova master: Removes getfattr from Quobyte Nova driver https://review.openstack.org/428646 | |
| 12:15:45 | openstackgerrit | Silvan Kaiser proposed openstack/nova master: Removes getfattr from Quobyte Nova driver https://review.openstack.org/428646 | |
| 12:28:52 | bauzas | jaypipes: you could be interested in https://review.openstack.org/#/c/446446/ | |
| 12:29:01 | bauzas | johnthetubaguy: too ^ | |
| 12:31:47 | johnthetubaguy | bauzas: hopefully will get there, mostly just cleaning up the queue this morning | |
| 12:32:10 | bauzas | johnthetubaguy: where are we discussing in the PTG etherpad about https://review.openstack.org/#/c/381912/10/specs/pike/approved/strict_isolation_of_group_of_hosts_for_image_and_flavor.rst ? | |
| 12:32:16 | bauzas | johnthetubaguy: I don't remember that | |
| 12:32:31 | johnthetubaguy | bauzas: search for the patch number, you should find it | |
| 12:33:29 | bauzas | johnthetubaguy: FWIW, it's an old discussion | |
| 12:38:56 | snikitin_ | jaypipes: I don't know why bot after your reapprove gate jobs weren't started. https://review.openstack.org/#/c/445855/ maybe try again with +2 | |
| 12:39:13 | snikitin_ | ? | |
| 12:41:48 | sdague | snikitin_: it needs both a +2 and a +W | |
| 12:42:22 | snikitin_ | sdague, thanks! | |
| 12:59:16 | johnthetubaguy | bauzas: sfinucan: can I ask you about this one: https://review.openstack.org/#/c/438787 | |
| 12:59:26 | sfinucan | johnthetubaguy: Hit me | |
| 12:59:36 | johnthetubaguy | bauzas: sfinucan: why can't we use the existing image property, basically? | |
| 13:00:19 | sfinucan | johnthetubaguy: I think the existing property refers to a different attribute of the emulated VGA dev | |
| 13:00:55 | johnthetubaguy | it seems to be video ram vs vga ram, they seem like the same thing to me | |
| 13:01:13 | bauzas | johnthetubaguy: AFAICU, that seems to be a separate property | |
| 13:01:28 | bauzas | looking at https://libvirt.org/formatdomain.html | |
| 13:01:29 | sfinucan | johnthetubaguy: That should be in the spec. If you haven't done it already, there should probably be a comment to the same | |
| 13:01:31 | johnthetubaguy | but why are they different? what does it mean? | |
| 13:01:43 | sfinucan | They are different properties though. Lemme find the QEMU docs | |
| 13:02:16 | sfinucan | johnthetubaguy: | |
| 13:02:18 | sfinucan | johnthetubaguy: https://www.ovirt.org/documentation/draft/video-ram/ | |
| 13:02:28 | sfinucan | vgamem is allocated within ram. Minimum ram size, as enforced by QEMU regardless the command line settings, is 2 * vgamem. vram is allocated separately. | |
| 13:02:38 | bauzas | johnthetubaguy: https://libvirt.org/formatdomain.html#elementsVideo | |
| 13:02:47 | bauzas | what sfinucan said | |
| 13:03:12 | bauzas | "Attribute ram ( since 1.0.2) specifies the size of the primary bar, while the attribute vram specifies the secondary bar size. If ram or vram are not supplied a default value is used. The ram should also be rounded to power of two as vram. There is also optional attribute vgamem (since 1.2.11) to set the size of VGA framebuffer for fallback mode of QXL device | |
| 13:03:26 | bauzas | honestly, it's a very specific libvirt thing | |
| 13:03:31 | sfinucan | fwiw, it's just another libvirt knob | |
| 13:03:35 | sfinucan | bauzas: jinx :) | |
| 13:03:46 | bauzas | I'm not really happy with that, hence my comment saying to maybe wait for custom RCs :) | |
| 13:03:53 | sfinucan | bauzas: well, technically QEMU - libvirt just wraps it | |
| 13:06:20 | bauzas | johnthetubaguy: sfinucan: just -1d it again | |
| 13:08:09 | openstackgerrit | Artom Lifshitz proposed openstack/nova-specs master: SPICE native client support https://review.openstack.org/442040 | |
| 13:09:59 | johnthetubaguy | bauzas: don't feel we are being great at giving suggestions how to fix the spec right now though | |
| 13:14:20 | bauzas | johnthetubaguy: I tried to explain my preferrence about providing custom resource classes against very specific image props | |
| 13:16:30 | openstackgerrit | Sergey Nikitin proposed openstack/nova master: Added PCI NUMA policies https://review.openstack.org/390520 | |
| 13:28:58 | mriedem | is everyone excited for a fun spec review day?! | |
| 13:29:39 | peter-hamilton | mriedem: indeed! | |
| 13:29:39 | artom | Doc's appointment this morning, but will try to participate this afternoon | |
| 13:29:40 | mriedem | i see johnthetubaguy has been having fun | |
| 13:30:22 | johnthetubaguy | yeah, I abandoned a bunch of left alone specs | |
| 13:30:39 | johnthetubaguy | only three pages of joy now | |
| 13:30:44 | mriedem | awesome | |
| 13:39:16 | openstackgerrit | Maciej Szankin proposed openstack/nova-specs master: Add remove-nova-cert spec https://review.openstack.org/448882 | |
| 13:45:54 | openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-manage: Deprecate 'shell' commands https://review.openstack.org/432280 | |
| 13:45:54 | openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-manage: Deprecate 'agent' commands https://review.openstack.org/450795 | |
| 13:45:55 | openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-manage: Deprecate 'host' commands https://review.openstack.org/453175 | |
| 13:45:55 | openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-manage: Deprecate 'project', 'account' commands https://review.openstack.org/453174 | |
| 13:45:56 | openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-manage: Deprecate 'log' commands https://review.openstack.org/453176 | |
| 13:48:36 | bauzas | mriedem: I guess you don't know https://en.wikipedia.org/wiki/L%C3%A9onard_%28comics%29 ? | |
| 13:48:56 | mriedem | nope | |
| 13:49:38 | bauzas | mriedem: because the disciple always says "I serve science and it's my joy" | |
| 13:52:58 | dane-fichter | peter-hamilton: anything relevant so far? | |
| 13:53:18 | peter-hamilton | dane-fichter: not yet :) | |
| 13:53:36 | dane-fichter | peter-hamilton: cool. got caught in traffic | |
| 13:54:15 | mriedem | peter-hamilton: dane-fichter: you guys might want to review https://review.openstack.org/#/c/448882 | |
| 13:54:23 | mriedem | for dropping the nova-cert service | |
| 13:54:32 | peter-hamilton | mriedem: looking now | |
| 13:54:37 | dane-fichter | mriedem: we'll take a look, thanks | |
| 13:58:02 | tdurakov | hi, folks, there will be no live-migration meeting this week | |
| 13:58:10 | bhagyashris | johnthetubaguy: Hi, Could you please review patch https://review.openstack.org/#/c/409644/ ? if you have free time. | |
| 14:01:03 | sdague | johnthetubaguy: why did you think 400 instead of 404 - https://review.openstack.org/#/c/448882 ? | |
| 14:01:48 | johnthetubaguy | sdague: didn't consider 404 honestly | |
| 14:02:02 | johnthetubaguy | sdague: I was seeing this as the implementation removed, not the API changed | |