)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"change_message_id":"3329fa69b8fe78165f20d6b0453fedd3c6403fef","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Select from a pool of hosts"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"*Add a pool of best hosts to choose a host from. The host subset size is configurable. Setting the subset size to 1 will mimic the current behavior, which is \"best_host \u003d weighed_hosts[0]\""},{"line_number":10,"context_line":"*changing property name to host_subset_size, was host_pool_size"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Icc6f8a7ca7cf992d99b872a45e241f13fa6e1536"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"AAAAM3%2F%2Ffhk%3D","line":9,"updated":"2013-01-18 18:00:08.000000000","message":"Some formatting love is needed here.  Check HACKING.rst","commit_id":"9c7cd2aaab1498c943fd6674a04ebeaadfc3ef88"},{"author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"change_message_id":"44b0d42336cb3624c575d878f11cbb350a626ce8","unresolved":false,"context_lines":[{"line_number":7,"context_line":"filter_scheduler: Select from a subset of hosts."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Adding a subset of best hosts to choose a host from, instead"},{"line_number":10,"context_line":"of chosing the host with the highest weight. The subset size is"},{"line_number":11,"context_line":"configurable. Setting the subset size to 1 will mimic the old"},{"line_number":12,"context_line":"behavior."},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"AAAAM3%2F%2FJV8%3D","line":10,"updated":"2013-02-05 17:41:28.000000000","message":"s/chosing/choosing/","commit_id":"b925e1a99a32c99b0d898d45ebda7643ce0d295f"}],"nova/scheduler/filter_scheduler.py":[{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"e11e88a85b4170d588f094e09a1776fd31cafa52","unresolved":false,"context_lines":[{"line_number":306,"context_line":""},{"line_number":307,"context_line":"            weighed_hosts \u003d self.host_manager.get_weighed_hosts(hosts,"},{"line_number":308,"context_line":"                    filter_properties)"},{"line_number":309,"context_line":"            host_subset_size \u003d CONF.host_subset_size if \\"},{"line_number":310,"context_line":"                CONF.host_subset_size \u003c len(weighed_hosts)\\"},{"line_number":311,"context_line":"                else len(weighed_hosts)"},{"line_number":312,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAM3%2F%2FPmc%3D","line":309,"updated":"2013-01-31 18:07:46.000000000","message":"Please don\u0027t use \\ for line continuation. The convention in openstack is to use something like (.","commit_id":"cd82f08fa62cd4d86a67426996311f7e5e36a2ee"},{"author":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"change_message_id":"fec1dbc24459ae39cf617595d17ed9cd997c2e72","unresolved":false,"context_lines":[{"line_number":306,"context_line":""},{"line_number":307,"context_line":"            weighed_hosts \u003d self.host_manager.get_weighed_hosts(hosts,"},{"line_number":308,"context_line":"                    filter_properties)"},{"line_number":309,"context_line":"            host_subset_size \u003d CONF.host_subset_size if \\"},{"line_number":310,"context_line":"                CONF.host_subset_size \u003c len(weighed_hosts)\\"},{"line_number":311,"context_line":"                else len(weighed_hosts)"},{"line_number":312,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAM3%2F%2FPZE%3D","line":309,"in_reply_to":"AAAAM3%2F%2FPmc%3D","updated":"2013-01-31 19:53:21.000000000","message":"or just don\u0027t bother with the ternary statement if it gets long enough to have to be split like this.  i\u0027d just go with multi-line normal if/else","commit_id":"cd82f08fa62cd4d86a67426996311f7e5e36a2ee"},{"author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"change_message_id":"44b0d42336cb3624c575d878f11cbb350a626ce8","unresolved":false,"context_lines":[{"line_number":37,"context_line":"                 default\u003d1,"},{"line_number":38,"context_line":"                 help\u003d\u0027New instances will be scheduled on a host chosen\u0027"},{"line_number":39,"context_line":"                      \u0027randomly from a subset of the N best hosts. This\u0027"},{"line_number":40,"context_line":"                      \u0027property defines the subst size that a host is\u0027"},{"line_number":41,"context_line":"                      \u0027chosen from. A value of 1 chooses the\u0027"},{"line_number":42,"context_line":"                      \u0027first host returned by the weighing functions.\u0027"},{"line_number":43,"context_line":"                      \u0027This value must be at least 1. Any value less than 1\u0027"}],"source_content_type":"text/x-python","patch_set":8,"id":"AAAAM3%2F%2FJV0%3D","line":40,"updated":"2013-02-05 17:41:28.000000000","message":"s/subst/subset/","commit_id":"b925e1a99a32c99b0d898d45ebda7643ce0d295f"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"8651d29a9103907140b6ec557e2593951cd31e3a","unresolved":false,"context_lines":[{"line_number":34,"context_line":""},{"line_number":35,"context_line":"filter_scheduler_pool_opts \u003d ["},{"line_number":36,"context_line":"    cfg.IntOpt(\u0027host_subset_size\u0027,"},{"line_number":37,"context_line":"                 default\u003d1,"},{"line_number":38,"context_line":"                 help\u003d\u0027New instances will be scheduled on a host chosen\u0027"},{"line_number":39,"context_line":"                      \u0027randomly from a subset of the N best hosts. This\u0027"},{"line_number":40,"context_line":"                      \u0027property defines the subset size that a host is\u0027"}],"source_content_type":"text/x-python","patch_set":10,"id":"AAAAM3%2F%2FG6w%3D","line":37,"updated":"2013-02-06 22:40:50.000000000","message":"femtonit: I\u0027d prefer this be indented to line up just after the open parenthesis—i.e., reduce the indentation by 2 spaces.","commit_id":"e1c047d194c65acccd01d89311cc3043d31156ff"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"8651d29a9103907140b6ec557e2593951cd31e3a","unresolved":false,"context_lines":[{"line_number":41,"context_line":"                      \u0027chosen from. A value of 1 chooses the\u0027"},{"line_number":42,"context_line":"                      \u0027first host returned by the weighing functions.\u0027"},{"line_number":43,"context_line":"                      \u0027This value must be at least 1. Any value less than 1\u0027"},{"line_number":44,"context_line":"                      \u0027will be ignored, and 1 will be used instead\u0027)"},{"line_number":45,"context_line":"]"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"CONF.register_opts(filter_scheduler_pool_opts)"}],"source_content_type":"text/x-python","patch_set":10,"id":"AAAAM3%2F%2FG6k%3D","line":44,"updated":"2013-02-06 22:40:50.000000000","message":"You have no spaces between successive lines.  This will end up being formatted as: \"chosenrandomly\", \"Thisproperty\", \"ischosen\", etc.  Please insert appropriate spaces; I prefer at the end of the lines, i.e.:\n\n    help\u003d\u0027New instances will be scheduled on a host chosen \u0027\n         \u0027randomly from a subset of the N best hosts. This \u0027\n    …","commit_id":"e1c047d194c65acccd01d89311cc3043d31156ff"}],"nova/tests/scheduler/test_filter_scheduler.py":[{"author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"change_message_id":"44b0d42336cb3624c575d878f11cbb350a626ce8","unresolved":false,"context_lines":[{"line_number":344,"context_line":"                         filter_properties[\u0027retry\u0027][\u0027hosts\u0027])"},{"line_number":345,"context_line":""},{"line_number":346,"context_line":"    def test_schedule_host_subset(self):"},{"line_number":347,"context_line":"        \"\"\"Make sure the host_subset_size property works"},{"line_number":348,"context_line":"        properly\"\"\""},{"line_number":349,"context_line":""},{"line_number":350,"context_line":"        self.flags(host_subset_size\u003d2)"}],"source_content_type":"text/x-python","patch_set":8,"id":"AAAAM3%2F%2FJXs%3D","line":347,"updated":"2013-02-05 17:41:28.000000000","message":"doc line split seems unnecessary?","commit_id":"b925e1a99a32c99b0d898d45ebda7643ce0d295f"},{"author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"change_message_id":"44b0d42336cb3624c575d878f11cbb350a626ce8","unresolved":false,"context_lines":[{"line_number":401,"context_line":"        self.assertEqual(len(hosts), 1)"},{"line_number":402,"context_line":""},{"line_number":403,"context_line":"    def test_schedule_chooses_best_host(self):"},{"line_number":404,"context_line":"        \"\"\"If host_subset_size is 1, the larget host with greatest"},{"line_number":405,"context_line":"        weightshould be returned\"\"\""},{"line_number":406,"context_line":""},{"line_number":407,"context_line":"        self.flags(host_subset_size\u003d1)"}],"source_content_type":"text/x-python","patch_set":8,"id":"AAAAM3%2F%2FJXQ%3D","line":404,"updated":"2013-02-05 17:41:28.000000000","message":"typos: \n s/larget/largest/\n s/weightshould/weight should/","commit_id":"b925e1a99a32c99b0d898d45ebda7643ce0d295f"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"faddce448dd6a0e4e16b93b3734b17e59c984c78","unresolved":false,"context_lines":[{"line_number":480,"context_line":"        self.assertRaises(exception.MigrationError,"},{"line_number":481,"context_line":"                self.driver._assert_compute_node_has_enough_memory,"},{"line_number":482,"context_line":"                self.context, instance, dest)"},{"line_number":483,"context_line":"        "},{"line_number":484,"context_line":"    def test_schedule_host_pool(self):"},{"line_number":485,"context_line":"        \"\"\"Make sure the scheduler_pool_size property works properly.\"\"\""},{"line_number":486,"context_line":""}],"source_content_type":"text/x-python","patch_set":12,"id":"AAAAM3%2F%2FAYk%3D","line":483,"updated":"2013-02-11 20:55:32.000000000","message":"Whitespace on a blank line (PEP8 error)","commit_id":"50aa6530108085c32cd91d8deb3c0199876f5c3d"}]}