Skip to content

Conversation

@belimawr
Copy link
Contributor

What does this PR do?

Use unique index for TestKubernetesJournaldInputOtel

Why is it important?

Closes #11220

Checklist

  • I have read and understood the pull request guidelines of this project.
  • 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/fragments using the changelog tool
  • I have added an integration test or an E2E test

## Disruptive User Impact

How to test this PR locally

# Package DEV=true SNAPSHOT=true EXTERNAL=true DOCKER_VARIANTS="elastic-otel-collector" PACKAGES="docker" PLATFORMS=linux/amd64 mage -v package

# Run the tests
SNAPSHOT=true INSTANCE_PROVISIONER=kind mage -v integration:testKubernetesSingle TestKubernetesJournaldInputOtel

Related issues

Questions to ask yourself

  • How are we going to support this in production?
  • How are we going to measure its adoption?
  • How are we going to debug this?
  • What are the metrics I should take care of?
  • ...

@belimawr belimawr self-assigned this Nov 20, 2025
@belimawr belimawr requested a review from a team as a code owner November 20, 2025 22:43
@belimawr belimawr added the Team:Elastic-Agent-Control-Plane Label for the Agent Control Plane team label Nov 20, 2025
@belimawr belimawr added backport-active-9 Automated backport with mergify to all the active 9.[0-9]+ branches backport-active-8 Automated backport with mergify to all the active 8.[0-9]+ branches labels Nov 20, 2025
@elasticmachine
Copy link
Contributor

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

Copy link
Contributor

@michalpristas michalpristas left a comment

Choose a reason for hiding this comment

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

approved, data plane should be probably owner of this test though as it seems integration specific

@belimawr
Copy link
Contributor Author

approved, data plane should be probably owner of this test though as it seems integration specific

That's a very good point @michalpristas. This test was created to ensure we can always ingest Journald logs when running Elastic Agent in a container, specially now that we depend on jouranlctl existing in the Docker image.

@belimawr belimawr merged commit efafc2a into elastic:main Nov 21, 2025
26 of 30 checks passed
@github-actions
Copy link
Contributor

@Mergifyio backport 8.19 9.1 9.2

@mergify
Copy link
Contributor

mergify bot commented Nov 21, 2025

backport 8.19 9.1 9.2

✅ Backports have been created

mergify bot pushed a commit that referenced this pull request Nov 21, 2025
(cherry picked from commit efafc2a)

# Conflicts:
#	testing/integration/k8s/journald_test.go
mergify bot pushed a commit that referenced this pull request Nov 21, 2025
mergify bot pushed a commit that referenced this pull request Nov 21, 2025
belimawr added a commit that referenced this pull request Nov 21, 2025
(cherry picked from commit efafc2a)

Co-authored-by: Tiago Queiroz <tiago.queiroz@elastic.co>
hayotbisonai pushed a commit to hayotbisonai/elastic-agent that referenced this pull request Nov 23, 2025
belimawr added a commit that referenced this pull request Nov 24, 2025
(cherry picked from commit efafc2a)

Co-authored-by: Tiago Queiroz <tiago.queiroz@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-active-8 Automated backport with mergify to all the active 8.[0-9]+ branches backport-active-9 Automated backport with mergify to all the active 9.[0-9]+ branches skip-changelog Team:Elastic-Agent-Control-Plane Label for the Agent Control Plane team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Flaky Test]: TestKubernetesJournaldInputOtelCondition never satisfied

4 participants