| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-08-16 | |||
| 19:11:49 | cdent | that fix looks right on for the ironic case | |
| 19:12:23 | cdent | (because it is only randomize those hosts in the weighted list that match the first weight) | |
| 19:14:01 | melwitt | cdent: I'm not certain how much they love it but it's currently possible to pack and after this change I think it's not, correct? the use case for fill-first is the ability to schedule large instances. in spread-first, you could easily lose the ability to schedule large instances | |
| 19:14:32 | melwitt | yeah, I understand for ironic it would be fine. but this is changing it across the board, right? | |
| 19:15:01 | cdent | if you want a resource provider to be for large instances, just set min_unit > 1 on it’s resource? | |
| 19:15:33 | cdent | I’m slightly kidding on that. I know that we have to maintain existing behaviors and expectations, but still… also s/'// | |
| 19:15:35 | melwitt | at the very least, there should be a release note that explains that fill-first is no longer possible to signal the change in behavior | |
| 19:16:38 | melwitt | this is the sort of thing we'd usually email the operators ML about to get a quick check if anyone cares about fill-first anymore | |
| 19:18:27 | melwitt | and if there's a way to address the use case with resource provider settings, then that's probably fine. something to include in the release note | |
| 19:19:04 | dansmith | cdent: spreading uses more power, means lower utilization on your hosts, and in the case of windows, makes you bleed on the license fee like crazy | |
| 19:20:25 | cdent | there’s an easy fix to that windows problem :) no, I get it, I just, meh, reality... | |
| 19:21:35 | cdent | hmm, so right now in libvirt at least, we provide now way to control min_unit. I guess we’ll want to change that eventually | |
| 19:22:23 | melwitt | yeah, and I want to know why setting host_subset_size isn't sufficient. that's the knob for adjusting the spread or pack | |
| 19:22:41 | melwitt | it's mentioned in the commit message but I didn't get what it means | |
| 19:26:21 | cdent | melwitt: do jay and pavlo cover that in their discussion on ps2? | |
| 19:27:35 | cdent | heh, and simultaneously jay said the same thing on the review | |
| 19:33:15 | melwitt | yeah, my bad | |
| 19:40:35 | dansmith | melwitt: from briefly looking, I'd expect that we're basically using the natural ordering of the resource providers as returned from placement, | |
| 19:40:47 | dansmith | which means the order isn't predictable, but it is consistent | |
| 19:41:07 | dansmith | which would mean that if you don't have any weights configured and you just want to pack, this would completely break your desired behavior | |
| 19:42:25 | melwitt | dansmith: okay, so the question there would be, could we get around by not random shuffling if weights are all 0 or can 0 not be treated as a special thing? is there a way to detect no weights have been configured | |
| 19:42:33 | cdent | where does the order that makes things pack come from is there are no weighers? | |
| 19:42:49 | cdent | if there | |
| 19:43:16 | dansmith | melwitt: maybe? I'm not sure I really see the problem here anyway | |
| 19:43:39 | dansmith | melwitt: if you want to spread, then spread. If you want to pack, you're _choosing_ less desirable placement in terms of performance | |
| 19:44:34 | melwitt | dansmith: I mean, we could keep this change if in the case of no weights configured, we don't do the random shuffle | |
| 19:45:25 | melwitt | what do you mean by you don't see the problem? you don't see the use case for the change? | |
| 19:45:26 | dansmith | I get that, I'm just not sure I see the point | |
| 19:45:29 | dansmith | ues | |
| 19:45:32 | melwitt | k | |
| 19:45:53 | cdent | the bug report goes into some detail on why it is bad for ironic, yeah? | |
| 19:46:00 | melwitt | I think the only point is the shuffling of same weights only vs shuffling of all (using host_subset_size) | |
| 19:46:08 | dansmith | it's basically "if the weighers don't know the difference between two things, then spread regardless of what you want" | |
| 19:46:24 | melwitt | yeah | |
| 19:46:31 | dansmith | like, two things that score the same are not equal if you want packing | |
| 19:46:39 | melwitt | right | |
| 19:46:47 | dansmith | and AFAICT, this is just choosing one default over the other (reversing the current default) right? | |
| 19:46:59 | melwitt | but the change wants the ability to shuffle things that score the same | |
| 19:47:23 | melwitt | that is, they can't get what they want with host_subset_size | |
| 19:47:49 | dansmith | yeah, but they're breaking pack people because they want to shuffle a smaller set of the top results | |
| 19:48:08 | melwitt | yeah I know. I was trying to think if there's some way to get them what they want and also not break pack people | |
| 19:48:36 | dansmith | sorry, I realize you think I mean I don't understand what they want | |
| 19:48:37 | dansmith | I do, | |
| 19:48:53 | melwitt | no, I know that you understand | |
| 19:48:54 | dansmith | I just think it's not the automatic applies-to-everybody performance optimization that they think it is | |
| 19:49:03 | dansmith | ack | |
| 19:49:18 | melwitt | I was just trying to explain what I was saying, that I was brainstorming if there's way to fulfill both use cases | |
| 19:49:42 | dansmith | yep yep | |
| 19:50:48 | dansmith | making 0.0 a magic weight seems less desirable to me | |
| 19:51:08 | dansmith | it's more confusing than a boolean called shuffle_same_weight_hosts=True, IMHO | |
| 19:51:18 | melwitt | yeah. the more I think about it the more I think that would be a bad idea. because weights can be negative too, so 0.0 is a real weight | |
| 19:51:23 | dansmith | because you think you disabled all weights and you get completely random behavior | |
| 19:51:36 | dansmith | yep | |
| 19:54:36 | openstackgerrit | Merged openstack/nova master: Add documentation for documentation contributions https://review.openstack.org/492124 | |
| 19:55:13 | openstackgerrit | Merged openstack/nova master: Clean up *most* ec2 / euca2ools references https://review.openstack.org/492166 | |
| 20:03:11 | cdent | dansmith: I’m done for the night. I it looks like https://review.openstack.org/#/c/493963/ is going to need a recheck. I put the hash on the backport manually. | |
| 20:40:38 | oomichi | sdake: | |
| 20:40:56 | oomichi | oh, sorry. ^^^ is my mistake | |
| 21:17:20 | openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285 | |
| 21:17:20 | openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Tweak connection_info translation for the new Cinder attach/detach API https://review.openstack.org/493324 | |
| 21:20:43 | openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Add attachment_complete call to volume/cinder.py https://review.openstack.org/493323 | |
| 21:20:44 | openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285 | |
| 21:20:44 | openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Tweak connection_info translation for the new Cinder attach/detach API https://review.openstack.org/493324 | |
| 21:35:09 | openstackgerrit | Merged openstack/nova master: remove extension param and usage https://review.openstack.org/481491 | |
| 21:38:51 | openstackgerrit | Eric Fried proposed openstack/nova master: [Trivial] docstrings, typos, minor refactoring https://review.openstack.org/493701 | |
| 22:42:19 | oomichi | melwitt: hi, can you take a look at https://review.openstack.org/#/c/494277 ? That is just a follow-up patch for https://review.openstack.org/#/c/492124 | |
| 22:42:48 | oomichi | I hesitated to clear many +1 with updating, then created another one | |
| 22:42:56 | melwitt | oomichi: sure, looking | |
| 22:43:02 | oomichi | thanks :) | |
| 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 | |