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

[Backport 2.x] Enhance tika document parsing tests #13708

Merged
merged 1 commit into from
May 16, 2024

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport f217270 from #13618.

* Update tika document parsing bwc tests.

Signed-off-by: Carroll <[email protected]>

* Skip sample tika files which do not parse consistently.

Signed-off-by: Carroll <[email protected]>

* Formatting for spotlessJavaCheck.

Signed-off-by: Carroll <[email protected]>

* Use fixed locale for consistent tika parsing.

Signed-off-by: Carroll <[email protected]>

* Move sha1 map to .checksums file.

Signed-off-by: Carroll <[email protected]>

* For locale dependant files do not verify contents with hash.

Signed-off-by: Carroll <[email protected]>

* Remove strict checksum validation for additional locale dependant files.

Signed-off-by: Carroll <[email protected]>

---------

Signed-off-by: Carroll <[email protected]>
(cherry picked from commit f217270)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Copy link
Contributor

❌ Gradle check result for 876d22a: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@reta
Copy link
Collaborator

reta commented May 16, 2024

❌ Gradle check result for 876d22a: FAILURE

#10006
#13473
#10983

Copy link
Contributor

✅ Gradle check result for 876d22a: SUCCESS

@reta reta merged commit 90265f3 into 2.x May 16, 2024
49 of 50 checks passed
@github-actions github-actions bot deleted the backport/backport-13618-to-2.x branch May 16, 2024 16:47
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