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

Better error handling in sage-cleaner #21800

Closed
jdemeyer opened this issue Nov 2, 2016 · 6 comments
Closed

Better error handling in sage-cleaner #21800

jdemeyer opened this issue Nov 2, 2016 · 6 comments

Comments

@jdemeyer
Copy link

jdemeyer commented Nov 2, 2016

My sage-cleaner instance is randomly killing jobs. Reason:

Checking PIDs [18654]
Process 18654 is no longer running, so we clean up
Killing 18654's spawned jobs
--> Killing 'gp' with PID 18743 and parent PID 18654
--> Killing 'gp' with PID 18759 and parent PID 18654
--> Killing 'gp' with PID 18841 and parent PID 18654
--> Killing 'gp' with PID 18851 and parent PID 18654
--> Killing 'gp' with PID 18868 and parent PID 18654
--> Killing 'gp' with PID 18878 and parent PID 18654
--> Killing 'gp' with PID 18982 and parent PID 18654
--> Killing 'gp' with PID 19333 and parent PID 18654
Exception while cleaning up PID 18654:
Traceback (most recent call last):
  File "/usr/local/src/sage-config/src/bin/sage-cleaner", line 94, in cleanup
    or kill_spawned_jobs(spawned_processes, parent_pid):
  File "/usr/local/src/sage-config/src/bin/sage-cleaner", line 106, in kill_spawned_jobs
    pid, cmd = job.strip().split(' ', 1)
ValueError: need more than 1 value to unpack

Probably the jobfile got corrupted somehow and we need to handle this gracefully.

Component: scripts

Author: Jeroen Demeyer

Branch/Commit: 7940870

Reviewer: Frédéric Chapoton

Issue created by migration from https://trac.sagemath.org/ticket/21800

@jdemeyer jdemeyer added this to the sage-7.5 milestone Nov 2, 2016
@jdemeyer
Copy link
Author

jdemeyer commented Nov 2, 2016

@jdemeyer
Copy link
Author

jdemeyer commented Nov 2, 2016

Commit: 7940870

@jdemeyer
Copy link
Author

jdemeyer commented Nov 2, 2016

New commits:

7940870Better error handling in sage-cleaner

@fchapoton
Copy link
Contributor

Reviewer: Frédéric Chapoton

@fchapoton
Copy link
Contributor

comment:3

ok, let it be

@vbraun
Copy link
Member

vbraun commented Nov 9, 2016

Changed branch from u/jdemeyer/better_error_handling_in_sage_cleaner to 7940870

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants