abra app restart help & --all #605

New Issue

decentral1se · 2024-04-04T13:05:30Z

decentral1se commented

2024-04-04 13:05:30 +00:00

It takes a <service> argument, it doesn't restart every service. The help doesn't reflect that.

See https://codeberg.org/eotl/sextant/issues/94#issuecomment-1732145 for confusion.

I believe it could be useful to cycle through all services if --all is passed perhaps @basebuilder?

abra app restart -h
NAME:
abra app restart - Restart an app

USAGE:
abra app restart [command options]

DESCRIPTION:
This command restarts a service within a deployed app.

OPTIONS:
--debug, -d Show DEBUG messages
--offline, -o Prefer offline & filesystem access when possible

It takes a `<service>` argument, it doesn't restart every service. The help doesn't reflect that. See https://codeberg.org/eotl/sextant/issues/94#issuecomment-1732145 for confusion. I believe it could be useful to cycle through all services if `--all` is passed perhaps @basebuilder? > abra app restart -h > NAME: > abra app restart - Restart an app > > USAGE: > abra app restart [command options] <domain> > > DESCRIPTION: > This command restarts a service within a deployed app. > > OPTIONS: > --debug, -d Show DEBUG messages > --offline, -o Prefer offline & filesystem access when possible

👍 1

decentral1se added the

bug

abra

labels 2024-04-04 13:05:30 +00:00

basebuilder commented

2024-04-04 16:33:36 +00:00

It takes a argument, it doesn't restart every service.

What a <service> arg means in this case is just a guess for me... I think I tried all of the following:

sextant.eotl.dev
sextant.eotl.dev:app
sextant:app

In the case of sextant there is only app service.

🔴 Fail - What I imagine would be intuitive for apps with multiple services is to restart all run:

abra app restart sextant.eotl.dev
ERRO[0000] missing service?

🔴 Fail - Or to restart the specific badger service within that app deployment

abra app restart sextant.eotl.dev:badger
FATA[0000] cannot find app with name sextant.eotl.dev:app

✅ Success - with the <service> as a separate word after the app

abra app restart sextant.eotl.dev app
INFO[0016] app service successfully restarted

So yah, I think the syntax is fine, just the helper needs to make this clear

> It takes a <service> argument, it doesn't restart every service. What a `<service>` arg means in this case is just a guess for me... I think I tried all of the following: - `sextant.eotl.dev` - `sextant.eotl.dev:app` - `sextant:app` In the case of `sextant` there is only `app` service. 🔴 Fail - What I imagine would be intuitive for apps with multiple services is to restart all run: ``` abra app restart sextant.eotl.dev ERRO[0000] missing service? ``` 🔴 Fail - Or to restart the specific `badger` service within that app deployment ``` abra app restart sextant.eotl.dev:badger FATA[0000] cannot find app with name sextant.eotl.dev:app ``` ✅ Success - with the `<service>` as a separate word after the app ``` abra app restart sextant.eotl.dev app INFO[0016] app service successfully restarted ``` So yah, I think the syntax is fine, just the helper needs to make this clear

decentral1se referenced this issue from a commit

2024-07-07 19:52:59 +00:00

fix: "abra app restart" docs + --all-services

decentral1se referenced this issue from coop-cloud/abra

2024-07-07 19:53:22 +00:00

fix: "abra app restart" improvements #427

decentral1se closed this issue

2024-07-07 19:54:59 +00:00

Sign in to join this conversation.