Skip to content

Don't update reason for disabled tape in cleaner

In CleanerSession, a tape is disabled if the cleaner fails to clean the drive with a mounted tape. However, the tape's state should not be changed if it is already in the REPACKING_DISABLED state, because it will override previous disabledReason. We want to keep the initial disabling message. Therefore, we should remove the corresponding condition:

        if (curr_state == Tape::REPACKING || curr_state == Tape::REPACKING_DISABLED) {
          m_catalogue.Tape()->modifyTapeState(admin, m_vid, Tape::REPACKING_DISABLED, curr_state, disabledReason);
        }