Skip to content
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.

Sign up for GitHub

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

Jump to bottom

CI does not report critical failures #45

Closed
fuchsto opened this issue Nov 2, 2016 · 1 comment · Fixed by #47
Closed

CI does not report critical failures #45

fuchsto opened this issue Nov 2, 2016 · 1 comment · Fixed by #47
Assignees
@fmoessbauer
Labels
bug
Milestone
dash-0.3.0

Comments

@fuchsto
Copy link
Member

fuchsto commented Nov 2, 2016

See log output of several recent CI tests, for example:
https://travis-ci.org/dash-project/dash/builds/171929653

Critical failures are recognized by the CI script but overall result is "PASSED".

Also, there is currently a corrupt free in dart-mpi/dart_segment.
@fuchsto fuchsto added the bug label Nov 2, 2016
@fuchsto fuchsto added this to the dash-0.3.0 milestone Nov 2, 2016
@fuchsto fuchsto mentioned this issue Nov 2, 2016
Closed
fmoessbauer added a commit that referenced this issue Nov 2, 2016
@fmoessbauer
validate ci logs outside docker container. See #45 for details
fd5ff20
@fmoessbauer fmoessbauer mentioned this issue Nov 2, 2016
Merged
@fmoessbauer
Copy link
Member

Fixed in PR #47 . Can be closed after merge.

fuchsto pushed a commit that referenced this issue Nov 3, 2016
@fmoessbauer @fuchsto
validate ci logs outside docker container. See #45 for details
ed170bc
fuchsto pushed a commit that referenced this issue Nov 3, 2016
@fmoessbauer @fuchsto
Bug 45 travisci (#47)
893f504 
* validate ci logs outside docker container. See #45 for details

* added missing travisci CI script
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@fmoessbauer fmoessbauer

Labels
bug
Projects
None yet
Milestone
dash-0.3.0
Development

Successfully merging a pull request may close this issue.

Bug 45 travisci dash-project/dash
2 participants
@fuchsto @fmoessbauer