)]}'
{"neutron/objects/router.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"fc7aff310ff0694020348170dc4ef1edb18ec0ae","unresolved":false,"context_lines":[{"line_number":352,"context_line":""},{"line_number":353,"context_line":"        # Just hit the first row of each group"},{"line_number":354,"context_line":"        for key, value in group_iterator:"},{"line_number":355,"context_line":"            row \u003d [r for r in next(value)]"},{"line_number":356,"context_line":"            yield (cls._load_object(context, row[0]), row[1])"},{"line_number":357,"context_line":""},{"line_number":358,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"ff570b3c_62d5b843","line":355,"updated":"2020-05-22 20:52:53.000000000","message":"pep8: R1708: Do not raise StopIteration in generator, use return statement instead (stop-iteration-return)","commit_id":"d0c6ad81bc7602d9129d3df65b3bb187671ceade"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"a85c5b758c67d076e23981493b8896c7f17bc79b","unresolved":false,"context_lines":[{"line_number":352,"context_line":""},{"line_number":353,"context_line":"        # Just hit the first row of each group"},{"line_number":354,"context_line":"        for key, value in group_iterator:"},{"line_number":355,"context_line":"            row \u003d [r for r in next(value)]"},{"line_number":356,"context_line":"            yield (cls._load_object(context, row[0]), row[1])"},{"line_number":357,"context_line":""},{"line_number":358,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"ff570b3c_e4c1b1e6","line":355,"in_reply_to":"ff570b3c_62d5b843","updated":"2020-05-26 16:48:19.000000000","message":"This one is more interesting, think I have a way to fix it.","commit_id":"d0c6ad81bc7602d9129d3df65b3bb187671ceade"}],"neutron/tests/unit/agent/l2/extensions/test_fdb_population.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"a85c5b758c67d076e23981493b8896c7f17bc79b","unresolved":false,"context_lines":[{"line_number":49,"context_line":"    def _get_existing_device(self):"},{"line_number":50,"context_line":"        device_mappings \u003d helpers.parse_mappings("},{"line_number":51,"context_line":"            cfg.CONF.FDB.shared_physical_device_mappings, unique_keys\u003dFalse)"},{"line_number":52,"context_line":"        DEVICES \u003d next(device_mappings.values())"},{"line_number":53,"context_line":"        return DEVICES[0]"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"    def _get_fdb_extension(self, mock_execute, fdb_table):"}],"source_content_type":"text/x-python","patch_set":1,"id":"ff570b3c_c4010db2","line":52,"updated":"2020-05-26 16:48:19.000000000","message":"I guess that same next(iter(values)) here as well","commit_id":"d0c6ad81bc7602d9129d3df65b3bb187671ceade"}],"neutron/tests/unit/extensions/test_timestamp.py":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"e91dd3f4efc280d3a1b49c1367cb123453ff0bdd","unresolved":false,"context_lines":[{"line_number":103,"context_line":"    def _list_resources_with_changed_since(self, resource):"},{"line_number":104,"context_line":"        # assert list results contain the net info when"},{"line_number":105,"context_line":"        # changed_since equal with the net updated time."},{"line_number":106,"context_line":"        resource_type \u003d next(resource.keys())"},{"line_number":107,"context_line":"        if resource_type in [\u0027network\u0027, \u0027port\u0027]:"},{"line_number":108,"context_line":"            self._set_timestamp_by_show(resource, resource_type)"},{"line_number":109,"context_line":"        resources \u003d self._get_resp_with_changed_since(resource_type,"}],"source_content_type":"text/x-python","patch_set":1,"id":"ff570b3c_e12b637f","line":106,"range":{"start_line":106,"start_character":24,"end_line":106,"end_character":45},"updated":"2020-05-26 15:34:52.000000000","message":"Be careful wit this you will have:\nTypeError: \u0027dict_keys\u0027 object is not an iterator\n\nso you need next(iter(resource.keys())) or something equivalent.","commit_id":"d0c6ad81bc7602d9129d3df65b3bb187671ceade"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"a85c5b758c67d076e23981493b8896c7f17bc79b","unresolved":false,"context_lines":[{"line_number":103,"context_line":"    def _list_resources_with_changed_since(self, resource):"},{"line_number":104,"context_line":"        # assert list results contain the net info when"},{"line_number":105,"context_line":"        # changed_since equal with the net updated time."},{"line_number":106,"context_line":"        resource_type \u003d next(resource.keys())"},{"line_number":107,"context_line":"        if resource_type in [\u0027network\u0027, \u0027port\u0027]:"},{"line_number":108,"context_line":"            self._set_timestamp_by_show(resource, resource_type)"},{"line_number":109,"context_line":"        resources \u003d self._get_resp_with_changed_since(resource_type,"}],"source_content_type":"text/x-python","patch_set":1,"id":"ff570b3c_817c2721","line":106,"range":{"start_line":106,"start_character":24,"end_line":106,"end_character":45},"in_reply_to":"ff570b3c_e12b637f","updated":"2020-05-26 16:48:19.000000000","message":"Thanks for the info, that wasn\u0027t obvious.","commit_id":"d0c6ad81bc7602d9129d3df65b3bb187671ceade"}]}
