[QC-619] Force the facility name to be less than 32 characters (#771)

* QC-619 - Force the facility name to be less than 32 characters

* Avoid reinventing the wheel.

* format

* format

* format

Author: Barthelemy

Framework/include/QualityControl/CheckRunner.h

@@ -121,6 +121,7 @@ class CheckRunner : public framework::Task
   static std::string createCheckRunnerIdString() { return "QC-CHECK-RUNNER"; };
   static std::string createCheckRunnerName(std::vector<Check> checks);
   static std::string createSinkCheckRunnerName(o2::framework::InputSpec input);
+  static std::string createCheckRunnerFacility(std::string deviceName);
 
  private:
   /**

Framework/src/CheckRunner.cxx

@@ -108,6 +108,14 @@ std::string CheckRunner::createCheckRunnerName(std::vector<Check> checks)
   return name;
 }
 
+std::string CheckRunner::createCheckRunnerFacility(std::string deviceName)
+{
+  // it starts with "check/" and is followed by the unique part of the device name truncated to a maximum of 32 characters.f
+  string facilityName = "check/" + deviceName.substr(CheckRunner::createCheckRunnerIdString().length() + 1, string::npos);
+  facilityName = facilityName.substr(0, 32);
+  return facilityName;
+}
+
 std::string CheckRunner::createSinkCheckRunnerName(InputSpec input)
 {
   std::string name(CheckRunner::createCheckRunnerIdString() + "-sink-");
@@ -186,7 +194,7 @@ void CheckRunner::init(framework::InitContext& iCtx)
   }
 
   try {
-    ILOG_INST.init("check/" + mDeviceName, mConfigFile->getRecursive(), ilContext);
+    ILOG_INST.init(createCheckRunnerFacility(mDeviceName), mConfigFile->getRecursive(), ilContext);
     initDatabase();
     initMonitoring();
     initServiceDiscovery();

Framework/test/testCheckRunner.cxx

@@ -33,4 +33,9 @@ BOOST_AUTO_TEST_CASE(test_check_runner_static)
   BOOST_CHECK(CheckRunner::createCheckRunnerDataDescription("qwertyuiop") == DataDescription("qwertyuiop-chk"));
   BOOST_CHECK(CheckRunner::createCheckRunnerDataDescription("012345678901234567890") == DataDescription("012345678901-chk"));
   BOOST_CHECK_THROW(CheckRunner::createCheckRunnerDataDescription(""), AliceO2::Common::FatalException);
+
+  // facility name
+  BOOST_CHECK(CheckRunner::createCheckRunnerFacility(CheckRunner::createCheckRunnerIdString() + "-test") == "check/test");
+  BOOST_CHECK(CheckRunner::createCheckRunnerFacility(CheckRunner::createCheckRunnerIdString() + "-abcdefghijklmnopqrstuvwxyz") == "check/abcdefghijklmnopqrstuvwxyz");
+  BOOST_CHECK(CheckRunner::createCheckRunnerFacility(CheckRunner::createCheckRunnerIdString() + "-abcdefghijklmnopqrstuvwxyz123456789") == "check/abcdefghijklmnopqrstuvwxyz");
 }