Skip to content

Add otelbeat binary for test #44855

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.

Sign up for GitHub

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

Jump to bottom
Merged
khushijain21 merged 41 commits into from
Jun 18, 2025
Merged

Add otelbeat binary for test #44855

khushijain21 merged 41 commits into from
Jun 18, 2025

Conversation

@khushijain21
Copy link
Contributor

@khushijain21 khushijain21 commented Jun 17, 2025 •
edited
Loading

Proposed commit message

This PR adds otel test binary that helps us write integration tests close to beats.

Checklist

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have made corresponding change to the default configuration files
  • I have added tests that prove my fix is effective or that my feature works
  • I have added an entry in CHANGELOG.next.asciidoc or CHANGELOG-developer.next.asciidoc.

How to test this PR locally

cd x-pack/filebeat/
go test -c -tags otelbeat
mage docker:composeUp
go test   -tags "integration" -run TestFilebeatOTelE2E ./tests/integration -v

Related issues

khushijain21 and others added 30 commits June 9, 2025 16:10
@khushijain21
add otel test binary
59d3e2e
@khushijain21
improve collector framework
bb6ef5e
@khushijain21
get es client and framework improvement
188bd7c
@khushijain21
otel e2e test works
6e3780a
@khushijain21
address review comments
b897c9d
@khushijain21
Merge branch 'main' into oteltestbinary
ac6aa52
@khushijain21
refactor paths
7b4e809
@khushijain21
check update
4a6c2eb
@khushijain21
check update
4fca789
@khushijain21
Merge branch 'main' into oteltestbinary
4204da4
@khushijain21 @belimawr
Update libbeat/tests/integration/framework.go
c4e3afe 
Co-authored-by: Tiago Queiroz <[email protected]>
@khushijain21 @belimawr
Update libbeat/tests/integration/framework.go
85042a8 
Co-authored-by: Tiago Queiroz <[email protected]>
@khushijain21
Merge branch 'main' into oteltestbinary
4d76a26
@khushijain21
increase timeout
7065f2a
@khushijain21
add collector wait method
3d9a15a
@khushijain21
Merge branch 'main' into oteltestbinary
dcc16f1
@khushijain21
add verbose error logging
8504982
@khushijain21
change context background
c06f914
@khushijain21
add logs
a48f4f8
@khushijain21
add logs
d98b134
@khushijain21
remove home
93fb778
@khushijain21
Merge branch 'main' into oteltestbinary
0735ac7
@khushijain21
changed getESCLient
59a66b5
@khushijain21
Merge branch 'main' into oteltestbinary
ee0e1ab
@khushijain21
use previous test file + GetESClient
8535993
@khushijain21
add error
e1dfc18
@khushijain21
Merge branch 'main' into oteltestbinary
4effb73
@khushijain21
use previous client
1866a4b
@khushijain21
change v7 to v8
3d268e2
@khushijain21
remove err
b0c49ff
@khushijain21 khushijain21 marked this pull request as ready for review June 17, 2025 08:47
@khushijain21 khushijain21 requested review from a team as code owners June 17, 2025 08:47
VihasMakwana
VihasMakwana reviewed Jun 17, 2025
View reviewed changes
@pierrehilbert pierrehilbert added the Team:Elastic-Agent-Data-Plane Label for the Agent Data Plane team label Jun 17, 2025
@botelastic botelastic bot removed the needs_team Indicates that the issue/PR needs a Team:* label label Jun 17, 2025
@elasticmachine
Copy link
Contributor

elasticmachine commented Jun 17, 2025

Pinging @elastic/elastic-agent-data-plane (Team:Elastic-Agent-Data-Plane)

@pierrehilbert pierrehilbert requested review from leehinman and mauri870 and removed request for rdner June 17, 2025 09:17
v1v
v1v approved these changes Jun 17, 2025
View reviewed changes
Copy link
Member

@v1v v1v left a comment

Choose a reason for hiding this comment

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

LGTM, just added a minor comment for the key, other than that, I reviewed only the BK pipeline

mauri870
mauri870 reviewed Jun 17, 2025
View reviewed changes
@khushijain21 khushijain21 merged commit a70c0de into elastic:main Jun 18, 2025
203 checks passed
@khushijain21 khushijain21 added the backport-8.19 Automated backport to the 8.19 branch label Jun 19, 2025
mergify bot pushed a commit that referenced this pull request Jun 19, 2025
@khushijain21 @mergify
Add otelbeat binary for test (#44855)
064f545 
* add otel test binary

(cherry picked from commit a70c0de)
@mergify mergify bot mentioned this pull request Jun 19, 2025
Merged
6 tasks
khushijain21 pushed a commit that referenced this pull request Jun 19, 2025
@mergify
Add otelbeat binary for test (#44855) (#44919)
3716465 
* add otel test binary
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mauri870 mauri870 mauri870 approved these changes

@VihasMakwana VihasMakwana VihasMakwana left review comments

@v1v v1v v1v approved these changes

@leehinman leehinman leehinman approved these changes

Assignees

@khushijain21 khushijain21

Labels

backport-8.19 Automated backport to the 8.19 branch Team:Elastic-Agent-Data-Plane Label for the Agent Data Plane team

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@khushijain21 @elasticmachine @v1v @mauri870 @leehinman @VihasMakwana @pierrehilbert