Earlier  
Posted Nick Remark
#openstack-nova - 2017-12-20
03:13:20 alex_xu oomichi: ^ remove extensions module entirely
03:24:10 openstackgerrit Alex Xu proposed openstack/nova master: Remove the LoadedExtensionInfo object https://review.openstack.org/529207
03:56:07 openstackgerrit blkart proposed openstack/nova master: Force delete an instance in any vm_state/task_state https://review.openstack.org/529212
04:22:21 openstackgerrit Alex Xu proposed openstack/nova master: Merge ResourceV21 obj into Resource obj https://review.openstack.org/529217
04:22:22 openstackgerrit Alex Xu proposed openstack/nova master: Remove the inherits parameter for the Resource object https://review.openstack.org/529218
05:43:32 jichen mriedem: hi, canyou please help to take a look at https://review.openstack.org/#/c/523387/ ?
06:12:58 openstackgerrit Chen Hanxiao proposed openstack/nova master: trivial: conf: libvirt: fix a typo https://review.openstack.org/529229
06:16:07 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/524795
06:38:15 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add migration db and object pagination support. https://review.openstack.org/514904
06:38:54 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add cross cell sort support for get_migrations https://review.openstack.org/517273
06:42:25 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add pagination and Changes-since filter support for os-migrations. https://review.openstack.org/330406
06:47:08 openstackgerrit Minho Ban proposed openstack/nova master: prevent removing numa_topology usage info from DB https://review.openstack.org/529236
07:09:49 openstackgerrit Ameed Ashour proposed openstack/nova master: change instance_system_metadata column type https://review.openstack.org/526900
07:25:45 openstackgerrit TommyLike proposed openstack/nova master: Remove redundant try/except block when authorize https://review.openstack.org/528991
07:32:14 openstackgerrit 龚肖 proposed openstack/nova master: compute: Catch binding failed exception while init host https://review.openstack.org/529249
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: libvirt: Introduce disk encryption config classes https://review.openstack.org/464008
09:36:11 openstackgerrit Lee Yarwood proposed openstack/nova master: WIP libvirt: QEMU native LUKS decryption for volumes https://review.openstack.org/523958
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: Refactor encryptor attach and detach calls https://review.openstack.org/460243
12:22:54 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Introduce disk encryption config classes https://review.openstack.org/464008
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: Remove server group sched filter support caching https://review.openstack.org/529200
13:37:13 openstackgerrit Jay Pipes proposed openstack/nova master: WIP Support aggregate affinity filters https://review.openstack.org/529201
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

Earlier   Later