Skip to content

[internal/tools] use native Go tool approach#1792

Merged
MrAlias merged 7 commits into
open-telemetry:mainfrom
florianl:go-mod-tool
Apr 10, 2026
Merged

[internal/tools] use native Go tool approach#1792
MrAlias merged 7 commits into
open-telemetry:mainfrom
florianl:go-mod-tool

Conversation

@florianl

@florianl florianl commented Apr 10, 2026

Copy link
Copy Markdown
Member

Summary

Migrate tooling from internal/tools/tools.go to a more Go generic approach so that they can be used with go tool ...

If this change is accepted, I will look into migrating the rest of the tools.

Validation

Signed-off-by: Florian Lehner <florian.lehner@elastic.co>
@codecov

codecov Bot commented Apr 10, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 45.06%. Comparing base (c46a635) to head (e1be18b).
⚠️ Report is 12 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1792   +/-   ##
=======================================
  Coverage   45.06%   45.06%           
=======================================
  Files         339      339           
  Lines       37481    37481           
=======================================
  Hits        16890    16890           
+ Misses      19519    19517    -2     
- Partials     1072     1074    +2     
Flag Coverage Δ
integration-test 21.22% <ø> (-0.03%) ⬇️
integration-test-arm 0.00% <ø> (ø)
integration-test-vm-x86_64-5.15.152 ?
integration-test-vm-x86_64-6.10.6 ?
k8s-integration-test 2.10% <ø> (-0.01%) ⬇️
oats-test 0.00% <ø> (ø)
unittests 48.58% <ø> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@florianl florianl marked this pull request as ready for review April 10, 2026 10:27
@florianl florianl requested a review from a team as a code owner April 10, 2026 10:27
Comment thread Makefile
Signed-off-by: Florian Lehner <florian.lehner@elastic.co>
Signed-off-by: Florian Lehner <florian.lehner@elastic.co>
Signed-off-by: Florian Lehner <florian.lehner@elastic.co>
Signed-off-by: Florian Lehner <florian.lehner@elastic.co>
Signed-off-by: Florian Lehner <florian.lehner@elastic.co>
Signed-off-by: Florian Lehner <florian.lehner@elastic.co>

@mariomac mariomac left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Amazing!

@grcevski grcevski left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

@mmat11 mmat11 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm!

@skl skl mentioned this pull request Apr 10, 2026
2 tasks
@MrAlias MrAlias merged commit dd7fdbc into open-telemetry:main Apr 10, 2026
69 checks passed
pinoOgni pushed a commit to coralogix/opentelemetry-ebpf-instrumentation that referenced this pull request Apr 13, 2026
* [internal/tools] use native Go tool approach

Signed-off-by: Florian Lehner <florian.lehner@elastic.co>

* migrate gotestsum

Signed-off-by: Florian Lehner <florian.lehner@elastic.co>

* migrate ginkgo

Signed-off-by: Florian Lehner <florian.lehner@elastic.co>

* migrate go-licenses

Signed-off-by: Florian Lehner <florian.lehner@elastic.co>

* migrate go-offsets-tracker

Signed-off-by: Florian Lehner <florian.lehner@elastic.co>

* migrate multimod

Signed-off-by: Florian Lehner <florian.lehner@elastic.co>

* fix CI jobs for gotestsum

Signed-off-by: Florian Lehner <florian.lehner@elastic.co>

---------

Signed-off-by: Florian Lehner <florian.lehner@elastic.co>
pinoOgni pushed a commit to coralogix/opentelemetry-ebpf-instrumentation that referenced this pull request Apr 13, 2026
* [internal/tools] use native Go tool approach

Signed-off-by: Florian Lehner <florian.lehner@elastic.co>

* migrate gotestsum

Signed-off-by: Florian Lehner <florian.lehner@elastic.co>

* migrate ginkgo

Signed-off-by: Florian Lehner <florian.lehner@elastic.co>

* migrate go-licenses

Signed-off-by: Florian Lehner <florian.lehner@elastic.co>

* migrate go-offsets-tracker

Signed-off-by: Florian Lehner <florian.lehner@elastic.co>

* migrate multimod

Signed-off-by: Florian Lehner <florian.lehner@elastic.co>

* fix CI jobs for gotestsum

Signed-off-by: Florian Lehner <florian.lehner@elastic.co>

---------

Signed-off-by: Florian Lehner <florian.lehner@elastic.co>
@MrAlias MrAlias added this to the v0.8.0 milestone Apr 16, 2026
@MrAlias MrAlias mentioned this pull request Apr 16, 2026
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.

5 participants