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

[Suggestion] axiom-rm "\*" deletes more than you would expect 🙄 #674

Closed
joaopfcruz opened this issue Jan 23, 2023 · 2 comments
Closed

Comments

@joaopfcruz
Copy link
Contributor

axiom-rm can sometimes produce this error:

No instance supplied, use '\*' if you want to delete all instances

I was expecting that this would only delete any fleet created beforehand. Unfortunately I've learned the hard way that on DigitalOcean (at least) this deletes all fleets BUT ALSO ANY OTHER DROPLETS 😅

I would suggest to change the error message to warn the user about this.

@0xtavian
Copy link
Collaborator

@joaopfcruz lol for what it’s worth you’re not the first one to do this. The error message does say use '\*' if you want to delete all instances, you also likely added -f so you didn’t see any confirmation prompts. This happens with all providers. If you want to submit a PR to change the warning, I’ll merge it. :)

@joaopfcruz
Copy link
Contributor Author

#677

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