)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":10475,"name":"Alexander Maretskiy","email":"amaretskiy@mirantis.com","username":"maretskiy"},"change_message_id":"f8955320864b68d2b5a3e1d9abad1e9eb3767441","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Added boot-server-attach-created-volume-and-resize nova benchmark."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Partially-Implements: blueprint benchmark-scenarios-for-nova"},{"line_number":12,"context_line":"Change-Id: I83782ebd58e155e20b37ab3e12bc24be60bcbc98"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"1a26ad4f_cd96d63e","line":11,"updated":"2015-11-02 13:43:11.000000000","message":"Can not find mentioned blueprint:\n\nhttps://blueprints.launchpad.net/openstack/?searchtext\u003dbenchmark-scenarios-for-nova","commit_id":"6ed5c3c67126833b9ba2cc482e75487a9a2ceac7"}],"rally/plugins/openstack/scenarios/nova/servers.py":[{"author":{"_account_id":12395,"name":"Roman Vasylets","email":"pomeo92@gmail.com","username":"rvasilets"},"change_message_id":"b75ea41d179935aefc717a94afc6725add28b8f1","unresolved":false,"context_lines":[{"line_number":396,"context_line":"        :param kwargs: Optional additional arguments for server creation"},{"line_number":397,"context_line":"        \"\"\""},{"line_number":398,"context_line":"        server \u003d self._boot_server(image, flavor, **kwargs)"},{"line_number":399,"context_line":"        volume \u003d self._create_volume(volume_size)"},{"line_number":400,"context_line":""},{"line_number":401,"context_line":"        self._attach_volume(server, volume)"},{"line_number":402,"context_line":"        self.sleep_between(min_sleep, max_sleep)"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a2ca52d_35657ee0","line":399,"updated":"2015-10-15 11:46:52.000000000","message":"https://github.com/openstack/rally/blob/master/rally/plugins/openstack/scenarios/cinder/utils.py#L119 I think we should enable possibility to pass kwargs here","commit_id":"cb7a92bab4870c967a7df015c97535553b1034ac"},{"author":{"_account_id":12395,"name":"Roman Vasylets","email":"pomeo92@gmail.com","username":"rvasilets"},"change_message_id":"b75ea41d179935aefc717a94afc6725add28b8f1","unresolved":false,"context_lines":[{"line_number":402,"context_line":"        self.sleep_between(min_sleep, max_sleep)"},{"line_number":403,"context_line":"        self._resize(server, to_flavor)"},{"line_number":404,"context_line":""},{"line_number":405,"context_line":"        # By default we confirm"},{"line_number":406,"context_line":"        confirm \u003d kwargs.get(\"confirm\", True)"},{"line_number":407,"context_line":"        if confirm:"},{"line_number":408,"context_line":"            self._resize_confirm(server)"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a2ca52d_f5ee26b3","line":405,"updated":"2015-10-15 11:46:52.000000000","message":"Its redundant obvious comment. Remove it please.","commit_id":"cb7a92bab4870c967a7df015c97535553b1034ac"},{"author":{"_account_id":6835,"name":"Kun Huang","email":"gareth@openstacker.org","username":"academicgareth"},"change_message_id":"a359f1a649f8c376cb9c828ec02fd48aa6f34939","unresolved":false,"context_lines":[{"line_number":411,"context_line":""},{"line_number":412,"context_line":"        self._detach_volume(server, volume)"},{"line_number":413,"context_line":"        self._delete_volume(volume)"},{"line_number":414,"context_line":"        self._delete_server(server, force\u003dforce_delete)"},{"line_number":415,"context_line":""},{"line_number":416,"context_line":"    @types.set(image\u003dtypes.ImageResourceType,"},{"line_number":417,"context_line":"               flavor\u003dtypes.FlavorResourceType)"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a2ca52d_dc3caf95","line":414,"updated":"2015-10-14 09:04:37.000000000","message":"L412~L414:\n\nyou don\u0027t need delete those here, because rally has cleanup module working on this.","commit_id":"cb7a92bab4870c967a7df015c97535553b1034ac"}],"samples/tasks/scenarios/nova/boot-server-attach-created-volume-and-resize.json":[{"author":{"_account_id":6835,"name":"Kun Huang","email":"gareth@openstacker.org","username":"academicgareth"},"change_message_id":"a359f1a649f8c376cb9c828ec02fd48aa6f34939","unresolved":false,"context_lines":[{"line_number":13,"context_line":"                    \"name\": \"m1.small\""},{"line_number":14,"context_line":"                },"},{"line_number":15,"context_line":"                \"confirm\": true,"},{"line_number":16,"context_line":"                \"volume_size\": 10,"},{"line_number":17,"context_line":"                \"force_delete\": false"},{"line_number":18,"context_line":"            },"},{"line_number":19,"context_line":"            \"runner\": {"}],"source_content_type":"application/json","patch_set":1,"id":"5a2ca52d_5c7adf25","line":16,"updated":"2015-10-14 09:04:37.000000000","message":"volume_size in rally-job/rally.yaml is 1, if CI passed, you could use 1 instead here:)","commit_id":"cb7a92bab4870c967a7df015c97535553b1034ac"}],"tests/unit/plugins/openstack/scenarios/nova/test_servers.py":[{"author":{"_account_id":12395,"name":"Roman Vasylets","email":"pomeo92@gmail.com","username":"rvasilets"},"change_message_id":"b75ea41d179935aefc717a94afc6725add28b8f1","unresolved":false,"context_lines":[{"line_number":423,"context_line":"        scenario._resize \u003d mock.MagicMock()"},{"line_number":424,"context_line":"        scenario._detach_volume \u003d mock.MagicMock()"},{"line_number":425,"context_line":"        scenario._delete_volume \u003d mock.MagicMock()"},{"line_number":426,"context_line":"        scenario._delete_server \u003d mock.MagicMock()"},{"line_number":427,"context_line":"        scenario.sleep_between \u003d mock.MagicMock()"},{"line_number":428,"context_line":""},{"line_number":429,"context_line":"        volume_size \u003d 5"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a2ca52d_98d4c720","line":426,"updated":"2015-10-15 11:46:52.000000000","message":"As Kun said we don;t need to use delete methods.","commit_id":"cb7a92bab4870c967a7df015c97535553b1034ac"},{"author":{"_account_id":12395,"name":"Roman Vasylets","email":"pomeo92@gmail.com","username":"rvasilets"},"change_message_id":"b75ea41d179935aefc717a94afc6725add28b8f1","unresolved":false,"context_lines":[{"line_number":454,"context_line":"        self._test_boot_server_attach_created_volume_and_resize(confirm\u003dTrue)"},{"line_number":455,"context_line":""},{"line_number":456,"context_line":"    def test_boot_server_attach_created_volume_and_resize_with_revert(self):"},{"line_number":457,"context_line":"        self._test_boot_server_attach_created_volume_and_resize(confirm\u003dFalse)"},{"line_number":458,"context_line":""},{"line_number":459,"context_line":"    def test_boot_and_live_migrate_server(self):"},{"line_number":460,"context_line":"        fake_server \u003d mock.MagicMock()"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a2ca52d_f8513bd9","line":457,"updated":"2015-10-15 11:46:52.000000000","message":"we could use ddt module. And therefore above two methods(453, 456) could be deleted. https://github.com/openstack/rally/blob/master/tests/unit/plugins/openstack/scenarios/neutron/test_network.py#L633 this is an example","commit_id":"cb7a92bab4870c967a7df015c97535553b1034ac"}]}