We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
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
This originally started as a PR comment: #1863 (comment)
The context hierarchy is a bit unexpected, we have:
parentCtx -> maxDurationCtx -> regDurationCtx \-> maxDurationCtxWithScenarioState -> vuCtx
So someone expecting to have the ScenarioState in regDurationCtx might be surprised. Something like this might be better:
ScenarioState
regDurationCtx
parentCtx -> parentCtxWithScenarioState -> maxDurationCtx -> regDurationCtx \-> vuCtx
Unfortunately, there are some implementation chicken-and-egg problems that were described in the PR thread 😞
Other somewhat related things:
Run()
The text was updated successfully, but these errors were encountered:
k6/x/execution
No branches or pull requests
This originally started as a PR comment: #1863 (comment)
The context hierarchy is a bit unexpected, we have:
So someone expecting to have the
ScenarioState
inregDurationCtx
might be surprised. Something like this might be better:Unfortunately, there are some implementation chicken-and-egg problems that were described in the PR thread 😞
Other somewhat related things:
Run()
methods and other optimizations and cleanups (Refactor the externally-controlled executor and add tests #1524, Refactor the module structure for the new executors #1302, Optimize ramping-vus #1499)The text was updated successfully, but these errors were encountered: