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.

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

Add error handling #70

Merged
merged 1 commit into from
Jul 18, 2016
Merged

Add error handling #70

merged 1 commit into from
Jul 18, 2016

Conversation

mestery
Copy link
Collaborator

@mestery mestery commented Jul 15, 2016

Add some initial error handling into the CI scripts

Closes Issue #69

Signed-off-by: Kyle Mestery [email protected]

@mestery
Copy link
Collaborator Author

mestery commented Jul 15, 2016

This needs a bit more work, but it's a start. Seeing how the travis-ci job run looks now.

@@ -4,6 +4,9 @@
XTRACE=$(set +o | grep xtrace)
set -o xtrace

# Track failures
FAILED=0
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this variable used to indicate how many tests the CI job fails?

Copy link
Collaborator Author

@mestery mestery Jul 15, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's mainly used to detect if a specific command in docker-rally.sh fails. If run fails, we want to ensure we run cleanup, but we need to remember run failed so we can return that to travis-ci. Make sense?

Add some initial error handling into the CI scripts

Closes Issue ovn-org#69

Signed-off-by: Kyle Mestery <[email protected]>
@huikang
Copy link
Collaborator

huikang commented Jul 16, 2016

@mestery is the PR ready for review or more to add? Thanks.

@mestery
Copy link
Collaborator Author

mestery commented Jul 17, 2016

@huikang I think so, I plan to add more error backout in the other scripts, but this can merge as-is now. Thanks!

@huikang huikang merged commit 03f1da1 into ovn-org:master Jul 18, 2016
@mestery mestery deleted the error-handling branch July 21, 2016 13:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants