Skip to content
This repository has been archived by the owner on May 6, 2020. It is now read-only.

image: Increase image size. #28

Merged
merged 1 commit into from
Sep 25, 2017

Conversation

jcvenegas
Copy link
Contributor

The default image size is very small leting the agent to
do some operations in storage.

Increase to allow the agent run.

Fixes: #27

Signed-off-by: Jose Carlos Venegas Munoz [email protected]

@@ -51,7 +51,7 @@ exit 1
# Image file to be created:
IMAGE="container.img"
# Image contents source folder
IMG_SIZE=${IMG_SIZE:-50M}
IMG_SIZE=${IMG_SIZE:-80M}
Copy link
Contributor

Choose a reason for hiding this comment

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

The README.md still mentions 50M as the default so can you add an update for that too? It might be an idea to add a make settings rule or something to avoid having to update the docs every time we tweak this? That way, we can document, "run make settings to see what settings will be used to create the rootfs/image".

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I like the idea!, working on it.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hi @jodh-intel updated docs, and captured #34

@jcvenegas jcvenegas mentioned this pull request Sep 25, 2017
The default image size is very small leting the agent to
do some operations in storage.

Increase to allow the agent run.

Fixes: clearcontainers#27

Signed-off-by: Jose Carlos Venegas Munoz <[email protected]>
@jodh-intel
Copy link
Contributor

lgtm

@jodh-intel jodh-intel merged commit ddf8570 into clearcontainers:master Sep 25, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants