Skip to content

Revert "[dashboard] Fix RAY_RAYLET_PID KeyError on Windows"#13572

Merged
edoakes merged 1 commit intomasterfrom
revert-12948-fix-dashboard-error
Jan 22, 2021
Merged

Revert "[dashboard] Fix RAY_RAYLET_PID KeyError on Windows"#13572
edoakes merged 1 commit intomasterfrom
revert-12948-fix-dashboard-error

Conversation

@amogkam
Copy link
Copy Markdown
Contributor

@amogkam amogkam commented Jan 20, 2021

Reverts #12948

As per @fyrestone on #13253, reverting this PR should fix #13199

Closes #13199

@amogkam
Copy link
Copy Markdown
Contributor Author

amogkam commented Jan 20, 2021

Hey @fyrestone here is a PR to revert- but am curious, is there a reason this revert and the fix in #13253 need to be done separately as opposed to in the same PR?

Also, I am not familiar with this code, so I will leave it to @edoakes to review and merge. Thanks!

@fyrestone
Copy link
Copy Markdown
Contributor

Hey @fyrestone here is a PR to revert- but am curious, is there a reason this revert and the fix in #13253 need to be done separately as opposed to in the same PR?

Also, I am not familiar with this code, so I will leave it to @edoakes to review and merge. Thanks!

Thanks. It's better to reverts the #12948 in a separate PR.

@amogkam
Copy link
Copy Markdown
Contributor Author

amogkam commented Jan 22, 2021

Hey bump on this @edoakes - it'll be great to get #13199 fixed

@edoakes edoakes merged commit c4a7103 into master Jan 22, 2021
@edoakes edoakes deleted the revert-12948-fix-dashboard-error branch January 22, 2021 20:10
amogkam added a commit that referenced this pull request Jan 25, 2021
rkooo567 pushed a commit that referenced this pull request Jan 25, 2021
fishbone pushed a commit to fishbone/ray that referenced this pull request Feb 16, 2021
fishbone added a commit to fishbone/ray that referenced this pull request Feb 16, 2021
fishbone added a commit to fishbone/ray that referenced this pull request Feb 16, 2021
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.

new_dashboard metrics agent crashed in Windows CI

3 participants