Base layer manipulation #1637
Merged
Base layer manipulation #1637
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
gabriel-samfira
force-pushed
the
base-layer-manipulation-rebased
branch
2 times, most recently
from
0f2ae9b
to
417c365
Compare
helsaawy
reviewed
Jan 31, 2023
helsaawy
reviewed
Jan 31, 2023
test/internal/scratch.go
Outdated
| func CreateWCOWBlankBaseLayer(ctx context.Context, t *testing.T) []string { | ||
| t.Helper() | ||
| tempDir := t.TempDir() | ||
| if err := wclayer.ConvertToBaseLayer(context.Background(), tempDir); err != nil { |
There was a problem hiding this comment.
wclayer.ConvertToBaseLayer(ctx tempDir)
gabriel-samfira
force-pushed
the
base-layer-manipulation-rebased
branch
from
aca4cc6
to
255f402
Compare
This is the inverse of the baseLayerWriter: It walks Files/ and UtilityVM/Files/ (if present) and ignores the rest of the layer data, as it will be recreated when the layer is imported. Signed-off-by: Paul "TBBle" Hampson <Paul.Hampson@Pobox.com>
This API allows turning any collection of files into a WCOW base layer. It will create the necessary files in Files/ for hcsshim.ProcessBaseLayer to function, validate the necessary files for hcsshim.ProcessUtilityVMImage if UtilityVM/ exists, and then call those two APIs to complete the process. Calling this on a directory containing an untarred base layer OCI tarball, gives a very similar outcome to passing the tar stream through ociwclayer.ImportLayer. The new API is used in `TestSCSIAddRemoveWCOW` to create nearly-empty base layers for the scratch layers attached and removed from the utility VM. A wclayer command is also introduced: `makebaselayer` for testing and validation purposes. Signed-off-by: Paul "TBBle" Hampson <Paul.Hampson@Pobox.com>
Signed-off-by: Paul "TBBle" Hampson <Paul.Hampson@Pobox.com>
This change adds functions to generate valid, empty hives. Signed-off-by: Gabriel Adrian Samfira <gsamfira@cloudbasesolutions.com>
Signed-off-by: Gabriel Adrian Samfira <gsamfira@cloudbasesolutions.com>
Signed-off-by: Gabriel Adrian Samfira <gsamfira@cloudbasesolutions.com>
Signed-off-by: Gabriel Adrian Samfira <gsamfira@cloudbasesolutions.com>
gabriel-samfira
force-pushed
the
base-layer-manipulation-rebased
branch
from
255f402
to
99c9146
Compare
Signed-off-by: Gabriel Adrian Samfira <gsamfira@cloudbasesolutions.com>
helsaawy
reviewed
Feb 7, 2023
Signed-off-by: Gabriel Adrian Samfira <gsamfira@cloudbasesolutions.com>
gabriel-samfira
force-pushed
the
base-layer-manipulation-rebased
branch
from
50a9616
to
077be0f
Compare
helsaawy
approved these changes
Feb 8, 2023
ambarve
reviewed
Feb 27, 2023
* We can use string instead of *uint16 in the //sys definition and allow mksyscall to generate the proper boilerplate. * do not shadow err if it's not nil Signed-off-by: Gabriel Adrian Samfira <gsamfira@cloudbasesolutions.com>
Signed-off-by: Gabriel Adrian Samfira <gsamfira@cloudbasesolutions.com>
gabriel-samfira
force-pushed
the
base-layer-manipulation-rebased
branch
from
31e246a
to
b2acd4e
Compare
Signed-off-by: Gabriel Adrian Samfira <gsamfira@cloudbasesolutions.com>
ambarve
approved these changes
Feb 28, 2023
This was referenced Mar 2, 2023
jsturtevant
added a commit
to jsturtevant/hcsshim
that referenced
this pull request
Mar 7, 2023
If a project creates a base layer image and wants to directly import the base then this ensure the baseline file are in place. It is a follow up to microsoft#1637 and enables containerd with ctr image import image.tar of a base layer. Signed-off-by: James Sturtevant <jstur@microsoft.com>
princepereira
pushed a commit
to princepereira/hcsshim
that referenced
this pull request
Aug 29, 2024
* Simple baseLayerReader to export parentless layers This is the inverse of the baseLayerWriter: It walks Files/ and UtilityVM/Files/ (if present) and ignores the rest of the layer data, as it will be recreated when the layer is imported. Signed-off-by: Paul "TBBle" Hampson <Paul.Hampson@Pobox.com> * Introduce hcsshim.ConvertToBaseLayer This API allows turning any collection of files into a WCOW base layer. It will create the necessary files in Files/ for hcsshim.ProcessBaseLayer to function, validate the necessary files for hcsshim.ProcessUtilityVMImage if UtilityVM/ exists, and then call those two APIs to complete the process. Calling this on a directory containing an untarred base layer OCI tarball, gives a very similar outcome to passing the tar stream through ociwclayer.ImportLayer. The new API is used in `TestSCSIAddRemoveWCOW` to create nearly-empty base layers for the scratch layers attached and removed from the utility VM. A wclayer command is also introduced: `makebaselayer` for testing and validation purposes. Signed-off-by: Paul "TBBle" Hampson <Paul.Hampson@Pobox.com> * Include hard-linked files as hard-links in the tarstream Signed-off-by: Paul "TBBle" Hampson <Paul.Hampson@Pobox.com> * Use offline registry library to generate min hive This change adds functions to generate valid, empty hives. Signed-off-by: Gabriel Adrian Samfira <gsamfira@cloudbasesolutions.com> * Rename ofreg.go and close key Signed-off-by: Gabriel Adrian Samfira <gsamfira@cloudbasesolutions.com> * Fix temp dir creation Signed-off-by: Gabriel Adrian Samfira <gsamfira@cloudbasesolutions.com> * Cleanup tests Signed-off-by: Gabriel Adrian Samfira <gsamfira@cloudbasesolutions.com> * Fix ORCloseHive definition Signed-off-by: Gabriel Adrian Samfira <gsamfira@cloudbasesolutions.com> * Remove unused ctx from baseLayerReader Signed-off-by: Gabriel Adrian Samfira <gsamfira@cloudbasesolutions.com> * Use string in sys definition and check for err * We can use string instead of *uint16 in the //sys definition and allow mksyscall to generate the proper boilerplate. * do not shadow err if it's not nil Signed-off-by: Gabriel Adrian Samfira <gsamfira@cloudbasesolutions.com> * Close the r.proceed channel Signed-off-by: Gabriel Adrian Samfira <gsamfira@cloudbasesolutions.com> * Return if backup reader is nil Signed-off-by: Gabriel Adrian Samfira <gsamfira@cloudbasesolutions.com> --------- Signed-off-by: Paul "TBBle" Hampson <Paul.Hampson@Pobox.com> Signed-off-by: Gabriel Adrian Samfira <gsamfira@cloudbasesolutions.com> Co-authored-by: Paul "TBBle" Hampson <Paul.Hampson@Pobox.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR is just a rebase of #901
@TBBle is currently on sabbatical so I will try to move the two PRs (this one and the one on containerd), forward.
All needed context for this PR is available on #901