investigate the possibility of droping ssh-askpass requirement for provisioning docker #320

New Issue

knoflook · 2022-05-11T09:43:22Z

knoflook commented

2022-05-11 09:43:22 +00:00

this is not installed by default on debian, so to install docker with -p you still need to ssh to the server and sudo apt install ssh-askpass, might install docker manually too at this point. Maybe abra can try ssh'ing as root when provisioning, so you don't have to type your password in? or maybe there's an alternative to ssh-askpass?

this is not installed by default on debian, so to install docker with `-p` you still need to ssh to the server and `sudo apt install ssh-askpass`, might install docker manually too at this point. Maybe abra can try ssh'ing as root when provisioning, so you don't have to type your password in? or maybe there's an alternative to `ssh-askpass`?

knoflook added the

enhancement

abra

labels 2022-05-11 09:43:23 +00:00

decentral1se commented

2022-05-11 10:08:23 +00:00

Yeh good point 🤔

I think we could refactor cd8137a7d8/cli/server/add.go (L207-L220) to install instead of erroring out? It's a bit of a chicken/egg situation with --ask-sudo-pass because the only way abra can install the required dependency is by having the dependency already 🙃

I'd love to know an alternative to ssh-askpass... does it exist?

Trying to write code that deals with this bootstrapping which handles both the root & non-root (sudo required) cases is quite error prone so I was a bit cautious to go further (not to even mention distro-portability 🙃).

Yeh good point 🤔 I think we could refactor https://git.coopcloud.tech/coop-cloud/abra/src/commit/cd8137a7d87cce902949153e03e1699d767ff9b4/cli/server/add.go#L207-L220 to install instead of erroring out? It's a bit of a chicken/egg situation with `--ask-sudo-pass` because the only way `abra` can install the required dependency is by having the dependency already 🙃 I'd love to know an alternative to `ssh-askpass`... does it exist? Trying to write code that deals with this bootstrapping which handles both the root & non-root (sudo required) cases is quite error prone so I was a bit cautious to go further (not to even mention distro-portability 🙃).

decentral1se added this to the (deleted) project 2022-08-05 11:48:30 +00:00

decentral1se commented

2023-02-08 22:36:04 +00:00

"Solved" in coop-cloud/abra#255 (we don't provision).

"Solved" in https://git.coopcloud.tech/coop-cloud/abra/pulls/255 (we don't provision).

decentral1se closed this issue

2023-02-08 22:36:04 +00:00

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: toolshed/organising#320