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

[9.0](backport #6966) Fix config used in Otel manager test #6993

Open
wants to merge 1 commit into
base: 9.0
Choose a base branch
from

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented Feb 24, 2025

What does this PR do?

Otel collector exposes metrics on port 8888 by default, which can cause conflicts with other applications in tests. Disable this.

Use the debugexporter and nopreceiver in the test instead of the otlpexporter and otlpreceiver. This is faster and doesn't require binding to any ports as well.

Why is it important?

Tests should not be flaky.

Checklist

  • I have read and understood the pull request guidelines of this project.
  • My code follows the style guidelines of this project

Related issues


This is an automatic backport of pull request #6966 done by [Mergify](https://mergify.com).

Otel collector exposes metrics on port 8888 by default, which can cause
conflicts with other applications in tests. Disable this.

Use the nopexporter and debugpreceiver in the test instead of the
otlpexporter and otlpreceiver. This is faster and doesn't require
binding to any ports as well.

(cherry picked from commit d68ade9)
@mergify mergify bot requested a review from a team as a code owner February 24, 2025 15:49
@mergify mergify bot added the backport label Feb 24, 2025
@mergify mergify bot requested review from michel-laterman and pchila and removed request for a team February 24, 2025 15:49
@elasticmachine
Copy link
Contributor

Pinging @elastic/elastic-agent-control-plane (Team:Elastic-Agent-Control-Plane)

@swiatekm swiatekm enabled auto-merge (squash) February 24, 2025 15:51
Copy link

@elasticmachine
Copy link
Contributor

💔 Build Failed

Failed CI Steps

cc @swiatekm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants