-
Notifications
You must be signed in to change notification settings - Fork 118
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
Tests failing with constant keywords in transforms with LogsDB #2218
Comments
@kpollich this would be kind of urgent because there are many tests failing in the daily builds because of this. |
Thanks @jsoriano I added this to our current sprint. |
@chrisberkhout I was working on this issue, using the
This can be reproduced with:
So there seems to be an actual issue. We would need to work on two things:
|
Tried to add a pipeline but I receive the same failure, even if logsdb is not enabled 🤔 elastic/integrations#11750 |
Ah, I think the problem was on the type, I was setting it to boolean |
There is already an open issue about testing transforms: #859 |
@jsoriano Where/how was this originally detected? I don't see any automated issues being opened in elastic/integrations for this failure? Or was this detected via some under-development EP version? Trying to see what we are missing that would have highlighted this sooner (obviously #859 is part of that). It is a real failure for logsdb where synthetic _source adds the constant_keyword value into the _source while the value would not normally be present outside of logsdb since it is only defined in the constant_keyword mapping. This did result in failures for TI integration users in Serverless (where logsdb is on by default). As of elastic/elasticsearch#117182, Elasticsearch has now been changed to NOT insert constant_keywords into synthesized _source. |
Oh ok, so this will get fixed when this change reaches the snapshot versions used in CI. |
Issue is indeed fixed now with latest snapshots https://buildkite.com/elastic/integrations-schedule-daily/builds/401 Closing this. |
Some packages are failing when logsdb is enabled with:
For testing we use the preview transform API, where the value actually doesn't match with the definition, but this doesn't seem to be an issue in a real deployment. Confirm if this is actually not an issue, and adjust the tests so they don't fail on this case.
The text was updated successfully, but these errors were encountered: