SEZ9 commented on PR #12298: URL: https://github.com/apache/seatunnel/pull/12298#issuecomment-5853027041
@DanielLeens Fixed the remaining checkpoint-test port lookup in `27cc91518e9c`, following Option A from review `5328937823`: `testCheckpointOverviewAndHistoryApi` now calls `buildHttpBaseUrl(httpPort(node1))` instead of reading `node1Config.getEngineConfig().getHttpConfig().getPort()`. This is the only change from the reviewed head `6f00c681b385`, folded into the existing feature commit. Production code and the test assertions are unchanged. Verification on the updated tree: - Spotless and full-reactor `./mvnw -q -DskipTests verify` passed with OpenJDK 11. - I reserved port 8080 before running the existing `RestApiIT#testCheckpointOverviewAndHistoryApi`. Jetty's startup logs confirmed node1 on 8081 and node2 on 8082. The complete selected integration test passed: 1 test, 0 failures, 0 errors, 0 skips. This directly exercises the occupied-configured-port condition behind your finding. [Fresh Build run for `27cc91518e9c`](https://github.com/SEZ9/seatunnel/actions/runs/36297430798) is now queued/running and under monitoring. The prior head's successful jobs are only historical evidence; I am not calling the new head green. #12489 is still the independent prerequisite for the remaining OceanBase packaging failure. -- 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]
