Skip to content
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

Use xk6-client-tracing in examples to generate traces #1920

Merged
merged 5 commits into from
Dec 1, 2022

Conversation

stoewer
Copy link
Contributor

@stoewer stoewer commented Nov 30, 2022

What this PR does:

Replace the synthetic-load-generator with xk6-client-tracing in docker-compose examples.

The grafana7.4 example still uses the load generator because it requires the generated trace IDs to be logged for querying.

Upgrade Grafana used in the examples to the latest version.

Which issue(s) this PR fixes:

Service graphs are now working in all docker-compose examples that use xk6-client-tracing.

Fixes #902

Note
It would be great if a reviewer could test the updated examples with a M1 Macbook to make sure #902 is solved

Checklist

  • Tests updated
  • Documentation added
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

Replace the synthetic-load-generator with xk6-client-tracing
in docker-compose examples

The grafana7.4 example still uses the load generator because
it requires the generated trace IDs to be logged for querying
The file is no longer used by multiple examples
@stoewer stoewer force-pushed the examples-replace-load-generator branch from 08eb4f2 to 8428d88 Compare December 1, 2022 02:49
@stoewer stoewer marked this pull request as ready for review December 1, 2022 03:06
@joe-elliott
Copy link
Member

Nice! Now I can get a macbook pro ;)

@joe-elliott joe-elliott merged commit f5109c5 into grafana:main Dec 1, 2022
@stoewer stoewer deleted the examples-replace-load-generator branch December 1, 2022 22:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Docker Compose examples do not work on Apple M1 Hardware
2 participants