)]}'
{"nova/tests/virt/disk/test_api.py":[{"author":{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},"change_message_id":"c4476cbc555b70422a73f51ce8dfb77c9855fd3a","unresolved":false,"context_lines":[{"line_number":16,"context_line":"#    under the License."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"import fixtures"},{"line_number":20,"context_line":"import os"},{"line_number":21,"context_line":"import tempfile"},{"line_number":22,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAM3%2F%2F3Xg%3D","line":19,"updated":"2012-12-29 10:56:05.000000000","message":"fixtures is a third-party library, so should be in a separate section.\n\n\u0027import os\nimport tempfile\n\nimport fixtures\n\nfrom nova.openstack.common...\u0027","commit_id":"c7ec48b3155dc98f131b64a9e3cfa249719f8c6b"},{"author":{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},"change_message_id":"c4476cbc555b70422a73f51ce8dfb77c9855fd3a","unresolved":false,"context_lines":[{"line_number":25,"context_line":"from nova.virt.disk import api"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"class APITestCase(test.TestCase):"},{"line_number":29,"context_line":"    def test_can_resize_need_fs_type_specified(self):"},{"line_number":30,"context_line":"        # NOTE(mikal): Bug 1094373 saw a regression where we failed to"},{"line_number":31,"context_line":"        # treat a failure to mount as a failure to be able to resize the"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAM3%2F%2F3Xc%3D","line":28,"updated":"2012-12-29 10:56:05.000000000","message":"Missing newline here.","commit_id":"c7ec48b3155dc98f131b64a9e3cfa249719f8c6b"},{"author":{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},"change_message_id":"eb1c64575c5fea822722cfe8aa7ab93018e28db3","unresolved":false,"context_lines":[{"line_number":23,"context_line":""},{"line_number":24,"context_line":"from nova.openstack.common import importutils"},{"line_number":25,"context_line":"from nova import test"},{"line_number":26,"context_line":"from nova.virt.disk import api"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"class APITestCase(test.TestCase):"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAM3%2F%2F3P0%3D","line":26,"updated":"2012-12-29 23:32:02.000000000","message":"These should be sorted according to the openstack coding standards.\ne.g.\nCurrently you have nova.o, nova, nova.v. You could argue the middle is nova.t, in which case, its still not sorted ;)","commit_id":"2397c6b902174ef39bff5f9194a06f82c6746267"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"a0ecd466397ca3c58e4eba3a611279e4133ef4ab","unresolved":false,"context_lines":[{"line_number":23,"context_line":""},{"line_number":24,"context_line":"from nova.openstack.common import importutils"},{"line_number":25,"context_line":"from nova import test"},{"line_number":26,"context_line":"from nova.virt.disk import api"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"class APITestCase(test.TestCase):"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAM3%2F%2F2Ns%3D","line":26,"in_reply_to":"AAAAM3%2F%2F3P0%3D","updated":"2013-01-02 16:51:49.000000000","message":"I think this is sorted, o -\u003e t -\u003e v (and that\u0027s how our hacking enforces it IIRC)","commit_id":"2397c6b902174ef39bff5f9194a06f82c6746267"}],"nova/tests/virt/disk/test_nbd.py":[{"author":{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},"change_message_id":"c4476cbc555b70422a73f51ce8dfb77c9855fd3a","unresolved":false,"context_lines":[{"line_number":16,"context_line":"#    under the License."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"import fixtures"},{"line_number":20,"context_line":"import os"},{"line_number":21,"context_line":"import tempfile"},{"line_number":22,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAM3%2F%2F3XY%3D","line":19,"updated":"2012-12-29 10:56:05.000000000","message":"You have inconsistent imports here. They should be\n\nimport os\\n\nimport tempfile\n\nimport fixtures\n\nfrom nova import test\\n\nfrom nova.virt.disk.mount import nbd","commit_id":"c7ec48b3155dc98f131b64a9e3cfa249719f8c6b"}],"nova/virt/disk/vfs/localfs.py":[{"author":{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},"change_message_id":"c4476cbc555b70422a73f51ce8dfb77c9855fd3a","unresolved":false,"context_lines":[{"line_number":61,"context_line":"        self.mount \u003d None"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"    def setup(self):"},{"line_number":64,"context_line":"        print \u0027setup\u0027"},{"line_number":65,"context_line":"        self.imgdir \u003d tempfile.mkdtemp(prefix\u003d\"openstack-vfs-localfs\")"},{"line_number":66,"context_line":"        try:"},{"line_number":67,"context_line":"            if self.imgfmt \u003d\u003d \"raw\":"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAM3%2F%2F3XU%3D","line":64,"updated":"2012-12-29 10:56:05.000000000","message":"really?","commit_id":"c7ec48b3155dc98f131b64a9e3cfa249719f8c6b"}]}