Commit 28c7456a authored by Victor Kotlyar's avatar Victor Kotlyar
Browse files

Ported commits from castor/master:

da054c343578da49fad9792c32a9ed3068c6b6f8
  Removed redundant setting of the TPVID paramter while mounting the
  tape. This had the nasty side effect of removing the parameter after
  leaving the functions (for all the following logs).

a354b3ee7996494fb78fb0337e91cb567f0ae8f0
  CASTOR-5323 Tapeserver logs inconsistencies

  Removed duplicate setting of a scoped TPVID parameter in the logs.
  When going out of scope, the parameter
  made TPVID disappear from following logs, which is not desired.
parent 2c2f4a27
......@@ -112,8 +112,7 @@ protected:
*/
void mountTapeReadOnly(){
cta::log::ScopedParamContainer scoped(m_logContext);
scoped.add("TPVID",m_volInfo.vid)
.add("drive_Slot",m_drive.config.getLibrarySlot().str());
scoped.add("drive_Slot",m_drive.config.getLibrarySlot().str());
try {
cta::utils::Timer timer;
m_mc.mountTapeReadOnly(m_volInfo.vid, m_drive.config.getLibrarySlot());
......@@ -137,8 +136,7 @@ protected:
*/
void mountTapeReadWrite(){
cta::log::ScopedParamContainer scoped(m_logContext);
scoped.add("TPVID",m_volInfo.vid)
.add("drive_Slot",m_drive.config.getLibrarySlot().str());
scoped.add("drive_Slot",m_drive.config.getLibrarySlot().str());
try {
cta::utils::Timer timer;
m_mc.mountTapeReadWrite(m_volInfo.vid, m_drive.config.getLibrarySlot());
......
......@@ -66,7 +66,6 @@ castor::tape::tapeserver::daemon::TapeWriteSingleThread::openWriteSession() {
std::unique_ptr<castor::tape::tapeFile::WriteSession> writeSession;
ScopedParam sp[]={
ScopedParam(m_logContext, Param("TPVID",m_vid)),
ScopedParam(m_logContext, Param("lastFseq", m_lastFseq)),
ScopedParam(m_logContext, Param("compression", m_compress)),
ScopedParam(m_logContext, Param("useLbp", m_useLbp))
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment