dongjoon-hyun opened a new pull request, #979:
URL: https://github.com/apache/spark-kubernetes-operator/pull/979

   ### What changes were proposed in this pull request?
   
   This PR aims to add the name getters of the master/worker `Service`s and the 
worker `NetworkPolicy` of `SparkCluster` to `SparkClusterResourceSpec`, and to 
use them when building these resources.
   
   - `getMasterServiceName` returns `<clusterName>-master-svc`.
   - `getWorkerServiceName` returns `<clusterName>-worker-svc`.
   - `getWorkerNetworkPolicyName` returns `<clusterName>-worker`.
   
   ### Why are the changes needed?
   
   This PR is a preparation step for releasing the resources of a 
`SparkCluster` which fails with `SchedulingFailure`. Like the name getters of 
the master/worker `StatefulSet`s and the worker 
`HorizontalPodAutoscaler`/`PodDisruptionBudget`, which SPARK-59750 added for 
`ClusterSuspendStep`, the new getters will be used to delete these resources by 
name.
   
   ### Does this PR introduce _any_ user-facing change?
   
   No. This is a refactoring and the resource names are unchanged.
   
   ### How was this patch tested?
   
   Pass the CIs. The existing `SparkClusterResourceSpecTest` already verifies 
these names, e.g., `cluster-name-master-svc`, `cluster-name-worker-svc`, and 
`cluster-name-worker`.
   
   ### 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]

Reply via email to