Client deletion is delayed (202), deleted displayed memonetarily

sensu / sensu-dashboard

A dashboard for Sensu, for displaying & managing events & clients.

http://sensuapp.org

MIT License

96 stars 44 forks source link

Open portertech opened 11 years ago

portertech commented 11 years ago

Perhaps client deletion should mark the model w/ deleted = true, and always filter those out?

amdprophet commented 11 years ago

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

portertech commented 11 years ago

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

amdprophet commented 11 years ago

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

portertech commented 11 years ago

Will test shortly.

amdprophet commented 11 years ago

Is this working correctly?