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

fix: use block device mount to format scratch #2215

Merged
merged 1 commit into from
Jul 24, 2024

Conversation

anmaxvl
Copy link
Contributor

@anmaxvl anmaxvl commented Jul 22, 2024

By default, create LCOW will set the number of SCSI controllers to 4,
when VPMem isn't used, which is the case when formatting scratch.
This makes the device path lookup non-determenistic, since the
SCSI controllers in the guest may have different indices.

To workaround that, use a block device mount and
use the mount-path to format the scratch VHDX
without needing to lookup the actual dev node path.

@anmaxvl anmaxvl requested a review from a team as a code owner July 22, 2024 17:40
By default create LCOW will set the number of SCSI controllers to 4, when
VPMem isn't used, which is the case when formatting scratch. This makes the
device path lookup non-determenistic, since the SCSI controllers in the guest
may have different indices.

To workaround that, use a block device mount and use the mount-path to format
the scratch VHDX without needing to lookup the actual dev node path.

Signed-off-by: Maksim An <maksiman@microsoft.com>
@anmaxvl anmaxvl force-pushed the fix/runhcs-scratch-formatting branch from 80ce665 to 716c45a Compare July 22, 2024 17:42
Copy link
Contributor

@katiewasnothere katiewasnothere left a comment

Choose a reason for hiding this comment

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

LGTM

@anmaxvl anmaxvl merged commit 5c81656 into microsoft:main Jul 24, 2024
19 checks passed
@anmaxvl anmaxvl deleted the fix/runhcs-scratch-formatting branch July 24, 2024 22:17
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