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

Script to uninstall Daytona #640

Open
osslate opened this issue Jun 5, 2024 · 2 comments
Open

Script to uninstall Daytona #640

osslate opened this issue Jun 5, 2024 · 2 comments
Labels
Discussion Needs consensus enhancement New feature or request

Comments

@osslate
Copy link

osslate commented Jun 5, 2024

Is your feature request related to a problem? Please describe.
As a Daytona user, I want to entirely remove Daytona and any directories/files it may have created during installation or runtime.

Describe the solution you'd like
A one-line script I can run to uninstall Daytona entirely from the system. It should remove the binary, config, any images, etc. returning the system to the state it was in before Daytona was installed.

Additional context
In reference to documentation issue daytonaio/docs#54

I've considered documenting it manually for users in the docs, but this doesn't provide a great user experience. It also requires maintenance of instructions in the docs that could easily be scripted as the installation procedure is.

When using a package manager, there's a one-line method of uninstalling a package and cleaning up any leftover files/directories. e.g. brew uninstall git, dnf uninstall podman, etc. With the recommended installation of Daytona, this doesn't exist.

Since we have an installation script, we should make uninstalling Daytona just as straightforward for the user.

@osslate osslate added the enhancement New feature or request label Jun 5, 2024
@Tpuljak
Copy link
Member

Tpuljak commented Jun 5, 2024

Just a note, this would probably just include running daytona purge and then removing the binary from the system.

It might be just worthwhile writing this in the README/docs and not introducing a script for it.

@osslate
Copy link
Author

osslate commented Jun 5, 2024

Maybe it doesn't warrant an entirely separate script, but it could be a function in the existing install.sh script (e.g. install.sh uninstall).

Currently, to install:

  • run ./install.sh

and to uninstall:

  • run daytona purge
  • locate binary on your machine based on your OS
  • rm <binary>

It might seem small, but from a UX perspective, having ./install.sh uninstall (or ./uninstall.sh) as a one-liner is much more straightforward. It should be just as straightforward to uninstall Daytona as it is to install Daytona; it's why package managers have an uninstall command, rather than expecting the user to issue multiple commands. Even if the uninstall commands are small.

@idagelic idagelic added the Discussion Needs consensus label Jul 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Discussion Needs consensus enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants