Skip to content

Add code to format disk as ext4 in guest#1717

Merged
katiewasnothere merged 1 commit intomicrosoft:mainfrom
katiewasnothere:kabaldau/add_back_ext4_formatting
Apr 19, 2023
Merged

Add code to format disk as ext4 in guest#1717
katiewasnothere merged 1 commit intomicrosoft:mainfrom
katiewasnothere:kabaldau/add_back_ext4_formatting

Conversation

@katiewasnothere
Copy link
Copy Markdown

@katiewasnothere katiewasnothere commented Apr 4, 2023

This PR adds a new call to format a disk as ext4 in the guest.

This PR does NOT hook up the code flow, that will come in later PRs.

@katiewasnothere katiewasnothere requested a review from a team as a code owner April 4, 2023 19:55
Comment thread internal/guest/storage/ext4/format.go Outdated
@katiewasnothere katiewasnothere force-pushed the kabaldau/add_back_ext4_formatting branch from 754ff7b to 06fc6cc Compare April 11, 2023 22:53
Comment thread internal/guest/storage/ext4/format.go Outdated
Comment thread internal/guest/storage/ext4/format.go Outdated
@kevpar
Copy link
Copy Markdown
Member

kevpar commented Apr 11, 2023

PR/commit description are outdated.

@katiewasnothere katiewasnothere force-pushed the kabaldau/add_back_ext4_formatting branch from 06fc6cc to c89b56c Compare April 11, 2023 23:07
@katiewasnothere katiewasnothere changed the title Add back code to format disk as ext4 in guest and update test code Add back code to format disk as ext4 in guest Apr 11, 2023
@katiewasnothere katiewasnothere changed the title Add back code to format disk as ext4 in guest Add code to format disk as ext4 in guest Apr 11, 2023
@katiewasnothere katiewasnothere force-pushed the kabaldau/add_back_ext4_formatting branch from c89b56c to 696699d Compare April 11, 2023 23:09
@katiewasnothere
Copy link
Copy Markdown
Author

Updated to fix comment, move to use fmt.Errorf instead of "errors" package, and updated PR description and title.

Copy link
Copy Markdown
Member

@kevpar kevpar left a comment

Choose a reason for hiding this comment

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

LGTM

@katiewasnothere
Copy link
Copy Markdown
Author

Test failures are unrelated

Comment thread internal/guest/storage/ext4/format.go Outdated
Comment thread internal/guest/storage/ext4/format.go Outdated
Signed-off-by: Kathryn Baldauf <kabaldau@microsoft.com>
@katiewasnothere katiewasnothere force-pushed the kabaldau/add_back_ext4_formatting branch from 696699d to daa723f Compare April 18, 2023 18:47
Copy link
Copy Markdown
Contributor

@msscotb msscotb left a comment

Choose a reason for hiding this comment

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

LGTM

@katiewasnothere katiewasnothere merged commit 1143934 into microsoft:main Apr 19, 2023
@katiewasnothere katiewasnothere deleted the kabaldau/add_back_ext4_formatting branch April 19, 2023 18:59
princepereira pushed a commit to princepereira/hcsshim that referenced this pull request Aug 29, 2024
…ack_ext4_formatting

Add code to format disk as ext4 in guest
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