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 #6228) otel: add test for filebeatreceiver delivery guarantees during a collector restart #6788

Merged
merged 2 commits into from
Feb 11, 2025

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented Feb 10, 2025

What does this PR do?

This commit adds a integration test that restarts the collector a couple times during the log ingestion and confirms that the data was fully written to elasticsearch.

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/fragments using the changelog tool
  • I have added an integration test or an E2E test

Related issues

…ector restart (#6228)

* otel: add test for filebeatreceiver delivery guarantees for a collector restart

This commit adds a integration test that restarts the collector a couple
times during a log ingestion and confirms that the data was fully written
to elasticsearch.

* check that input lines increase after each restart

* use json for docs, use json parser

* add debug exporter, remove fake component from fixture

* fix typo in json input

* add newlines to input file, fixes for recent fingerprint changes

* assert unique log lines

* fix restart counter logic

* update test to restart collector only once

* remove some leftover code

(cherry picked from commit 346a2fe)
@mergify mergify bot requested a review from a team as a code owner February 10, 2025 15:44
@mergify mergify bot added the backport label Feb 10, 2025
@mergify mergify bot requested review from michalpristas and blakerouse and removed request for a team February 10, 2025 15:44
@mauri870 mauri870 requested review from mauri870 and removed request for michalpristas and blakerouse February 10, 2025 16:18
@mauri870 mauri870 enabled auto-merge (squash) February 10, 2025 16:19
Copy link

@mauri870 mauri870 merged commit 2140820 into 9.0 Feb 11, 2025
14 checks passed
@mauri870 mauri870 deleted the mergify/bp/9.0/pr-6228 branch February 11, 2025 16:26
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.

1 participant