-
Notifications
You must be signed in to change notification settings - Fork 5
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.
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
Refactoring and template trace generator #5
Conversation
Move trace genration from the function GenerateResource into the the ParameterizedGenerator
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Like the direction of the PR, nice work. Left a few comments
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm good to merge. I'll wait to see if @dgzlopes has any thoughts.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Thanks for the work @stoewer 🙇
Refactored the code and introduce the concept of different trace generators. The existing functionality to generate traces is now provided by tracing.ParameterizedGenerator Implement a new generator that creates traces and spans from a configurable template with the goal to generate more realistiacally looking traces
What this PR does:
The purpose of this change is create more realistiacally looking traces with
xk6-client-tracing
.tracing.ParameterizedGenerator
.Which issue(s) this PR fixes:
May help to solve grafana/tempo#902 because
xk6-client-tracing
could be used as a replacement for thesynthetic-load-generator