search

docker / app

Make your Docker Compose applications reusable, and share them on Docker Hub
Apache License 2.0
1.57k stars 177 forks source link

Add debug logs #590

Closed rumpl closed 5 years ago

rumpl commented 5 years ago

- What I did

This is WIP because I want to update the cnab-to-oci dependency once this PR is merged

Fixed some logs that didn't log the error buffer from the invocation image.

- How I did it

- How to verify it

Run docker --log-level debug app <COMMAND>

- A picture of a cute animal (not mandatory but encouraged) image

codecov[bot] commented 5 years ago

Codecov Report

Merging #590 into master will increase coverage by 0.83%. The diff coverage is 40%.

Impacted file tree graph

@@           Coverage Diff            @@
##           master   #590      +/-   ##
========================================
+ Coverage   71.16%    72%   +0.83%     
========================================
  Files          53     54       +1     
  Lines        2802   2975     +173     
========================================
+ Hits         1994   2142     +148     
- Misses        552    569      +17     
- Partials      256    264       +8
Impacted Files Coverage Δ
internal/commands/bundle.go 67.2% <40%> (+4.7%) :arrow_up:
internal/log/log.go 100% <0%> (ø)
internal/packager/cnab.go 98.25% <0%> (+0.88%) :arrow_up:
internal/store/bundle.go 72.13% <0%> (+3.24%) :arrow_up:
internal/commands/push.go 50.37% <0%> (+4.14%) :arrow_up:
internal/commands/install.go 77.17% <0%> (+11.45%) :arrow_up:
internal/packager/packing.go 40% <0%> (+12.04%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 49ae99f...21daac1. Read the comment docs.