Page MenuHomeSoftware Heritage

fuse: use logging.exception() instead of .debug()
ClosedPublic

Authored by haltode on Nov 16 2020, 1:26 PM.

Details

Summary

During development of new features, enabling the DEBUG log-level every
time to hunt down a runtime exception is unpractical since the DEBUG
level is very verbose and does not include the exception traceback.
Using the logging.exception() method fixes all these issues.

Diff Detail

Repository
rDFUSE FUSE virtual file system
Branch
feature/logging-exception
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 17098
Build 26390: Phabricator diff pipeline on jenkinsJenkins console · Jenkins
Build 26389: arc lint + arc unit

Event Timeline

This revision is now accepted and ready to land.Nov 16 2020, 1:27 PM

Build is green

Patch application report for D4478 (id=15890)

Rebasing onto 604665ab40...

Current branch diff-target is up to date.
Changes applied before test
commit 1e09b9b1b2ca30cde4fc60f8d4742e9fb443e21a
Author: Thibault Allançon <haltode@gmail.com>
Date:   Mon Nov 16 13:19:02 2020 +0100

    fuse: use logging.exception() instead of .debug()
    
    During development of new features, enabling the DEBUG log-level every
    time to hunt down a runtime exception is unpractical since the DEBUG
    level is very verbose and does not include the exception traceback.
    Using the logging.exception() method fixes all these issues.

See https://jenkins.softwareheritage.org/job/DFUSE/job/tests-on-diff/152/ for more details.