Lowered log level where not useful.

Avoided counting an error for files hitting end of tape.

objectstore/BackendPopulator.cpp

@@ -70,7 +70,7 @@ BackendPopulator::~BackendPopulator() throw() {
       agent.commit();
       params.add("agentObject", agent.getAddressIfSet())
             .add("ownedObjectCount", agent.getOwnershipList().size());
-      m_lc.log(log::WARNING, "In BackendPopulator::~BackendPopulator(): not deleting non-empty agent object, left for garbage collection.");
+      m_lc.log(log::INFO, "In BackendPopulator::~BackendPopulator(): not deleting non-empty agent object, left for garbage collection.");
       return;
     }
     agent.removeAndUnregisterSelf(m_lc);

tapeserver/castor/tape/tapeserver/daemon/TapeWriteTask.cpp

@@ -236,19 +236,21 @@ namespace daemon {
       // We also change the log level to INFO for the case of end of tape.
       int errorCode = 666; // TODO - Remove error code
       int errorLevel = cta::log::ERR;
+      bool doReportJobError = true;
       try {
         const cta::exception::Errnum & errnum = 
             dynamic_cast<const cta::exception::Errnum &> (e);
         if (ENOSPC == errnum.errorNumber()) {
           errorCode = ENOSPC;
           errorLevel = cta::log::INFO;
+          doReportJobError = false;
         }
       } catch (...) {}
       LogContext::ScopedParam sp(lc, Param("exceptionCode",errorCode));
       LogContext::ScopedParam sp1(lc, Param("exceptionMessage", e.getMessageValue()));
       lc.log(errorLevel,"An error occurred for this file. End of migrations.");
       circulateMemBlocks();
-      reportPacker.reportFailedJob(std::move(m_archiveJob),e, lc);
+      if (doReportJobError) reportPacker.reportFailedJob(std::move(m_archiveJob),e, lc);
   
       //we throw again because we want TWST to stop all tasks from execution 
       //and go into a degraded mode operation.