)]}'
{"tempest/scenario/test_volume_boot_pattern.py":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"a9c409f977a8cf294b5beb6cdb50231c782b0657","unresolved":false,"context_lines":[{"line_number":49,"context_line":"                                     source_type,"},{"line_number":50,"context_line":"                                     keypair\u003dNone,"},{"line_number":51,"context_line":"                                     security_group\u003dNone,"},{"line_number":52,"context_line":"                                     delete_on_termination\u003dFalse):"},{"line_number":53,"context_line":"        create_kwargs \u003d dict()"},{"line_number":54,"context_line":"        if keypair:"},{"line_number":55,"context_line":"            create_kwargs[\u0027key_name\u0027] \u003d keypair[\u0027name\u0027]"}],"source_content_type":"text/x-python","patch_set":2,"id":"3f79a3b5_25609c01","line":52,"range":{"start_line":52,"start_character":64,"end_line":52,"end_character":65},"updated":"2018-08-08 02:55:33.000000000","message":"here you can add new kwargs name\u003dNone which can be added on create_kwargs to pass name in self.create_server() request at L64","commit_id":"2ce18843c7a8fe9a398f48b324737fc28b806f6f"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"a9c409f977a8cf294b5beb6cdb50231c782b0657","unresolved":false,"context_lines":[{"line_number":206,"context_line":"    def test_image_defined_boot_from_volume(self):"},{"line_number":207,"context_line":"        # create an instance from image-backed volume"},{"line_number":208,"context_line":"        volume_origin \u003d self._create_volume_from_image()"},{"line_number":209,"context_line":"        instance1 \u003d self._boot_instance_from_resource("},{"line_number":210,"context_line":"            source_id\u003dvolume_origin[\u0027id\u0027],"},{"line_number":211,"context_line":"            source_type\u003d\u0027volume\u0027,"},{"line_number":212,"context_line":"            delete_on_termination\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":2,"id":"3f79a3b5_6546947f","line":209,"range":{"start_line":209,"start_character":53,"end_line":209,"end_character":54},"updated":"2018-08-08 02:55:33.000000000","message":"let\u0027s pass the specific name here too like volume to make debugging easy in future.\n\nname \u003d data_utils.rand_name(self.__class__.__name__ + \"volume-backed-server\")","commit_id":"2ce18843c7a8fe9a398f48b324737fc28b806f6f"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"a9c409f977a8cf294b5beb6cdb50231c782b0657","unresolved":false,"context_lines":[{"line_number":220,"context_line":"        # about the volume snapshot. The compute service will use this to"},{"line_number":221,"context_line":"        # create a volume from the volume snapshot and use that as the root"},{"line_number":222,"context_line":"        # disk for the server."},{"line_number":223,"context_line":"        instance2 \u003d self.create_server(image_id\u003dimage[\u0027id\u0027])"},{"line_number":224,"context_line":""},{"line_number":225,"context_line":"        # Verify the server was created from the image-defined BDM."},{"line_number":226,"context_line":"        volume_attachments \u003d instance2[\u0027os-extended-volumes:volumes_attached\u0027]"}],"source_content_type":"text/x-python","patch_set":2,"id":"3f79a3b5_653bf4ea","line":223,"range":{"start_line":223,"start_character":59,"end_line":223,"end_character":60},"updated":"2018-08-08 02:55:33.000000000","message":"let\u0027s use specific name here too\n\nname \u003d data_utils.rand_name(self.__class__.__name__ + \"image-snapshot-server\")\ninstance2 \u003d self.create_server(name\u003dname, image_id\u003dimage[\u0027id\u0027])","commit_id":"2ce18843c7a8fe9a398f48b324737fc28b806f6f"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"a9c409f977a8cf294b5beb6cdb50231c782b0657","unresolved":false,"context_lines":[{"line_number":250,"context_line":"        # Delete the volume snapshot. We must do this before deleting the first"},{"line_number":251,"context_line":"        # server created in this test because the snapshot depends on the first"},{"line_number":252,"context_line":"        # instance\u0027s underlying volume (volume_origin)."},{"line_number":253,"context_line":"        snapshot \u003d self.snapshots_client.list_snapshots()[\u0027snapshots\u0027][0]"},{"line_number":254,"context_line":"        self._delete_snapshot(snapshot)"},{"line_number":255,"context_line":""},{"line_number":256,"context_line":"        # Now, delete the first server which will also delete the first"}],"source_content_type":"text/x-python","patch_set":2,"id":"3f79a3b5_90252cae","line":253,"range":{"start_line":253,"start_character":19,"end_line":253,"end_character":73},"updated":"2018-08-08 02:55:33.000000000","message":"this can return some other snapshot also which are being created by other tests running in parallel. and this tests is not running with LockFixture also. \n\nWe can get the created snapshot_id from base create method[1] at L216 and use that here\n\n[1] https://github.com/openstack/tempest/blob/b98bf994369fd94c869a304782d46884604aaab4/tempest/scenario/manager.py#L517","commit_id":"2ce18843c7a8fe9a398f48b324737fc28b806f6f"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"c832d57001e619ab73aff7040f9c9d87906a5b61","unresolved":false,"context_lines":[{"line_number":259,"context_line":"        # Delete the volume snapshot. We must do this before deleting the first"},{"line_number":260,"context_line":"        # server created in this test because the snapshot depends on the first"},{"line_number":261,"context_line":"        # instance\u0027s underlying volume (volume_origin)."},{"line_number":262,"context_line":"        bdms \u003d jsonutils.loads(image[\u0027block_device_mapping\u0027])"},{"line_number":263,"context_line":"        snapshot_id \u003d bdms[0][\u0027snapshot_id\u0027]"},{"line_number":264,"context_line":"        self._delete_snapshot(snapshot_id)"},{"line_number":265,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"3f79a3b5_c921acfd","line":262,"updated":"2018-08-08 23:11:22.000000000","message":"This failed the Ocata job because I assumed glance v2 here. Need to duplicate the functionality in create_server_snapshot to handle glance v1 too.","commit_id":"3a855db390c358fc7acee63802c931e5925592d1"}]}