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

Harden dynamic job destination handling #18241

Open
6 tasks
mvdbeek opened this issue May 28, 2024 · 0 comments
Open
6 tasks

Harden dynamic job destination handling #18241

mvdbeek opened this issue May 28, 2024 · 0 comments

Comments

@mvdbeek
Copy link
Member

mvdbeek commented May 28, 2024

Earlier today galaxyproject/tpv-shared-database#61 was merged. When updating usegalaxy.org this broke a number of jobs, since this broke calculating the job destination (which we also do when a job is being handled, not just when we submit a job).

  • explore if we can (at least temporarily) persist the job destination, maybe in the working directory ?
  • raise a well-defined exception indicating a temporary failure in calculating the job destination (Probably in TPV)
  • make it so that we don’t fail the job when we encounter that exception … just retry later if possible
  • produce some kind of type interface for a given variable and validate all rules / code blocks against them (TPV)
  • Version the shared database and don't deploy from main (TPV + usegalaxy playbook)
  • There might be a bug in TPV that led to this situation, that also needs fixing (and a test ideally) before a new attempt at Add default resource params and slurm destination tpv-shared-database#61 can be made
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

2 participants