Skip to content

Add paths to windows metadataFiles - #212

Merged
kzys merged 1 commit into
containerd:mainfrom
gabriel-samfira:add-paths-to-metadata-files
Feb 6, 2023
Merged

Add paths to windows metadataFiles#212
kzys merged 1 commit into
containerd:mainfrom
gabriel-samfira:add-paths-to-metadata-files

Conversation

@gabriel-samfira

Copy link
Copy Markdown
Contributor

When creating a new empty base layer on Windows, hcsshim creates a set of empty registry hives in C:\Windows\System32\config. These files are needed in order for the new layer to be considered valid.

Add these locations to the metadataFiles, which excludes them when comparing the diffs of two layers.

This will be used in the snapshot tests on Windows in: containerd/containerd#8043

Signed-off-by: Gabriel Adrian Samfira gsamfira@cloudbasesolutions.com

@gabriel-samfira

gabriel-samfira commented Feb 4, 2023

Copy link
Copy Markdown
Contributor Author

Comment thread fs/fstest/compare_windows.go Outdated
When creating a new empty base layer on Windows, hcsshim creates a set
of empty registry hives in C:\Windows\System32\config. These files are
needed in order for the new layer to be considered valid.

Add these locations to the metadataFiles, which excludes them when
comparing the diffs of two layers.

Signed-off-by: Gabriel Adrian Samfira <gsamfira@cloudbasesolutions.com>
@gabriel-samfira
gabriel-samfira force-pushed the add-paths-to-metadata-files branch from a1433d9 to 68f7b34 Compare February 6, 2023 10:28

@estesp estesp left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants