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.

Sign up for GitHub

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

Jump to bottom

Add a missing eval trigger reason #4712

Merged
merged 3 commits into from
Sep 25, 2018
Merged

Add a missing eval trigger reason #4712

merged 3 commits into from
Sep 25, 2018

Commits on Sep 25, 2018

  1. Added logging around nacked evals in the scheduler worker

    @preetapan @dadgar
    preetapan authored and dadgar committed Sep 25, 2018
    Configuration menu
    Copy the full SHA
    e9c7dc1 View commit details
    Browse the repository at this point in the history

  2. Add failed follow up to the list of allowed eval trigger reasons 

    needs unit test
    @preetapan @dadgar
    preetapan authored and dadgar committed Sep 25, 2018
    Configuration menu
    Copy the full SHA
    47e22f6 View commit details
    Browse the repository at this point in the history

  3. fix logging

    @dadgar
    dadgar committed Sep 25, 2018
    Configuration menu
    Copy the full SHA
    b3e8555 View commit details
    Browse the repository at this point in the history