Skip to content

Add manticore header to unit tests to avoid transient error#205

Merged
alexcams merged 2 commits into
logstash-plugins:mainfrom
alexcams:fix-unit-tests-transient-error-uninitialized-manticore
Oct 17, 2025
Merged

Add manticore header to unit tests to avoid transient error#205
alexcams merged 2 commits into
logstash-plugins:mainfrom
alexcams:fix-unit-tests-transient-error-uninitialized-manticore

Conversation

@alexcams

@alexcams alexcams commented Oct 17, 2025

Copy link
Copy Markdown
Contributor

There was a CI transient error while executing elasticsearch_spec.rb unit tests that was making pipelines fail from time to time:
Uninitialized Manticore error
image

This is a concurrency error caused by the indirect inclusion of Manticore library while calling this function that is called by many tests in elasticsearch_spec.rb file. Since the tests execution does not always follow the same order, the problem comes if any test that use Manticore::Client class is executed before calling this function that indirectly includes the Manticore library, it causes this class to be "uninitialized".

To solve this transient error, I've just included the Manticore lib at the beginning of the file.

Thanks for contributing to Logstash! If you haven't already signed our CLA, here's a handy link: https://www.elastic.co/contributor-agreement/

@jsvd

jsvd commented Oct 17, 2025

Copy link
Copy Markdown
Member

Tests are failing due to fixture certs expiring, please run spec/filters/fixtures/generate_test_certs.sh and commit the new certs to this PR

@alexcams alexcams force-pushed the fix-unit-tests-transient-error-uninitialized-manticore branch from 5295e29 to f9d7b79 Compare October 17, 2025 14:43
@alexcams alexcams requested a review from jsvd October 17, 2025 15:25

@jsvd jsvd left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@alexcams alexcams merged commit 2a7bfa4 into logstash-plugins:main Oct 17, 2025
3 checks passed
alexcams pushed a commit to alexcams/logstash-filter-elasticsearch that referenced this pull request Jan 22, 2026
…#234)

Provide field value tracking, persisted to disk on each search_after page.

Adds `:last_value` and `:present` placeholders, allowing the plugin to inject the cursor value and now-30 seconds, respectively, in the query string.

Useful to track new data being written to an index or series of indices.

Works best with nano second precision timestamps added by Elasticsearch's Ingest Pipelines.

---------

Co-authored-by: Joel Andritsch <joel.andritsch@gmail.com>
Co-authored-by: Rob Bavey <rob.bavey@elastic.co>
Co-authored-by: Karen Metts <35154725+karenzone@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants