)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"8e58eb9d1e139894c735c605b42e6f94d2e62828","unresolved":false,"context_lines":[{"line_number":16,"context_line":"        cls.clear_snapshots()"},{"line_number":17,"context_line":"        cls.clear_volumes()"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"So it will delete snapshots first and then delete volumes,"},{"line_number":20,"context_line":"but dst_vol is created based on snapshot, so deleting"},{"line_number":21,"context_line":"snapshot will end with status \u0027error-deleting\u0027 because"},{"line_number":22,"context_line":"the volume created based on it(dst_vol) is not deleted yet."},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"So we should add immediate cleanup of volume dst_vol to"},{"line_number":25,"context_line":"avoid deleting snapshot failure."},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"Change-Id: I0d9ad9556748dafddefe3680417290f9b798488d"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5a74a57a_5e6484e4","line":25,"range":{"start_line":19,"start_character":0,"end_line":25,"end_character":32},"updated":"2016-11-29 18:04:09.000000000","message":"If this failure scenario is common, I guess we should face this problem on the gate testing. Are there any bug report about this?\nOr is this scenario some vender specific?","commit_id":"5ab6a62c43a4cc0b8e33b865a3a0f9128cccfd4b"}],"tempest/api/volume/test_volumes_snapshots.py":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"b9e8fdbc24019e59827f3b97f905ed50b1e76366","unresolved":false,"context_lines":[{"line_number":140,"context_line":"        # Destination volume bigger than source snapshot"},{"line_number":141,"context_line":"        dst_vol \u003d self.create_volume(snapshot_id\u003dsrc_snap[\u0027id\u0027],"},{"line_number":142,"context_line":"                                     size\u003dsrc_size + 1)"},{"line_number":143,"context_line":"        self.addCleanup(self.delete_volume, self.volumes_client, dst_vol[\u0027id\u0027])"},{"line_number":144,"context_line":""},{"line_number":145,"context_line":"        volume \u003d self.volumes_client.show_volume(dst_vol[\u0027id\u0027])[\u0027volume\u0027]"},{"line_number":146,"context_line":"        # Should allow"}],"source_content_type":"text/x-python","patch_set":2,"id":"3a71b18c_ab2f6486","line":143,"updated":"2016-12-06 07:04:52.000000000","message":"I think it\u0027s good to add a NOTE here like your commit message. Because at L.138, you don\u0027t add addCleanup() there, so I think people may be weird that why addCleanup() is only here.","commit_id":"252b3b4de1ae2e881ef1bca0f2e9f3b09bdc6de4"}]}