| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-12-20 | |||
| 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/ | |
| 14:23:47 | stephenfin | ameeda: If you could add a little more context to that, it would make it much easier to review :) | |
| 14:29:11 | stephenfin | mriedem: Two questions left on https://review.openstack.org/#/c/529185/. I can ignore them if you really want to get it in now (or I've misunderstood something) | |
| 14:29:18 | stephenfin | The first one is a-ok | |
| 14:31:41 | ameeda | stephenfin: thanks for comment, I change the commit and I will submit it now | |
| 14:32:25 | mriedem | stephenfin: replied | |
| 14:34:41 | stephenfin | Cool. +2 to that too, in that case | |
| 14:39:45 | mriedem | stephenfin: thanks | |
| 14:39:51 | openstack | Launchpad bug 1739323 in OpenStack Compute (nova) pike "KeyError in host_manager for _get_host_states" [High,Confirmed] | |
| 14:39:51 | mriedem | mnaser: i'm at a loss for how you're hittin this https://bugs.launchpad.net/nova/+bug/1739323 | |
| 14:40:19 | mriedem | mnaser: dug through it again and the entries in seen_nodes should all also be in host_state_map because of https://github.com/openstack/nova/blob/3ebc358002d3f62a9ff0a3b0f50fea4f2a5c2c91/nova/scheduler/host_manager.py#L698 | |
| 14:40:50 | mriedem | and i don't see anything else in the HostStateManager that modifies that dict, so no locking issues that i can see, unless you have a multiple workers patch or something | |
| 14:41:57 | mriedem | mdbooth: maybe your keen eye can spot an obvious issue in here ^ | |
| 14:50:04 | openstackgerrit | Ameed Ashour proposed openstack/nova master: detach instance volumes when VM creation fails https://review.openstack.org/528385 | |
| 14:50:04 | openstackgerrit | Ameed Ashour proposed openstack/nova master: detaching volumes when deleted VM is in error state https://review.openstack.org/528069 | |
| 14:55:41 | ameeda | stephenfin: please check the comment and let me know, Thanks | |
| 14:57:30 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Cleanups for the scheduler code https://review.openstack.org/529343 | |
| 15:00:52 | mdbooth | mriedem_away: I'm a little lacking in scheduler context. It seems to me that we would hit this if the scheduler was capable of evaluating filters 'concurrently'. | |
| 15:01:11 | mdbooth | For some pythonic/eventlet definition of concurrently, obviously. | |
| 15:01:36 | stephenfin | lyarwood, mdbooth: Think you could take a look at this at some point? BDM'y things https://review.openstack.org/#/c/528069 | |
| 15:01:39 | mdbooth | Does the scheduler serialize all scheduling requests? | |
| 15:04:51 | jaypipes | mdbooth: no | |