abra app undeploy fails to fetch recipe #573

New Issue

2025-06-09T10:00:00Z

decentral1se commented

2025-06-09 10:00:00 +00:00

(Regression from the large refactor last release...)

$ abra app undeploy vault.foo.com
FATA stat /home/d/.abra/recipes/vaultwarden/compose.yml: no such file or directory

(Regression from the large refactor last release...) ``` $ abra app undeploy vault.foo.com FATA stat /home/d/.abra/recipes/vaultwarden/compose.yml: no such file or directory ```

decentral1se added the bug label 2025-06-09 10:00:00 +00:00

decentral1se self-assigned this 2025-07-08 09:59:13 +00:00

decentral1se added this to the Abra v0.11.x project 2025-07-08 10:06:51 +00:00

decentral1se referenced this issue from a commit

2025-08-13 09:43:11 +00:00

fix: ensure recipe with undeploy

decentral1se referenced this issue

2025-08-13 09:43:17 +00:00

fix: ensure recipe with undeploy #595

decentral1se moved this to In Progress in Abra v0.11.x on 2025-08-13 09:45:15 +00:00

decentral1se commented

2025-08-13 09:59:03 +00:00

Oh dear, this one is also a bit of a rabbit hole!

This is a result of toolshed/organising#628. Contrary to what we thought then, yes, abra app undeploy does need to access the check-out (and have it present on the local FS) because it needs to know the TIMEOUT value.

@simon how do you feel about --chaos/-C being added to abra app undeploy? The proposed code change is here: #595 Would this disrupt your workflow?

I guess this might be a slight annoyance for those not expecting it in the next release of abra but passing -C is so ingrained now, I hope it will be obvious how to proceed when people run into the issue?

Oh dear, this one is also a bit of a rabbit hole! This is a result of https://git.coopcloud.tech/toolshed/organising/issues/628. Contrary to what we thought then, yes, `abra app undeploy` does need to access the check-out (and have it present on the local FS) because it needs to know the `TIMEOUT` value. @simon how do you feel about `--chaos/-C` being added to `abra app undeploy`? The proposed code change is here: https://git.coopcloud.tech/toolshed/abra/pulls/595 Would this disrupt your workflow? I guess this might be a slight annoyance for those not expecting it in the next release of `abra` but passing `-C` is so ingrained now, I hope it will be obvious how to proceed when people run into the issue?

simon commented

2025-08-14 10:22:11 +00:00

Hey, thanks for investigating!

I think adding the -C flag should be fine I guess. Do you have thoughts on this, @moritz?

I was wondering whether I should be able to undeploy an app even if I don't have the recipe available, e.g. by having a fallback to the default TIMEOUT with a Warning or a --force option.
But I guess thats also a generall discussion of what is or should be possible should I lose access to my recipe whysoever

Hey, thanks for investigating! I think adding the `-C` flag should be fine I guess. Do you have thoughts on this, @moritz? I was wondering whether I should be able to undeploy an app even if I don't have the recipe available, e.g. by having a fallback to the default `TIMEOUT` with a Warning or a `--force` option. But I guess thats also a generall discussion of what is or should be possible should I lose access to my recipe whysoever

👍 1

decentral1se referenced this issue from a commit

2025-08-17 09:29:46 +00:00

fix: ensure recipe with undeploy

decentral1se commented

2025-08-17 09:35:29 +00:00

Oh, wait a second! I can just ensure the recipe is present but not update it / check if there are changes 😌 We don't need to add a -C flag. It seems the timeouts are becoming a bit of a controversial aspect of our (un)deployment strategy 🙃 See #596. Moving on 🤸

Oh, wait a second! I can just ensure the recipe is present but not update it / check if there are changes 😌 We don't need to add a `-C` flag. It seems the timeouts are becoming a bit of a controversial aspect of our (un)deployment strategy 🙃 See https://git.coopcloud.tech/toolshed/abra/issues/596. Moving on 🤸

❤️ 1

decentral1se referenced this issue from a commit

2025-08-17 09:36:49 +00:00

fix: ensure recipe with undeploy

decentral1se closed this issue

2025-08-17 09:36:56 +00:00

decentral1se moved this to Done in Abra v0.11.x on 2025-08-17 09:37:20 +00:00

Sign in to join this conversation.