NBomber icon indicating copy to clipboard operation
NBomber copied to clipboard

Add threshold for statistics

Open AntyaDev opened this issue 3 years ago • 8 comments

https://k6.io/docs/using-k6/thresholds/

AntyaDev avatar May 06 '21 12:05 AntyaDev

Hey @AntyaDev, do you mean here something like the following (conceptually), or something more sophisticated?

image

pavlogrushetsky avatar May 11 '22 20:05 pavlogrushetsky

Wow, looks really sexy!!!

AntyaDev avatar May 13 '22 16:05 AntyaDev

I think these thresholds should be applied after the test is finished. NodeStats |> applyTresholds [myTresholds]

AntyaDev avatar May 13 '22 16:05 AntyaDev

Otherwise, it should be somehow integrated into the whole execution pipeline

AntyaDev avatar May 13 '22 16:05 AntyaDev

Okay, great! Just to be clear, this is just a prototype of how I feel I'd like to be able to configure this as a user (not a real implementation in the pipeline) 😄 By the way, do you have any guidelines/recommendations/requirements for the contributors? I noticed that to be able to expose these new models, I need to make changes in NBomber.Contracts, and NBomber itself references NBomber.Contracts as a Nuget package. How should the changes be coordinated between these two projects? Do you have some sort of a prerelease concept for the NBomber.Contracts package that could be used in the scope of a feature development in the NBomber project? I hope, I'm not missing something 🤞 but I would appreciate any guidance. I'd like to dig deeper into this if I have some spare time 😅

pavlogrushetsky avatar May 13 '22 18:05 pavlogrushetsky

And this is a prototype of how this functionality could look like in C#. Please, let me know if this seems good to you.

image

pavlogrushetsky avatar May 14 '22 18:05 pavlogrushetsky

Hey @AntyaDev, here is a draft PR with a rough implementation of the core thresholds logic - https://github.com/PragmaticFlow/NBomber/pull/465. It is rather for review and conversation, because the work is still in progress, and there are some other things I may need to implement/fix. But I would appreciate if you could take a look at it.

pavlogrushetsky avatar May 15 '22 19:05 pavlogrushetsky

Ok, looks amazing. I will take a look and comment.

AntyaDev avatar May 16 '22 04:05 AntyaDev