)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5652,"name":"David Ripton","email":"dripton@redhat.com","username":"dripton"},"change_message_id":"cbc4d3136a0d968f19e5c8a5d832c91009893edd","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     gtt116 \u003cgtt116@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2013-05-09 02:10:37 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Extract getting instance\u0027 AZ into a helper module."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Since get an instance\u0027s availability zone logic may used for many modules,"},{"line_number":10,"context_line":"so move it from extend API into helper module `nova.availability_zones`."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"AAAAOn%2F%2Fb%2BQ%3D","line":7,"updated":"2013-05-09 17:50:53.000000000","message":"s/instance\u0027/instance\u0027s/","commit_id":"6e433d360a63f304f918f593c0e2b0562a1cadf7"},{"author":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"change_message_id":"a429e96e9ddf1fe5e34ec5e4b5b9e3faf9944f20","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Since get an instance\u0027s availability zone logic may used for many modules,"},{"line_number":10,"context_line":"so move it from extend API into helper module `nova.availability_zones`."},{"line_number":11,"context_line":"And add some tests for the new method."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"part of bp: different-availability-zone-filter"},{"line_number":14,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"AAAAOn%2F%2FZU0%3D","line":11,"updated":"2013-05-10 22:55:35.000000000","message":"I would reword this paragraph a bit.  Here is a proposed replacement:\n\nAn instance\u0027s availability zone logic may used by many modules, so move it from extend API into helper module `nova.availability_zones`. Also add some tests for the new method.","commit_id":"be8cfb071b02a1e781d5cbb279b5734b7f2f076a"},{"author":{"_account_id":1678,"name":"Tiantian Gao","email":"gtt116@gmail.com","username":"gtt116"},"change_message_id":"4f102c170f76a7e29b13686e203e42c653610362","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Since get an instance\u0027s availability zone logic may used for many modules,"},{"line_number":10,"context_line":"so move it from extend API into helper module `nova.availability_zones`."},{"line_number":11,"context_line":"And add some tests for the new method."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"part of bp: different-availability-zone-filter"},{"line_number":14,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"AAAAOn%2F%2FZMM%3D","line":11,"in_reply_to":"AAAAOn%2F%2FZU0%3D","updated":"2013-05-11 01:12:30.000000000","message":"Done\n\nThanks Russell, The commit message looks better now :-)","commit_id":"be8cfb071b02a1e781d5cbb279b5734b7f2f076a"}],"nova/availability_zones.py":[{"author":{"_account_id":4395,"name":"Aaron Rosen","email":"aaronorosen@gmail.com","username":"arosen"},"change_message_id":"636896a4e6a8b2acc27e2b7ec4990b5964cd4a68","unresolved":false,"context_lines":[{"line_number":93,"context_line":"    \"\"\"Return availability zone of specified instance.\"\"\""},{"line_number":94,"context_line":"    host \u003d str(instance.get(\u0027host\u0027))"},{"line_number":95,"context_line":"    if not host:"},{"line_number":96,"context_line":"        return None"},{"line_number":97,"context_line":""},{"line_number":98,"context_line":"    cache_key \u003d \"azcache-%s\" % host"},{"line_number":99,"context_line":"    az \u003d MC.get(cache_key)"}],"source_content_type":"text/x-python","patch_set":7,"id":"AAAAOn%2F%2FYMo%3D","line":96,"updated":"2013-05-13 02:04:54.000000000","message":"nit: None is implied. (but OK as i see you just moved that method here. )","commit_id":"488fcb4ad345bfb4fd614a140b317b606e008872"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"66ec47159cfb03751156123be09c70fade1194ed","unresolved":false,"context_lines":[{"line_number":93,"context_line":"    \"\"\"Return availability zone of specified instance.\"\"\""},{"line_number":94,"context_line":"    host \u003d str(instance.get(\u0027host\u0027))"},{"line_number":95,"context_line":"    if not host:"},{"line_number":96,"context_line":"        return None"},{"line_number":97,"context_line":""},{"line_number":98,"context_line":"    cache_key \u003d \"azcache-%s\" % host"},{"line_number":99,"context_line":"    az \u003d MC.get(cache_key)"}],"source_content_type":"text/x-python","patch_set":7,"id":"AAAAO3%2F%2F%2Bow%3D","line":96,"in_reply_to":"AAAAOn%2F%2FYMo%3D","updated":"2013-05-15 18:18:03.000000000","message":"I like to have the value to return be explicit, if there are non-None returns elsewhere in a function or method.  Probably my C programming background speaking :)","commit_id":"488fcb4ad345bfb4fd614a140b317b606e008872"}]}