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
[21.01] Initial fix for quay.io repo query issue #13468
Conversation
Fixes not getting a full list of images in a repo. Without this fix some container images are not resolvable.
Co-authored-by: Sergey Golitsynskiy <sgolitsynskiy@gmail.com>
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.
Confirmed this fixes the resolution issue for me.
Do we know an example repo that was affected by this ? We could add this to the tests then. |
On anvil, we were able to replicate with fastqc and multiqc. |
if resolution_cache is not None and cache_key in resolution_cache: | ||
repo_names = resolution_cache.get(cache_key) |
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.
Shouldn't the cache have different repo_names
for different namespace
s?
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 think so, this would be a problem if we we're using more than one namespace, but so far it's only biocontainers.
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.
Yep, added to #13460 for tracking.
Fixes not getting a full list of images in a repo. Without this fix some container images are not resolvable.
I understand there's more to be done, but this is a major fix we should get out ASAP.
How to test the changes?
(Select all options that apply)
License