Allow to remove volumes on their own conveniently #124

New Issue

decentral1se · 2021-04-02T11:54:40Z

decentral1se commented

2021-04-02 11:54:40 +00:00

hmmm seems like we need a command to wipe volumes on their own?
yeah I almost added one recently but I got hung up on the UI
e.g. should it be abra app ... volumes rm or abra volume ... rm or abra app ... rm (--config|--volumes|--secrets|--all)?

Also see https://matrix.to/#/!JSVYWCRXSVMrAzgeKB:autonomic.zone/$k0y8oXTS5xt5kEQOuncdZ9b2_u4tsjGNV7OdRjM-qDs?via=autonomic.zone&via=1312.media&via=matrix.org

> hmmm seems like we need a command to wipe volumes on their own? > yeah I almost added one recently but I got hung up on the UI > e.g. should it be abra app ... volumes rm <volume> or abra volume ... rm or abra app ... rm (--config|--volumes|--secrets|--all)? Also see https://matrix.to/#/!JSVYWCRXSVMrAzgeKB:autonomic.zone/$k0y8oXTS5xt5kEQOuncdZ9b2_u4tsjGNV7OdRjM-qDs?via=autonomic.zone&via=1312.media&via=matrix.org

decentral1se added this to the Beta release milestone 2021-04-02 11:54:40 +00:00

decentral1se added the

enhancement

label 2021-04-02 11:54:40 +00:00

decentral1se commented

2021-04-02 11:57:22 +00:00

Maybe extending abra app <app> rm with (--config|--volumes|--secrets|--all) is pretty consistent? I think that is the winner for me.

Maybe extending `abra app <app> rm` with `(--config|--volumes|--secrets|--all)` is pretty consistent? I think that is the winner for me.

3wordchant commented

2021-04-02 15:32:08 +00:00

Yeah abra app <app> rm (--config|--volumes|--secrets|--all) seems like the smallest change to the command line UI.

My only hesitation is that it might be a bit semantically confusing: abra app <app> rm implies to me that it's gonna do something involving removing the app, which the current behaviour does, but I'm not sure I'd expect to find "delete the volumes, leave the app alone" under the same command 🤔

Fine to put it there and move it later, though.

Yeah `abra app <app> rm (--config|--volumes|--secrets|--all)` seems like the smallest change to the command line UI. My only hesitation is that it might be a bit semantically confusing: `abra app <app> rm` implies to me that it's gonna do something involving removing the app, which the current behaviour does, but I'm not sure I'd expect to find "delete the volumes, leave the app alone" under the same command 🤔 Fine to put it there and move it later, though.

👍 1

3wordchant referenced this issue

2021-05-30 18:20:48 +00:00

Add `app .. volume` commands, working vol deletion #166

decentral1se commented

2021-05-30 20:05:06 +00:00

#166

https://git.autonomic.zone/coop-cloud/abra/pulls/166

decentral1se closed this issue

2021-05-30 20:05:06 +00:00

This repo is archived. You cannot comment on issues.