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

copy rootfs and hash VHD to bundle directory for SNP #2110

Merged
merged 2 commits into from
Apr 29, 2024

Conversation

anmaxvl
Copy link
Contributor

@anmaxvl anmaxvl commented Apr 17, 2024

Make sure that there's no potential race in accessing
rootfs.vhd and rootfs.hash.vhd when multiple SNP pods
are created concurrently with verity boot.

@anmaxvl anmaxvl force-pushed the copy-rootfs-hash-vhd branch from e6e04f1 to 39b120c Compare April 24, 2024 06:33
@anmaxvl anmaxvl changed the title Copy rootfs hash vhd copy rootfs and hash VHD to bundle directory for SNP Apr 24, 2024
@anmaxvl anmaxvl marked this pull request as ready for review April 24, 2024 06:34
@anmaxvl anmaxvl requested a review from a team as a code owner April 24, 2024 06:35
Make sure that there's no potential race in accessing
rootfs.vhd and rootfs.hash.vhd when multiple SNP pods
are created concurrently with verity boot.

Signed-off-by: Maksim An <maksiman@microsoft.com>
@anmaxvl anmaxvl force-pushed the copy-rootfs-hash-vhd branch from 39b120c to e98e2dc Compare April 24, 2024 06:35
internal/uvm/create_lcow.go Outdated Show resolved Hide resolved
Signed-off-by: Maksim An <maksiman@microsoft.com>
@anmaxvl anmaxvl force-pushed the copy-rootfs-hash-vhd branch from d7ed4c6 to eae47c2 Compare April 26, 2024 22:42
@anmaxvl anmaxvl merged commit 530d0f3 into microsoft:main Apr 29, 2024
19 checks passed
@anmaxvl anmaxvl deleted the copy-rootfs-hash-vhd branch April 29, 2024 17:25
princepereira pushed a commit to princepereira/hcsshim that referenced this pull request Aug 29, 2024
Make sure that there's no potential race in accessing
rootfs.vhd and rootfs.hash.vhd when multiple SNP pods
are created concurrently with verity boot.

Signed-off-by: Maksim An <maksiman@microsoft.com>
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.

3 participants