loadtest-controller: improve logging

Author: ivchicano

loadtest-controller/src/main/java/io/openvidu/loadtest/monitoring/ElasticSearchClient.java

@@ -83,11 +83,11 @@ public void init() {
                 log.info("Connection to Elasticsearch established at {}", elasticsearchHost);
             }
         } catch (Exception e) {
-            throw new LoadTestInitializationException(
-                    "Connection to Elasticsearch failed at " + loadTestConfig.getElasticsearchHost()
-                            + " (" + e.getMessage()
-                            + "). If property 'ELASTICSEARCH_HOST' is defined, then it is mandatory that OpenVidu Load Test is able to connect to it",
-                    e);
+            String message = "Connection to Elasticsearch failed at " + loadTestConfig.getElasticsearchHost()
+                    + " (" + e.getMessage()
+                    + "). If property 'ELASTICSEARCH_HOST' is defined, then it is mandatory that OpenVidu Load Test is able to connect to it";
+            log.error(message);
+            throw new LoadTestInitializationException(message, e);
         }
     }
 

loadtest-controller/src/main/java/io/openvidu/loadtest/services/BrowserEmulatorClient.java

@@ -139,7 +139,11 @@ public void ping(String workerUrl) {
             Thread.currentThread().interrupt();
             log.error("Ping interrupted: {}", e.getMessage());
         } catch (Exception e) {
-            log.error(e.getMessage());
+            if (e.getMessage() != null) {
+                log.error(e.getMessage());
+            } else {
+                log.debug("Error doing ping", e);
+            }
             log.error("Error doing ping. Retry...");
             ping(workerUrl);
         }