dongjoon-hyun opened a new pull request, #929: URL: https://github.com/apache/spark-kubernetes-operator/pull/929
### What changes were proposed in this pull request? This PR aims to fix the Helm command examples that fail as written, following [SPARK-53639](https://github.com/apache/spark-kubernetes-operator/pull/323). - `docs/operations.md`: Add the missing release name `spark` to the `--set` example. - `tests/e2e/helm/helm-test-values/*/values.yaml`: Fix the `Usage:` comments to match the CI, i.e., run from the repository root with the release name `spark`. ### Why are the changes needed? SPARK-53639 unified the Helm release name to `spark`, but missed these examples. - The `--set` example fails with `must either provide a name or specify --generate-name`. - The `Usage:` comments use `spark-operator` and mix the chart path `.` with repository-root `-f` paths, so they fail in any directory. `configmap-metadata` also refers to a non-existent file. ### Does this PR introduce _any_ user-facing change? No behavior change. Only a documentation example and test comments are fixed. ### How was this patch tested? Manually ran the fixed commands with `helm install --dry-run=client` from the repository root. ### Was this patch authored or co-authored using generative AI tooling? Generated-by: Claude Opus 5.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]
