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

StatsD output scheduled removal #1673

Merged
merged 4 commits into from
Aug 2, 2024
Merged

Conversation

olegbespalov
Copy link
Contributor

@olegbespalov olegbespalov commented Jul 26, 2024

What?

In the upcoming k6 v0.53 we've defined a version where we will remove StatsD output. This PR is adjusting the docs to reflect that.

Checklist

  • I have used a meaningful title for the PR.
  • I have described the changes I've made in the "What?" section above.
  • I have performed a self-review of my changes.
  • I have run the npm start command locally and verified that the changes look good.
  • I have made my changes in the docs/sources/next folder of the documentation.

Related PR(s)/Issue(s)

@olegbespalov olegbespalov added the Area: OSS Content Improvements or additions to community/oss documentation label Jul 26, 2024
@olegbespalov olegbespalov requested a review from a team July 26, 2024 11:49
@olegbespalov olegbespalov self-assigned this Jul 26, 2024
@olegbespalov olegbespalov requested review from mstoykov and removed request for a team July 26, 2024 11:49
@olegbespalov olegbespalov requested a review from codebien July 26, 2024 11:49

{{% admonition type="warning" %}}

The built-in StatsD output has been deprecated on k6 v0.47.0 and [scheduled to remove in v0.55.0](https://github.com/grafana/k6/pull/3849). You can continue to use this feature by using the [xk6-output-statsd extension](https://github.com/LeonAdato/xk6-output-statsd), and this guide has been updated to include instructions for how to use it.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Maybe also mention the OpenTelemetry output, WDYT?

Copy link
Contributor

Choose a reason for hiding this comment

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

Probably good idea, but also probably a good idea. Not certain I will do it in the admonition as it goes for all of the specific products (amazon cloudwatch, datadog ...)

But it might actually be better if they are edited to recommend using somethign that is not statsd. Like for example the opentelemetry output

Copy link
Contributor

@codebien codebien left a comment

Choose a reason for hiding this comment

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

Most of the other 3rd-party outputs should already support OTel. We should suggest using directly it.

@olegbespalov olegbespalov requested a review from codebien July 30, 2024 14:26
Copy link
Collaborator

@heitortsergent heitortsergent left a comment

Choose a reason for hiding this comment

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

Made some small suggestions, thanks for creating the shared file! :shipit:

docs/sources/next/shared/statsd-output.md Outdated Show resolved Hide resolved
docs/sources/next/shared/statsd-output.md Outdated Show resolved Hide resolved
docs/sources/next/shared/statsd-output.md Outdated Show resolved Hide resolved
docs/sources/next/shared/statsd-output.md Outdated Show resolved Hide resolved
olegbespalov and others added 2 commits August 2, 2024 08:51
Co-authored-by: Heitor Tashiro Sergent <heitortsergent@gmail.com>
@heitortsergent heitortsergent merged commit 9eb02ec into main Aug 2, 2024
5 checks passed
@heitortsergent heitortsergent deleted the feat/statsd-output-removal branch August 2, 2024 16:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: OSS Content Improvements or additions to community/oss documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants