-
Notifications
You must be signed in to change notification settings - Fork 1k
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
published histories do not load #17622
Comments
I was not able to reproduce it locally, yet. I tried to track it down on sentry, it might be related to: Looking at the dataset states in the schema, it seems that |
I think We might have some broken/ancient data in database? Carl's comment on that method 🫣 galaxy/lib/galaxy/managers/histories.py Line 867 in bc0d075
|
Yes, #17475. I'll take a look. |
Hmm, so deleted isn't actually a valid state, at all , and instead a boolean column. We have a bunch of datasets from 2008 doing that, but also a few from february of this year:
I wonder if the ones from february have been manually set to that state, do you have any recollection of that @natefoo ? |
Fixed this with |
What do you mean by manual thing? Something other than the app modifying the db? |
yes |
I think we need to have data migrations (or something of that sort), so that other admins can apply such fixes to their instances. It's separate from db schema migrations. Also, that would document all such fixes. |
getting 500 from
https://usegalaxy.org/api/histories?limit=25&offset=0&search=&sort_by=update_time&sort_desc=true&show_own=false&show_published=true&show_shared=false
Galaxy Version and/or server at which you observed the bug
Galaxy Version: Main 24.0
Commit: 8b44fde05fe0a2aab1fb7f7e164f2adf96ce2387
To Reproduce
Steps to reproduce the behavior:
The text was updated successfully, but these errors were encountered: