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

devicemapper: Change default basesize to 100G #14709

Merged
merged 1 commit into from
Jul 17, 2015

Conversation

rhvgoyal
Copy link
Contributor

Fixes issue #14678

Current default basesize is 10G. Change it to 100G. Reason being that for
some people 10G is turning out to be too small and we don't have capabilities
to grow it dyamically.

This is just overcommitting and no real space is allocated till container
actually writes data. And this is no different then fs based graphdrivers
where virtual size of a container root is unlimited.

Signed-off-by: Vivek Goyal vgoyal@redhat.com

Current default basesize is 10G. Change it to 100G. Reason being that for
some people 10G is turning out to be too small and we don't have capabilities
to grow it dyamically.

This is just overcommitting and no real space is allocated till container
actually writes data. And this is no different then fs based graphdrivers
where virtual size of a container root is unlimited.

Signed-off-by: Vivek Goyal <vgoyal@redhat.com>
@icecrime
Copy link
Contributor

I heard people complaining in the past about that 10G limitations, which I had no idea was on our end. I think it's a good thing to do yes: LGTM.

@cpuguy83
Copy link
Member

LGTM

cpuguy83 added a commit that referenced this pull request Jul 17, 2015
devicemapper: Change default basesize to 100G
@cpuguy83 cpuguy83 merged commit e939a30 into moby:master Jul 17, 2015
rhatdan pushed a commit to rhatdan/moby1 that referenced this pull request Jul 17, 2015
devicemapper: Change default basesize to 100G

Docker-DCO-1.1-Signed-off-by: Brian Goff <cpuguy83@gmail.com> (github: rhatdan)
@k0105 k0105 mentioned this pull request Nov 20, 2015
@thaJeztah thaJeztah added this to the 1.8.0 milestone Dec 1, 2015
@rhvgoyal
Copy link
Contributor Author

rhvgoyal commented Jan 6, 2016

I guess going back to 10G (or possibly 20G) size is reasonable. And those who need bigger sizes will use dm.basesize=100G for now.

Will be good to also provide capability to specify container size during creation. That way those who need bigger containers can specify per container size. Shishir has a working patch to specify per container size during create time.

@shishir-a412ed can you please generate a PR with that patch.

@rhvgoyal
Copy link
Contributor Author

rhvgoyal commented Jan 6, 2016

I wanted to put my comments in issue #18314

@rootsongjc
Copy link
Contributor

How cloud I assign the container mapper device size more than 10G?

@rhvgoyal
Copy link
Contributor Author

rhvgoyal commented Jul 1, 2016

@rootsongjc Use dm.basesize=100G

@warmchang
Copy link

mark. 👍

@littleday
Copy link

I have a question regarding the resize. After changing the size to 20 GB in the configuration file and restarting docker daemon, I can see the default device size been changed using "docker info". However, after pulling image and re-running the container, the container size is still 10GB (default). Does anyone have an idea about this?

@rootsongjc
Copy link
Contributor

@littleday I think you can try to use Overlay rather than device mapper, that have no container local storage limit.

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.

8 participants