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.

Sign up for GitHub

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

Jump to bottom

[BUG] Conditional routing test failure #3139

Closed
dlvenable opened this issue Aug 10, 2023 · 0 comments · Fixed by #3152
Closed

[BUG] Conditional routing test failure #3139

dlvenable opened this issue Aug 10, 2023 · 0 comments · Fixed by #3152
Assignees
@dlvenable
Labels
bug Something isn't working
Milestone
v2.4

Comments

@dlvenable
Copy link
Member

Describe the bug

The conditional routing integration test failed.

Router_SingleRouteIT > sending_alpha_events_sends_to_the_sink_with_alpha_only_routes() FAILED
    java.util.ConcurrentModificationException at Router_SingleRouteIT.java:79

https://github.com/opensearch-project/data-prepper/actions/runs/5824982516/job/15795716699
@dlvenable dlvenable added bug Something isn't working untriaged labels Aug 10, 2023
@dlvenable dlvenable self-assigned this Aug 10, 2023
@dlvenable dlvenable moved this from Unplanned to To do in Data Prepper Tracking Board Aug 10, 2023
@dlvenable dlvenable added this to the v2.4 milestone Aug 10, 2023
dlvenable added a commit to dlvenable/data-prepper that referenced this issue Aug 14, 2023
@dlvenable
Fix flaky integration test by wrapping a list in a new list to avoid …
0b3ad53 
…a ConcurrentModificationException. Resolves opensearch-project#3139.

Signed-off-by: David Venable <[email protected]>
@dlvenable dlvenable mentioned this issue Aug 14, 2023
Merged
4 tasks
dlvenable added a commit to dlvenable/data-prepper that referenced this issue Aug 14, 2023
@dlvenable
Fix flaky integration test by wrapping a list in a new list to avoid …
ea37d48 
…a ConcurrentModificationException. Resolves opensearch-project#3139.

Signed-off-by: David Venable <[email protected]>
dlvenable added a commit that referenced this issue Aug 14, 2023
@dlvenable
Fix flaky integration test by wrapping a list in a new list to avoid …
0f70d52 
…a ConcurrentModificationException. Resolves #3139. (#3152)

Signed-off-by: David Venable <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dlvenable dlvenable

Labels
bug Something isn't working
Projects
Data Prepper Tracking Board
Archived in project
Milestone
v2.4
Development

Successfully merging a pull request may close this issue.

Fix flaky integration test in conditional routing integration tests dlvenable/data-prepper
1 participant
@dlvenable