-
Notifications
You must be signed in to change notification settings - Fork 16
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
Add pull-through caching support #97
Conversation
86993af
to
b43a5ee
Compare
0283528
to
a0ff316
Compare
pytest.skip("gem not installed on test machine") | ||
content_before = gem_content_api_client.list().count | ||
|
||
remote = gem_remote_factory(url="https://rubygems.org") |
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.
Where's the repository to receive all the cached content?
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.
You don't need it. Pulp itself is the repository that is storing the cached content. An admin user would need to list the content and manually add it to a repository if they want to.
We could add a feature request in pulpcore to try to add the pull-through content to an attached repository.
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.
So the caching is actually only done until someone calls orphan cleanup.
Yes, I think we should handle the case when a pull-through distribution has a repository and a remote. You could use that e.g. to collect all the packages needed to build one container and then move it into an immutable set.
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.
Not this PR then...
fixes: #94
Required pulpcore: pulp/pulpcore#3819
TODO: