Skip to content

[AMBARI-22805] Improve Blueprints error handling in case of timeout#179

Merged
adoroszlai merged 2 commits intoapache:branch-2.6from
adoroszlai:AMBARI-22805_timeout_branch-2.6
Jan 24, 2018
Merged

[AMBARI-22805] Improve Blueprints error handling in case of timeout#179
adoroszlai merged 2 commits intoapache:branch-2.6from
adoroszlai:AMBARI-22805_timeout_branch-2.6

Conversation

@adoroszlai
Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

This is a small addendum for #146 to improve behavior in case of timeout:

  • pass last seen error if present to callback, instead of TimeoutException
  • add message to the exception thrown when not all required hosts have joined the cluster
  • subtract retry delay from time left for subsequent attempt (since time left is calculated before scheduling with the delay)

How was this patch tested?

Tested cluster provisioning failure with various timeouts. Added unit test.

@asfgit
Copy link
Copy Markdown

asfgit commented Jan 24, 2018

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/256/
Test PASSed.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants