Adding server manually without server add creates unclear error on deploy #675

Closed
opened 2025-09-30 18:56:31 +00:00 by decentral1se · 2 comments
Owner

If you are syncing in folders or adding them manually but you don't abra server add <server>, then the docker client doesn't get created and it appears like abra ignores it. When you deploy, it shows: FATA unknown server, run "abra server add <server>" which does fix it.

However, this error kind of pops up out of nowhere on a deploy. Also, you think you've added the server correctly. So, we could probably detect if there is a client configured correctly or not and adjust the "unknown server" part to "missing client for server".

/cc @kolaente

If you are syncing in folders or adding them manually but you don't `abra server add <server>`, then the docker client doesn't get created and it appears like `abra` ignores it. When you deploy, it shows: `FATA unknown server, run "abra server add <server>"` which does fix it. However, this error kind of pops up out of nowhere on a deploy. Also, you think you've added the server correctly. So, we could probably detect if there is a client configured correctly or not and adjust the "unknown server" part to "missing client for server". /cc @kolaente
decentral1se added the
enhancement
label 2025-09-30 18:56:31 +00:00
decentral1se changed title from Adding server manually without `server add` ignores server to Adding server manually without `server add` creates unclear error on deploy 2025-09-30 19:00:08 +00:00
decentral1se added this to the Abra "next" project 2025-09-30 19:03:35 +00:00
decentral1se moved this to Prioritised in Abra "next" on 2025-09-30 19:03:42 +00:00
decentral1se moved this to In Progress in Abra "next" on 2025-10-01 09:27:49 +00:00
decentral1se started working 2025-10-01 09:28:02 +00:00
decentral1se worked for 22 minutes 2025-10-01 09:50:58 +00:00
decentral1se moved this to Done in Abra "next" on 2025-10-01 09:51:09 +00:00
Member

ty for improving the handling!

just had a quick thought, how about not throwing that error but prompting whether abra should try to add the specific server?
e.g.
"server missing context, proceed to try running "abra server add %s" (Y/n)?
and then just run that command?

ty for improving the handling! just had a quick thought, how about not throwing that error but prompting whether abra should try to add the specific server? e.g. "server missing context, proceed to try running \"abra server add %s\" (Y/n)? and then just run that command?
Author
Owner

@simon thanks for the feedback! I thought about it but since the server name is customisable via ~/.ssh/config and these names might change while syncing across work stations (the reason why this bug happens in the first place), we might end up in a situation where we're recommending a server name which isn't valid. Which is still a problem with this new handling but it's less work than doing an interactive prompt, handling --no-input and further handling the failure when the server is wrong... 🙃 I have a feeling less is more in this case...

@simon thanks for the feedback! I thought about it but since the server name is customisable via `~/.ssh/config` and these names might change while syncing across work stations (the reason why this bug happens in the first place), we might end up in a situation where we're recommending a server name which isn't valid. Which is still a problem with this new handling but it's less work than doing an interactive prompt, handling `--no-input` and further handling the failure when the server is wrong... 🙃 I have a feeling less is more in this case...
Sign in to join this conversation.
2 Participants
Notifications
Total Time Spent: 22 minutes
decentral1se
22 minutes
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: toolshed/abra#675
No description provided.