)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"7031a39b8ce003f527ddecc8e3c45dbc50274c3a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"1a6eadb0_cfe7d7aa","updated":"2016-12-19 17:29:50.000000000","message":"You\u0027re also adding a brand new scenario, which you don\u0027t mention in the commit message.","commit_id":"f67c02c5ca9d0a11509e3543e32f80b2703d57ba"},{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"7031a39b8ce003f527ddecc8e3c45dbc50274c3a","unresolved":false,"context_lines":[{"line_number":15,"context_line":"* Detach the volume from instance"},{"line_number":16,"context_line":"* Re-attach the volume to instance and get timestamp."},{"line_number":17,"context_line":"* Detach the volume."},{"line_number":18,"context_line":"*"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Change-Id: Id7ec07110b6246a8f2cd8ad6c98e9d7b8b431eff"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"1a6eadb0_6fe243a1","line":18,"updated":"2016-12-19 17:29:50.000000000","message":"Empty line","commit_id":"f67c02c5ca9d0a11509e3543e32f80b2703d57ba"}],"tempest/scenario/manager.py":[{"author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"change_message_id":"52f54f95334c78a5a3cd6e1a54e95affaadab57e","unresolved":false,"context_lines":[{"line_number":496,"context_line":"        return snapshot_image"},{"line_number":497,"context_line":""},{"line_number":498,"context_line":"    def nova_volume_attach(self, server, volume_to_attach,"},{"line_number":499,"context_line":"                           device\u003dCONF.compute.volume_device_name):"},{"line_number":500,"context_line":"        volume \u003d self.servers_client.attach_volume("},{"line_number":501,"context_line":"            server[\u0027id\u0027], volumeId\u003dvolume_to_attach[\u0027id\u0027], device\u003d\u0027/dev/%s\u0027"},{"line_number":502,"context_line":"            % device)[\u0027volumeAttachment\u0027]"}],"source_content_type":"text/x-python","patch_set":5,"id":"ba5201f7_9c91184a","line":499,"range":{"start_line":499,"start_character":27,"end_line":499,"end_character":65},"updated":"2017-01-05 09:01:03.000000000","message":"Why add this in this patch? It seems in this patch device parameter is not used. Maybe you can add it when it is really needed.","commit_id":"178a54790462d0d216e961a8233ca91b5788f4c1"},{"author":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"change_message_id":"225d2fcfa1e58a72d77b0441efc9baa77fd5c85e","unresolved":false,"context_lines":[{"line_number":496,"context_line":"        return snapshot_image"},{"line_number":497,"context_line":""},{"line_number":498,"context_line":"    def nova_volume_attach(self, server, volume_to_attach,"},{"line_number":499,"context_line":"                           device\u003dCONF.compute.volume_device_name):"},{"line_number":500,"context_line":"        volume \u003d self.servers_client.attach_volume("},{"line_number":501,"context_line":"            server[\u0027id\u0027], volumeId\u003dvolume_to_attach[\u0027id\u0027], device\u003d\u0027/dev/%s\u0027"},{"line_number":502,"context_line":"            % device)[\u0027volumeAttachment\u0027]"}],"source_content_type":"text/x-python","patch_set":5,"id":"7a3c09a3_c6479f49","line":499,"range":{"start_line":499,"start_character":27,"end_line":499,"end_character":65},"in_reply_to":"ba5201f7_9c91184a","updated":"2017-01-15 07:03:14.000000000","message":"IMO the device should be as param here ,  will allow us more flexability\nBenny","commit_id":"178a54790462d0d216e961a8233ca91b5788f4c1"}],"tempest/scenario/test_encrypted_cinder_volumes.py":[{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"7031a39b8ce003f527ddecc8e3c45dbc50274c3a","unresolved":false,"context_lines":[{"line_number":46,"context_line":"        security_group \u003d self._create_security_group()"},{"line_number":47,"context_line":"        security_groups \u003d [{\u0027name\u0027: security_group[\u0027name\u0027]}]"},{"line_number":48,"context_line":"        return self.create_server("},{"line_number":49,"context_line":"            image_id\u003dCONF.compute.image_ref, key_name\u003dkey_name,"},{"line_number":50,"context_line":"            security_groups\u003dsecurity_groups,"},{"line_number":51,"context_line":"            wait_until\u003d\u0027ACTIVE\u0027)"},{"line_number":52,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"1a6eadb0_2ff5ab82","line":49,"range":{"start_line":49,"start_character":12,"end_line":49,"end_character":43},"updated":"2016-12-19 17:29:50.000000000","message":"NIT: this is what you\u0027ll get if you don\u0027t specify anything, you should use image_id only in case you need to override the default.","commit_id":"f67c02c5ca9d0a11509e3543e32f80b2703d57ba"},{"author":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"change_message_id":"3ae25fbd8a135139e9e9c410fa164e2944132d7d","unresolved":false,"context_lines":[{"line_number":46,"context_line":"        security_group \u003d self._create_security_group()"},{"line_number":47,"context_line":"        security_groups \u003d [{\u0027name\u0027: security_group[\u0027name\u0027]}]"},{"line_number":48,"context_line":"        return self.create_server("},{"line_number":49,"context_line":"            image_id\u003dCONF.compute.image_ref, key_name\u003dkey_name,"},{"line_number":50,"context_line":"            security_groups\u003dsecurity_groups,"},{"line_number":51,"context_line":"            wait_until\u003d\u0027ACTIVE\u0027)"},{"line_number":52,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"1a6eadb0_a5a53348","line":49,"range":{"start_line":49,"start_character":12,"end_line":49,"end_character":43},"in_reply_to":"1a6eadb0_2ff5ab82","updated":"2016-12-20 12:31:35.000000000","message":"done , \nI will remove image_id but keeep keypair as was before .\nMayve we can have another patch for fine tunning these issues\nBenny","commit_id":"f67c02c5ca9d0a11509e3543e32f80b2703d57ba"},{"author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"change_message_id":"52f54f95334c78a5a3cd6e1a54e95affaadab57e","unresolved":false,"context_lines":[{"line_number":73,"context_line":"            private_key\u003dkeypair[\u0027private_key\u0027])"},{"line_number":74,"context_line":"        # verify timestamp"},{"line_number":75,"context_line":"        self.assertEqual(timestamp_set, timestamp_get)"},{"line_number":76,"context_line":"        self.nova_volume_detach(server, attached_volume)"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"    @test.idempotent_id(\u002779165fb4-5534-4b9d-8429-97ccffb8f86e\u0027)"},{"line_number":79,"context_line":"    @test.services(\u0027compute\u0027, \u0027volume\u0027, \u0027image\u0027)"}],"source_content_type":"text/x-python","patch_set":5,"id":"ba5201f7_9c7bd8f4","line":76,"range":{"start_line":76,"start_character":8,"end_line":76,"end_character":56},"updated":"2017-01-05 09:01:03.000000000","message":"This should be used as addCleanup immediately after self.nova_volume_attach, othersize if the testcase failes, this step may not be executed.","commit_id":"178a54790462d0d216e961a8233ca91b5788f4c1"},{"author":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"change_message_id":"225d2fcfa1e58a72d77b0441efc9baa77fd5c85e","unresolved":false,"context_lines":[{"line_number":73,"context_line":"            private_key\u003dkeypair[\u0027private_key\u0027])"},{"line_number":74,"context_line":"        # verify timestamp"},{"line_number":75,"context_line":"        self.assertEqual(timestamp_set, timestamp_get)"},{"line_number":76,"context_line":"        self.nova_volume_detach(server, attached_volume)"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"    @test.idempotent_id(\u002779165fb4-5534-4b9d-8429-97ccffb8f86e\u0027)"},{"line_number":79,"context_line":"    @test.services(\u0027compute\u0027, \u0027volume\u0027, \u0027image\u0027)"}],"source_content_type":"text/x-python","patch_set":5,"id":"7a3c09a3_266f83c6","line":76,"range":{"start_line":76,"start_character":8,"end_line":76,"end_character":56},"in_reply_to":"ba5201f7_9c7bd8f4","updated":"2017-01-15 07:03:14.000000000","message":"Hi , Thanks , fixing","commit_id":"178a54790462d0d216e961a8233ca91b5788f4c1"}]}