)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"08977d254b5c7c10e31909398094fe450bec61d7","unresolved":false,"context_lines":[{"line_number":11,"context_line":"another test just to update the metadata is not worth it."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I2f1093616dc7c4268a0bda5a4854da23423f4cab"},{"line_number":14,"context_line":"Related-To: https://bugs.launchpad.net/nova/+bug/1717547"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ff570b3c_1a655d46","line":14,"range":{"start_line":14,"start_character":0,"end_line":14,"end_character":56},"updated":"2020-05-22 15:34:48.000000000","message":"Related-Bug: #1717547","commit_id":"95c5bd7937c5e3383b1fa86ff285af49ee22cd0f"},{"author":{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},"change_message_id":"33191ce736dceac49d9bb9c9833d1a6acbafe7d6","unresolved":false,"context_lines":[{"line_number":11,"context_line":"another test just to update the metadata is not worth it."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I2f1093616dc7c4268a0bda5a4854da23423f4cab"},{"line_number":14,"context_line":"Related-To: https://bugs.launchpad.net/nova/+bug/1717547"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ff570b3c_c6ac9f37","line":14,"range":{"start_line":14,"start_character":0,"end_line":14,"end_character":56},"in_reply_to":"ff570b3c_1a655d46","updated":"2020-05-22 15:40:51.000000000","message":"Oh right, will fix this","commit_id":"95c5bd7937c5e3383b1fa86ff285af49ee22cd0f"},{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"76dcc680a3bef94f7f4ff2978d90ac24d7437fd5","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Add image \"version\" metadata in test_snapshot_pattern"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Add image \"version\" metadata to the snapshot image in"},{"line_number":10,"context_line":"test_snapshot_pattern as shown in the related bug. Ccreating"},{"line_number":11,"context_line":"another test just to update the metadata is not worth it."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I2f1093616dc7c4268a0bda5a4854da23423f4cab"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"ff570b3c_ab0cd33b","line":10,"range":{"start_line":10,"start_character":51,"end_line":10,"end_character":60},"updated":"2020-05-22 16:13:12.000000000","message":"Creating","commit_id":"b0f60ba24a98e39bb7069fe778421ed792915303"},{"author":{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},"change_message_id":"405426361b029a07c55fa880daeb6dd6d5745a1d","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Add image \"version\" metadata in test_snapshot_pattern"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Add image \"version\" metadata to the snapshot image in"},{"line_number":10,"context_line":"test_snapshot_pattern as shown in the related bug. Ccreating"},{"line_number":11,"context_line":"another test just to update the metadata is not worth it."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I2f1093616dc7c4268a0bda5a4854da23423f4cab"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"ff570b3c_ae2501ae","line":10,"range":{"start_line":10,"start_character":51,"end_line":10,"end_character":60},"in_reply_to":"ff570b3c_ab0cd33b","updated":"2020-05-22 16:55:54.000000000","message":"Done","commit_id":"b0f60ba24a98e39bb7069fe778421ed792915303"}],"tempest/scenario/test_snapshot_pattern.py":[{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"08977d254b5c7c10e31909398094fe450bec61d7","unresolved":false,"context_lines":[{"line_number":46,"context_line":"    @testtools.skipUnless(CONF.network.public_network_id,"},{"line_number":47,"context_line":"                          \u0027The public_network_id option must be specified.\u0027)"},{"line_number":48,"context_line":"    @utils.services(\u0027compute\u0027, \u0027network\u0027, \u0027image\u0027)"},{"line_number":49,"context_line":"    def test_snapshot_pattern(self, image_version\u003d\u00278.0\u0027):"},{"line_number":50,"context_line":"        # create image and update version metadata"},{"line_number":51,"context_line":"        image \u003d self.glance_image_create()"},{"line_number":52,"context_line":"        if image_version:"}],"source_content_type":"text/x-python","patch_set":1,"id":"ff570b3c_66db1321","line":49,"range":{"start_line":49,"start_character":36,"end_line":49,"end_character":55},"updated":"2020-05-22 15:34:48.000000000","message":"This isn\u0027t required, no one calls this directly.","commit_id":"95c5bd7937c5e3383b1fa86ff285af49ee22cd0f"},{"author":{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},"change_message_id":"c20e6ea7dcfd67c17820584cc3a2b3ad9e2ac072","unresolved":false,"context_lines":[{"line_number":46,"context_line":"    @testtools.skipUnless(CONF.network.public_network_id,"},{"line_number":47,"context_line":"                          \u0027The public_network_id option must be specified.\u0027)"},{"line_number":48,"context_line":"    @utils.services(\u0027compute\u0027, \u0027network\u0027, \u0027image\u0027)"},{"line_number":49,"context_line":"    def test_snapshot_pattern(self, image_version\u003d\u00278.0\u0027):"},{"line_number":50,"context_line":"        # create image and update version metadata"},{"line_number":51,"context_line":"        image \u003d self.glance_image_create()"},{"line_number":52,"context_line":"        if image_version:"}],"source_content_type":"text/x-python","patch_set":1,"id":"ff570b3c_e66a8319","line":49,"range":{"start_line":49,"start_character":36,"end_line":49,"end_character":55},"in_reply_to":"ff570b3c_66db1321","updated":"2020-05-22 15:37:23.000000000","message":"yes that\u0027s what i need input for, so i guess this 8.0 value is good to be set for this test","commit_id":"95c5bd7937c5e3383b1fa86ff285af49ee22cd0f"},{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"08977d254b5c7c10e31909398094fe450bec61d7","unresolved":false,"context_lines":[{"line_number":49,"context_line":"    def test_snapshot_pattern(self, image_version\u003d\u00278.0\u0027):"},{"line_number":50,"context_line":"        # create image and update version metadata"},{"line_number":51,"context_line":"        image \u003d self.glance_image_create()"},{"line_number":52,"context_line":"        if image_version:"},{"line_number":53,"context_line":"            self.image_client.update_image("},{"line_number":54,"context_line":"                image, [dict(add\u003d\u0027/version\u0027,"},{"line_number":55,"context_line":"                                   value\u003dimage_version)])"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"        # prepare for booting an instance"},{"line_number":58,"context_line":"        keypair \u003d self.create_keypair()"}],"source_content_type":"text/x-python","patch_set":1,"id":"ff570b3c_86eaa7a9","line":55,"range":{"start_line":52,"start_character":8,"end_line":55,"end_character":57},"updated":"2020-05-22 15:34:48.000000000","message":"Drop the if and just set a version property everytime.","commit_id":"95c5bd7937c5e3383b1fa86ff285af49ee22cd0f"},{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"08977d254b5c7c10e31909398094fe450bec61d7","unresolved":false,"context_lines":[{"line_number":52,"context_line":"        if image_version:"},{"line_number":53,"context_line":"            self.image_client.update_image("},{"line_number":54,"context_line":"                image, [dict(add\u003d\u0027/version\u0027,"},{"line_number":55,"context_line":"                                   value\u003dimage_version)])"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"        # prepare for booting an instance"},{"line_number":58,"context_line":"        keypair \u003d self.create_keypair()"}],"source_content_type":"text/x-python","patch_set":1,"id":"ff570b3c_c6781f2d","line":55,"range":{"start_line":55,"start_character":29,"end_line":55,"end_character":35},"updated":"2020-05-22 15:34:48.000000000","message":"This needs to be in-line with the add\u003d above.","commit_id":"95c5bd7937c5e3383b1fa86ff285af49ee22cd0f"},{"author":{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},"change_message_id":"c04034645b7cece54f1b98d3e5c745172d3c309f","unresolved":false,"context_lines":[{"line_number":46,"context_line":"    @testtools.skipUnless(CONF.network.public_network_id,"},{"line_number":47,"context_line":"                          \u0027The public_network_id option must be specified.\u0027)"},{"line_number":48,"context_line":"    @utils.services(\u0027compute\u0027, \u0027network\u0027, \u0027image\u0027)"},{"line_number":49,"context_line":"    def test_snapshot_pattern(self, image_version\u003d\u00278.0\u0027):"},{"line_number":50,"context_line":"        # create image and update version metadata"},{"line_number":51,"context_line":"        image \u003d self.glance_image_create()"},{"line_number":52,"context_line":"        if image_version:"},{"line_number":53,"context_line":"            self.image_client.update_image("},{"line_number":54,"context_line":"                image, [dict(add\u003d\u0027/version\u0027,"},{"line_number":55,"context_line":"                                   value\u003dimage_version)])"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"        # prepare for booting an instance"},{"line_number":58,"context_line":"        keypair \u003d self.create_keypair()"}],"source_content_type":"text/x-python","patch_set":1,"id":"ff570b3c_dc976844","line":55,"range":{"start_line":49,"start_character":36,"end_line":55,"end_character":57},"updated":"2020-05-21 16:38:43.000000000","message":"need input on how to put this: should this be a CONF option, or just any fake version number would do?","commit_id":"95c5bd7937c5e3383b1fa86ff285af49ee22cd0f"},{"author":{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},"change_message_id":"03f4151209c6cdc1d79f3b20dc68b22cced5556f","unresolved":false,"context_lines":[{"line_number":46,"context_line":"    @testtools.skipUnless(CONF.network.public_network_id,"},{"line_number":47,"context_line":"                          \u0027The public_network_id option must be specified.\u0027)"},{"line_number":48,"context_line":"    @utils.services(\u0027compute\u0027, \u0027network\u0027, \u0027image\u0027)"},{"line_number":49,"context_line":"    def test_snapshot_pattern(self, image_version\u003d\u00278.0\u0027):"},{"line_number":50,"context_line":"        # create image and update version metadata"},{"line_number":51,"context_line":"        image \u003d self.glance_image_create()"},{"line_number":52,"context_line":"        if image_version:"},{"line_number":53,"context_line":"            self.image_client.update_image("},{"line_number":54,"context_line":"                image, [dict(add\u003d\u0027/version\u0027,"},{"line_number":55,"context_line":"                                   value\u003dimage_version)])"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"        # prepare for booting an instance"},{"line_number":58,"context_line":"        keypair \u003d self.create_keypair()"}],"source_content_type":"text/x-python","patch_set":1,"id":"ff570b3c_97f6698e","line":55,"range":{"start_line":49,"start_character":36,"end_line":55,"end_character":57},"in_reply_to":"ff570b3c_570a111f","updated":"2020-05-21 17:15:40.000000000","message":"\u003e I would think adding a CONF.option with a generic default would be\n \u003e satisfactory for something like this.\n\nyes but if any version works then adding a CONF option would be unnecessary work IMO, but ill let others vote \n \u003e \n \u003e Also nit, just for consistency you can change \u0027dict(add\u003d\u0027/version\u0027,...\u0027\n \u003e to match the approach of passing dictionaries throughout the\n \u003e method, i.e. change to [{\u0027add\u0027: \u0027/version\u0027, \u0027value\u0027:\n \u003e image_version}].\n\nSo throughout tempest, we use dict() for such methods AFAIK. Here are some examples:\n\n1. https://github.com/openstack/tempest/blob/7ad6abd520cb8d3c2919409f28ab6f60611be29d/tempest/api/image/v2/admin/test_images.py#L48-L49\n2. https://github.com/openstack/tempest/blob/0fbce172204a0b4edd5de74934f64d374238c3c4/tempest/api/compute/servers/test_server_rescue.py#L134-L141","commit_id":"95c5bd7937c5e3383b1fa86ff285af49ee22cd0f"},{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"08977d254b5c7c10e31909398094fe450bec61d7","unresolved":false,"context_lines":[{"line_number":46,"context_line":"    @testtools.skipUnless(CONF.network.public_network_id,"},{"line_number":47,"context_line":"                          \u0027The public_network_id option must be specified.\u0027)"},{"line_number":48,"context_line":"    @utils.services(\u0027compute\u0027, \u0027network\u0027, \u0027image\u0027)"},{"line_number":49,"context_line":"    def test_snapshot_pattern(self, image_version\u003d\u00278.0\u0027):"},{"line_number":50,"context_line":"        # create image and update version metadata"},{"line_number":51,"context_line":"        image \u003d self.glance_image_create()"},{"line_number":52,"context_line":"        if image_version:"},{"line_number":53,"context_line":"            self.image_client.update_image("},{"line_number":54,"context_line":"                image, [dict(add\u003d\u0027/version\u0027,"},{"line_number":55,"context_line":"                                   value\u003dimage_version)])"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"        # prepare for booting an instance"},{"line_number":58,"context_line":"        keypair \u003d self.create_keypair()"}],"source_content_type":"text/x-python","patch_set":1,"id":"ff570b3c_468d2f30","line":55,"range":{"start_line":49,"start_character":36,"end_line":55,"end_character":57},"in_reply_to":"ff570b3c_97f6698e","updated":"2020-05-22 15:34:48.000000000","message":"Yeah it needs to be dict().","commit_id":"95c5bd7937c5e3383b1fa86ff285af49ee22cd0f"},{"author":{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},"change_message_id":"c20e6ea7dcfd67c17820584cc3a2b3ad9e2ac072","unresolved":false,"context_lines":[{"line_number":52,"context_line":"        if image_version:"},{"line_number":53,"context_line":"            self.image_client.update_image("},{"line_number":54,"context_line":"                image, [dict(add\u003d\u0027/version\u0027,"},{"line_number":55,"context_line":"                                   value\u003dimage_version)])"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"        # prepare for booting an instance"},{"line_number":58,"context_line":"        keypair \u003d self.create_keypair()"}],"source_content_type":"text/x-python","patch_set":1,"id":"ff570b3c_861107b0","line":55,"range":{"start_line":55,"start_character":29,"end_line":55,"end_character":35},"in_reply_to":"ff570b3c_c6781f2d","updated":"2020-05-22 15:37:23.000000000","message":"Yeah idk how it got extra indented, wasn\u0027t the same locally.","commit_id":"95c5bd7937c5e3383b1fa86ff285af49ee22cd0f"},{"author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"change_message_id":"5ae596cf672fbb891645895a3c699bd030034b9f","unresolved":false,"context_lines":[{"line_number":46,"context_line":"    @testtools.skipUnless(CONF.network.public_network_id,"},{"line_number":47,"context_line":"                          \u0027The public_network_id option must be specified.\u0027)"},{"line_number":48,"context_line":"    @utils.services(\u0027compute\u0027, \u0027network\u0027, \u0027image\u0027)"},{"line_number":49,"context_line":"    def test_snapshot_pattern(self, image_version\u003d\u00278.0\u0027):"},{"line_number":50,"context_line":"        # create image and update version metadata"},{"line_number":51,"context_line":"        image \u003d self.glance_image_create()"},{"line_number":52,"context_line":"        if image_version:"},{"line_number":53,"context_line":"            self.image_client.update_image("},{"line_number":54,"context_line":"                image, [dict(add\u003d\u0027/version\u0027,"},{"line_number":55,"context_line":"                                   value\u003dimage_version)])"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"        # prepare for booting an instance"},{"line_number":58,"context_line":"        keypair \u003d self.create_keypair()"}],"source_content_type":"text/x-python","patch_set":1,"id":"ff570b3c_570a111f","line":55,"range":{"start_line":49,"start_character":36,"end_line":55,"end_character":57},"in_reply_to":"ff570b3c_dc976844","updated":"2020-05-21 16:54:15.000000000","message":"I would think adding a CONF.option with a generic default would be satisfactory for something like this.  \n\nAlso nit, just for consistency you can change \u0027dict(add\u003d\u0027/version\u0027,...\u0027 to match the approach of passing dictionaries throughout the method, i.e. change to [{\u0027add\u0027: \u0027/version\u0027, \u0027value\u0027: image_version}].","commit_id":"95c5bd7937c5e3383b1fa86ff285af49ee22cd0f"},{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"08977d254b5c7c10e31909398094fe450bec61d7","unresolved":false,"context_lines":[{"line_number":72,"context_line":"        # snapshot the instance"},{"line_number":73,"context_line":"        snapshot_image \u003d self.create_server_snapshot(server\u003dserver)"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"        # boot a second instance from the snapshot"},{"line_number":76,"context_line":"        server_from_snapshot \u003d self.create_server("},{"line_number":77,"context_line":"            image_id\u003dsnapshot_image[\u0027id\u0027],"},{"line_number":78,"context_line":"            key_name\u003dkeypair[\u0027name\u0027],"},{"line_number":79,"context_line":"            security_groups\u003d[{\u0027name\u0027: security_group[\u0027name\u0027]}])"},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"        # check the existence of the timestamp file in the second instance"},{"line_number":82,"context_line":"        server_from_snapshot_ip \u003d self.get_server_ip(server_from_snapshot)"}],"source_content_type":"text/x-python","patch_set":1,"id":"ff570b3c_46924f4d","line":79,"range":{"start_line":75,"start_character":8,"end_line":79,"end_character":63},"updated":"2020-05-22 15:34:48.000000000","message":"Alternatively you could decorate the snapshot image, boot the second instance from it and then snapshot the instance one final time. I think this is more realistic tbh.","commit_id":"95c5bd7937c5e3383b1fa86ff285af49ee22cd0f"},{"author":{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},"change_message_id":"ca3af03b59b5581fb2caadd39314f20caa664085","unresolved":false,"context_lines":[{"line_number":72,"context_line":"        # snapshot the instance"},{"line_number":73,"context_line":"        snapshot_image \u003d self.create_server_snapshot(server\u003dserver)"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"        # boot a second instance from the snapshot"},{"line_number":76,"context_line":"        server_from_snapshot \u003d self.create_server("},{"line_number":77,"context_line":"            image_id\u003dsnapshot_image[\u0027id\u0027],"},{"line_number":78,"context_line":"            key_name\u003dkeypair[\u0027name\u0027],"},{"line_number":79,"context_line":"            security_groups\u003d[{\u0027name\u0027: security_group[\u0027name\u0027]}])"},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"        # check the existence of the timestamp file in the second instance"},{"line_number":82,"context_line":"        server_from_snapshot_ip \u003d self.get_server_ip(server_from_snapshot)"}],"source_content_type":"text/x-python","patch_set":1,"id":"ff570b3c_0893455e","line":79,"range":{"start_line":75,"start_character":8,"end_line":79,"end_character":63},"in_reply_to":"ff570b3c_46924f4d","updated":"2020-05-22 15:55:28.000000000","message":"so insert the version here instead of above? can do","commit_id":"95c5bd7937c5e3383b1fa86ff285af49ee22cd0f"},{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"e767d9e023ebdc42bfd455d7c7d2e2835f7bb999","unresolved":false,"context_lines":[{"line_number":67,"context_line":"        # add version metadata to the snapshot image"},{"line_number":68,"context_line":"        self.image_client.update_image("},{"line_number":69,"context_line":"            snapshot_image[\u0027id\u0027], [dict(add\u003d\u0027/version\u0027,"},{"line_number":70,"context_line":"                                   value\u003d\u00278.0\u0027)])"},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"        # boot a second instance from the snapshot"},{"line_number":73,"context_line":"        server_from_snapshot \u003d self.create_server("}],"source_content_type":"text/x-python","patch_set":4,"id":"ff570b3c_c812ed62","line":70,"range":{"start_line":70,"start_character":35,"end_line":70,"end_character":48},"updated":"2020-05-22 16:12:28.000000000","message":"This needs to be in-line with add to make pep8 happy.\n\n        # add version metadata to the snapshot image\n        self.image_client.update_image(\n            snapshot_image[\u0027id\u0027], [dict(add\u003d\u0027/version\u0027,\n                                        value\u003d\u00278.0\u0027)])\n\nor break it out onto another line:\n\n        # add version metadata to the snapshot image\n        self.image_client.update_image(\n            snapshot_image[\u0027id\u0027], [\n                dict(add\u003d\u0027/version\u0027, value\u003d\u00278.0\u0027)])","commit_id":"b0f60ba24a98e39bb7069fe778421ed792915303"},{"author":{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},"change_message_id":"80cfb1d79742828ec4e43cd48cb8e86eca3cb97e","unresolved":false,"context_lines":[{"line_number":67,"context_line":"        # add version metadata to the snapshot image"},{"line_number":68,"context_line":"        self.image_client.update_image("},{"line_number":69,"context_line":"            snapshot_image[\u0027id\u0027], [dict(add\u003d\u0027/version\u0027,"},{"line_number":70,"context_line":"                                   value\u003d\u00278.0\u0027)])"},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"        # boot a second instance from the snapshot"},{"line_number":73,"context_line":"        server_from_snapshot \u003d self.create_server("}],"source_content_type":"text/x-python","patch_set":4,"id":"ff570b3c_4e1b8572","line":70,"range":{"start_line":70,"start_character":35,"end_line":70,"end_character":48},"in_reply_to":"ff570b3c_c812ed62","updated":"2020-05-22 16:55:41.000000000","message":"Done","commit_id":"b0f60ba24a98e39bb7069fe778421ed792915303"},{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"e767d9e023ebdc42bfd455d7c7d2e2835f7bb999","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"ff570b3c_c844ad77","line":84,"updated":"2020-05-22 16:12:28.000000000","message":"You need to snapshot the instance again here to trigger the bug.\n\n  # snapshot the instance\n  snapshot_image \u003d self.create_server_snapshot(server\u003dserver)","commit_id":"b0f60ba24a98e39bb7069fe778421ed792915303"},{"author":{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},"change_message_id":"80cfb1d79742828ec4e43cd48cb8e86eca3cb97e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"ff570b3c_2e18d173","line":84,"in_reply_to":"ff570b3c_c844ad77","updated":"2020-05-22 16:55:41.000000000","message":"Done","commit_id":"b0f60ba24a98e39bb7069fe778421ed792915303"}]}