dongjoon-hyun opened a new pull request, #882: URL: https://github.com/apache/spark-kubernetes-operator/pull/882
### What changes were proposed in this pull request? This PR aims to consolidate the `Kueue` integration tests into the `kueue` mode of the `K8s Integration Tests` job. - Run `helm test spark` after installing the operator in the `kueue` mode, so that `test-rbac.yaml` asserts the opt-in Kueue RBAC rules there. - Move the check that Kueue RBAC is denied by default from `Helm Tests` to the `kueue` mode. It runs after the `chainsaw` test. - Remove the `kueue` group and the `Install Kueue` step from the `Helm Tests` job. - Remove `tests/e2e/helm/helm-test-values/kueue/values.yaml` because `tests/e2e/helm/kueue-config-values.yaml` already sets `operatorRbac.kueue.enabled: true`. ### Why are the changes needed? Currently, the Kueue integration tests are split into two jobs that use the same K8s and Kueue versions. | Job | Kueue Test | | - | - | | `K8s Integration Tests` (`kueue` mode) | `tests/e2e/kueue` Chainsaw test | | `Helm Tests` (`kueue` group) | Kueue RBAC check with `helm test` and the default denial check | Only the `kueue` group of `Helm Tests` needs Kueue, but the whole `Helm Tests` job installs it and waits for it. After this PR, only one job downloads and installs Kueue, and all Kueue tests run in the same place. ### Does this PR introduce _any_ user-facing change? No. This is a test-only change. ### How was this patch tested? Pass the CIs. ### 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]
