-
Notifications
You must be signed in to change notification settings - Fork 18
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
Validate id run #669
Validate id run #669
Conversation
…run before updating the DB
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.
We should be setting run folder name for run sin as many statuses as we can up to the point when run reaches 'analysis pending', otherwise we will will be unable to automatically delete cancelled and similar runs. Temporal indexes and run state comparison should be used for this purpose. We should not be updating very old runs, ie whenthe current status of the run is well in the past, say, over a week old.
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.
Adding some comments with the rationale for limiting to run pending
.
Co-authored-by: dkj <david.jackson@sanger.ac.uk>
No description provided.