| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-08-16 | |||
| 22:43:49 | melwitt | +2 | |
| 22:44:09 | oomichi | melwitt: thanks again | |
| 22:44:14 | melwitt | np | |
| 22:44:46 | melwitt | thanks for fixing those up | |
| #openstack-nova - 2017-08-17 | |||
| 01:52:58 | openstackgerrit | zhangbailin proposed openstack/nova-specs master: Modify spelling error in nova-specs document https://review.openstack.org/494094 | |
| 01:57:45 | openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi: cached images should be cleaned up by time https://review.openstack.org/465954 | |
| 02:15:53 | openstackgerrit | Kevin Zhao proposed openstack/nova master: Add video type virtio for AArch64 https://review.openstack.org/493822 | |
| 02:15:55 | openstackgerrit | melanie witt proposed openstack/nova master: Request zero root disk for boot-from-volume instances https://review.openstack.org/428481 | |
| 02:15:56 | openstackgerrit | melanie witt proposed openstack/nova master: Claim and report zero root disk for boot-from-volume instances https://review.openstack.org/428505 | |
| 02:20:20 | openstackgerrit | melanie witt proposed openstack/nova master: Claim and report zero root disk for boot-from-volume instances https://review.openstack.org/428505 | |
| 02:51:45 | openstackgerrit | Merged openstack/nova master: Make scheduler.utils.merge_resources ignore zero values https://review.openstack.org/493963 | |
| 04:35:46 | masber | hi, I am getting this error when I try to resize vm openstack "Insufficient compute resources: Requested instance NUMA topology cannot fit the given host NUMA topology." any idea? | |
| 04:46:43 | masber | hi, I am getting this error when I try to resize vm openstack "Insufficient compute resources: Requested instance NUMA topology cannot fit the given host NUMA topology." any idea? | |
| 04:55:58 | openstackgerrit | Merged openstack/nova master: Fix contributor documentation https://review.openstack.org/494277 | |
| 05:47:15 | openstackgerrit | Jianghua Wang proposed openstack/nova master: xenapi: cached images should be cleaned up by time https://review.openstack.org/465954 | |
| 07:23:10 | openstackgerrit | jichenjc proposed openstack/nova master: update comment for dropping support https://review.openstack.org/494061 | |
| 07:37:24 | openstackgerrit | jichenjc proposed openstack/nova master: Move common definition into common layer https://review.openstack.org/489491 | |
| 07:37:25 | openstackgerrit | jichenjc proposed openstack/nova master: WIP:check query param for service's index function https://review.openstack.org/489492 | |
| 07:41:49 | gibi | hm, interesting, freenode klined me during the night due to spamming | |
| 07:43:49 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955 | |
| 07:45:54 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964 | |
| 07:46:10 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042 | |
| 07:47:02 | openstackgerrit | Michael Still proposed openstack/nova master: Cleanup mount / umount and associated rmdir calls https://review.openstack.org/494423 | |
| 07:48:10 | openstackgerrit | Rikimaru Honjo proposed openstack/nova master: compute: Sweep duplicated bdm records https://review.openstack.org/290793 | |
| 08:36:32 | openstackgerrit | jichenjc proposed openstack/nova master: WIP: use conf for keystone session creation https://review.openstack.org/485121 | |
| 09:06:41 | openstackgerrit | Takashi NATSUME proposed openstack/nova-specs master: Abort Cold Migration https://review.openstack.org/334732 | |
| 09:16:18 | openstackgerrit | Pavlo Shchelokovskyy proposed openstack/nova master: Allow shuffling best hosts from weighed hosts https://review.openstack.org/494136 | |
| 09:23:05 | yongwc | I have a system volume and boot as a nova instance with FC connection, however when I shutdown the instance, the FC connection still be there, never do the terminate_connection operation, anyone know how I can terminate those FC connection on the nova compute host with openstack operation? | |
| 09:37:47 | openstackgerrit | jichenjc proposed openstack/nova master: WIP: fix race condition https://review.openstack.org/494458 | |
| 10:21:57 | trinaths | hi, I get this error when I spawn VM, Instance failed to spawn: libvirtError: monitor socket did not show up: No such file or directory. http://paste.openstack.org/show/618651/ | |
| 10:22:04 | trinaths | how to resolve this issue | |
| 10:22:06 | trinaths | please help me | |
| 10:34:31 | openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi: Live migration failed in xapi pool https://review.openstack.org/489451 | |
| 10:38:06 | jaypipes | trinaths: https://docs.fedoraproject.org/en-US/Fedora_Draft_Documentation/0.1/html/Virtualization_Deployment_and_Administration_Guide/App_Domain_Monitor_Socket.html | |
| 10:41:02 | trinaths | jaypipes: thanks for the link. I dont get that error. | |
| 10:44:09 | trinaths | jaypipes: here is the output for instance qemu log, http://paste.openstack.org/show/618655/ | |
| 10:44:18 | trinaths | jaypipes: there were no errors reported. | |
| 11:01:35 | openstackgerrit | Pavlo Shchelokovskyy proposed openstack/nova master: Allow shuffling hosts with the same best weight https://review.openstack.org/494136 | |
| 11:10:48 | trinaths | jaypipes: i get this error in the instance qemu log, http://paste.openstack.org/show/618660/ | |
| 11:12:46 | jaypipes | trinaths: I'm afraid I don't know what the solution to that is. It's definitely not a Nova issue, thouhg. | |
| 11:13:01 | trinaths | jaypipes: agree. | |
| 11:33:59 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add __str__ method to Host* objects https://review.openstack.org/493082 | |
| 11:37:46 | openstackgerrit | Michael Still proposed openstack/nova master: Cleanup mount / umount and associated rmdir calls https://review.openstack.org/494423 | |
| 12:21:21 | openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285 | |
| 12:23:34 | openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285 | |
| 13:39:51 | maciejjozefczyk | dansmith: Hello, could you take a look on this https://review.openstack.org/#/c/491808/ ? | |
| 13:51:26 | dansmith | maciejjozefczyk: I think you're missing a test case there, comments in the review | |
| 14:08:46 | maciejjozefczyk | dansmith: I dont get your comment. You mean to add asserting that fileters dictonary is dict with specific key set according to test case? | |
| 14:10:26 | dansmith | maciejjozefczyk: you need to assert that the filters dict passed to get_by_filters() looks like you expect and includes the host value you are changing (moving) | |
| 14:10:47 | dansmith | admittedly the test doesn't do a good job of this now, but given that you're moving the code you should make sure you're covered | |
| 14:11:14 | dansmith | because before it was asserting that host was in filters as passed to the _get_instances_on_driver() method, which can no longer be asserted | |
| 14:23:04 | dansmith | maciejjozefczyk: does that make sense? if necessary I can just pull it down and fix it myself | |
| 14:26:47 | maciejjozefczyk | dansmith: I'm back, give me a sec | |
| 14:30:56 | maciejjozefczyk | dansmith: so in test_get_instances_on_driver_fallback() It could be better to add little asserting that filters are the same like in L1451? I thought its already done while checking mock_instance_list mock, but I can add assertEqual too | |
| 14:31:12 | maciejjozefczyk | dansmith: you mean to solve it this way or I'm wrong too? | |
| 14:31:59 | dansmith | maciejjozefczyk: L1451 isn't an assertion | |
| 14:34:08 | dansmith | maciejjozefczyk: your assertion on L1487 is asserting that the thing you passed to _get_instances_on_driver() (which modifies it) is the thing that gets passed to get_by_filters(), but you don't assert what is in it | |
| 14:43:10 | maciejjozefczyk | dansmith: Yhmmm, right... | |
| 14:43:59 | dansmith | maciejjozefczyk: I think this is what you want: https://pastebin.com/egQnq8cH | |
| 14:44:18 | maciejjozefczyk | dansmith: what do you think about remove host key from filters in line 1452, and then asserting that it has been added after _get_instances_on_driver ? | |
| 14:44:49 | maciejjozefczyk | yes, exactly | |
| 14:45:05 | dansmith | maciejjozefczyk: asserting that it has been added to the thing you passed isn't good, but asserting that it was passed to the query method is, yes | |
| 14:45:24 | maciejjozefczyk | dansmith: okey | |
| 14:48:52 | maciejjozefczyk | dansmith: Thanks a lot for you time, sorry that I'm so retarted today | |
| 14:49:08 | dansmith | maciejjozefczyk: no problem :) | |
| 14:52:45 | gibi | dansmith: hi! Could you check the bugfix for the missing cleanup at evacuation https://review.openstack.org/#/c/493037/ ? Jay is +2 about it. | |
| 14:58:04 | dansmith | gibi: this is another thing we need for pike I guess? | |
| 14:58:15 | dansmith | sorry I hadn't noticed this yet | |
| 14:58:36 | cdent | gibi: why is remove_provider… what is wanted there? is the instance destroyed globally, or just on the compute node? | |
| 14:59:06 | cdent | “evacuate” is a very confusing term | |
| 14:59:54 | cdent | ah, I see from the tests, we want some allocations left over | |
| 15:00:23 | gibi | dansmith: I guess this is something that worked in Ocata but doesn't in Pike RC1 so yes we need a backport | |
| 15:00:50 | gibi | cdent: we want to remove the allocation from the source host of the allocation | |
| 15:00:57 | gibi | cdent: we want to remove the allocation from the source host of the evacuation | |
| 15:01:17 | gibi | cdent: so the source host will be the removed provider | |
| 15:01:47 | cdent | gibi: yeah, I figured that out after looking at the tests. I was struggling to remember the meaning of evacuate | |
| 15:02:09 | gibi | cdent: I agree that evacuate should be renamed to recreate | |
| 15:02:22 | gibi | dansmith: we have a similar bug in shelve offload https://bugs.launchpad.net/nova/+bug/1710249 | |
| 15:02:23 | openstack | Launchpad bug 1710249 in OpenStack Compute (nova) "nova doesn't clean up the resources after shelve offload" [High,In progress] - Assigned to Balazs Gibizer (balazs-gibizer) | |
| 15:02:34 | dansmith | gdi gibi stop finding bug! :) | |
| 15:02:36 | dansmith | *bugs | |
| 15:03:08 | gibi | dansmith: I don't want to make you mad but I'm currently looking at soft delete + periodic reclaim and that seems buggy as well... | |
| 15:03:15 | cdent | woot! | |
| 15:03:25 | dansmith | gibi: nooooo :P | |
| 15:04:20 | gibi | dansmith: but on the plus side simple migrate confirm / revert works based on https://review.openstack.org/#/c/493865/ | |
| 15:04:29 | dansmith | that's cool | |
| 15:04:48 | openstackgerrit | Maciej Jozefczyk proposed openstack/nova master: Remove host filter for _cleanup_running_deleted_instances periodic task https://review.openstack.org/491808 | |
| 15:06:16 | gibi | dansmith: but I will be on vacation for a week starting at next Tuesday so my bug flow will decrease ;) | |
| 15:08:24 | dansmith | gibi: good :) | |
| 15:08:30 | dansmith | gibi: can you propose that against pike? | |
| 15:08:50 | cdent | gibi: I rebased matt’s https://review.openstack.org/#/c/490733/ yesterday, and because of all your bug finding and fixing it needs a pretty manual rebase, but once we finally get going with shared providers, it will be handy | |
| 15:09:48 | gibi | dansmith: do you mean the evacuate one or both the evac and the shelve offload patches? | |
| 15:10:00 | dansmith | jaypipes: still around? | |
| 15:10:37 | dansmith | gibi: the evacuate one for now since it's on the way to the gate. I'm looking at the shelve one now | |
| 15:10:48 | gibi | cdent: ack, I will review that | |
| 15:11:03 | gibi | dansmith: OK | |
| 15:12:08 | jaypipes | dansmith: yyup | |
| 15:12:10 | maciejjozefczyk | dansmith: I've updated https://review.openstack.org/#/c/491808 ; looks good now? | |
| 15:12:29 | dansmith | jaypipes: can you go over this one too? I'm doing so as we speak: https://review.openstack.org/#/c/493834 | |
| 15:12:37 | dansmith | maciejjozefczyk: will look in a sec | |