Skip to content

Deflake cluster heartbeat test#5552

Merged
ericl merged 6 commits intoray-project:masterfrom
ericl:deflake-cluster
Sep 11, 2019
Merged

Deflake cluster heartbeat test#5552
ericl merged 6 commits intoray-project:masterfrom
ericl:deflake-cluster

Conversation

@ericl
Copy link
Copy Markdown
Contributor

@ericl ericl commented Aug 27, 2019

Why are these changes needed?

Related issue number

Linter

  • I've run scripts/format.sh to lint the changes in this PR.

@AmplabJenkins
Copy link
Copy Markdown

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Ray-PRB/16576/
Test FAILed.

@AmplabJenkins
Copy link
Copy Markdown

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Ray-PRB/16574/
Test PASSed.

@AmplabJenkins
Copy link
Copy Markdown

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Ray-PRB/16578/
Test PASSed.

@ericl ericl changed the title [WIP] deflake cluster heartbeat test Deflake cluster heartbeat test Aug 30, 2019
@ericl ericl added tests-ok The tagger certifies test failures are unrelated and assumes personal liability. and removed tests-ok The tagger certifies test failures are unrelated and assumes personal liability. labels Aug 30, 2019
@ericl
Copy link
Copy Markdown
Contributor Author

ericl commented Aug 30, 2019

            raise ValueError("Timeout. {} != {}".format(
              resource_usage, expected_resource_usage))

E ValueError: Timeout. (3.0, {'CPU': 3.0}, {'CPU': 4.0}) != (4, {'CPU': 4}, {'CPU': 4})

@AmplabJenkins
Copy link
Copy Markdown

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Ray-PRB/16959/
Test PASSed.

@ericl ericl merged commit faeaa34 into ray-project:master Sep 11, 2019
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.

3 participants