)]}'
{"neutron/plugins/ml2/drivers/ovn/mech_driver/mech_driver.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"fb00a56e9503ced765a52d649eab290cd9975347","unresolved":false,"context_lines":[{"line_number":515,"context_line":"        LOG.error(\u0027The following extra DHCP options for port %(port_id)s \u0027"},{"line_number":516,"context_line":"                  \u0027are not supported by OVN. IPv4: \"%(ipv4_opts)s\" and \u0027"},{"line_number":517,"context_line":"                  \u0027IPv6: \"%(ipv6_opts)s\"\u0027, {\u0027port_id\u0027: port[\u0027id\u0027],"},{"line_number":518,"context_line":"                  \u0027ipv4_opts\u0027: ipv4_opts, \u0027ipv6_opts\u0027: ipv6_opts})"},{"line_number":519,"context_line":""},{"line_number":520,"context_line":"    def create_port_precommit(self, context):"},{"line_number":521,"context_line":"        \"\"\"Allocate resources for a new port."}],"source_content_type":"text/x-python","patch_set":1,"id":"bf51134e_9c5d7e17","line":518,"updated":"2020-07-23 13:03:14.000000000","message":"Some questions/comments:\n\n1) IMO, this is not an error if this is going to be part of a normal deployment use case. If we need to support those tags, we can\u0027t log an error here.\n\n2) If we don\u0027t raise an exception and continue the processing, when we create/update the port, we get the DHCP options in [1]. If an option is not valid, we are already logging a warning message. IMO, that\u0027s enough.\n\nConclusion: that should be just an INFO message. I don\u0027t know if you agree with me.\n\n[1]https://github.com/openstack/neutron/blob/9717fea647b3836ba305450bb10989d0d25af7ed/neutron/common/ovn/utils.py#L154","commit_id":"634e3d8210570d11531fc46a336693699f5c1c45"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"c19d319ab7cd0d56406e8fef91e297e4e9b92312","unresolved":false,"context_lines":[{"line_number":515,"context_line":"        LOG.error(\u0027The following extra DHCP options for port %(port_id)s \u0027"},{"line_number":516,"context_line":"                  \u0027are not supported by OVN. IPv4: \"%(ipv4_opts)s\" and \u0027"},{"line_number":517,"context_line":"                  \u0027IPv6: \"%(ipv6_opts)s\"\u0027, {\u0027port_id\u0027: port[\u0027id\u0027],"},{"line_number":518,"context_line":"                  \u0027ipv4_opts\u0027: ipv4_opts, \u0027ipv6_opts\u0027: ipv6_opts})"},{"line_number":519,"context_line":""},{"line_number":520,"context_line":"    def create_port_precommit(self, context):"},{"line_number":521,"context_line":"        \"\"\"Allocate resources for a new port."}],"source_content_type":"text/x-python","patch_set":1,"id":"bf51134e_bc06821f","line":518,"in_reply_to":"bf51134e_9c5d7e17","updated":"2020-07-23 13:53:26.000000000","message":"Hi Rodolfo, \n\nYou have a great point, indeed I think ERROR might be misleading here cause the code actually won\u0027t \"error out\" or anything.\n\nI do agree with you, I will change the log level to INFO as you suggested.\n\nThanks for that.","commit_id":"634e3d8210570d11531fc46a336693699f5c1c45"}]}