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

Removes statsd output #3849

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Removes statsd output #3849

wants to merge 2 commits into from

Conversation

olegbespalov
Copy link
Contributor

What?

Removes StatsD output from k6 in favor of https://github.com/LeonAdato/xk6-output-statsd

Why?

An external extension exists https://github.com/LeonAdato/xk6-output-statsd.

Checklist

  • I have performed a self-review of my code.
  • I have added tests for my changes.
  • I have run linter locally (make lint) and all checks pass.
  • I have run tests locally (make tests) and all tests pass.
  • I have commented on my code, particularly in hard-to-understand areas.

Related PR(s)/Issue(s)

#3848

Closes #2982

@olegbespalov olegbespalov added breaking change for PRs that need to be mentioned in the breaking changes section of the release notes documentation-needed A PR which will need a separate PR for documentation dependencies Pull requests that update a dependency file metrics-output Issues related to the metrics output (a.k.a. just output) labels Jul 15, 2024
@olegbespalov olegbespalov added this to the v0.55.0 milestone Jul 15, 2024
@olegbespalov olegbespalov self-assigned this Jul 15, 2024
@olegbespalov olegbespalov marked this pull request as ready for review October 11, 2024 06:57
@olegbespalov olegbespalov requested a review from a team as a code owner October 11, 2024 06:57
@olegbespalov olegbespalov requested review from mstoykov and joanlopez and removed request for a team October 11, 2024 06:57
mstoykov
mstoykov previously approved these changes Oct 11, 2024
Copy link
Contributor

@mstoykov mstoykov left a comment

Choose a reason for hiding this comment

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

LGTM

But am not certain what is going on with windows packaging

cmd/outputs.go Outdated Show resolved Hide resolved
@joanlopez
Copy link
Contributor

But am not certain what is going on with windows packaging

Looks like it was a temporary error, something wrong with the NuGet backend made it impossible to download & install one of the dependencies we use for packaging:

[NuGet] Response status code does not indicate success: 503 (Service Unavailable: Back-end server is at capacity).

I just clicked on retry, and it worked now. Apparently they managed whatever was wrong with their backend.

joanlopez
joanlopez previously approved these changes Oct 11, 2024
Copy link
Contributor

@joanlopez joanlopez left a comment

Choose a reason for hiding this comment

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

Except for the tiny typo, it looks good! 🎉

Co-authored-by: Joan López de la Franca Beltran <5459617+joanlopez@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking change for PRs that need to be mentioned in the breaking changes section of the release notes dependencies Pull requests that update a dependency file documentation-needed A PR which will need a separate PR for documentation metrics-output Issues related to the metrics output (a.k.a. just output)
Projects
Development

Successfully merging this pull request may close these issues.

Future of StatsD outputs
3 participants