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

Update configuration for the buildah #8

Merged
merged 5 commits into from
Mar 6, 2020
Merged

Update configuration for the buildah #8

merged 5 commits into from
Mar 6, 2020

Conversation

vitaliy-guliy
Copy link
Contributor

Signed-off-by: Vitaliy Gulyy [email protected]

Switches to the new stable buildah release.
Makes it possible to build the image inside running container.

Partially fixes eclipse-che/che#15035

Screenshot from 2020-03-03 15-42-20

Dockerfile Outdated Show resolved Hide resolved
etc/storage.conf Outdated Show resolved Hide resolved
etc/storage.conf Outdated Show resolved Hide resolved
etc/storage.conf Outdated

# Path to an helper program to use for mounting the file system instead of mounting it
# directly.
#mount_program = "/usr/bin/fuse-overlayfs"
Copy link
Contributor

Choose a reason for hiding this comment

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

If we are not using fuse-overlayfs it's probably best to remove this.

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 have copied the default configuration file and then changed necessary properties. I think it's a good practice, isn't 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.

But I'm also Ok to cleanup the file and left here only few lines.

Copy link
Contributor

Choose a reason for hiding this comment

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

Let's keep it clean just for sake of consistency and making it easier to read.

Dockerfile Outdated Show resolved Hide resolved
Copy link
Contributor

@ericwill ericwill left a comment

Choose a reason for hiding this comment

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

Nice. Let's do the same for the 1.1.0 branch too.

@ericwill ericwill merged commit 82c704e into che-dockerfiles:1.0.9 Mar 6, 2020
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