Skip to content

Windows: Bump RW layer size#35925

Merged
johnstep merged 1 commit intomoby:masterfrom
microsoft:jjh/builder-default-rw-size-bump
Jan 5, 2018
Merged

Windows: Bump RW layer size#35925
johnstep merged 1 commit intomoby:masterfrom
microsoft:jjh/builder-default-rw-size-bump

Conversation

@lowenna
Copy link
Copy Markdown
Member

@lowenna lowenna commented Jan 4, 2018

Signed-off-by: John Howard [email protected]

@johnstep @darrenstahlmsft As mentioned offline... PTAL

This bumps the size of the RW layer (sandbox.vhdx) from the default (20GB, hard coded in the platform) to 127GB (purely as the default size of a VHD in Hyper-V) for the builder RUN scenario where users are adding a lot of data during a RUN statement (eg while debugging #32838). Done local verification using diskmgmt.msc watching disk being attached/detached to verify it's operating as expected.

image

It does not change the default size for a docker run (which can be controlled anyway through --storage-opt)

@friism FYI.

Signed-off-by: John Howard <[email protected]>
Copy link
Copy Markdown
Member

@tonistiigi tonistiigi left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

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

LGTM

@mikebolland
Copy link
Copy Markdown

Is this the first part of a fix for #33586 ?

@lowenna
Copy link
Copy Markdown
Member Author

lowenna commented Jan 5, 2018

@mikebolland No, that is unrelated.

@lowenna lowenna deleted the jjh/builder-default-rw-size-bump branch January 5, 2018 15:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants