Closed yelijing18 closed 3 months ago
Thanks for opening this pull request! Please check out our contributing guidelines. (https://github.com/apache/dolphinscheduler/blob/dev/docs/docs/en/contribute/join/pull-request.md)
Thank you for your contribution. LGTM. But please make CI happy first.
Please execute ./mvnw validate -P helm-doc -pl :dolphinscheduler
in your fork
Thank you for your contribution. LGTM. But please make CI happy first.
Please execute
./mvnw validate -P helm-doc -pl :dolphinscheduler
in your fork
Already done. Thanks for your guidance.
Issues
0 New issues
0 Accepted issues
Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code
Awesome work, congrats on your first merged pull request!
Purpose of the pull request
fix #16410
Brief change log
fix wrong env in values.yaml and Readme for K8s deploy.
Verify this pull request
This pull request is code cleanup without any test coverage.