Skip to content
This repository has been archived by the owner on May 18, 2021. It is now read-only.

Client deletion is delayed (202), deleted displayed memonetarily #119

Open
portertech opened this issue Mar 26, 2013 · 5 comments · Fixed by #121
Open

Client deletion is delayed (202), deleted displayed memonetarily #119

portertech opened this issue Mar 26, 2013 · 5 comments · Fixed by #121
Assignees
@amdprophet
Labels
Bug Feature

Comments

@portertech
Copy link
Contributor

Perhaps client deletion should mark the model w/ deleted = true, and always filter those out?
@amdprophet
Copy link
Member

That will work. It may also be possible to pass {wait: true} as an option to the remove function. I'll do some testing.

@ghost ghost assigned amdprophet Mar 26, 2013
@portertech
Copy link
Contributor Author

@amdprophet The API currently immediately returns w/ a 202, but uses a timer to delete the actual client after 5 sec.

@amdprophet amdprophet mentioned this issue Mar 27, 2013
Merged
@amdprophet
Copy link
Member

@portertech Can you please test this as well? I want to be sure it's working for you before we close this.

@portertech
Copy link
Contributor Author

Will test shortly.

@amdprophet
Copy link
Member

Is this working correctly?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@amdprophet amdprophet

Labels
Bug Feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Wait for clients & events to be removed amdprophet/sensu-dashboard
2 participants
@amdprophet @portertech