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

Adds pool name to several log messages #1217

Merged
merged 2 commits into from Sep 24, 2019

Conversation

@dposada
Copy link
Member

commented Sep 23, 2019

Changes proposed in this PR

  • adding the pool name to several logs in scheduler.clj

Why are we making these changes?

To make troubleshooting scheduling issues easier in multi-pool environments.

@dposada dposada added the pools label Sep 23, 2019
@dposada dposada self-assigned this Sep 23, 2019
@scrosby scrosby closed this Sep 24, 2019
@scrosby scrosby reopened this Sep 24, 2019
@twosigma twosigma deleted a comment from scrosby Sep 24, 2019
@dposada dposada requested a review from nsinkov Sep 24, 2019
@nsinkov nsinkov merged commit 5c2b367 into master Sep 24, 2019
2 checks passed
2 checks passed
Mergeable Mergeable Run has been Completed!
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@nsinkov nsinkov deleted the pool-logs branch Sep 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.