Kurtiscwright opened a new pull request, #18216:
URL: https://github.com/apache/iceberg/pull/18216

   The REST Compatibility Kit tests start their local `RESTCatalogServer` on 
the fixed default port 8181. With Gradle running test tasks in parallel on a 
host that has `docker compose`, the kafka-connect integration stack publishes 
its REST catalog container on host port 8181 at the same time, and 
`:iceberg-open-api:test` fails with `java.net.BindException: Address already in 
use` (hit while verifying the 1.12.0 RC1 tarball). Any other process on 8181 
does the same. This registers the extension with 
`RESTServerExtension.FREE_PORT`, as the Spark and Flink catalog test bases 
already do, and builds the RCK clients from the extension's resolved config. 
With `rck.local=false` no server starts, so the extension drops the unresolved 
placeholder instead of pointing clients at port 0. Complements #17109. Closes 
#18215.
   
   Test plan: `./gradlew :iceberg-open-api:test` with a dummy listener holding 
`127.0.0.1:8181`, which fails on main and passes with this change.
   
   ---
   **AI Disclosure**
   - Model: Claude Fable 5.1 (claude-fable-5.1)
   - Platform/Tool: Kiro
   - Human Oversight: fully reviewed
   - Prompt Summary: Diagnose why :iceberg-open-api:test failed to bind 8181 
during a parallel Gradle build of the 1.12.0 RC1 tarball and make the RCK local 
server use a free port. The diagnosis, the change and the test run are 
generated; the decision to fix the RCK side rather than the kafka-connect port 
mapping is the author's.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to