chaostoolkit / chaostoolkit-lib

The Chaos Toolkit core library
https://chaostoolkit.org/
Apache License 2.0
77 stars 46 forks source link

after_activity_control state.status is "succeeded" but probe tolerance wasn't met #245

Closed WixoLeo closed 1 year ago

WixoLeo commented 2 years ago

Whenever "after_activity_control" being invoked, doesn't matter if probe's tolerance is met or not, the status is always "succeeded".

image

Question is if it's intended to be this way or should it be failed only if a probe raises and exception? If it's intended, then it would be great to have a tolerance status as well.

Runtime versions

Python 3.9.2
NAME                VERSION   
CLI                 1.10.0    
Core library        1.22.1

To Reproduce

Expected behavior If tolerance of a probe did not met, expected to see the "status" to be "tolerance-not-met" or something like that.

Additional context In the example repo I've provided minimal experiment with minimal plugin in order for you to run locally with ease. It contains the following:

Playground plugin contains the following:

github-actions[bot] commented 1 year ago

This Issue has not been active in 365 days. To re-activate this Issue, remove the Stale label or comment on it. If not re-activated, this Issue will be closed in 7 days.

github-actions[bot] commented 1 year ago

This Issue was closed because it was not reactivated after 7 days of being marked Stale.