)]}'
{"tempest/scenario/test_network_basic_ops.py":[{"author":{"_account_id":19642,"name":"Zou Keke","email":"zoukeke@cmss.chinamobile.com","username":"zoukeke"},"change_message_id":"49558062f58814458d5ba90ff06f8f441d178ec3","unresolved":false,"context_lines":[{"line_number":322,"context_line":"        # which is always IPv4, so we must only test connectivity to"},{"line_number":323,"context_line":"        # external IPv4 IPs if the external network is dualstack."},{"line_number":324,"context_line":"        floating_ip, _ \u003d self.floating_ip_tuple"},{"line_number":325,"context_line":"        router \u003d self.routers_client.show_router("},{"line_number":326,"context_line":"            floating_ip[\u0027router_id\u0027])[\u0027router\u0027]"},{"line_number":327,"context_line":""},{"line_number":328,"context_line":"        ext_ips \u003d router[\u0027external_gateway_info\u0027][\u0027external_fixed_ips\u0027]"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa31d9ce_11f84be5","line":325,"range":{"start_line":325,"start_character":22,"end_line":325,"end_character":36},"updated":"2017-02-10 01:45:15.000000000","message":"It seems that we should use admin_routers_client here.","commit_id":"9cfde2e0977c88fb3eebdf0354cda14641e13436"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"d75b8bce500660847ffc833f4af15d6c0795de9d","unresolved":false,"context_lines":[{"line_number":325,"context_line":"        router \u003d self.routers_client.show_router("},{"line_number":326,"context_line":"            floating_ip[\u0027router_id\u0027])[\u0027router\u0027]"},{"line_number":327,"context_line":""},{"line_number":328,"context_line":"        ext_ips \u003d router[\u0027external_gateway_info\u0027][\u0027external_fixed_ips\u0027]"},{"line_number":329,"context_line":"        sub_ids \u003d [sub[\u0027subnet_id\u0027] for sub in ext_ips]"},{"line_number":330,"context_line":"        v4_subnets \u003d [s for s in self._list_subnets("},{"line_number":331,"context_line":"            network_id\u003dCONF.network.public_network_id)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a430d35_52b3c497","line":328,"updated":"2017-02-09 09:33:28.000000000","message":"Was the problem that self.router was None or was it that the router had no \u0027external_gateway_info\u0027 ?","commit_id":"9cfde2e0977c88fb3eebdf0354cda14641e13436"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"32d70ac5a7065d300998e41f10913bc2fa543325","unresolved":false,"context_lines":[{"line_number":81,"context_line":"    @classmethod"},{"line_number":82,"context_line":"    def setup_clients(cls):"},{"line_number":83,"context_line":"        super(TestNetworkBasicOps, cls).setup_clients()"},{"line_number":84,"context_line":"        cls.admin_routers_client \u003d cls.os_adm.routers_client"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"    @classmethod"},{"line_number":87,"context_line":"    def skip_checks(cls):"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa31d9ce_d3e7d3a9","line":84,"updated":"2017-02-15 09:46:59.000000000","message":"Tempest can run without admin credentials. The dependency on  os_adm.routers_client added here means that the whole class won\u0027t run if admin creds are not available. That\u0027s suboptimal. Isn\u0027t there another way to solve this ?","commit_id":"42e5f992575d1713eb06b095216d01d408698103"}]}