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

rpmostreepayload: Reuse the local repo as a cache #1119

Merged
merged 1 commit into from Jul 3, 2017

Conversation

cgwalters
Copy link
Contributor

@cgwalters cgwalters commented Jun 29, 2017

When doing kickstart installs (can happen for both bare metal PXE as well as
building cloud images), we currently end up redownloading all of the ostree
content, even though we have a version in the squashfs already.

However, the version we have may not be the version we want; the installer image
is typically built at a different/slower cycle from the ostree content.

But this is where a strength of libostree's content-addressed storage shines; we
can use the local repo as an object cache, and avoid fetching what we already
have.

This just enables the logic that lives in libostree: ostreedev/ostree#982

Closes: https://github.com/rhinstaller/anaconda/issues/1117

@cgwalters
Copy link
Contributor Author

This is a WIP since the API hasn't landed in libostree yet, but I tested this and it works nicely!

@cgwalters cgwalters changed the title rpmostreepayload: Reuse the local repo as a cache WIP: rpmostreepayload: Reuse the local repo as a cache Jun 29, 2017
When doing kickstart installs (can happen for both bare metal PXE as well as
building cloud images), we currently end up redownloading all of the ostree
content, even though we have a version in the squashfs already.

However, the version we have may not be the version we want; the installer image
is typically built at a different/slower cycle from the ostree content.

But this is where a strength of libostree's content-addressed storage shines; we
can use the local repo as an object cache, and avoid fetching what we already
have.

This just enables the logic that lives in libostree.

Closes: https://github.com/rhinstaller/anaconda/issues/1117
Copy link
Member

@jkonecny12 jkonecny12 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. Please write a comment here when the required feature will arrive to Rawhide.
Thanks!

@jkonecny12
Copy link
Member

Jenkins, it's ok to test.

@cgwalters
Copy link
Contributor Author

The upstream PR was merged, removing WIP. Note that I added a version check, so we'll work with both old and new versions. Hence this should be safe to merge at any time.

libostree releases usually are on a 3-4 week cadence, so it'll be a week or two.

@cgwalters cgwalters changed the title WIP: rpmostreepayload: Reuse the local repo as a cache rpmostreepayload: Reuse the local repo as a cache Jun 30, 2017
Copy link
Contributor

@M4rtinK M4rtinK left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me as well. :)

@M4rtinK M4rtinK merged commit eeca18d into rhinstaller:master Jul 3, 2017
@cgwalters
Copy link
Contributor Author

For reference the OSTree PR is: ostreedev/ostree#982

@cgwalters cgwalters mentioned this pull request Jul 24, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants