)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"change_message_id":"c91dd252a00729298d0c8663c9e75312981a3545","unresolved":false,"context_lines":[{"line_number":15,"context_line":"parts of the request_spec).   Also instance uuids are"},{"line_number":16,"context_line":"not logged making it hard to search for instance entries."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Thsi set of changes allows it to be used in a very"},{"line_number":19,"context_line":"large install with debug turned off for everything"},{"line_number":20,"context_line":"except filter_scheduler itself"},{"line_number":21,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAAOn%2F%2FfjM%3D","line":18,"updated":"2013-05-07 23:12:09.000000000","message":"Thsi \u003d\u003e This","commit_id":"1a308e8dfd011578d9913add3ac739546ec8ab51"},{"author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"change_message_id":"c91dd252a00729298d0c8663c9e75312981a3545","unresolved":false,"context_lines":[{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Thsi set of changes allows it to be used in a very"},{"line_number":19,"context_line":"large install with debug turned off for everything"},{"line_number":20,"context_line":"except filter_scheduler itself"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Specifically:"},{"line_number":23,"context_line":" - Log the instance_uuids at the start of schedule"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAAOn%2F%2FfjE%3D","line":20,"updated":"2013-05-07 23:12:09.000000000","message":"What do you think about removing the debug logs from some of the individual filters that overlap with these changes.","commit_id":"1a308e8dfd011578d9913add3ac739546ec8ab51"},{"author":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"change_message_id":"7e1772415eac259f27e95f16d0f437f169572fcd","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Adds useful debug logging to filter_scheduler"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"On a large system (\u003e500 hosts) the amount of logging"},{"line_number":10,"context_line":"information if everything is at DEBUG is vasts, due to"},{"line_number":11,"context_line":"the number of hosts checked for each filter"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"However at the moment some useful information is only"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"AAAAOn%2F%2FZTc%3D","line":10,"updated":"2013-05-10 23:08:44.000000000","message":"s/vasts/vast/","commit_id":"76f24ac1c750d4e04673db201c315e226b2aa596"},{"author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"change_message_id":"8ab2cd586d05480acdc06c0d6a3dfd3dcfed473f","unresolved":false,"context_lines":[{"line_number":31,"context_line":"changed to generate a list after each filter rather than"},{"line_number":32,"context_line":"building a recursive generator object.   Although this"},{"line_number":33,"context_line":"new approach is less elegant it does provide simple and"},{"line_number":34,"context_line":"useful insight into the behaviour of the filters, and s"},{"line_number":35,"context_line":"the impact on execution time on a system with several"},{"line_number":36,"context_line":"hundred hosts and 10 filters was negligible"},{"line_number":37,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"AAAAOn%2F%2FWj8%3D","line":34,"updated":"2013-05-13 21:31:40.000000000","message":"typo: \"and s the impact\"","commit_id":"76f24ac1c750d4e04673db201c315e226b2aa596"}],"nova/scheduler/filter_scheduler.py":[{"author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"change_message_id":"7e9d19b2aa2e1e22c86e630bdfd3fefd49d93fae","unresolved":false,"context_lines":[{"line_number":70,"context_line":"                        \u0027scheduler.run_instance.start\u0027, notifier.INFO, payload)"},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"        instance_uuids \u003d request_spec.pop(\u0027instance_uuids\u0027)"},{"line_number":73,"context_line":"        LOG.debug(_(\"Attempting to build %(num_instances)d instance(s) \" +\\"},{"line_number":74,"context_line":"                    \"uuids: %(instance_uuids)s\"),"},{"line_number":75,"context_line":"                  {\u0027num_instances\u0027: len(instance_uuids),"},{"line_number":76,"context_line":"                   \u0027instance_uuids\u0027: instance_uuids})"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAOn%2F%2FjTs%3D","line":73,"updated":"2013-05-04 19:03:19.000000000","message":"The + and \\ are not needed.","commit_id":"1a308e8dfd011578d9913add3ac739546ec8ab51"},{"author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"change_message_id":"7e9d19b2aa2e1e22c86e630bdfd3fefd49d93fae","unresolved":false,"context_lines":[{"line_number":89,"context_line":"            try:"},{"line_number":90,"context_line":"                try:"},{"line_number":91,"context_line":"                    weighed_host \u003d weighed_hosts.pop(0)"},{"line_number":92,"context_line":"                    LOG.debug(_(\"Choosing host %(weighed_host)s \" +\\"},{"line_number":93,"context_line":"                                \"for instance %(instance_uuid)s\"),"},{"line_number":94,"context_line":"                              {\u0027weighed_host\u0027: weighed_host,"},{"line_number":95,"context_line":"                               \u0027instance_uuid\u0027: instance_uuid})"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAOn%2F%2FjTo%3D","line":92,"updated":"2013-05-04 19:03:19.000000000","message":"Same here.  Also: there\u0027s already a \u0027Choosing host\u0027 log message in _scheduler().  Seems like that one should be removed.","commit_id":"1a308e8dfd011578d9913add3ac739546ec8ab51"}]}