Skip to content

dind: remove zfs dependency - #586

Open
thaJeztah wants to merge 1 commit into
docker-library:masterfrom
thaJeztah:rm_zfs
Open

dind: remove zfs dependency#586
thaJeztah wants to merge 1 commit into
docker-library:masterfrom
thaJeztah:rm_zfs

Conversation

@thaJeztah

Copy link
Copy Markdown
Contributor

The zfs utilities were used by the zfs graph-driver. containerd provides an optional zfs-snapshotter, but it's not installed by default. Remove the dependency, as it's not used in the default configuration.

The zfs utilities were used by the zfs graph-driver. containerd provides
an optional zfs-snapshotter, but it's not installed by default. Remove
the dependency, as it's not used in the default configuration.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah

Copy link
Copy Markdown
Contributor Author

Size comparison;

docker image ls dind
IMAGE           ID             DISK USAGE   CONTENT SIZE   EXTRA
dind:before     66a85fa1e49a        502MB          131MB
dind:ctr        b42924822491        471MB          122MB

@thaJeztah

Copy link
Copy Markdown
Contributor Author

Windows failure is unrelated;

failed to connect to the docker API at npipe:////./pipe/docker_engine; check if the path is correct and if the daemon is running: open //./pipe/docker_engine: The system cannot find the file specified.
Error: Process completed with exit code 1.

@thaJeztah

Copy link
Copy Markdown
Contributor Author

cc @tianon

@tianon tianon left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

I think we should give this a little time to "soak" in case there are users who notice and can justify the continued inclusion of ZFS, but I also think we'll probably find those most effectively by merging and releasing this, so I think we should do that sometime next week. 👍

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.

2 participants