)]}'
{"nova/api/openstack/compute/volumes.py":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"d0fde70676ee2a6ab7e0870ae67718a38608f756","unresolved":false,"context_lines":[{"line_number":221,"context_line":"    \"\"\"Maps keys for attachment summary view.\"\"\""},{"line_number":222,"context_line":"    d \u003d {}"},{"line_number":223,"context_line":""},{"line_number":224,"context_line":"    # NOTE(justinsb): We use the volume id as he id of the attachment object"},{"line_number":225,"context_line":"    d[\u0027id\u0027] \u003d volume_id"},{"line_number":226,"context_line":""},{"line_number":227,"context_line":"    d[\u0027volumeId\u0027] \u003d volume_id"}],"source_content_type":"text/x-python","patch_set":8,"id":"dada55a8_710e113c","line":224,"range":{"start_line":224,"start_character":46,"end_line":224,"end_character":48},"updated":"2016-07-21 06:06:59.000000000","message":"i think unrelated change.","commit_id":"7b7270fa2aab9ffc90fee1655c68c2644d21206f"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"8b2743150deb382b13fc3831c000ed6e8c36da96","unresolved":false,"context_lines":[{"line_number":221,"context_line":"    \"\"\"Maps keys for attachment summary view.\"\"\""},{"line_number":222,"context_line":"    d \u003d {}"},{"line_number":223,"context_line":""},{"line_number":224,"context_line":"    # NOTE(justinsb): We use the volume id as he id of the attachment object"},{"line_number":225,"context_line":"    d[\u0027id\u0027] \u003d volume_id"},{"line_number":226,"context_line":""},{"line_number":227,"context_line":"    d[\u0027volumeId\u0027] \u003d volume_id"}],"source_content_type":"text/x-python","patch_set":8,"id":"dada55a8_eee72e3e","line":224,"range":{"start_line":224,"start_character":46,"end_line":224,"end_character":48},"in_reply_to":"dada55a8_70116faf","updated":"2016-07-25 06:10:26.000000000","message":"oops, will revert this.","commit_id":"7b7270fa2aab9ffc90fee1655c68c2644d21206f"},{"author":{"_account_id":6062,"name":"jichenjc","email":"jichenjc@cn.ibm.com","username":"jichenjc"},"change_message_id":"9fe3047499868bf91be460e668466d75479b5b9b","unresolved":false,"context_lines":[{"line_number":221,"context_line":"    \"\"\"Maps keys for attachment summary view.\"\"\""},{"line_number":222,"context_line":"    d \u003d {}"},{"line_number":223,"context_line":""},{"line_number":224,"context_line":"    # NOTE(justinsb): We use the volume id as he id of the attachment object"},{"line_number":225,"context_line":"    d[\u0027id\u0027] \u003d volume_id"},{"line_number":226,"context_line":""},{"line_number":227,"context_line":"    d[\u0027volumeId\u0027] \u003d volume_id"}],"source_content_type":"text/x-python","patch_set":8,"id":"dada55a8_70116faf","line":224,"range":{"start_line":224,"start_character":46,"end_line":224,"end_character":48},"in_reply_to":"dada55a8_710e113c","updated":"2016-07-23 16:50:56.000000000","message":"yeah, a typo","commit_id":"7b7270fa2aab9ffc90fee1655c68c2644d21206f"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"d251ee68ad769b1cfe71ee7ebf8a7f45ed8b8260","unresolved":false,"context_lines":[{"line_number":214,"context_line":"    \"\"\"Maps keys for attachment summary view.\"\"\""},{"line_number":215,"context_line":"    d \u003d {}"},{"line_number":216,"context_line":""},{"line_number":217,"context_line":"    # NOTE(justinsb): We use the volume id as the id of the attachment object"},{"line_number":218,"context_line":"    d[\u0027id\u0027] \u003d volume_id"},{"line_number":219,"context_line":""},{"line_number":220,"context_line":"    d[\u0027volumeId\u0027] \u003d volume_id"}],"source_content_type":"text/x-python","patch_set":9,"id":"dada55a8_cd585dfd","side":"PARENT","line":217,"range":{"start_line":217,"start_character":46,"end_line":217,"end_character":47},"updated":"2016-07-25 14:14:54.000000000","message":"new typo added?","commit_id":"654ec151db7cbc7fd8a5b3912b6bd20144b4dfff"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"2ee6af125096763a92c406c6921d923769523714","unresolved":false,"context_lines":[{"line_number":214,"context_line":"    \"\"\"Maps keys for attachment summary view.\"\"\""},{"line_number":215,"context_line":"    d \u003d {}"},{"line_number":216,"context_line":""},{"line_number":217,"context_line":"    # NOTE(justinsb): We use the volume id as the id of the attachment object"},{"line_number":218,"context_line":"    d[\u0027id\u0027] \u003d volume_id"},{"line_number":219,"context_line":""},{"line_number":220,"context_line":"    d[\u0027volumeId\u0027] \u003d volume_id"}],"source_content_type":"text/x-python","patch_set":11,"id":"dada55a8_10ba7823","side":"PARENT","line":217,"range":{"start_line":217,"start_character":46,"end_line":217,"end_character":49},"updated":"2016-07-25 21:23:08.000000000","message":"nit: why did you add a typo here?","commit_id":"c7efa66ce9a2e84a0c0c1be75a4a9204a47c0dca"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"ad0731ac6ad26b44cf7880d83db4087d4a330d27","unresolved":false,"context_lines":[{"line_number":214,"context_line":"    \"\"\"Maps keys for attachment summary view.\"\"\""},{"line_number":215,"context_line":"    d \u003d {}"},{"line_number":216,"context_line":""},{"line_number":217,"context_line":"    # NOTE(justinsb): We use the volume id as the id of the attachment object"},{"line_number":218,"context_line":"    d[\u0027id\u0027] \u003d volume_id"},{"line_number":219,"context_line":""},{"line_number":220,"context_line":"    d[\u0027volumeId\u0027] \u003d volume_id"}],"source_content_type":"text/x-python","patch_set":11,"id":"dada55a8_58f6d2f9","side":"PARENT","line":217,"range":{"start_line":217,"start_character":46,"end_line":217,"end_character":49},"in_reply_to":"dada55a8_104f18be","updated":"2016-07-25 23:34:43.000000000","message":"I\u0027m pretty sure it was a rebase miss with one of the spell checker changes.","commit_id":"c7efa66ce9a2e84a0c0c1be75a4a9204a47c0dca"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"3f06c37165ce1765e98a7dfcc17b8e7cbba46a03","unresolved":false,"context_lines":[{"line_number":214,"context_line":"    \"\"\"Maps keys for attachment summary view.\"\"\""},{"line_number":215,"context_line":"    d \u003d {}"},{"line_number":216,"context_line":""},{"line_number":217,"context_line":"    # NOTE(justinsb): We use the volume id as the id of the attachment object"},{"line_number":218,"context_line":"    d[\u0027id\u0027] \u003d volume_id"},{"line_number":219,"context_line":""},{"line_number":220,"context_line":"    d[\u0027volumeId\u0027] \u003d volume_id"}],"source_content_type":"text/x-python","patch_set":11,"id":"dada55a8_104f18be","side":"PARENT","line":217,"range":{"start_line":217,"start_character":46,"end_line":217,"end_character":49},"in_reply_to":"dada55a8_10ba7823","updated":"2016-07-25 21:36:28.000000000","message":"yeah we should fix that probably","commit_id":"c7efa66ce9a2e84a0c0c1be75a4a9204a47c0dca"}],"nova/tests/unit/api/openstack/compute/test_snapshots.py":[{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"2ee6af125096763a92c406c6921d923769523714","unresolved":false,"context_lines":[{"line_number":73,"context_line":""},{"line_number":74,"context_line":"    def test_snapshot_delete(self):"},{"line_number":75,"context_line":"        snapshot_id \u003d \u0027123\u0027"},{"line_number":76,"context_line":"        delete \u003d self.controller.delete"},{"line_number":77,"context_line":"        result \u003d delete(self.req, snapshot_id)"},{"line_number":78,"context_line":""},{"line_number":79,"context_line":"        # NOTE: on v2.1, http status code is set as wsgi_code of API"},{"line_number":80,"context_line":"        # method instead of status_int in a response object."}],"source_content_type":"text/x-python","patch_set":11,"id":"dada55a8_10e8b80f","line":77,"range":{"start_line":76,"start_character":8,"end_line":77,"end_character":46},"updated":"2016-07-25 21:23:08.000000000","message":"I am not sure why this change is necessary for this patch purpose.","commit_id":"3563a48847221da4d5131e1612eca1efab01f6b0"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"ad0731ac6ad26b44cf7880d83db4087d4a330d27","unresolved":false,"context_lines":[{"line_number":73,"context_line":""},{"line_number":74,"context_line":"    def test_snapshot_delete(self):"},{"line_number":75,"context_line":"        snapshot_id \u003d \u0027123\u0027"},{"line_number":76,"context_line":"        delete \u003d self.controller.delete"},{"line_number":77,"context_line":"        result \u003d delete(self.req, snapshot_id)"},{"line_number":78,"context_line":""},{"line_number":79,"context_line":"        # NOTE: on v2.1, http status code is set as wsgi_code of API"},{"line_number":80,"context_line":"        # method instead of status_int in a response object."}],"source_content_type":"text/x-python","patch_set":11,"id":"dada55a8_38d03e53","line":77,"range":{"start_line":76,"start_character":8,"end_line":77,"end_character":46},"in_reply_to":"dada55a8_10e8b80f","updated":"2016-07-25 23:34:43.000000000","message":"it probably isn\u0027t, but it\u0027s also not a big deal. Seems pretty nit picky on this.","commit_id":"3563a48847221da4d5131e1612eca1efab01f6b0"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"4eed4941f4919a6b9ed744e472bd01b9673161e3","unresolved":false,"context_lines":[{"line_number":79,"context_line":"        # NOTE: on v2.1, http status code is set as wsgi_code of API"},{"line_number":80,"context_line":"        # method instead of status_int in a response object."},{"line_number":81,"context_line":"        if isinstance(self.controller, volumes_v21.SnapshotController):"},{"line_number":82,"context_line":"            status_int \u003d delete.wsgi_code"},{"line_number":83,"context_line":"        else:"},{"line_number":84,"context_line":"            status_int \u003d result.status_int"},{"line_number":85,"context_line":"        self.assertEqual(202, status_int)"}],"source_content_type":"text/x-python","patch_set":14,"id":"dada55a8_8dbbd3e4","line":82,"updated":"2016-07-26 10:20:27.000000000","message":"Nit: I mean, yeah, this shouldn\u0027t be here, but the rest is so much more important.","commit_id":"83bd45bbcf766be4aba7ad5dfaa5f3e66ac547e8"}],"nova/tests/unit/api/openstack/compute/test_volumes.py":[{"author":{"_account_id":6062,"name":"jichenjc","email":"jichenjc@cn.ibm.com","username":"jichenjc"},"change_message_id":"9fe3047499868bf91be460e668466d75479b5b9b","unresolved":false,"context_lines":[{"line_number":1001,"context_line":"            self.controller.detail, self.req)"},{"line_number":1002,"context_line":""},{"line_number":1003,"context_line":""},{"line_number":1004,"context_line":"class TestSnapshotAPIDeprecation(test.NoDBTestCase):"},{"line_number":1005,"context_line":""},{"line_number":1006,"context_line":"    def setUp(self):"},{"line_number":1007,"context_line":"        super(TestSnapshotAPIDeprecation, self).setUp()"}],"source_content_type":"text/x-python","patch_set":8,"id":"dada55a8_f001df3a","line":1004,"range":{"start_line":1004,"start_character":6,"end_line":1004,"end_character":32},"updated":"2016-07-23 16:50:56.000000000","message":"better to move to test_snapshots.py","commit_id":"7b7270fa2aab9ffc90fee1655c68c2644d21206f"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"8b2743150deb382b13fc3831c000ed6e8c36da96","unresolved":false,"context_lines":[{"line_number":1001,"context_line":"            self.controller.detail, self.req)"},{"line_number":1002,"context_line":""},{"line_number":1003,"context_line":""},{"line_number":1004,"context_line":"class TestSnapshotAPIDeprecation(test.NoDBTestCase):"},{"line_number":1005,"context_line":""},{"line_number":1006,"context_line":"    def setUp(self):"},{"line_number":1007,"context_line":"        super(TestSnapshotAPIDeprecation, self).setUp()"}],"source_content_type":"text/x-python","patch_set":8,"id":"dada55a8_cee22a2f","line":1004,"range":{"start_line":1004,"start_character":6,"end_line":1004,"end_character":32},"in_reply_to":"dada55a8_f001df3a","updated":"2016-07-25 06:10:26.000000000","message":"Done","commit_id":"7b7270fa2aab9ffc90fee1655c68c2644d21206f"}]}