)]}'
{"tempest/api/compute/admin/test_aggregates_negative.py":[{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"06e0db66008d38ccedfc9d60f60005c7c2b5fb96","unresolved":false,"context_lines":[{"line_number":34,"context_line":"        super(AggregatesAdminNegativeTestJSON, cls).resource_setup()"},{"line_number":35,"context_line":"        cls.aggregate_name_prefix \u003d \u0027test_aggregate\u0027"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"        hyper_list \u003d cls.hyper_client.list_hypervisors()[\u0027hypervisors\u0027]"},{"line_number":38,"context_line":"        cls.host \u003d hyper_list[0][\u0027hypervisor_hostname\u0027]"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    def _create_test_aggregate(self):"}],"source_content_type":"text/x-python","patch_set":2,"id":"df7087c5_469ed7b8","line":37,"updated":"2018-03-21 16:52:48.000000000","message":"If you do self.hyper_list here you can avoid the redundant call on L125","commit_id":"97fe6097bcbb8e314d1af0f2abbefc606fabe084"},{"author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"change_message_id":"d263f373759bfc4c0f308732f95d7b65950a13db","unresolved":false,"context_lines":[{"line_number":34,"context_line":"        super(AggregatesAdminNegativeTestJSON, cls).resource_setup()"},{"line_number":35,"context_line":"        cls.aggregate_name_prefix \u003d \u0027test_aggregate\u0027"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"        hyper_list \u003d cls.hyper_client.list_hypervisors()[\u0027hypervisors\u0027]"},{"line_number":38,"context_line":"        cls.host \u003d hyper_list[0][\u0027hypervisor_hostname\u0027]"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    def _create_test_aggregate(self):"}],"source_content_type":"text/x-python","patch_set":2,"id":"df7087c5_856cd836","line":37,"in_reply_to":"df7087c5_469ed7b8","updated":"2018-03-22 05:10:51.000000000","message":"Thanks, done","commit_id":"97fe6097bcbb8e314d1af0f2abbefc606fabe084"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"00f9a48002d7785564c7496e5fc5580756e33405","unresolved":false,"context_lines":[{"line_number":35,"context_line":"        cls.aggregate_name_prefix \u003d \u0027test_aggregate\u0027"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"        hyper_list \u003d cls.hyper_client.list_hypervisors()[\u0027hypervisors\u0027]"},{"line_number":38,"context_line":"        cls.host \u003d hyper_list[0][\u0027hypervisor_hostname\u0027]"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    def _create_test_aggregate(self):"},{"line_number":41,"context_line":"        aggregate_name \u003d data_utils.rand_name(self.aggregate_name_prefix)"}],"source_content_type":"text/x-python","patch_set":2,"id":"df7087c5_a6e6d315","line":38,"updated":"2018-03-21 16:58:57.000000000","message":"One more thought: I\u0027m curious if this can break if cls.host is disabled or down. For example, see L150. I can\u0027t see any constraints around adding a disabled host to an aggregate but list_hypervisors does return `status` and `state` info that might be useful to select host whose hypervisor has status\u003denabled and state\u003dup. https://developer.openstack.org/api-ref/compute/#id303","commit_id":"97fe6097bcbb8e314d1af0f2abbefc606fabe084"},{"author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"change_message_id":"d263f373759bfc4c0f308732f95d7b65950a13db","unresolved":false,"context_lines":[{"line_number":35,"context_line":"        cls.aggregate_name_prefix \u003d \u0027test_aggregate\u0027"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"        hyper_list \u003d cls.hyper_client.list_hypervisors()[\u0027hypervisors\u0027]"},{"line_number":38,"context_line":"        cls.host \u003d hyper_list[0][\u0027hypervisor_hostname\u0027]"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    def _create_test_aggregate(self):"},{"line_number":41,"context_line":"        aggregate_name \u003d data_utils.rand_name(self.aggregate_name_prefix)"}],"source_content_type":"text/x-python","patch_set":2,"id":"df7087c5_6571840c","line":38,"in_reply_to":"df7087c5_a6e6d315","updated":"2018-03-22 05:10:51.000000000","message":"done","commit_id":"97fe6097bcbb8e314d1af0f2abbefc606fabe084"}]}