Skip to content

Added EJB-over-HTTP test, deduplication#5

Open
honza-kasik wants to merge 18 commits intomodcluster:mainfrom
honza-kasik:ejb
Open

Added EJB-over-HTTP test, deduplication#5
honza-kasik wants to merge 18 commits intomodcluster:mainfrom
honza-kasik:ejb

Conversation

@honza-kasik
Copy link
Copy Markdown
Contributor

@honza-kasik honza-kasik commented Mar 26, 2026

This PR adds tests for EJB-over-HTTP functionality.

This never worked with httpd balancer before so one test still has undertow tag.

I'm also trying to resolve some related failover issues. This helped to significantly reduce flakiness also in other scenarios.

@honza-kasik honza-kasik marked this pull request as draft March 26, 2026 12:36
@rhusar
Copy link
Copy Markdown
Member

rhusar commented Mar 26, 2026

@rachmatowicz Please have a look.

@honza-kasik honza-kasik marked this pull request as ready for review April 14, 2026 10:31
@honza-kasik honza-kasik changed the title Draft: Added EJB-over-HTTP test, deduplication Added EJB-over-HTTP test, deduplication Apr 14, 2026
@honza-kasik
Copy link
Copy Markdown
Contributor Author

Ready to be merged.

Comment thread src/test/java/org/jboss/modcluster/test/ejb/EjbViaHttpTest.java Outdated
Comment thread src/test/java/org/jboss/modcluster/test/utils/WildFlyJGroupsManager.java Outdated
@marekkopecky
Copy link
Copy Markdown

I added just two minor comments, otherwise LGTM.

@rhusar Do you prefer to have another review from @rachmatowicz (or maybe @tadamski if Richard doesn't have time)?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants