install: check for docker #193
Closed
knoflook
wants to merge 1 commits from
knoflook/abra:installer into main
pull from: knoflook/abra:installer
merge into: :main
:main
:fix/492
:local-integration-testing
:renovate/github.com-charmbracelet-lipgloss-2.x
:renovate/otel-weaver-0.x
:renovate/codespell-2.x
:renovate/github.com-charmbracelet-bubbletea-2.x
:renovate/tonistiigi-xx-1.x
:renovate/alpine-3.x
:renovate/github.com-charmbracelet-log-2.x
:chore-deps
:fix/deps
:fix/613
No Reviewers
Labels
Clear labels
bug
build
ci/cd
critical fix
design
documentation
duplicate
easy-first-issue
enhancement
help wanted
i10n
i18n
installer
invalid
question
release
release-candidate
security
tech-debt
test
wontfix
Something is not working
go build related issues
Building things with CI/CD
https://docs.coopcloud.tech/federation/resolutions/passed/010/
UI/UX
Documenting all the things
This issue or pull request already exists
Something for new people to get stuck into. We hope it's easy!
New feature
Need some help
Everything to do with localisation
Everything to do with internationalisation
Everything to do with the install script.
Something is wrong
More information is needed
Release management
Related to the new release candidate
Security related
Unit/integration testing
This won't be fixed
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
3wordchant
aadil (Aadil Ayub)
abra-bot (Abra Bot)
ammaratef45
amras (Sarma)
Apfelwurm
BornDeleuze
Brooke
carla
cas (Cassowary)
coopcloud
cyrnel
decentral1se (d1)
dede
devydave
fauno (fauno)
iexos
jade (Jade Ambrose)
jjsfunhouse
jmakdah2 (Jackie Makdah)
joe-irving (Joe Irving)
kawaiipunk (KawaiiPunk)
knoflook
kolaente
lambdabundesverband
linnealovespie (April)
moosemower
moritz
notplants
oxaliq (sorrel)
p4u1
pharaohgraphy (Andrew 🐦🔥❤️🔥✴️)
renovate-bot (Comrade Renovate Bot)
ripclap
simon
sixsmith (Sixsmith)
stevensting
trav (Trav Fryer)
val (val (he/him))
yksflip
Clear assignees
No Assignees
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: toolshed/abra#193
Reference in New Issue
Block a user
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
if you install abra when docker is not present on your local machine or your user isn't added to the docker group, it shows strange errors. This checks if docker is present on your local machine and proposes to install it if needed.
@knoflook
That's strange, could you also copy/pasta in some of the errors?
I thought we only needed to install Docker on the server-side, not on the local machine?
Also, I would guess that some people who use
abrawon't ever drop down into the more fine-grained Docker command-line commands, so unsure if we should have this check as default?sorry i was rushing to put this issue here. this happens when you run
abra server addAFAIU abra can only control docker, not replace it. Which means it still needs the docker engine to run on the local machine. It's actual docker that manages contexts and makes connections to the servers, and we're just talking to it.
you can't add a server if the daemon is not present on your local machine.
abralargely re-uses upstream Docker CLI libs, so it is basically a drop-in replacement for a bunch of functionality.abramakes calls to the remote Docker daemon, not a local one. When youabra server add ...,abraruns the same code that the Docker CLI would run, to create the local context.I can on my machine:
The error you posted above could have a number of causes but I don't think it is about a missing local Docker daemon. So many things can go wrong when dealing with a new server from
abraside, anything SSH, provisioning fails, user permissions and the list goes on... if you can smoke out your issue, then we can improve the error message or try to handle it inabra?Let's re-group to discuss this, closing off for now.
Pull request closed