-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Revert "[Dashboard][event] Basic event module" #17068
Conversation
This reverts commit f1faa79.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1
@amogkam Where can i find the build error log? |
You can see it by clicking on the bug icon on the flaky test tracker https://flakey-tests.ray.io/ |
There are many errors in the build log, but none of them related to this PR. P.S. many failed builds have no output.
|
@amogkam I found some worker crashes in this build log: https://travis-ci.com/github/ray-project/ray/jobs/524307882
I am not sure if it is the root cause of the failure. Some other errors are caused by missing
The build log also shows the redis failure:
|
@amogkam @zhe-thoughts The |
@amogkam I noticed that the mac wheel build still fails after the revert. I think it's better to show the error details in the build log, so that we can fix it quickly. Currently, I don't know which error caused the build to fail. |
You're right @fyrestone this isn't causing the failures. We can merge back in. |
Thanks. |
…)" (#17107) This reverts commit c17e171. Co-authored-by: 刘宝 <[email protected]>
Reverts #16985
Sorry I have to revert this again @fyrestone. This is causing Mac Wheel Build to fail