)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},"change_message_id":"99f7e08f2c048f2a58ccde945a58985de29dd3c5","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"If your deployment has a slave configured, poll_*instance* periodic"},{"line_number":10,"context_line":"tasks will send their reads to the slave. If you have no slave"},{"line_number":11,"context_line":"configured, their is no change to behavior."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Note for developers, I have targeted all the easy tasks. By this I"},{"line_number":14,"context_line":"mean tasks that were already using the InstanceList object and the"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"AAAATn%2F9Fs4%3D","line":11,"updated":"2013-12-16 21:59:14.000000000","message":"It\u0027s a nit but - \u0027there\u0027  not \u0027their\u0027 for \u0027there is no change to behaviour\u0027. Oh and I think behavior is misspelt, but since I don\u0027t actually speak \u0027merican, I may be wrong about that.","commit_id":"c779c69a176e2cd07bbf09f92780780caf31315f"},{"author":{"_account_id":4912,"name":"Michael H Wilson","email":"geekinutah@gmail.com","username":"geekinutah"},"change_message_id":"96c6f8fc4cbd8f30c4776d8ee25c275a752b1153","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"If your deployment has a slave configured, poll_*instance* periodic"},{"line_number":10,"context_line":"tasks will send their reads to the slave. If you have no slave"},{"line_number":11,"context_line":"configured, their is no change to behavior."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Note for developers, I have targeted all the easy tasks. By this I"},{"line_number":14,"context_line":"mean tasks that were already using the InstanceList object and the"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"AAAATn%2F851s%3D","line":11,"in_reply_to":"AAAATn%2F9Fs4%3D","updated":"2013-12-19 00:10:00.000000000","message":"Happy to fix nits. I\u0027m going to stand firm on my \u0027merican spelling though :-). Thx for this.","commit_id":"c779c69a176e2cd07bbf09f92780780caf31315f"}],"nova/tests/api/openstack/fakes.py":[{"author":{"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},"change_message_id":"6e81080d134b43af73a719da1e4e4e2ece71c012","unresolved":false,"context_lines":[{"line_number":456,"context_line":"        if \"limit\" in kwargs:"},{"line_number":457,"context_line":"            limit \u003d kwargs[\"limit\"]"},{"line_number":458,"context_line":""},{"line_number":459,"context_line":"        if \u0027columns_to_join\u0027 in kwargs:"},{"line_number":460,"context_line":"            kwargs.pop(\u0027columns_to_join\u0027)"},{"line_number":461,"context_line":"        for i in xrange(num_servers):"},{"line_number":462,"context_line":"            uuid \u003d get_fake_uuid(i)"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAATn%2F9GTE%3D","line":459,"updated":"2013-12-16 20:08:44.000000000","message":"By copying this for use_slave you may be able to avoid the extra argument below.","commit_id":"c779c69a176e2cd07bbf09f92780780caf31315f"},{"author":{"_account_id":4912,"name":"Michael H Wilson","email":"geekinutah@gmail.com","username":"geekinutah"},"change_message_id":"96c6f8fc4cbd8f30c4776d8ee25c275a752b1153","unresolved":false,"context_lines":[{"line_number":456,"context_line":"        if \"limit\" in kwargs:"},{"line_number":457,"context_line":"            limit \u003d kwargs[\"limit\"]"},{"line_number":458,"context_line":""},{"line_number":459,"context_line":"        if \u0027columns_to_join\u0027 in kwargs:"},{"line_number":460,"context_line":"            kwargs.pop(\u0027columns_to_join\u0027)"},{"line_number":461,"context_line":"        for i in xrange(num_servers):"},{"line_number":462,"context_line":"            uuid \u003d get_fake_uuid(i)"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAATn%2F851o%3D","line":459,"in_reply_to":"AAAATn%2F9GTE%3D","updated":"2013-12-19 00:10:00.000000000","message":"Much cleaner, thx.","commit_id":"c779c69a176e2cd07bbf09f92780780caf31315f"},{"author":{"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},"change_message_id":"2444c0dc0408bdb7f36a74d76ea242f40a1fc8c6","unresolved":false,"context_lines":[{"line_number":487,"context_line":"                  launched_at\u003dtimeutils.utcnow(),"},{"line_number":488,"context_line":"                  terminated_at\u003dtimeutils.utcnow(),"},{"line_number":489,"context_line":"                  availability_zone\u003d\u0027\u0027, locked_by\u003dNone, cleaned\u003dFalse,"},{"line_number":490,"context_line":"                  use_slave\u003dFalse):"},{"line_number":491,"context_line":""},{"line_number":492,"context_line":"    if user_id is None:"},{"line_number":493,"context_line":"        user_id \u003d \u0027fake_user\u0027"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAATn%2F9ghk%3D","line":490,"updated":"2013-12-10 20:04:28.000000000","message":"I may be overlooking something but I can\u0027t see the reason for this change.","commit_id":"c779c69a176e2cd07bbf09f92780780caf31315f"},{"author":{"_account_id":4912,"name":"Michael H Wilson","email":"geekinutah@gmail.com","username":"geekinutah"},"change_message_id":"f5bfd531bc9320793c31720d4f88234a93e690b5","unresolved":false,"context_lines":[{"line_number":487,"context_line":"                  launched_at\u003dtimeutils.utcnow(),"},{"line_number":488,"context_line":"                  terminated_at\u003dtimeutils.utcnow(),"},{"line_number":489,"context_line":"                  availability_zone\u003d\u0027\u0027, locked_by\u003dNone, cleaned\u003dFalse,"},{"line_number":490,"context_line":"                  use_slave\u003dFalse):"},{"line_number":491,"context_line":""},{"line_number":492,"context_line":"    if user_id is None:"},{"line_number":493,"context_line":"        user_id \u003d \u0027fake_user\u0027"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAATn%2F9Gvs%3D","line":490,"in_reply_to":"AAAATn%2F9HJ4%3D","updated":"2013-12-16 18:49:43.000000000","message":"Actually it became very apparent why I added this when I ran tox again, a bunch of the API tests are calling stub_instance. Even though I straight up ignore use_slave it seems like I need to at least accept the arg since all the stubs pass kwargs through to stub_instance.","commit_id":"c779c69a176e2cd07bbf09f92780780caf31315f"},{"author":{"_account_id":4912,"name":"Michael H Wilson","email":"geekinutah@gmail.com","username":"geekinutah"},"change_message_id":"2bde72d611fb0d01727461c1a894d82517473135","unresolved":false,"context_lines":[{"line_number":487,"context_line":"                  launched_at\u003dtimeutils.utcnow(),"},{"line_number":488,"context_line":"                  terminated_at\u003dtimeutils.utcnow(),"},{"line_number":489,"context_line":"                  availability_zone\u003d\u0027\u0027, locked_by\u003dNone, cleaned\u003dFalse,"},{"line_number":490,"context_line":"                  use_slave\u003dFalse):"},{"line_number":491,"context_line":""},{"line_number":492,"context_line":"    if user_id is None:"},{"line_number":493,"context_line":"        user_id \u003d \u0027fake_user\u0027"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAATn%2F9HJ4%3D","line":490,"in_reply_to":"AAAATn%2F9ghk%3D","updated":"2013-12-16 17:37:13.000000000","message":"I think you are right, I believe this is left over from another approach that I was going to take.","commit_id":"c779c69a176e2cd07bbf09f92780780caf31315f"}]}