From 65a70b899d4d4624a7c3391c636c48204a60b571 Mon Sep 17 00:00:00 2001
From: Alexander Alexandrov <alexander.s.alexandrov@gmail.com>
Date: Fri, 9 Sep 2016 14:06:07 +0200
Subject: [PATCH] Moved `hostname` to `config` package.

---
 .../org/peelframework/core/cli/Peel.scala      |  8 +-------
 .../peelframework/core/config/package.scala    | 18 ++++++++++++------
 2 files changed, 13 insertions(+), 13 deletions(-)

diff --git a/peel-core/src/main/scala/org/peelframework/core/cli/Peel.scala b/peel-core/src/main/scala/org/peelframework/core/cli/Peel.scala
index bc9c4a59..f34ad6bd 100644
--- a/peel-core/src/main/scala/org/peelframework/core/cli/Peel.scala
+++ b/peel-core/src/main/scala/org/peelframework/core/cli/Peel.scala
@@ -24,11 +24,10 @@ import net.sourceforge.argparse4j.internal.HelpScreenException
 import org.apache.log4j.{Level, PatternLayout, RollingFileAppender}
 import org.peelframework.core.PeelApplicationContext
 import org.peelframework.core.cli.command.Command
-import org.peelframework.core.util.shell
+import org.peelframework.core.config.hostname
 import org.slf4j.{Logger, LoggerFactory}
 
 import scala.collection.JavaConversions._
-import scala.sys.process.Process
 
 object Peel {
 
@@ -164,11 +163,6 @@ object Peel {
     parser
   }
 
-  lazy val hostname = {
-    val name = Process("/bin/bash", Seq("-c", "CLASSPATH=;echo $HOSTNAME")).!!
-    if (name.nonEmpty) name.trim else "localhost"
-  }
-
   def printHeader(logger: Logger) {
     logger.info("############################################################")
     logger.info("#           PEEL EXPERIMENTS EXECUTION FRAMEWORK           #")
diff --git a/peel-core/src/main/scala/org/peelframework/core/config/package.scala b/peel-core/src/main/scala/org/peelframework/core/config/package.scala
index adfb58e6..bcbc67db 100644
--- a/peel-core/src/main/scala/org/peelframework/core/config/package.scala
+++ b/peel-core/src/main/scala/org/peelframework/core/config/package.scala
@@ -24,9 +24,10 @@ import org.peelframework.core.beans.experiment.Experiment
 import org.peelframework.core.beans.system.System
 import org.peelframework.core.graph.{DependencyGraph, Node}
 import org.peelframework.core.util.console.ConsoleColorise
-import org.peelframework.core.util.shell
 import org.slf4j.LoggerFactory
 
+import scala.sys.process.Process
+
 /** Package Object to handle the loading of configuration files
   *
   * Configuration files are loaded and resolved according to the
@@ -71,7 +72,7 @@ package object config {
 
     // load current runtime config
     logger.info(s"+-- Loading current runtime values as configuration")
-    cb.append(currentRuntimeConfig())
+    cb.append(currentRuntimeConfig)
 
     // load system properties
     logger.info(s"+-- Loading system properties as configuration")
@@ -122,7 +123,7 @@ package object config {
 
     // load current runtime config
     logger.info(s"+-- Loading current runtime values as configuration")
-    cb.append(currentRuntimeConfig())
+    cb.append(currentRuntimeConfig)
 
     // load system properties
     logger.info(s"+-- Loading system properties as configuration")
@@ -176,7 +177,7 @@ package object config {
 
     // load current runtime config
     logger.info(s"+-- Loading current runtime values as configuration")
-    cb.append(currentRuntimeConfig())
+    cb.append(currentRuntimeConfig)
 
     // load system properties
     logger.info(s"+-- Loading system properties as configuration")
@@ -204,18 +205,23 @@ package object config {
     *
     * @return current Config Object
     */
-  private def currentRuntimeConfig() = {
+  lazy val currentRuntimeConfig = {
     // initial empty configuration
     val runtimeConfig = new java.util.HashMap[String, Object]()
     // add current runtime values to configuration
     runtimeConfig.put("runtime.cpu.cores", Runtime.getRuntime.availableProcessors().asInstanceOf[Object])
     runtimeConfig.put("runtime.memory.max", Runtime.getRuntime.maxMemory().asInstanceOf[Object])
-    runtimeConfig.put("runtime.hostname", (shell !! "echo $HOSTNAME").trim())
+    runtimeConfig.put("runtime.hostname", hostname)
     runtimeConfig.put("runtime.disk.size", new File("/").getTotalSpace.asInstanceOf[Object])
     // return a config object
     ConfigFactory.parseMap(runtimeConfig)
   }
 
+  lazy val hostname = {
+    val name = Process("/bin/bash", Seq("-c", "CLASSPATH=;echo $HOSTNAME")).!!
+    if (name.nonEmpty) name.trim else "localhost"
+  }
+
   private class ConfigBuilder {
 
     // initial empty configuration