Compile and run tests in Claude cloud environment #3644
Open
luis100 wants to merge 1 commit intodevelopmentfrom
Open
Compile and run tests in Claude cloud environment #3644luis100 wants to merge 1 commit intodevelopmentfrom
luis100 wants to merge 1 commit intodevelopmentfrom
Conversation
The default zkConnectTimeout of 15s in SolrJ causes SolrZkClient to call ZooKeeper.close() if the ZK session isn't established in time. That close() sends a CLOSESESSION packet then waits indefinitely in Object.wait() with no timeout, because the ZK background threads are already gone — hanging the test JVM forever. Changes: - RodaCoreFactory.instantiateSolr(): add withZkConnectTimeout(300s) to the CloudSolrClient.Builder so SolrZkClient has enough time to establish the ZK session before panicking - roda-core.properties: raise connect timeout to 300s, document zk.client.timeout_ms - TestContainersManager: set zkConnectTimeout and RODA_CORE_SOLR_CLOUD_ZK_CONNECT_TIMEOUT_MS system properties as belt-and-suspenders fallback - roda-core-tests/pom.xml: add 172.*/192.* to http.nonProxyHosts in surefire argLine so Docker container IPs bypass the HTTPS proxy - testng-single.xml: single-class TestNG suite file for fast targeted test runs - CLAUDE.md: document cloud environment workflow (Docker startup, build commands, single-test shortcut, proxy quirks, ZK/Solr notes, pre-PR checklist) Verified: IndexServiceTest 17/17 tests pass with this fix. https://claude.ai/code/session_01XRhDcKRQtCBua46PGom5dx
dosubot
bot
added
the
size:XXL
luis100
changed the title
dosubot
bot
added
size:L
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
No description provided.