Apply suggestions from code review

Author: GutoVeronezi

api/src/main/java/com/cloud/agent/api/storage/OVFHelper.java

@@ -541,7 +541,7 @@ private void matchNicsToNets(Map<String, OVFNetworkTO> nets, Node systemElement)
                 nicCount++;
                 String name = e.getTextContent(); // should be in our nets
                 if(nets.get(name) == null) {
-                    logger.info("Found a nic definition without a network definition byname {}, adding it to the list.", name);
+                    logger.info("Found a nic definition without a network definition by name {}, adding it to the list.", name);
                     nets.put(name, new OVFNetworkTO());
                 }
                 OVFNetworkTO thisNet = nets.get(name);

api/src/main/java/org/apache/cloudstack/api/command/user/firewall/CreateEgressFirewallRuleCmd.java

@@ -255,7 +255,7 @@ public void create() {
             }
         } catch (NetworkRuleConflictException ex) {
             String message = "Network rule conflict: ";
-            logger.error( "Network rule conflict: {}", ex.getMessage());
+            logger.error("{}{}", message, ex.getMessage());
             logger.trace(message, ex);
             throw new ServerApiException(ApiErrorCode.NETWORK_RULE_CONFLICT_ERROR, ex.getMessage());
         }

api/src/main/java/org/apache/cloudstack/context/LogContext.java

@@ -205,7 +205,7 @@ public static void unregister() {
         LogContext context = s_currentContext.get();
         if (context != null) {
             s_currentContext.remove();
-            LOGGER.trace("Unregistered: {}", () -> context);
+            LOGGER.trace("Unregistered: {}", context);
         }
         ThreadContext.clearMap();
     }