)]}'
{"neutron/agent/ovn/metadata/agent.py":[{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"bb56b56f9f67d70ddfd9c8e1f89b8ff891125e76","unresolved":true,"context_lines":[{"line_number":629,"context_line":"        if not port.mac:"},{"line_number":630,"context_line":"            LOG.warning(\"Port %s MAC column is empty, cannot retrieve IP \""},{"line_number":631,"context_line":"                        \"addresses\", port.uuid)"},{"line_number":632,"context_line":"            return [], False"},{"line_number":633,"context_line":"        mac, ips \u003d ovn_utils.get_mac_and_ips_from_port_binding(port)"},{"line_number":634,"context_line":"        if not ips:"},{"line_number":635,"context_line":"            LOG.debug(\"Port %s IP addresses were not retrieved from the \""}],"source_content_type":"text/x-python","patch_set":1,"id":"62f9690b_ce4a7a9f","line":632,"updated":"2026-04-28 12:29:34.000000000","message":"perhaps we should add unittests that validate the returned values in both cases","commit_id":"52428ff282471a71d8941ca0016b44bc19b1394e"},{"author":{"_account_id":31291,"name":"Eduardo Olivares","email":"eolivare@redhat.com","username":"eolivare"},"change_message_id":"0605245c9f9fcc626dabfcb8da8f3c63f8e746c1","unresolved":false,"context_lines":[{"line_number":629,"context_line":"        if not port.mac:"},{"line_number":630,"context_line":"            LOG.warning(\"Port %s MAC column is empty, cannot retrieve IP \""},{"line_number":631,"context_line":"                        \"addresses\", port.uuid)"},{"line_number":632,"context_line":"            return [], False"},{"line_number":633,"context_line":"        mac, ips \u003d ovn_utils.get_mac_and_ips_from_port_binding(port)"},{"line_number":634,"context_line":"        if not ips:"},{"line_number":635,"context_line":"            LOG.debug(\"Port %s IP addresses were not retrieved from the \""}],"source_content_type":"text/x-python","patch_set":1,"id":"da74b261_deca21d8","line":632,"in_reply_to":"05cd97d4_5ac6305f","updated":"2026-04-28 13:06:16.000000000","message":"Done","commit_id":"52428ff282471a71d8941ca0016b44bc19b1394e"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"bf7b127f2992ddde81432cd525b4231b85c43d65","unresolved":true,"context_lines":[{"line_number":629,"context_line":"        if not port.mac:"},{"line_number":630,"context_line":"            LOG.warning(\"Port %s MAC column is empty, cannot retrieve IP \""},{"line_number":631,"context_line":"                        \"addresses\", port.uuid)"},{"line_number":632,"context_line":"            return [], False"},{"line_number":633,"context_line":"        mac, ips \u003d ovn_utils.get_mac_and_ips_from_port_binding(port)"},{"line_number":634,"context_line":"        if not ips:"},{"line_number":635,"context_line":"            LOG.debug(\"Port %s IP addresses were not retrieved from the \""}],"source_content_type":"text/x-python","patch_set":1,"id":"05cd97d4_5ac6305f","line":632,"in_reply_to":"62f9690b_ce4a7a9f","updated":"2026-04-28 12:47:59.000000000","message":"yes, maybe something to append to https://review.opendev.org/c/openstack/neutron/+/861124/7/neutron/tests/unit/agent/ovn/metadata/test_agent.py","commit_id":"52428ff282471a71d8941ca0016b44bc19b1394e"}]}