New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Output logs from s3-npm-publish Docker container #318

Merged
merged 1 commit into from Mar 29, 2017

Conversation

Projects
None yet
3 participants
@robertknight
Contributor

robertknight commented Mar 28, 2017

To help debug publishing failures, output logs from the container. See
https://hypothes-is.slack.com/archives/C076LQFA4/p1490705667081734

See https://go.cloudbees.com/docs/cloudbees-documentation/cje-user-guide/index.html#docker-workflow-sect-run for docs on the Jenkins plugin that we use to run Docker. Note that unlike the example they give, I use --follow to make sure that we output all of the logs until the task exits.

Output logs from s3-npm-publish Docker container
To help debug publishing failures, output logs from the container.  See
https://hypothes-is.slack.com/archives/C076LQFA4/p1490705667081734
@codecov-io

This comment has been minimized.

Show comment
Hide comment
@codecov-io

codecov-io Mar 28, 2017

Codecov Report

Merging #318 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           master    #318   +/-   ##
======================================
  Coverage    76.6%   76.6%           
======================================
  Files         120     120           
  Lines        5887    5887           
  Branches      959     959           
======================================
  Hits         4510    4510           
  Misses       1377    1377

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 54519c6...2a7a425. Read the comment docs.

codecov-io commented Mar 28, 2017

Codecov Report

Merging #318 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           master    #318   +/-   ##
======================================
  Coverage    76.6%   76.6%           
======================================
  Files         120     120           
  Lines        5887    5887           
  Branches      959     959           
======================================
  Hits         4510    4510           
  Misses       1377    1377

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 54519c6...2a7a425. Read the comment docs.

@chdorner

If the Jenkins build output is what you expect, then please go ahead an merge.

@robertknight robertknight merged commit 55cc1f0 into master Mar 29, 2017

5 checks passed

codecov/patch Coverage not affected when comparing 54519c6...2a7a425
Details
codecov/project 76.6% remains the same compared to 54519c6
Details
continuous-integration/jenkins/branch This commit looks good
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@robertknight robertknight deleted the docker-publish-logs branch Mar 29, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment