dongjoon-hyun opened a new pull request, #57632:
URL: https://github.com/apache/spark/pull/57632
### What changes were proposed in this pull request?
This PR improves the K8s integration tests `README.md` to match the current
scripts and build:
- Document the default mode which builds Docker images directly from the
current source tree, and
remove the stale `TODO` which was already implemented.
- Add a new `Running the YuniKorn Integration Tests` section, symmetric to
the Volcano section.
- Document missing script flags (`--image-repo`, `--include-tags`,
`--exclude-tags`,
`--default-exclude-tags`, `--java-version`, `--hadoop-profile`,
`--skip-building-dependencies`)
and properties (`test.include.tags`, `test.exclude.tags`,
`test.default.exclude.tags`).
- Fix the default value of `spark.kubernetes.test.dockerFile` and clarify
the Minikube version
requirement (1.28.0 enforced, 1.38.0 tested), and remove an outdated note.
### Why are the changes needed?
The README has drifted from the actual behavior. Accurate documentation
helps developers run the
K8s integration tests correctly.
### Does this PR introduce _any_ user-facing change?
No. This is a documentation-only change.
### How was this patch tested?
Manual review.
### Was this patch authored or co-authored using generative AI tooling?
Generated-by: Claude Fable 5
--
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]
