applicationset-progressive-sync icon indicating copy to clipboard operation
applicationset-progressive-sync copied to clipboard

Add the ability to cancel a rollout

Open sledigabel opened this issue 3 years ago • 2 comments

We need a way to signal to the APS that a progressive rollout is cancelled by a user.

That includes a new final state (Cancelled) and some way of notifying the APS object of the cancellation.

sledigabel avatar Jun 02 '21 11:06 sledigabel

What's the difference with https://github.com/Skyscanner/applicationset-progressive-sync/issues/14 ?

maruina avatar Jun 04 '21 07:06 maruina

mmmh good point. I think the difference is that pause implies a resume somehow, whereas cancel is actually cancelled for good. it could be restarted but not resumed.

It could be just semantics here, the idea was to provide some way for a user to cancel an ongoing deployment. If we want to make it part of pause, or expand pause with an extra use case, I'm all up for no duplicate effort

sledigabel avatar Jun 04 '21 08:06 sledigabel