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

Bump k6 to 0.48.1-973e442 #1142

Merged
merged 2 commits into from
Dec 19, 2023
Merged

Bump k6 to 0.48.1-973e442 #1142

merged 2 commits into from
Dec 19, 2023

Conversation

ka3de
Copy link
Collaborator

@ka3de ka3de commented Dec 18, 2023

What?

Bumps k6 version to v0.48.1-973e442.

Why?

To keep xk6-browser dependencies up to date and in line with k6. Additionally, in this case, k6 has upgraded the OTEL dependency version from v1.19 to v1.21 which is used directly in xk6-browser, so we have to comply with that, which in this case requires fixing the tracing integration test due to a breaking change in the tracer implementation.

Checklist

  • I have performed a self-review of my code
  • I have added tests for my changes
  • I have commented on my code, particularly in hard-to-understand areas

Related PR(s)/Issue(s)

grafana/k6#3503

k6 v0.48.0-973e442 upgrades the OTEL dependency from 1.19 to 1.21, which
includes changes in the tracer implementation that breaks our tracing
integration test.
@ka3de ka3de added maintenance dependencies Pull requests that update a dependency file and removed maintenance labels Dec 18, 2023
@ka3de ka3de marked this pull request as ready for review December 18, 2023 08:49
@ka3de ka3de changed the title Bump/k6 to 973e442 Bump k6 to 973e442 Dec 18, 2023
@ka3de ka3de changed the title Bump k6 to 973e442 Bump k6 to 0.48.1-973e442 Dec 18, 2023
Copy link
Collaborator

@ankur22 ankur22 left a comment

Choose a reason for hiding this comment

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

LGTM

@ka3de ka3de mentioned this pull request Dec 18, 2023
3 tasks
@ka3de ka3de self-assigned this Dec 18, 2023
@ka3de ka3de merged commit a3aafdf into main Dec 19, 2023
17 checks passed
@ka3de ka3de deleted the bump/k6-to-973e442 branch December 19, 2023 08:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants