)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"03e550ec6e3166e0e8e161b25120c7ecf33aa8a4","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2015-02-17 02:32:09 -0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Added missing test case for nova volume-update"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Closes-Bug: #1422575"},{"line_number":10,"context_line":"Change-Id: I8f052aed3b803ddca1a29263bb923bcc953cebc5"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"ba7be1f8_2bca7a6e","line":8,"updated":"2015-02-23 09:23:23.000000000","message":"The message body should contain some text for human reading, even if it just differs by 1 character (\".\") from the subject. Usually not difficult to write 2 sentence story.","commit_id":"35fb8329b66a7ed69574684566b8548373df0a25"}],"tempest/api/compute/volumes/test_attach_volume.py":[{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"2f0d359808077e64de9a2cc68ef5df673ae2129b","unresolved":false,"context_lines":[{"line_number":129,"context_line":"        self.assertEqual(self.volume[\u0027id\u0027], body[\u0027volumeId\u0027])"},{"line_number":130,"context_line":"        self.assertEqual(self.attachment[\u0027id\u0027], body[\u0027id\u0027])"},{"line_number":131,"context_line":""},{"line_number":132,"context_line":"    @decorators.skip_because(bug\u003d\u00271413610\u0027)"},{"line_number":133,"context_line":"    @test.attr(type\u003d\u0027gate\u0027)"},{"line_number":134,"context_line":"    def test_volume_update(self):"},{"line_number":135,"context_line":"        # Create Server, Volume and attach that Volume to Server"}],"source_content_type":"text/x-python","patch_set":2,"id":"da86d52c_eaa4b109","line":132,"updated":"2015-02-17 14:56:08.000000000","message":"I think this test is valuable but I read somewhere we shouldn\u0027t merge tests that are not run in the Gate... And this new test is skipped","commit_id":"35fb8329b66a7ed69574684566b8548373df0a25"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"2f0d359808077e64de9a2cc68ef5df673ae2129b","unresolved":false,"context_lines":[{"line_number":144,"context_line":""},{"line_number":145,"context_line":"        # Create a volume and wait for it to become ready"},{"line_number":146,"context_line":"        volume \u003d self.volumes_client.create_volume("},{"line_number":147,"context_line":"            1, display_name\u003d\u0027test\u0027)"},{"line_number":148,"context_line":"        self.addCleanup(self._delete_volume, volume[\u0027id\u0027])"},{"line_number":149,"context_line":"        self.volumes_client.wait_for_volume_status(volume[\u0027id\u0027],"},{"line_number":150,"context_line":"                                                   \u0027available\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"da86d52c_2af7a98d","line":147,"updated":"2015-02-17 14:56:08.000000000","message":"Shouldn\u0027t you random the display name ? I didn\u0027t check whether the Nova volume API accepts duplicate volume name. If not, you should random the display name here.","commit_id":"35fb8329b66a7ed69574684566b8548373df0a25"},{"author":{"_account_id":10644,"name":"David Paterson","email":"davpat2112@yahoo.com","username":"dpaterson"},"change_message_id":"f770b3c2359f888fea69544448abea27725ebd10","unresolved":false,"context_lines":[{"line_number":144,"context_line":""},{"line_number":145,"context_line":"        # Create a volume and wait for it to become ready"},{"line_number":146,"context_line":"        volume \u003d self.volumes_client.create_volume("},{"line_number":147,"context_line":"            1, display_name\u003d\u0027test\u0027)"},{"line_number":148,"context_line":"        self.addCleanup(self._delete_volume, volume[\u0027id\u0027])"},{"line_number":149,"context_line":"        self.volumes_client.wait_for_volume_status(volume[\u0027id\u0027],"},{"line_number":150,"context_line":"                                                   \u0027available\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"da86d52c_ca652d1e","line":147,"in_reply_to":"da86d52c_2af7a98d","updated":"2015-02-17 15:05:33.000000000","message":"I believe display_name can be duplicated but name cannot?","commit_id":"35fb8329b66a7ed69574684566b8548373df0a25"},{"author":{"_account_id":1773,"name":"oliver-leahy-l","email":"oliver.leahy@hpe.com","username":"oliver-leahy-l"},"change_message_id":"aab1f9b4c69ac4563e0a77ee458f25561e94a741","unresolved":false,"context_lines":[{"line_number":144,"context_line":""},{"line_number":145,"context_line":"        # Create a volume and wait for it to become ready"},{"line_number":146,"context_line":"        volume \u003d self.volumes_client.create_volume("},{"line_number":147,"context_line":"            1, display_name\u003d\u0027test\u0027)"},{"line_number":148,"context_line":"        self.addCleanup(self._delete_volume, volume[\u0027id\u0027])"},{"line_number":149,"context_line":"        self.volumes_client.wait_for_volume_status(volume[\u0027id\u0027],"},{"line_number":150,"context_line":"                                                   \u0027available\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"ba7be1f8_cea2059a","line":147,"in_reply_to":"da86d52c_ca652d1e","updated":"2015-02-20 11:35:35.000000000","message":"Duplicate names are fine here, the api uses the volume uuid","commit_id":"35fb8329b66a7ed69574684566b8548373df0a25"}]}