)]}'
{"nova/conf/scheduler.py":[{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"660e7f4b493833aebefe174ba44ce8d4dd458a77","unresolved":false,"context_lines":[{"line_number":157,"context_line":"possible race conditions when two similar requests were received close"},{"line_number":158,"context_line":"together. With the change that claims are now done in the scheduler instead of"},{"line_number":159,"context_line":"the compute service, this race is no longer an issue."},{"line_number":160,"context_line":"\"\"\","},{"line_number":161,"context_line":"        help\u003d\"\"\""},{"line_number":162,"context_line":"Size of subset of best hosts selected by scheduler."},{"line_number":163,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"ff0f0b1f_324a2f55","line":160,"updated":"2017-05-18 01:05:01.000000000","message":"This option also controls whether the scheduler will pack (host_subset_size\u003d1) or spread (host_subset_size \u003e 1). Without this option, the scheduler will only pack.\n\nI\u0027m not sure whether there is any use case for wanting to spread instances during scheduling but wanted to raise the fact that this will take away the ability to do that, as far as I can tell.","commit_id":"97ce3df68e85a4d4efbfb456ede2cdca0c0c610e"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"78c83a2b18a2ffec0d7bee5f1bfb7c2777f68504","unresolved":false,"context_lines":[{"line_number":157,"context_line":"possible race conditions when two similar requests were received close"},{"line_number":158,"context_line":"together. With the change that claims are now done in the scheduler instead of"},{"line_number":159,"context_line":"the compute service, this race is no longer an issue."},{"line_number":160,"context_line":"\"\"\","},{"line_number":161,"context_line":"        help\u003d\"\"\""},{"line_number":162,"context_line":"Size of subset of best hosts selected by scheduler."},{"line_number":163,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"ff0f0b1f_bf9ed8b5","line":160,"in_reply_to":"ff0f0b1f_324a2f55","updated":"2017-05-19 08:52:23.000000000","message":"Mmm, good point Melanie. Given the default is 1, it means that multiple instances will get the same host until it is full.\n\nI should probably stop deprecating that logic in https://review.openstack.org/#/c/465175/ and discuss on deprecating that option only when we merge all the other points.","commit_id":"97ce3df68e85a4d4efbfb456ede2cdca0c0c610e"}],"nova/scheduler/filter_scheduler.py":[{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"4bee773f1914cdb5c1031e16925d88e054d26dc1","unresolved":false,"context_lines":[{"line_number":121,"context_line":""},{"line_number":122,"context_line":"            chosen_host \u003d None"},{"line_number":123,"context_line":"            allocated \u003d False"},{"line_number":124,"context_line":"            while weighed_hosts and not allocated:"},{"line_number":125,"context_line":"                chosen_host \u003d weighed_hosts.pop(0)"},{"line_number":126,"context_line":"                LOG.debug(\"Selected host: %(host)s\", {\u0027host\u0027: chosen_host})"},{"line_number":127,"context_line":"                # Allocate the resources used by the instance"}],"source_content_type":"text/x-python","patch_set":7,"id":"ff0f0b1f_56f2309c","line":124,"updated":"2017-05-26 12:18:34.000000000","message":"that\u0027s certainly a ton easier to read\n\nI don\u0027t have the experience to comment on the pack and spread issue","commit_id":"52540a4cf0f5051557122f56c85bc48ba724327e"}]}