Chatops rollback check command should compare the last 2 successful deployments
@rspeicher I decided not to block https://gitlab.com/gitlab-com/chatops/-/merge_requests/180 with minor remarks.
I've applied one suggestion myself and opened this follow-up to discuss a couple of improvements/comments.
The following discussions from https://gitlab.com/gitlab-com/chatops/-/merge_requests/180 should be addressed:
- [ ] @nolith started a [discussion](https://gitlab.com/gitlab-com/chatops/-/merge_requests/180#note_508996560):
> It seems that we do not support ordering by `finished_at` :(
- [x] @nolith started a [discussion](https://gitlab.com/gitlab-com/chatops/-/merge_requests/180#note_508996567):
> I will suggest filtering by `status`, we mentioned quite a few times to start triggering the start of a deployment. It think will be safer here to be specific about what we are looking for.
>
> ```suggestion:-0+0
> sort: 'desc',
> status: 'success',
>
> ```
>
> WDYT?
issue