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

Missing/invalid schedulers generate many exceptions #2187

Open
cg2v opened this issue Sep 3, 2024 · 0 comments · May be fixed by #2214
Open

Missing/invalid schedulers generate many exceptions #2187

cg2v opened this issue Sep 3, 2024 · 0 comments · May be fixed by #2214

Comments

@cg2v
Copy link
Member

cg2v commented Sep 3, 2024

Your environment
Let us know details such as

Steps To Reproduce
Steps to reproduce the behavior:

  1. Define a new scheduler job with the action field containing a value that does not match an existing filename
  2. Save it
  3. Watch the exceptions start to be generated every [INTERVAL]

Screenshots

Date: Tue, 03 Sep 2024 07:08:08 -0400
Subject: [CMU-Autolab-Prod Error]  (LoadError) "cannot load such file -- /opt/autolab/courses/[REDACTED].r...
A LoadError occurred in background at 2024-09-03 07:08:08 -0400 :
-------------------------------
Backtrace:
-------------------------------

  /opt/autolab/app/controllers/application_controller.rb:281:in `require'
  /opt/autolab/app/controllers/application_controller.rb:281:in `block (3 levels) in run_scheduler'

Suggestions

  • Validations could require the action to be an existing file
  • The controller could require that a successful visual run occur before the job can be enabled
  • The scheduler could auto-disable or do exponential back-off for failing jobs.
    cannot load such file -- /opt/autolab/courses/[REDACTED].rb
    /opt/autolab/app/controllers/application_controller.rb:281:in `require'
  • if nothing else, scheduler errors that are likely to be course errors could be caught and logged but not passed to exception notifier
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants