)]}'
{"rally/plugins/openstack/scenarios/nova/aggregates.py":[{"author":{"_account_id":12395,"name":"Roman Vasylets","email":"pomeo92@gmail.com","username":"rvasilets"},"change_message_id":"aba413dd6ab9fc8aa41bfffa2ca3d8ce2c783777","unresolved":false,"context_lines":[{"line_number":54,"context_line":"@validation.required_openstack(admin\u003dTrue)"},{"line_number":55,"context_line":"@scenario.configure(context\u003d{\"admin_cleanup\": [\"nova\"]},"},{"line_number":56,"context_line":"                    name\u003d\"NovaAggregates.create_aggregate_and_add_host\")"},{"line_number":57,"context_line":"class CreateAggregateAndAddHost(utils.NovaScenario):"},{"line_number":58,"context_line":"    \"\"\"Scenario for add a host to and remove the host from an aggregate.\"\"\""},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"    def run(self, availability_zone):"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a95cdbc_ced6e126","line":57,"updated":"2016-10-05 23:29:07.000000000","message":"rename it to CreateAggregateAddAndRemoveHost because you include remove_host into scenario measures too. Or Left remove_host to cleanup. It depends do you want to measure it or not","commit_id":"be83c624fee2583fb17de8da0ab4573943fa1e06"},{"author":{"_account_id":18404,"name":"Ma Xinjian","email":"maxj.fnst@cn.fujitsu.com","username":"maxinjian"},"change_message_id":"09ce85b40c6180579429f6fbc193c327a811147d","unresolved":false,"context_lines":[{"line_number":54,"context_line":"@validation.required_openstack(admin\u003dTrue)"},{"line_number":55,"context_line":"@scenario.configure(context\u003d{\"admin_cleanup\": [\"nova\"]},"},{"line_number":56,"context_line":"                    name\u003d\"NovaAggregates.create_aggregate_and_add_host\")"},{"line_number":57,"context_line":"class CreateAggregateAndAddHost(utils.NovaScenario):"},{"line_number":58,"context_line":"    \"\"\"Scenario for add a host to and remove the host from an aggregate.\"\"\""},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"    def run(self, availability_zone):"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a95cdbc_69a7070b","line":57,"in_reply_to":"1a95cdbc_ced6e126","updated":"2016-10-10 03:09:28.000000000","message":"Done","commit_id":"be83c624fee2583fb17de8da0ab4573943fa1e06"},{"author":{"_account_id":10475,"name":"Alexander Maretskiy","email":"amaretskiy@mirantis.com","username":"maretskiy"},"change_message_id":"49e7652d404e3866b5dab1171ff0498f15460f97","unresolved":false,"context_lines":[{"line_number":83,"context_line":"        \"\"\""},{"line_number":84,"context_line":"        aggregate \u003d self._create_aggregate(availability_zone)"},{"line_number":85,"context_line":"        hosts \u003d self._list_hosts(zone\u003dNone)"},{"line_number":86,"context_line":"        host_name \u003d hosts[0].host_name"},{"line_number":87,"context_line":"        self._aggregate_add_host(aggregate, host_name)"},{"line_number":88,"context_line":"        self._aggregate_remove_host(aggregate, host_name)"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a95cdbc_b27a4fbb","line":86,"updated":"2016-10-10 09:38:59.000000000","message":"Can we be sure that hosts list is never empty?","commit_id":"6ac48e59d22c397eff3e447754d124507fae46c7"},{"author":{"_account_id":18404,"name":"Ma Xinjian","email":"maxj.fnst@cn.fujitsu.com","username":"maxinjian"},"change_message_id":"7cf796a5c2a061a6a3856ba9f0a9e456e42abc87","unresolved":false,"context_lines":[{"line_number":83,"context_line":"        \"\"\""},{"line_number":84,"context_line":"        aggregate \u003d self._create_aggregate(availability_zone)"},{"line_number":85,"context_line":"        hosts \u003d self._list_hosts(zone\u003dNone)"},{"line_number":86,"context_line":"        host_name \u003d hosts[0].host_name"},{"line_number":87,"context_line":"        self._aggregate_add_host(aggregate, host_name)"},{"line_number":88,"context_line":"        self._aggregate_remove_host(aggregate, host_name)"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a95cdbc_08c3d457","line":86,"in_reply_to":"1a95cdbc_b27a4fbb","updated":"2016-10-10 10:11:40.000000000","message":"\"self._list_hosts()\" list all hosts by service. An openstack environment  must exist a nova service, so I think it will never empty","commit_id":"6ac48e59d22c397eff3e447754d124507fae46c7"}]}