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

minor fix to prevent flaky tests in downstream plugins #804

Merged
merged 1 commit into from
Feb 21, 2023

Conversation

sbcd90
Copy link
Collaborator

@sbcd90 sbcd90 commented Feb 21, 2023

minor fix to prevent flaky tests in downstream plugins

Issue #, if available:
sample flaky run: https://github.com/phaseshiftg/security-analytics/actions/runs/4228665096/jobs/7344314835

Description of changes:
https://stackoverflow.com/questions/58644738/update-a-field-from-a-elasticsearch-document
https://www.elastic.co/guide/en/elasticsearch/reference/current/enabled.html

CheckList:
[X] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@sbcd90 sbcd90 requested review from a team and lezzago February 21, 2023 08:25
@sbcd90 sbcd90 merged commit c426173 into opensearch-project:main Feb 21, 2023
opensearch-trigger-bot bot pushed a commit that referenced this pull request Feb 21, 2023
Signed-off-by: Subhobrata Dey <[email protected]>
(cherry picked from commit c426173)
sbcd90 pushed a commit that referenced this pull request Feb 21, 2023
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.

3 participants