-
Notifications
You must be signed in to change notification settings - Fork 528
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
Bump apm-perf in systemtest and tools #15379
base: main
Are you sure you want to change the base?
Conversation
…default apm-server tools is using an outdated version of apm-perf that does not enable rewrite-timestamps and rewrite-ids in apmbench. Bump apm-perf dependency to enable rewrite-timestamps and rewrite-ids. Not rewriting ids is causing TBS to reuse trace ids and only output a subset of events.
This pull request does not have a backport label. Could you fix it @carsonip? 🙏
|
|
No urgency to this as #15380 is merged |
This pull request is now in conflicts. Could you fix it @carsonip? 🙏
|
This will fail CI as new apm-perf moved loadgen into an internal package. |
Motivation/summary
apm-server tools is using an outdated version of apm-perf that does not
enable rewrite-timestamps and rewrite-ids in apmbench. Bump apm-perf
dependency to enable rewrite-timestamps and rewrite-ids. Not rewriting
ids is causing TBS to reuse trace ids and only output a subset of
events.
Checklist
For functional changes, consider:
How to test these changes
Related issues