)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"a371d9bb3f3a3b39c35570c8788be6011e1903b8","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add os-availability-zone extension"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"* Query /os-availability-zone to get an object representing the configured"},{"line_number":10,"context_line":"  availability zones and their state"},{"line_number":11,"context_line":"* This implements a subset of Nova\u0027s os-availability-zone extension"},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"AAAAPn%2F%2Fg1E%3D","line":9,"updated":"2013-06-28 18:56:54.000000000","message":"Did you mean to have a \u0027/\u0027 here?  If not you may want to fix this if you upload another patch set.","commit_id":"993afc46c0fb674a89a620671a0d82f0c2daf0aa"},{"author":{"_account_id":1132,"name":"Brian Waldon","email":"brian@waldon.cc","username":"bcwaldon"},"change_message_id":"c7f501579208667b189a5f6c6987fb91418b7443","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add os-availability-zone extension"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"* Query /os-availability-zone to get an object representing the configured"},{"line_number":10,"context_line":"  availability zones and their state"},{"line_number":11,"context_line":"* This implements a subset of Nova\u0027s os-availability-zone extension"},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"AAAAPn%2F%2Fgrk%3D","line":9,"in_reply_to":"AAAAPn%2F%2Fg1E%3D","updated":"2013-06-28 21:06:09.000000000","message":"I did. It\u0027s supposed to represent the URL after /vX/\u003ctenant\u003e, but I get the confusion - will remove if I upload a new patch.","commit_id":"993afc46c0fb674a89a620671a0d82f0c2daf0aa"}],"cinder/api/contrib/availability_zones.py":[{"author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"change_message_id":"187eaa3524e82808879e46b1b7f40400df6a9da9","unresolved":false,"context_lines":[{"line_number":59,"context_line":""},{"line_number":60,"context_line":"    name \u003d \u0027AvailabilityZones\u0027"},{"line_number":61,"context_line":"    alias \u003d \u0027os-availability-zone\u0027"},{"line_number":62,"context_line":"    namespace \u003d \u0027http://docs.openstack.org/volume/ext/os-availability-zone/api/v1\u0027"},{"line_number":63,"context_line":"    updated \u003d \u00272013-06-27T00:00:00+00:00\u0027"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"    def get_resources(self):"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAPn%2F%2Fi3w%3D","line":62,"updated":"2013-06-28 03:25:20.000000000","message":"line too long","commit_id":"62b1bcac0a9c04ef0b03d09995f24c92c0c478c9"},{"author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"change_message_id":"187eaa3524e82808879e46b1b7f40400df6a9da9","unresolved":false,"context_lines":[{"line_number":64,"context_line":""},{"line_number":65,"context_line":"    def get_resources(self):"},{"line_number":66,"context_line":"        controller \u003d Controller()"},{"line_number":67,"context_line":"        res \u003d extensions.ResourceExtension(Availability_zones.alias, controller)"},{"line_number":68,"context_line":"        return [res]"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAPn%2F%2Fi30%3D","line":67,"updated":"2013-06-28 03:25:20.000000000","message":"line too long","commit_id":"62b1bcac0a9c04ef0b03d09995f24c92c0c478c9"}],"cinder/tests/api/v1/stubs.py":[{"author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"change_message_id":"63db1fbcec2f674b402dbcc6e7214a3c26f269e9","unresolved":false,"context_lines":[{"line_number":132,"context_line":""},{"line_number":133,"context_line":""},{"line_number":134,"context_line":"def stub_service_get_all_by_topic(context, topic):"},{"line_number":135,"context_line":"    return [{\u0027availability_zone\u0027: \"zone1:host1\", \"disabled\": 0}]"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAPn%2F%2Fi2A%3D","line":135,"updated":"2013-06-28 03:50:45.000000000","message":"You need this same add in the v2 version","commit_id":"62b1bcac0a9c04ef0b03d09995f24c92c0c478c9"}],"cinder/volume/api.py":[{"author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"change_message_id":"187eaa3524e82808879e46b1b7f40400df6a9da9","unresolved":false,"context_lines":[{"line_number":319,"context_line":"        topic \u003d CONF.volume_topic"},{"line_number":320,"context_line":"        ctxt \u003d context.get_admin_context()"},{"line_number":321,"context_line":"        services \u003d self.db.service_get_all_by_topic(ctxt, topic)"},{"line_number":322,"context_line":"        az_data \u003d [(s[\u0027availability_zone\u0027], s[\u0027disabled\u0027]) for s in services]"},{"line_number":323,"context_line":""},{"line_number":324,"context_line":"        disabled_map \u003d {}"},{"line_number":325,"context_line":"        for (az_name, disabled) in az_data:"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAPn%2F%2Fi3g%3D","line":322,"updated":"2013-06-28 03:25:20.000000000","message":"The stub for this is incorrect and doesn\u0027t have the disabled key, it actually only returns this:[{\u0027availability_zone\u0027: \"zone1:host1\"}]","commit_id":"62b1bcac0a9c04ef0b03d09995f24c92c0c478c9"},{"author":{"_account_id":1132,"name":"Brian Waldon","email":"brian@waldon.cc","username":"bcwaldon"},"change_message_id":"5120f3ac087f789b9e3b1890087bb1dd18a3f718","unresolved":false,"context_lines":[{"line_number":319,"context_line":"        topic \u003d CONF.volume_topic"},{"line_number":320,"context_line":"        ctxt \u003d context.get_admin_context()"},{"line_number":321,"context_line":"        services \u003d self.db.service_get_all_by_topic(ctxt, topic)"},{"line_number":322,"context_line":"        az_data \u003d [(s[\u0027availability_zone\u0027], s[\u0027disabled\u0027]) for s in services]"},{"line_number":323,"context_line":""},{"line_number":324,"context_line":"        disabled_map \u003d {}"},{"line_number":325,"context_line":"        for (az_name, disabled) in az_data:"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAPn%2F%2FhvQ%3D","line":322,"in_reply_to":"AAAAPn%2F%2Fi3g%3D","updated":"2013-06-28 14:21:00.000000000","message":"I fixed one of the stubs but apparently not all of them. That will fix the tests. I\u0027ll get another patch up w/ pep8 fixes, too.","commit_id":"62b1bcac0a9c04ef0b03d09995f24c92c0c478c9"}]}