| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-12-20 | |||
| 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. | |
| 14:00:19 | kashyap | jaypipes: Haha, I have the spreads at home; it's the Italian Ciabatta bread :P | |
| 14:00:26 | jaypipes | :) | |
| 14:00:31 | mriedem | "the spreads" | |
| 14:00:34 | mriedem | sounds like "the runs" | |
| 14:00:35 | jaypipes | ah, mriedem that's the ticket. thanks! | |
| 14:00:36 | kashyap | jaypipes: Gentleman, it's called Gent -- http://wikitravel.org/en/Gent | |
| 14:00:43 | kashyap | s/men/man/ | |
| 14:00:47 | jaypipes | heh | |
| 14:00:56 | kashyap | (It's in Flanders, Belgium :P) | |
| 14:01:58 | kashyap | It's late lunch; was debugging the Gate migration bug from hell | |
| 14:02:02 | kashyap | While I'm here; I'll update lyarwood too | |
| 14:02:10 | kashyap | lyarwood: On that migration bug we were talking about | |
| 14:02:19 | kashyap | I spent past 2 hours creating a reproducer -- | |
| 14:02:49 | kashyap | Migrate a guest with a `dd if=/dev/zero bs=1M | md5sum` workload in it, AND throttle the bandwitdh to 1MiB/s | |
| 14:02:59 | kashyap | And also set the clock back 20 minutes on the source | |
| 14:02:59 | stephenfin | jaypipes: Why do you ask? I was waiting on sean-k-mooney to wade in on that again before doing anything, personally | |
| 14:03:07 | kashyap | To see if those keepalives will kick in | |
| 14:03:19 | kashyap | s/keepalives/keep-alive timeout/ | |
| 14:03:54 | jaypipes | stephenfin: mostly I was just looking for a link to that other patch :) I couldn't remember what the commit summary was and couldn't find it on gerrit. | |
| 14:03:56 | kashyap | I couldn't reproduce it so far; the guest migrated slowly after a long while. So, the investigation to be conitnued (assuming it's still a problem for the stable gate) | |
| 14:11:46 | danpawlik | mriedem: thanks for review the patch for mkisofs calls | |
| 14:11:59 | mdbooth | Incoming | |
| 14:12:10 | openstackgerrit | Matthew Booth proposed openstack/nova master: Rename block_device_info_get_root https://review.openstack.org/529028 | |
| 14:12:11 | openstackgerrit | Matthew Booth proposed openstack/nova master: Expose driver_block_device fields as attributes https://review.openstack.org/528362 | |
| 14:12:11 | openstackgerrit | Matthew Booth proposed openstack/nova master: Add local_root to block_device_info https://review.openstack.org/529029 | |
| 14:12:12 | openstackgerrit | Matthew Booth proposed openstack/nova master: Use real block_device_info data in libvirt tests https://review.openstack.org/527916 | |
| 14:12:12 | openstackgerrit | Matthew Booth proposed openstack/nova master: Pass DriverBlockDevice to driver.attach_volume https://review.openstack.org/528363 | |
| 14:12:13 | openstackgerrit | Matthew Booth proposed openstack/nova master: Pass disk_info dict to libvirt_info https://review.openstack.org/529329 | |
| 14:12:13 | openstackgerrit | Matthew Booth proposed openstack/nova master: Fix libvirt volume tests passing invalid disk_info https://review.openstack.org/529328 | |
| 14:13:08 | openstackgerrit | Stephen Finucane proposed openstack/nova master: console: Provide an RFB security proxy implementation https://review.openstack.org/345399 | |
| 14:13:09 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Document TLS security setup for noVNC proxy https://review.openstack.org/500544 | |
| 14:16:48 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Workaround missing RequestSpec.project_id when moving an instance https://review.openstack.org/529185 | |
| 14:16:48 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Use instance.project_id when creating request specs for old instances https://review.openstack.org/529184 | |
| 14:16:49 | mriedem | jaypipes: i had to fix the test for py35 ^ otherwise unchanged | |
| 14:17:14 | mriedem | stephenfin: gibi: ^ contains a fix for a regression introduced in newton which breaks us later in pike, | |
| 14:17:21 | mriedem | so i'm hoping to get that backported | |
| 14:17:36 | mriedem | before we eol newton | |
| 14:17:43 | stephenfin | Looking | |
| 14:18:04 | mriedem | the workaround patch will go back to pike, | |
| 14:18:08 | mriedem | the other one can go to newton | |
| 14:20:18 | ameeda | Hello, Can you please review my code here ? https://review.openstack.org/#/c/528069/ | |