slimtoolkit / slim

Slim(toolkit): Don't change anything in your container image and minify it by up to 30x (and for compiled languages even more) making it secure too! (free and open source)
Apache License 2.0
18.97k stars 706 forks source link

[Bug]Slimtoolkit doesn't recognised docker engine comes buddled up with Docker Desktop #519

Open StarTrooper08 opened 1 year ago

StarTrooper08 commented 1 year ago

Expected Behavior

The slimtoolkit should connect with docker desktop and docker engine but it didn't!


Actual Behavior

I've installed Docker Desktop which comes bundled up with Docker Engine and then installed slimtoolkit but I think slimtoolkit didn't able to find docker engine which comes with Docker desktop. So It only connects with docker when we install docker engine and other docker peripherals when we install it seprately.


Steps to Reproduce the Problem

1. 1. 1. --- Specifications ================= - Version: Slimtoolkit version 1.40.1 - Platform: Ubuntu Linux 22.04 LTS
kcq commented 1 year ago

Thanks for sharing your problem @StarTrooper08 ! This might be linux desktop related. Not a problem with the Mac Docker Desktop. slim does need /var/run/docker.sock, which is the traditional docker socket path, but it's possible that it's not created in your Docker Desktop installation. According to https://github.com/docker/desktop-linux/issues/20 there's a linux Docker Desktop option for it: Settings -> Advanced -> check the option Enable default Docker socket (Requires password). Let me know if this does the trick for you.

StarTrooper08 commented 1 year ago

I didn't find any configuration to set up slimtoolkit with docker desktop. But no issue it's working fine with Docker CLI and using slimtoolkit container with docker desktop.

I think we can close this issue.

kcq commented 1 year ago

@StarTrooper08 adding the Docker Desktop notes to the README

kcq commented 1 year ago

here's the info: https://github.com/slimtoolkit/slim#docker-desktop

kcq commented 1 year ago

There's a small enhancement that can go along with the new Docker Desktop readme notes (to try ~/.docker/run/docker.sock if /var/run/docker.sock is not there. Once it's added this ticket can be closed.

kcq commented 1 year ago

the enhancement will be available in the next release (already in master)