Disable any debug logs by defaults (#231)

Author: awegrzyn

src/Backends/ApMonBackend.cxx

@@ -39,7 +39,7 @@ ApMonBackend::ApMonBackend(const std::string& path)
 {
   try {
     mApMon = std::make_unique<ApMon>(const_cast<char*>(path.c_str()));
-    MonLogger::Get() << "ApMon backend initialized" << MonLogger::End();
+    MonLogger::Get(Severity::Info) << "ApMon backend initialized" << MonLogger::End();
   } catch (std::runtime_error& e) {
     throw MonitoringException("ApMonBackend initialization", std::string(e.what()));
   }

src/Backends/InfluxDB.cxx

@@ -39,7 +39,7 @@ overloaded(Ts...)->overloaded<Ts...>;
 InfluxDB::InfluxDB(std::unique_ptr<transports::TransportInterface> transport)
   : mTransport(std::move(transport))
 {
-  MonLogger::Get() << "InfluxDB backend initialized" << MonLogger::End();
+  MonLogger::Get(Severity::Info) << "InfluxDB backend initialized" << MonLogger::End();
 }
 
 inline unsigned long InfluxDB::convertTimestamp(const std::chrono::time_point<std::chrono::system_clock>& timestamp)

src/Backends/StdOut.cxx

@@ -43,7 +43,7 @@ inline unsigned long StdOut::convertTimestamp(const std::chrono::time_point<std:
 StdOut::StdOut(const std::string& prefix) : mPrefix(prefix)
 {
   setVerbosisty(Verbosity::Debug);
-  MonLogger::Get() << "StdOut backend initialized" << MonLogger::End();
+  MonLogger::Get(Severity::Info) << "StdOut backend initialized" << MonLogger::End();
 }
 
 void StdOut::addGlobalTag(std::string_view name, std::string_view value)

src/MonLogger.h

@@ -29,7 +29,8 @@ namespace monitoring
 /// List of possible log severities
 enum class Severity { Error = 31,
                       Warn = 33,
-                      Info = 49 };
+                      Info = 49,
+                      Debug = 50};
 
 /// Simple Monitoring logging class
 class MonLogger
@@ -41,19 +42,21 @@ class MonLogger
   template <typename T>
   MonLogger& operator<<(const T& log)
   {
-    mStream << log;
+    if (MonLogger::globalSeverity >= severity) {
+      mStream << log;
+    }
     return *this;
   }
 
   /// Singleton
   /// Returns Logger instance with current date and given severity
   /// \param severity - severity level
   /// \return - logger instance
-  static MonLogger& Get(Severity severity = Severity::Info)
+  static MonLogger& Get(Severity severity = Severity::Debug)
   {
     static MonLogger loggerInstance;
+    loggerInstance.instanceSeverity(severity);
     loggerInstance.setDate();
-    loggerInstance.setSeverity(severity);
     return loggerInstance;
   }
 
@@ -64,7 +67,13 @@ class MonLogger
     return "\033[0m\n";
   }
 
+  /// Currently set severity
+  const Severity globalSeverity = Severity::Info;
+
  private:
+  /// Instance severity
+  Severity severity;
+
   /// Log stream
   std::ostream& mStream;
 
@@ -77,14 +86,15 @@ class MonLogger
   void setDate()
   {
     auto now = std::chrono::system_clock::to_time_t(std::chrono::system_clock::now());
-    mStream << std::put_time(std::localtime(&now), "%Y-%m-%d %X") << "\t" << "Monitoring" << "\t";
+    *this << std::put_time(std::localtime(&now), "%Y-%m-%d %X") << "\t" << "Monitoring" << "\t";
   }
 
   /// Sets log color based on severity
   /// \param severity - log severity
-  void setSeverity(Severity severity)
+  void instanceSeverity(Severity desiredSeverity)
   {
-    mStream << "\033[0;" << static_cast<int>(severity) << "m";
+    severity = desiredSeverity;
+    *this << "\033[0;" << static_cast<int>(severity) << "m";
   }
 
   /// Delete copy and move constructors

src/Monitoring.cxx

@@ -90,7 +90,7 @@ void Monitoring::enableProcessMonitoring(const unsigned int interval)
 #ifdef O2_MONITORING_OS_LINUX
   MonLogger::Get() << "Process Monitor : Automatic updates enabled" << MonLogger::End();
 #else
-  MonLogger::Get() << "!! Process Monitor : Limited metrics available" << MonLogger::End();
+  MonLogger::Get() << "Process Monitor : Limited metrics available" << MonLogger::End();
 #endif
 }
 

src/ProcessMonitor.cxx

@@ -102,7 +102,7 @@ std::vector<Metric> ProcessMonitor::getCpuAndContexts()
   auto timeNow = std::chrono::high_resolution_clock::now();
   double timePassed = std::chrono::duration_cast<std::chrono::microseconds>(timeNow - mTimeLastRun).count();
   if (timePassed < 950) {
-    MonLogger::Get() << "[WARN] Do not invoke Process Monitor more frequent then every 1s" << MonLogger::End();
+    MonLogger::Get(Severity::Warn) << "Do not invoke Process Monitor more frequent then every 1s" << MonLogger::End();
     metrics.emplace_back("processPerformance");
     return metrics;
   }