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

Clarify disable/delete in sensu-cli #2677

Closed
asachs01 opened this issue Feb 4, 2019 · 1 comment
Closed

Clarify disable/delete in sensu-cli #2677

asachs01 opened this issue Feb 4, 2019 · 1 comment
Assignees
Labels
bug component:sensuctl Sensu CLI improvements
Milestone

Comments

@asachs01
Copy link

asachs01 commented Feb 4, 2019

When disabling/deleting a user, the verbiage used in the cli tool seems unclear to me. IMO there's a difference between disabling and deleting a user. Disabling would imply that the user's account is disabled, but the account still exists. Deleting would imply that the account is removed entirely.

Expected Behavior

Sensu cli has a "delete" verb.

Current Behavior

Sensu cli currently has "disable" as a verb, which actually deletes the user resource:
disable-delete

Possible Solution

Change the verb to delete

@asachs01
Copy link
Author

asachs01 commented Feb 4, 2019

Just noticed #2540 thanks to @apaskulin

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug component:sensuctl Sensu CLI improvements
Projects
None yet
Development

No branches or pull requests

3 participants