| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-12-20 | |||
| 07:44:26 | openstackgerrit | Ameed Ashour proposed openstack/nova master: change instance_system_metadata column type https://review.openstack.org/526900 | |
| 07:47:29 | openstackgerrit | guanzuoyu proposed openstack/nova-specs master: Fix an spelling error of driver https://review.openstack.org/529251 | |
| 07:50:54 | openstackgerrit | Chen Hanxiao proposed openstack/nova master: InstanceInfo: store more runtime information of instance https://review.openstack.org/526348 | |
| 07:53:57 | openstackgerrit | Marc Koderer proposed openstack/nova master: Enhance error handling for InvalidBDMVolume https://review.openstack.org/529254 | |
| 07:56:35 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Fix duplicate allocation candidates https://review.openstack.org/528591 | |
| 08:16:49 | openstackgerrit | Minho Ban proposed openstack/nova master: prevent removing numa_topology usage info from DB https://review.openstack.org/529236 | |
| 08:17:38 | maciejjozefczyk | mriedem: jaypipes thanks for https://review.openstack.org/#/c/520024/ | |
| 08:22:17 | openstackgerrit | 龚肖 proposed openstack/nova master: compute: Catch binding failed exception while init host https://review.openstack.org/529249 | |
| 08:57:15 | ameeda | can you please review my code here ? https://review.openstack.org/#/c/528069/ also here https://review.openstack.org/#/c/528385/ | |
| 09:36:10 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Refactor encryptor attach and detach calls https://review.openstack.org/460243 | |
| 09:36:11 | openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP libvirt: QEMU native LUKS decryption for volumes https://review.openstack.org/523958 | |
| 09:36:11 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Introduce disk encryption config classes https://review.openstack.org/464008 | |
| 09:37:02 | lyarwood | mdbooth: ^ updated the config class change, also if you have time reviews of the WIP change would be great, some of the code in there is horrid but required to keep prior behaviour | |
| 09:38:03 | mdbooth | Did gerrit just die? | |
| 09:38:08 | mdbooth | Was working a second ago | |
| 09:38:19 | lyarwood | yeah I think so | |
| 09:38:30 | lyarwood | Didn't think my change was _that_ bad | |
| 09:39:28 | mdbooth | Hehe | |
| 09:39:35 | mdbooth | It's really not happy about something, though | |
| 09:41:33 | openstackgerrit | Chen Hanxiao proposed openstack/nova master: libvirt: implement NotImplementedError for _set_quiesced https://review.openstack.org/529278 | |
| 09:55:16 | danpawlik | Hi, I have small question: I try to run unit tests on newton, and it raises me an error described here: https://bugs.launchpad.net/nova/+bug/1630420 . My question is: should I cherry-pick patch: https://review.openstack.org/#/c/383524/ to newton or add genisoimage to bindeps? | |
| 09:55:18 | openstack | Launchpad bug 1630420 in OpenStack Compute (nova) "config_drive unit tests (libvirt driver) aren't mocking genisoimage" [Low,Fix released] - Assigned to Diana Clarke (diana-clarke) | |
| 09:58:09 | openstackgerrit | Daniel Pawlik proposed openstack/nova stable/newton: Patch mkisofs calls https://review.openstack.org/529284 | |
| 10:01:48 | ebbex | mdbooth: I've created an instance with swap, where both root and swap are on rbd, yet there's a big swap file under nova/instances/_base on the compute-node, and according to the nova-compute logs it get's touched about once every minute (which would prevent _remove_old_enough_file from acting upon it). Is any of this intended behaviour? | |
| 10:02:48 | mdbooth | It's intended behaviour that it exists, although it could be done much better. | |
| 10:03:13 | mdbooth | We initialise a swap device on the compute host, then copy it to rbd iirc | |
| 10:03:32 | mdbooth | It's in _base because that's the cache | |
| 10:03:48 | mdbooth | Caching swap and ephemeral disks is a bug imho | |
| 10:04:15 | mdbooth | Not sure why it's being touched every minute, though | |
| 10:04:19 | mdbooth | Any idea what's touching it? | |
| 10:07:23 | ebbex | 2017-12-20T10:06:22.143079+00:00 oscompute01 sudo: nova : TTY=unknown ; PWD=/ ; USER=root ; COMMAND=/opt/openstack/nova/venv/bin/nova-rootwrap /opt/openstack/nova/etc/nova/rootwrap.conf touch -c /opt/openstack/nova/instances/_base/swap_16384 | |
| 10:09:30 | mdbooth | ebbex: Ah... IIRC there's a periodic task which touches cache files of active instances to prevent them from being deleted. | |
| 10:09:37 | mdbooth | But of course that's irrelevant for rbd. | |
| 10:09:46 | mdbooth | Another reason that rbd should have its own cache... | |
| 10:10:15 | mdbooth | Also LVM. | |
| 10:10:19 | mdbooth | And flat | |
| 10:10:33 | mdbooth | Everything except qcow2, in fact | |
| 10:12:52 | ebbex | Super, cause the compute-node has about 120gb storage, while ceph has a couple TBs. And I can see this clog up the disk on compute-node after 5-6 created vms. | |
| 10:13:39 | ebbex | Any idea for a quick fix? Disable the periodic touching perhaps? | |
| 10:14:08 | mdbooth | ebbex: If you found the relevant periodic task you could disable it if images_type != qcow2 | |
| 10:14:23 | mdbooth | As long as you accompanied it with a sufficiently explanatory comment | |
| 10:15:41 | mdbooth | We should have refactored imagebackend first time we implemented a backend other than qcow2 | |
| 10:19:18 | mdbooth | Unfortunately getting sufficient review attention for large refactors is hard, which discourages doing them. | |
| 10:24:05 | ebbex | Yeah, I have yet to understand how everything hangs together, so can't really contribute much there yet. | |
| 10:32:17 | openstackgerrit | Ameed Ashour proposed openstack/nova master: change instance_system_metadata column type https://review.openstack.org/526900 | |
| 10:53:46 | openstackgerrit | Ameed Ashour proposed openstack/nova master: change instance_system_metadata column type https://review.openstack.org/526900 | |
| 11:00:18 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: doc: update supported drivers for cpu topology https://review.openstack.org/529294 | |
| 11:06:26 | lyarwood | mdbooth: FYI I'm dropping the code to handle rbd volumes from the native LUKS change | |
| 11:07:11 | lyarwood | mdbooth: turns out that c-vol will always attempt to attach the os-brick encryptors and thus fail for rbd when creating a volume from an image. | |
| 11:11:12 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: doc: update supported drivers for cpu topology https://review.openstack.org/529294 | |
| 12:15:30 | jianghuaw_ | bauzas, jaypipes: Hi you may be interested on the new *test-vgpu* CI job added in "Citrix XenServer CI": https://review.openstack.org/#/c/516899/ | |
| 12:16:54 | jianghuaw_ | basically it's monitoring changes on some files in openstack/nova and openstack/os-xenapi and run vGPU test in devstack env. | |
| 12:18:57 | jianghuaw_ | bauzas, do you have time to review the above patch sometime today? Once it's merged, we can make the above test-vgpu CI job to be a formal test for XenAPI. | |
| 12:22:54 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Introduce disk encryption config classes https://review.openstack.org/464008 | |
| 12:22:54 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Refactor encryptor attach and detach calls https://review.openstack.org/460243 | |
| 12:22:55 | openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP libvirt: QEMU native LUKS decryption for volumes https://review.openstack.org/523958 | |
| 12:39:45 | jaypipes | jianghuaw_: excellent. :) | |
| 12:43:15 | cdent | jaypipes: can you confirm that gerrit is being a big sow in the wallow down back the holler? | |
| 12:47:18 | jaypipes | cdent: confirmed. | |
| 12:47:30 | jaypipes | cdent: it doesn't just hate you. | |
| 12:48:46 | cdent | thanks | |
| 13:09:30 | mnaser | and i think someone just emailed the ml about gerrit issues ^ | |
| 13:20:29 | jaypipes | gibi: https://review.openstack.org/#/c/528591/ is ready to go, IMHO. | |
| 13:23:35 | jaypipes | jianghuaw_: I'm not seeing in the test-vgpu logs where a flavor is being modified to include a request for resources:VGPU=1? | |
| 13:24:03 | jaypipes | jianghuaw_: erp, never mind. found it :) | |
| 13:24:14 | jaypipes | | flavor:extra_specs | {"resources:VGPU": "1"} | |
| 13:25:58 | jianghuaw_ | jaypipes, yeah. it's set at here "+ ./test_vgpu.sh:main:175 : nova flavor-key 1 set resources:VGPU=1" | |
| 13:26:23 | jaypipes | jianghuaw_: yep, found it eventually :) | |
| 13:26:33 | jianghuaw_ | :-) | |
| 13:27:13 | jaypipes | jianghuaw_: I'm wondering... is there any way to functionally test that the VM (once started) has the vGPU resources properly allocated from the hypervisor? | |
| 13:28:29 | jianghuaw_ | the test script will query xapi to verify if the VGPU is created and attached to the VM. | |
| 13:30:00 | jianghuaw_ | jaypipes, + ./test_vgpu.sh:main:214 : result='other-config (MRW): nova-instance-uuid: 64d20778-055a-4b58-a615-87a467caf7a2' | |
| 13:30:41 | jaypipes | jianghuaw_: excellent! | |
| 13:30:55 | jaypipes | jianghuaw_: nice work on this. :) | |
| 13:31:11 | jianghuaw_ | jaypipes, :-) thanks. | |
| 13:33:42 | jianghuaw_ | jaypipes, what we need is to get another core review on the patch. Hope bauzas will have time to review it. I really don't want to leave it to the new year:-( | |
| 13:34:04 | jianghuaw_ | I mean this patch: https://review.openstack.org/#/c/516899/ | |
| 13:34:34 | jaypipes | gibi: ^^ | |
| 13:35:07 | jianghuaw_ | thanks. | |
| 13:35:43 | gibi | jaypipes: looking... | |
| 13:36:03 | gibi | jaypipes: I was about to +2 https://review.openstack.org/#/c/528591/ when gerrit disappeared | |
| 13:37:13 | openstackgerrit | Jay Pipes proposed openstack/nova master: WIP Support aggregate affinity filters https://review.openstack.org/529201 | |
| 13:37:13 | openstackgerrit | Jay Pipes proposed openstack/nova master: Remove server group sched filter support caching https://review.openstack.org/529200 | |
| 13:37:59 | maciejjozefczyk | Guys, maybe you know if there is similiar thing implemented like https://blueprints.launchpad.net/nova/+spec/boot-order-for-instance | |
| 13:38:09 | maciejjozefczyk | Boot order for instance | |
| 13:38:29 | maciejjozefczyk | Or this idea died in Kilo? | |
| 13:38:33 | jianghuaw_ | gibi, Thanks. Gerrit service was restarted to resolve the slowness issue. it's recovered now. | |
| 13:43:07 | gibi | bauzas: is there a chance that you can check https://review.openstack.org/#/c/516899/ today as you already familiar with that patch? If not then I can dig | |
| 13:53:26 | mriedem1 | bauzas: can you take a look at https://bugs.launchpad.net/nova/+bug/1739323 ? | |
| 13:53:27 | openstack | Launchpad bug 1739323 in OpenStack Compute (nova) pike "KeyError in host_manager for _get_host_states" [High,Confirmed] | |
| 13:53:48 | mriedem1 | i can't remember why we changed from an iterator there in ocata, but you must have had a reason | |
| 13:58:48 | jaypipes | bauzas, kashyap, mriedem: regarding https://review.openstack.org/#/c/529294/ ... do I remember some patch recently that disabled either CPU pinning or NUMA topology (or both?) for any hypervisor other than KVM? I have this strange feeling that we recently restricted NUMA/CPU pinning to just KVM... | |
| 13:59:13 | gibi | mriedem: I left some suggestion in https://review.openstack.org/#/c/529194/ | |
| 13:59:26 | gibi | mriedem: if you want to discuss then i'm still here for couple of hours today | |
| 13:59:39 | kashyap | jaypipes: Hey, just going to get a piece of bread for lunch | |
| 13:59:52 | mriedem | jaypipes: yes i can dig up the link | |
| 14:00:01 | jaypipes | kashyap: I think you can probably afford more than just a piece of bread for lunch. | |
| 14:00:02 | kashyap | Should be back soon | |
| 14:00:06 | mriedem | jaypipes: https://review.openstack.org/#/c/465160/ | |
| 14:00:17 | jaypipes | kashyap: clearly, Holland has become too expensive. | |