-
Notifications
You must be signed in to change notification settings - Fork 25
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
allow keeping input folder during job cleanup #199
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This changes the default behavior from "add input folder to tar archive" to "delete input folder". But I even slightly prefer the new behavior.
Right, should have mentioned this in the description. I'd expect that an input folder in the tar archive is hardly ever used. But we can also keep it in there if others prefer that. |
I'm not sure I fully understand. You don't need to open the |
Right, the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would mention in the description of the commit and PR that this changes the default behavior. But I'm fine with that.
I'm also fine with the other changes otherwise.
When sisyphus cleans up a job folder, the input folder is deleted. This can be impractical if you want to find some job's dependencies because instead of just going through the input folders, you need to open each
finished.tar.gz
file and check for input jobs in theinfo
file. I therefore propose to add the config optionJOB_CLEANUP_KEEP_INPUT
. It set to True, the input folder will remain even after cleaning.Edit: As suggested in the PR, we set
JOB_CLEANUP_KEEP_INPUT = True
as the default setting. This changes the default behavior, but is not expected to hurt anyone.