)]}'
{"openstackclient/network/v2/port.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"6666cbba3e8fc64e9475f4131c1f590f9cc949ec","unresolved":false,"context_lines":[{"line_number":665,"context_line":""},{"line_number":666,"context_line":"        _tag.get_tag_filtering_args(parsed_args, filters)"},{"line_number":667,"context_line":""},{"line_number":668,"context_line":"        data \u003d network_client.ports(fields\u003dcolumns, **filters)"},{"line_number":669,"context_line":""},{"line_number":670,"context_line":"        headers, attrs \u003d utils.calculate_header_and_attrs("},{"line_number":671,"context_line":"            column_headers, columns, parsed_args)"}],"source_content_type":"text/x-python","patch_set":7,"id":"9f560f44_4d368f51","line":668,"range":{"start_line":668,"start_character":36,"end_line":668,"end_character":52},"updated":"2020-10-06 10:16:39.000000000","message":"I assume the client will just ignore this if it doesn\u0027t support it?","commit_id":"b0405db62300a53acb3143830d44483c37cecbb0"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"3e18238caa30d0102005eb5dc91375548e7c8d7e","unresolved":false,"context_lines":[{"line_number":665,"context_line":""},{"line_number":666,"context_line":"        _tag.get_tag_filtering_args(parsed_args, filters)"},{"line_number":667,"context_line":""},{"line_number":668,"context_line":"        data \u003d network_client.ports(fields\u003dcolumns, **filters)"},{"line_number":669,"context_line":""},{"line_number":670,"context_line":"        headers, attrs \u003d utils.calculate_header_and_attrs("},{"line_number":671,"context_line":"            column_headers, columns, parsed_args)"}],"source_content_type":"text/x-python","patch_set":7,"id":"9f560f44_e18b2c14","line":668,"range":{"start_line":668,"start_character":36,"end_line":668,"end_character":52},"in_reply_to":"9f560f44_4d368f51","updated":"2020-10-06 13:35:03.000000000","message":"No indeed, good catch!\n\nI\u0027ll need to release a new SDK version but I\u0027ll need to wait until new branch (Wallaby) is open.","commit_id":"b0405db62300a53acb3143830d44483c37cecbb0"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"d5a235e257ed4c1edc962d1eff9142be545bbd75","unresolved":false,"context_lines":[{"line_number":665,"context_line":""},{"line_number":666,"context_line":"        _tag.get_tag_filtering_args(parsed_args, filters)"},{"line_number":667,"context_line":""},{"line_number":668,"context_line":"        data \u003d network_client.ports(fields\u003dcolumns, **filters)"},{"line_number":669,"context_line":""},{"line_number":670,"context_line":"        headers, attrs \u003d utils.calculate_header_and_attrs("},{"line_number":671,"context_line":"            column_headers, columns, parsed_args)"}],"source_content_type":"text/x-python","patch_set":10,"id":"1f621f24_7a064d54","line":668,"range":{"start_line":668,"start_character":36,"end_line":668,"end_character":51},"updated":"2020-11-12 15:11:34.000000000","message":"So we now have the SDK change noted in [1], but what happens if you run this on a old installation? Will neutron just ignore the extra kwarg? I assume so, since I think the neutron API is additive?\n\n[1] https://review.opendev.org/#/c/754117/7/openstackclient/network/v2/port.py@668","commit_id":"75ac501a180eedfbd3d1dce6931c6b8f2db2569e"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"ec2366827387327b749e0ecb7a28866c82cbc6fe","unresolved":false,"context_lines":[{"line_number":665,"context_line":""},{"line_number":666,"context_line":"        _tag.get_tag_filtering_args(parsed_args, filters)"},{"line_number":667,"context_line":""},{"line_number":668,"context_line":"        data \u003d network_client.ports(fields\u003dcolumns, **filters)"},{"line_number":669,"context_line":""},{"line_number":670,"context_line":"        headers, attrs \u003d utils.calculate_header_and_attrs("},{"line_number":671,"context_line":"            column_headers, columns, parsed_args)"}],"source_content_type":"text/x-python","patch_set":10,"id":"1f621f24_9a288997","line":668,"range":{"start_line":668,"start_character":36,"end_line":668,"end_character":51},"in_reply_to":"1f621f24_7a064d54","updated":"2020-11-12 15:19:16.000000000","message":"Ah yes, this \"fields\" column exists in Neutron at least since 2012 [1]. \"fields\" is a Neutron standard argument for queries.\n\n[1]https://review.opendev.org/#/c/15384/","commit_id":"75ac501a180eedfbd3d1dce6931c6b8f2db2569e"}],"openstackclient/tests/functional/network/v2/common.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"28bf5c9d98bbf0f07d973dc762261982d5260403","unresolved":false,"context_lines":[{"line_number":70,"context_line":"            net \u003d [n for n in cmd_output if n[\u0027Name\u0027] \u003d\u003d name][0]"},{"line_number":71,"context_line":"            try:"},{"line_number":72,"context_line":"                # self.assertEqual(set(tags), set(net[\u0027Tags\u0027]))"},{"line_number":73,"context_line":"                assert set(tags) \u003d\u003d  set(net[\u0027Tags\u0027])"},{"line_number":74,"context_line":"            except:"},{"line_number":75,"context_line":"                raise Exception(\u0027Tags differ, tags %s, cmd output: %s\u0027 % (tags, cmd_output))"},{"line_number":76,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"9f560f44_d1caa46d","line":73,"updated":"2020-09-28 18:37:46.000000000","message":"pep8: E222 multiple spaces after operator","commit_id":"ff3fe50b4f2f637c5289b2ad39255bf58ea1535b"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"28bf5c9d98bbf0f07d973dc762261982d5260403","unresolved":false,"context_lines":[{"line_number":71,"context_line":"            try:"},{"line_number":72,"context_line":"                # self.assertEqual(set(tags), set(net[\u0027Tags\u0027]))"},{"line_number":73,"context_line":"                assert set(tags) \u003d\u003d  set(net[\u0027Tags\u0027])"},{"line_number":74,"context_line":"            except:"},{"line_number":75,"context_line":"                raise Exception(\u0027Tags differ, tags %s, cmd output: %s\u0027 % (tags, cmd_output))"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"    def _create_resource_for_tag_test(self, name, args):"}],"source_content_type":"text/x-python","patch_set":4,"id":"9f560f44_b1c73066","line":74,"updated":"2020-09-28 18:37:46.000000000","message":"pep8: E722 do not use bare \u0027except\u0027","commit_id":"ff3fe50b4f2f637c5289b2ad39255bf58ea1535b"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"28bf5c9d98bbf0f07d973dc762261982d5260403","unresolved":false,"context_lines":[{"line_number":71,"context_line":"            try:"},{"line_number":72,"context_line":"                # self.assertEqual(set(tags), set(net[\u0027Tags\u0027]))"},{"line_number":73,"context_line":"                assert set(tags) \u003d\u003d  set(net[\u0027Tags\u0027])"},{"line_number":74,"context_line":"            except:"},{"line_number":75,"context_line":"                raise Exception(\u0027Tags differ, tags %s, cmd output: %s\u0027 % (tags, cmd_output))"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"    def _create_resource_for_tag_test(self, name, args):"}],"source_content_type":"text/x-python","patch_set":4,"id":"9f560f44_11c53c5d","line":74,"updated":"2020-09-28 18:37:46.000000000","message":"pep8: H201: no \u0027except:\u0027 at least use \u0027except Exception:\u0027","commit_id":"ff3fe50b4f2f637c5289b2ad39255bf58ea1535b"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"28bf5c9d98bbf0f07d973dc762261982d5260403","unresolved":false,"context_lines":[{"line_number":72,"context_line":"                # self.assertEqual(set(tags), set(net[\u0027Tags\u0027]))"},{"line_number":73,"context_line":"                assert set(tags) \u003d\u003d  set(net[\u0027Tags\u0027])"},{"line_number":74,"context_line":"            except:"},{"line_number":75,"context_line":"                raise Exception(\u0027Tags differ, tags %s, cmd output: %s\u0027 % (tags, cmd_output))"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"    def _create_resource_for_tag_test(self, name, args):"},{"line_number":78,"context_line":"        return json.loads(self.openstack("}],"source_content_type":"text/x-python","patch_set":4,"id":"9f560f44_f1c1a850","line":75,"updated":"2020-09-28 18:37:46.000000000","message":"pep8: E501 line too long (92 \u003e 79 characters)","commit_id":"ff3fe50b4f2f637c5289b2ad39255bf58ea1535b"}],"openstackclient/tests/unit/network/v2/test_port.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"c208076280103772da87a263b77289029d1a5875","unresolved":false,"context_lines":[{"line_number":29,"context_line":"LIST_FIELDS_TO_RETRIEVE \u003d (\u0027id\u0027, \u0027name\u0027, \u0027mac_address\u0027, \u0027fixed_ips\u0027, \u0027status\u0027)"},{"line_number":30,"context_line":"LIST_FIELDS_TO_RETRIEVE_LONG \u003d (\u0027security_group_ids\u0027, \u0027device_owner\u0027, \u0027tags\u0027)"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"class TestPort(network_fakes.TestNetworkV2):"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"    def setUp(self):"},{"line_number":35,"context_line":"        super(TestPort, self).setUp()"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_708a8d64","line":32,"updated":"2020-09-30 17:09:41.000000000","message":"pep8: E302 expected 2 blank lines, found 1","commit_id":"3419689798601ab548515ce86ceb991ff3c0d3a3"}],"requirements.txt":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"d5a235e257ed4c1edc962d1eff9142be545bbd75","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"1f621f24_fa7b7df1","updated":"2020-11-12 15:11:34.000000000","message":"This looks like a bad rebase","commit_id":"75ac501a180eedfbd3d1dce6931c6b8f2db2569e"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"ec2366827387327b749e0ecb7a28866c82cbc6fe","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"1f621f24_7a8bedc1","in_reply_to":"1f621f24_fa7b7df1","updated":"2020-11-12 15:19:16.000000000","message":"Horrible!","commit_id":"75ac501a180eedfbd3d1dce6931c6b8f2db2569e"}]}
