Skip to content

Conversation

@mauri870
Copy link
Member

@mauri870 mauri870 commented Nov 24, 2025

Proposed commit message

This PR is a follow up to #47338 and updates the remaining tests that still used otelbeat to use the in-process OTel testing collector. To avoid cyclic dependency issues, some tests were relocated and parts of the code were moved into a shared package.

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. Where relevant, I have used the stresstest.sh script to run them under stress conditions and race detector to verify their stability.
  • I have added an entry in ./changelog/fragments using the changelog tool.

How to test this PR locally

cd x-pack/filebeat
mage goIntegTest

Related issues

@mauri870 mauri870 self-assigned this Nov 24, 2025
@mauri870 mauri870 added cleanup Team:Elastic-Agent-Data-Plane Label for the Agent Data Plane team backport-active-all Automated backport with mergify to all the active branches skip-changelog labels Nov 24, 2025
@botelastic botelastic bot added needs_team Indicates that the issue/PR needs a Team:* label and removed needs_team Indicates that the issue/PR needs a Team:* label labels Nov 24, 2025
@github-actions
Copy link
Contributor

🤖 GitHub comments

Just comment with:

  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

@mauri870 mauri870 force-pushed the otelbeat-cyclic-deps-tests branch from 1824c56 to 33d55ca Compare November 24, 2025 20:04
@mauri870 mauri870 force-pushed the otelbeat-cyclic-deps-tests branch from 33d55ca to 76d83af Compare November 24, 2025 20:27
@mauri870 mauri870 marked this pull request as ready for review November 24, 2025 23:14
@mauri870 mauri870 requested review from a team as code owners November 24, 2025 23:14
@elasticmachine
Copy link
Contributor

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

@mauri870 mauri870 requested a review from efd6 November 25, 2025 15:42
@mauri870 mauri870 merged commit 7577d59 into elastic:main Nov 26, 2025
41 checks passed
@github-actions
Copy link
Contributor

@Mergifyio backport 8.19 9.1 9.2

@mergify
Copy link
Contributor

mergify bot commented Nov 26, 2025

backport 8.19 9.1 9.2

✅ Backports have been created

mergify bot pushed a commit that referenced this pull request Nov 26, 2025
(cherry picked from commit 7577d59)

# Conflicts:
#	x-pack/filebeat/input/gcppubsub/otel_test.go
#	x-pack/filebeat/input/gcppubsub/pubsub_test.go
#	x-pack/filebeat/tests/integration/otel_test.go
mergify bot pushed a commit that referenced this pull request Nov 26, 2025
(cherry picked from commit 7577d59)

# Conflicts:
#	x-pack/otel/processor/beatprocessor/integration_test.go
mergify bot pushed a commit that referenced this pull request Nov 26, 2025
(cherry picked from commit 7577d59)

# Conflicts:
#	x-pack/otel/processor/beatprocessor/integration_test.go
mauri870 added a commit that referenced this pull request Nov 26, 2025
…sting collector (#47796)

* otel: fix remaining tests to use in process testing collector (#47772)

(cherry picked from commit 7577d59)

# Conflicts:
#	x-pack/otel/processor/beatprocessor/integration_test.go

* add missing otelconsumer output

---------

Co-authored-by: Mauri de Souza Meneguzzo <mauri870@gmail.com>
@mauri870 mauri870 mentioned this pull request Nov 26, 2025
6 tasks
mauri870 added a commit that referenced this pull request Nov 26, 2025
…esting collector (#47795)

* otel: fix remaining tests to use in process testing collector (#47772)

(cherry picked from commit 7577d59)

# Conflicts:
#	x-pack/otel/processor/beatprocessor/integration_test.go

* add missing output definition to test

---------

Co-authored-by: Mauri de Souza Meneguzzo <mauri870@gmail.com>
andrzej-stencel pushed a commit to andrzej-stencel/beats that referenced this pull request Dec 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-active-all Automated backport with mergify to all the active branches cleanup skip-changelog Team:Elastic-Agent-Data-Plane Label for the Agent Data Plane team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants