Clean CLI config logging
ClosedPublic
Actions

Authored by haltode on Nov 26 2020, 10:55 AM.

Details

Reviewers

Group Reviewers

Commits

rDFUSE7c694bdcfe78: cli: add traceback in exception logging
rDFUSEf03fa96ce9bc: cli: remove redundante logging
rDFUSE3340589c04e9: cli: only show loaded config in debug and use yaml.dump()
rDFUSE1740ecca8a36: requirements: add missing explicit yaml dependency

Summary

requirements: add missing explicit yaml dependency
cli: only show loaded config in debug and use yaml.dump()
cli: remove redundante logging
cli: add traceback in exception logging

Diff Detail

Repository

rDFUSE FUSE virtual file system

Branch

feature/config-debug-print

Lint

No Linters Available

Unit

No Unit Test Coverage

Build Status

Buildable 17521
Build 27080: Phabricator diff pipeline on jenkins	Jenkins console · Jenkins
Build 27079: arc lint + arc unit

Event Timeline

haltode created this revision.Nov 26 2020, 10:55 AM

Herald added a reviewer: Reviewers. · View Herald TranscriptNov 26 2020, 10:55 AM

zack accepted this revision.Nov 26 2020, 10:56 AM

This revision is now accepted and ready to land.Nov 26 2020, 10:56 AM

Build is green

Patch application report for D4596 (id=16328)

Rebasing onto b0c3d77e7d...

Current branch diff-target is up to date.

Changes applied before test

commit 7c694bdcfe7852ef8e79133a24c153ed2a57dcfd
Author: Thibault Allançon <haltode@gmail.com>
Date:   Thu Nov 26 10:53:17 2020 +0100

    cli: add traceback in exception logging

commit f03fa96ce9bc8e56415ab5eb4af75f15fd9f052a
Author: Thibault Allançon <haltode@gmail.com>
Date:   Thu Nov 26 10:52:56 2020 +0100

    cli: remove redundante logging

commit 3340589c04e9fdf22de8e11525f8ca018a35affb
Author: Thibault Allançon <haltode@gmail.com>
Date:   Thu Nov 26 10:23:10 2020 +0100

    cli: only show loaded config in debug and use yaml.dump()

commit 1740ecca8a36e93863329181b9615e8606b714ad
Author: Thibault Allançon <haltode@gmail.com>
Date:   Thu Nov 26 10:22:48 2020 +0100

    requirements: add missing explicit yaml dependency

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

Harbormaster completed remote builds in B17521: Diff 16328.Nov 26 2020, 10:58 AM

Closed by commit rDFUSE1740ecca8a36: requirements: add missing explicit yaml dependency (authored by haltode). · Explain WhyNov 26 2020, 10:58 AM

This revision was automatically updated to reflect the committed changes.

haltode added a commit: rDFUSE1740ecca8a36: requirements: add missing explicit yaml dependency.

haltode added a commit: rDFUSE3340589c04e9: cli: only show loaded config in debug and use yaml.dump().

haltode added a commit: rDFUSEf03fa96ce9bc: cli: remove redundante logging.

haltode added a commit: rDFUSE7c694bdcfe78: cli: add traceback in exception logging.

zack mentioned this in T2811: FUSE: fix various paper cuts (user testing 2020-11-24).Nov 27 2020, 12:23 PM