Skip to content
Snippets Groups Projects

Add better exception handling for edge cases in Galaxy dataset fetching.

Summary of Changes

This mr fixes an edge case where some datasets from the Galaxy history would not have a state or creating_job parameter which would cause the loop checking for datasets to break prematurely. Now those datasets will be properly ignored, and everything legitimate will continue to be fetched, solving the issue where Ref1d for example would just never properly be detected by the launcher.

Checklist

  • The MR has a clear and concise title
  • Code is self-documented and follows style guidelines.
  • Automated tests are written and pass successfully.
  • Regression tests (e.g. manually triggered system tests, manual GUI/tool tests, ...) are performed to make sure the MR does not break anything (when applicable)
  • Readme file is present and up-to-date.

Documentation Updates

Additional Notes

Merge request reports

Pipeline #608397 skipped

Pipeline skipped for f809cbcf on fix-dataset-edgecase-no-state

Merged by Cage, GregoryCage, Gregory 4 months ago (Aug 27, 2024 8:43pm UTC)

Loading

Pipeline #608403 skipped

Pipeline skipped for 9f0c8860 on 40-implement-django-version-of-the-app-launcher

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading