Skip to content

Invoke fsformatter for container scratch - #2498

Merged
ambarve merged 1 commit into
microsoft:mainfrom
kiashok:invoke-fs-formatter
Sep 11, 2025
Merged

Invoke fsformatter for container scratch#2498
ambarve merged 1 commit into
microsoft:mainfrom
kiashok:invoke-fs-formatter

Conversation

@kiashok

@kiashok kiashok commented Aug 12, 2025

Copy link
Copy Markdown
Contributor

This PR makes the changes to format container scratch vhds with refs for c-wcow cases.

@kiashok
kiashok force-pushed the invoke-fs-formatter branch from 7fcc844 to c97839d Compare August 20, 2025 22:03
@kiashok
kiashok force-pushed the invoke-fs-formatter branch 2 times, most recently from 4069c2d to e2b7b54 Compare September 3, 2025 22:23
@kiashok
kiashok marked this pull request as ready for review September 3, 2025 22:24
@kiashok
kiashok requested a review from a team as a code owner September 3, 2025 22:24
@kiashok
kiashok force-pushed the invoke-fs-formatter branch from e2b7b54 to f183303 Compare September 3, 2025 22:32
Comment thread internal/uvm/types.go Outdated
Comment thread internal/uvm/scsi/backend.go Outdated
Comment thread pkg/cimfs/mount_cim.go Outdated
Comment thread internal/protocol/guestresource/resources.go
Comment thread internal/uvm/scsi/backend.go
@kiashok
kiashok force-pushed the invoke-fs-formatter branch 2 times, most recently from 38f8093 to 44a0d41 Compare September 10, 2025 18:44

@ambarve ambarve left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

There is one file named taq that seems to havee been included by mistake. Can you remove that?

Otherwise, this looks good to me.

- Add support to invoke refs formatter for
container scratch.
- Add tests to validate fsformatter

Signed-off-by: Kirtana Ashok <kiashok@microsoft.com>
@kiashok
kiashok force-pushed the invoke-fs-formatter branch from 44a0d41 to 07050ee Compare September 11, 2025 15:54
@ambarve
ambarve merged commit 64208bd into microsoft:main Sep 11, 2025
16 of 17 checks passed
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