Skip to content

Allow k6 scenario to be defined within k6 test #1797

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 30 commits into
base: master
Choose a base branch
from
Open

Allow k6 scenario to be defined within k6 test #1797

Changes from 1 commit
Commits
Show all changes
30 commits
Select commit Hold shift + click to select a range
7a3e253
Add K6 own get_load method
asos-benpearson Nov 29, 2023
9d8904b
add vscode to gitignore
asos-benpearson Jan 8, 2024
0ca4a2c
added change file
asos-benpearson Jan 8, 2024
bc33321
Merge branch 'master' of https://github.com/benpearson84/taurus into …
asos-benpearson Jan 30, 2024
8334e75
add tests
asos-benpearson Jan 30, 2024
32085e8
fix broken test
asos-benpearson Jan 30, 2024
5c6a3ee
Merge branch 'master' of https://github.com/benpearson84/taurus into …
asos-benpearson Mar 6, 2024
35a8536
fix tests to cover new get_load method
asos-benpearson Mar 6, 2024
11f6c4b
missing self parameter
asos-benpearson Mar 6, 2024
28bcb56
fix broken test
asos-benpearson Mar 6, 2024
dd259dd
fix test_get_load_str_fail test
asos-benpearson Mar 6, 2024
1c2d174
improve code coverage
asos-benpearson Mar 6, 2024
9334acd
Update docymentation to include example yaml
asos-benpearson Mar 7, 2024
3f27103
Merge branch 'Blazemeter:master' into fix/k6-scenario-within-k6-test
asos-benpearson Apr 18, 2024
06c7c47
Update __init__.py
asos-benpearson Apr 19, 2024
cd1b215
Merge branch 'master' of https://github.com/benpearson84/taurus into …
asos-benpearson Aug 12, 2024
3889e0f
Merge branch 'fix/k6-scenario-within-k6-test' of https://github.com/b…
asos-benpearson Aug 12, 2024
53b15f1
Merge pull request #1 from Blazemeter/master
asos-benpearson Jan 21, 2025
4c13ba1
Merge branch 'master' of https://github.com/benpearson84/taurus into …
asos-benpearson Jan 21, 2025
0a04df6
Merge pull request #2 from benpearson84/fix/k6-scenario-within-k6-test
asos-benpearson Jan 21, 2025
d62ecf8
Remove --out to csv file as not required
asos-benpearson Jan 21, 2025
ef851b4
fix broken kpi test
asos-benpearson Jan 21, 2025
98f9963
Allow KPI csv to be disabled via modifications block within scenario
asos-benpearson Mar 25, 2025
a1aad4a
Merge branch 'master' of https://github.com/benpearson84/taurus into …
asos-benpearson Mar 25, 2025
60ee59d
git ignore
asos-benpearson Mar 25, 2025
aa7019c
Merge branch 'master' into fix/k6-scenario-within-k6-test
benpearson84 Mar 25, 2025
e0344c5
add vscode gitignore
asos-benpearson Mar 25, 2025
72f34ef
use latest jmeter version
asos-benpearson Mar 27, 2025
3f414b1
Merge branch 'Blazemeter:master' into fix/k6-scenario-within-k6-test
benpearson84 Jul 31, 2025
890c7e7
Merge branch 'Blazemeter:master' into fix/k6-scenario-within-k6-test
asos-benpearson Sep 24, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion bzt/modules/k6.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ def prepare(self):
self.engine.aggregator.add_underling(self.reader)

def startup(self):
cmdline = [self.k6.tool_name, "run", "--out", f"csv={self.kpi_file}"]
cmdline = [self.k6.tool_name, "run"] #, "--out", f"csv={self.kpi_file}"]
Copy link

@aldrichksb aldrichksb Feb 11, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change breaks Taurus' handling of K6 results when more than one scenario is executed in parallel.

In the previous version, Taurus managed the creation of files like kpi-1.csv, kpi-2.csv, etc. automatically.
With this change, we now have to manually configure this setup.

Error:
Child Process Error: Empty results, most likely xxx (K6Executor) failed. Actual reason for this can be found in logs under...

(but the execution was a success. However, due to this error status, the execution is considered a failure in the pipeline)

For a complex execution involving multiple scenarios running in parallel, this increases the complexity of the user's code for something that was previously managed internally.

Therefore, I’d like to understand the reasoning behind removing this default functionality, if possible. Thank you.

Copy link
Author

@asos-benpearson asos-benpearson Mar 24, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah Im aware of this. However, this PR has been raised for over a year with no communication from the project maintainers at all so we have been using this Fork instead. We decided we dont need the results output to file as we output to an InfluxDB instead and we dont care about the overall Taurus result so this 'quick' fix works for us.

If this is going to be seriously considered for merging then Ill spend some more time looking at it but Im not sure the project maintainers are interested.

Copy link
Author

@asos-benpearson asos-benpearson Mar 25, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@aldrichksb this has now been made optional. Disabled via the modifications -> disable block inside the scenarios block.


load = self.get_load()
if load.concurrency:
Expand Down