Skip to content
Snippets Groups Projects
Commit c469879a authored by Eric Cano's avatar Eric Cano
Browse files

Fixed missing handling of cleanup errors in ArchiveMount::getNextJobBatch().

A benign exception failed to be caught and led to unnecessary errors.
parent d7861a81
No related branches found
No related tags found
No related merge requests found
...@@ -1828,6 +1828,12 @@ std::list<std::unique_ptr<SchedulerDatabase::ArchiveJob> > OStoreDB::ArchiveMoun ...@@ -1828,6 +1828,12 @@ std::list<std::unique_ptr<SchedulerDatabase::ArchiveJob> > OStoreDB::ArchiveMoun
.add("queueObject", aq.getAddressIfSet()) .add("queueObject", aq.getAddressIfSet())
.add("Message", ex.getMessageValue()); .add("Message", ex.getMessageValue());
logContext.log(log::INFO, "In ArchiveMount::getNextJobBatch(): could not de-referenced missing queue from root entry"); logContext.log(log::INFO, "In ArchiveMount::getNextJobBatch(): could not de-referenced missing queue from root entry");
} catch (RootEntry::NoSuchArchiveQueue & ex) {
// Somebody removed the queue in the mean time. Barely worth mentioning.
log::ScopedParamContainer params(logContext);
params.add("tapepool", mountInfo.tapePool)
.add("queueObject", aq.getAddressIfSet());
logContext.log(log::DEBUG, "In ArchiveMount::getNextJobBatch(): could not de-referenced missing queue from root entry: already done.");
} }
emptyQueueCleanupTime += t.secs(utils::Timer::resetCounter); emptyQueueCleanupTime += t.secs(utils::Timer::resetCounter);
continue; continue;
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment