-
Notifications
You must be signed in to change notification settings - Fork 785
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
vendor: update containers/storage #846
Conversation
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.
Change LGTM, just wondering if we should hold until containers/image#472 merges.
@TomSweeneyRedHat this changes only containers/storage not containers/image. I am wondering more about: containers/storage#137 that I'd like to include as well in our tools 😄 |
DOH! You're right, I must be having too much July 4th fun. A straight LGTM then. |
@giuseppe I merged the OSTree support, so update to pull that in. |
a217b50
to
179782c
Compare
Update to version 9cbb6cb3fc2044eae8b8fd8d8da081fe812858c4 Signed-off-by: Giuseppe Scrivano <[email protected]>
@rhatdan done ⬆️ |
📌 Commit 7c701e5 has been approved by |
☀️ Test successful - status-papr |
@giuseppe Could you write up a couple of blogs on why these versions of container/storage are important and useful. |
yes, I'll do that. In the last days I've done a lot of improvements in https://github.com/giuseppe/containerfs and solved all the design issues I didn't like. I think it is in a good shape for a tech preview. |
Update to version 58f557cf12df3e11437485f329c15b5c20bebb90
Signed-off-by: Giuseppe Scrivano [email protected]