Fixed Info_tapeFilledUp tape log parameter potentially appearing with a count greater that 1.

castor/tape/tapeserver/daemon/TapeWriteSingleThread.cpp

@@ -256,7 +256,7 @@ void castor::tape::tapeserver::daemon::TapeWriteSingleThread::run() {
         throw 0;
       }
       // This is indeed the end of the tape. Not an error.
-      m_watchdog.addToErrorCount("Info_tapeFilledUp");
+      m_watchdog.setErrorCount("Info_tapeFilledUp",1);
     } catch (...) {
       // The error is not an ENOSPC, so it is, indeed, an error.
       // If we got here with a new error, currentErrorToCount will be non-empty,

castor/tape/tapeserver/daemon/TapeWriteTask.cpp

@@ -167,7 +167,7 @@ namespace daemon {
           throw 0;
         }
         // This is indeed the end of the tape. Not an error.
-        watchdog.addToErrorCount("Info_tapeFilledUp");
+        watchdog.setErrorCount("Info_tapeFilledUp",1);
       } catch (...) {
         // The error is not an ENOSPC, so it is, indeed, an error.
         // If we got here with a new error, currentErrorToCount will be non-empty,

castor/tape/tapeserver/daemon/TaskWatchDog.hpp

@@ -320,6 +320,19 @@ protected:
     addParameter(log::Param(errorName, count));
   }
   
+  /**
+   * Set error count. This is used for once per session events that could
+   * be detected several times.
+   */
+  void setErrorCount (const std::string & errorName, uint32_t value) {
+    {
+      castor::server::MutexLocker locker(&m_mutex);
+      m_errorCounts[errorName] = value;
+    }
+    // We ship the new value ASAP to the main thread.
+    addParameter(log::Param(errorName, value));
+  }
+  
   /**
    * Start the thread
    */