Conversation
- Save/restore ~/.m2/wrapper in unit-test sanity-check and matrix jobs - Retry sanity-check ./mvnw spotless:check on wrapper download failures
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
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.
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.zipfrom Maven Central via Maven Wrapper. Central returns HTTP 429; Wrapper has no retry, soInstall Dependenciesfails before tests run.This reuses the existing
sanity-checkjob (the matrix alreadyneeds: sanity-check) to save~/.m2/wrapper, then restores the same cache in matrix jobs. Jobneedsandfail-fastare unchanged.Closes #18656
Example failure: https://github.com/apache/dolphinscheduler/actions/runs/35494682258/job/106035745127?pr=18585
Brief change log
~/.m2/wrapperinunit-test.ymlsanity-checkand matrix jobs (key frommaven-wrapper.properties, notpom.xml)./mvnw spotless:checkin the sanity-check composite action (backoff + jitter) so a single cold-start 429 can recoverVerify this pull request
This pull request is CI workflow/action YAML without unit-test coverage.
Testworkflow: Sanity Check saves Maven Wrapper cache; matrix jobs restore the same key and do not downloadapache-maven-*-bin.zipfrom CentralNon-goals
fail-fast: falsedistributionUrl./mvnw install/verify