det101 opened a new pull request, #18657: URL: https://github.com/apache/dolphinscheduler/pull/18657
## Was this PR generated or assisted by AI? YES. Implementation was assisted by AI (CI log analysis, change drafting, and PR text). The approach and scope were reviewed before landing. ## Purpose of the pull request Fix CI flake where the unit-test matrix (~44 parallel jobs) all download `apache-maven-3.8.4-bin.zip` from Maven Central via Maven Wrapper. Central returns HTTP 429; Wrapper has no retry, so `Install Dependencies` fails before tests run. This reuses the existing `sanity-check` job (the matrix already `needs: sanity-check`) to save `~/.m2/wrapper`, then restores the same cache in matrix jobs. Job `needs` and `fail-fast` are unchanged. Closes #18656 Example failure: https://github.com/apache/dolphinscheduler/actions/runs/35494682258/job/106035745127?pr=18585 ## Brief change log - Cache `~/.m2/wrapper` in `unit-test.yml` `sanity-check` and matrix jobs (key from `maven-wrapper.properties`, not `pom.xml`) - Retry `./mvnw spotless:check` in the sanity-check composite action (backoff + jitter) so a single cold-start 429 can recover ## Verify this pull request This pull request is CI workflow/action YAML without unit-test coverage. - YAML parses locally - Confirm on this PR's `Test` workflow: Sanity Check saves Maven Wrapper cache; matrix jobs restore the same key and do not download `apache-maven-*-bin.zip` from Central ## Non-goals - Do not set `fail-fast: false` - Do not change `distributionUrl` - Do not add retry around matrix `./mvnw install` / `verify` - Do not add a dedicated warmup job or change backend/e2e/api-test structure -- 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]
