Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add commands to remove or purge apps #8

Open
saltydk opened this issue Jan 3, 2023 · 3 comments
Open

add commands to remove or purge apps #8

saltydk opened this issue Jan 3, 2023 · 3 comments

Comments

@saltydk
Copy link
Member

saltydk commented Jan 3, 2023

Removal would just remove the container and optionally remove the dns record tied to the machine (unsure if we should mess with dns)
Purge would remove the container and prompt the user if any of the found opt folders (common ones excluded) should be removed as well. Prompt to confirm any data removal.

Suggested template for looking up the folders

docker inspect tautulli | jq -r '.[0].Mounts[] | select(.Source | startswith("/opt")).Source'

If we decide to manage the removal of dns records we could add additional label(s) to specify which are tied to the container instead of hoping traefik labels will stay in the same format.

As currently constructed this would be scoped to container based apps and multiple containers in the same app would still need some kind of label pairing (fairly easy to add going forward) to link them together for this tool to more easily find them in one go. Removal of the paired containers could be optional.

@keldian
Copy link

keldian commented Jan 3, 2023

Maybe have two commands, mimicking apt behavior.

sb remove foo
  • container(s)
  • DNS record(s) (if implemented)
  • ??
sb purge foo
  • container(s)
  • DNS record(s) (if implemented)
  • image and other Docker leftovers
  • app data (/opt/foo etc)
  • ??

When the command is issued, print a rundown of what exactly will be removed, awaiting confirmation.

@saltydk
Copy link
Member Author

saltydk commented Jan 3, 2023

I feel like we should leave the image cleanup to the docker role.

@saltydk
Copy link
Member Author

saltydk commented Jan 3, 2023

Or add a short hand command for it with sb maybe to pair with this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants