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.10] Backport #1325 to 2.11 #1363

Merged
merged 1 commit into from
Mar 11, 2024
Merged

[Backport 2.10] Backport #1325 to 2.11 #1363

merged 1 commit into from
Mar 11, 2024

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 6368979 from #1351.

* Set docData to empty string if actual is null (#1325)

Signed-off-by: Chase Engelbrecht <[email protected]>
(cherry picked from commit 008e076)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* Remove not yet introduced parameter fields

Signed-off-by: Chase Engelbrecht <[email protected]>

---------

Signed-off-by: Chase Engelbrecht <[email protected]>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
(cherry picked from commit 6368979)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@engechas
Copy link
Collaborator

engechas commented Jan 8, 2024

The same test is failing for both failing multi-node workflows. It passed for JDK 17 in Multi-node Test Workflow / Build and test Alerting (11) (pull_request). It also passed for JDK 11 locally:

./gradlew ':alerting:integTest' --tests "org.opensearch.alerting.resthandler.WorkflowRestApiIT.test chained alerts and audit alerts for workflows with query level monitor" -Dtests.seed=12297810C3FF245A -Dtests.security.manager=false -Dtests.locale=en-US -Dtests.timezone=America/Mexico_City -Druntime.java=11

...

BUILD SUCCESSFUL in 2m 47s

@eirsep eirsep merged commit ee15812 into 2.10 Mar 11, 2024
22 of 26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants