From 81863c959b82ca84df8775d16c0ba73a3d6ede9c Mon Sep 17 00:00:00 2001 From: Thomas Haller Date: Fri, 6 Nov 2020 19:44:55 +0100 Subject: [PATCH] core/ovs: rename logging output for _LOGT_call() The text should match the OvsdbCommand enum. If the enum value is named OVSDB_ADD_INTERFACE, then we should print "add-interface". Or alternatively, if you think spelling out interface is too long, then the enum should be renamed. I don't care, but name should correspond. --- src/devices/ovs/nm-ovsdb.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/devices/ovs/nm-ovsdb.c b/src/devices/ovs/nm-ovsdb.c index c706df466..2593b9b74 100644 --- a/src/devices/ovs/nm-ovsdb.c +++ b/src/devices/ovs/nm-ovsdb.c @@ -154,7 +154,7 @@ _LOGT_call_do(const char *comment, OvsdbMethodCall *call, json_t *msg) _LOGT("%s: monitor%s%s", comment, _QUOTE_MSG(msg, msg_as_str)); break; case OVSDB_ADD_INTERFACE: - _LOGT("%s: add-iface bridge=%s port=%s interface=%s%s%s", + _LOGT("%s: add-interface bridge=%s port=%s interface=%s%s%s", comment, nm_connection_get_interface_name(call->add_interface.bridge), nm_connection_get_interface_name(call->add_interface.port), @@ -162,13 +162,13 @@ _LOGT_call_do(const char *comment, OvsdbMethodCall *call, json_t *msg) _QUOTE_MSG(msg, msg_as_str)); break; case OVSDB_DEL_INTERFACE: - _LOGT("%s: del-iface interface=%s%s%s", + _LOGT("%s: del-interface interface=%s%s%s", comment, call->del_interface.ifname, _QUOTE_MSG(msg, msg_as_str)); break; case OVSDB_SET_INTERFACE_MTU: - _LOGT("%s: set-iface-mtu interface=%s mtu=%u%s%s", + _LOGT("%s: set-interface-mtu interface=%s mtu=%u%s%s", comment, call->set_interface_mtu.ifname, call->set_interface_mtu.mtu,