MODCLUSTER-794 mod_proxy_cluster gets stuck with 20 nodes #123
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Rebase done. Because the issue has not been fixed yet, I disabled this test as well (in CI). |
|
So this is a test case for MODCLUSTER-794 – I personally prefer to get test cases merged as disabled and the person fixing the issue fixes and enables the test – @jfclere can weigh in. |
Yes. |
|
Converted to draft PR since there are 'wip' commits. |
jajik
force-pushed
the
MODCLUSTER-794
branch
2 times, most recently
from
75cb207
to
2e82a7b
Compare
|
Marking ready for review. It's no longer the test only, but there is a fix too. |
rhusar
changed the title
Great. @jfclere please review. |
* change workers indexing * simplify code a bit
jfclere
approved these changes
Aug 31, 2023
rhusar
reviewed
Sep 4, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fix and new test for https://issues.redhat.com/browse/MODCLUSTER-794.
Add test for https://issues.redhat.com/browse/MODCLUSTER-794.This branch is based on #120 so I'm opening this as a draft until it is merged.