Skip to content

Conversation

@dongjoon-hyun
Copy link
Member

@dongjoon-hyun dongjoon-hyun commented Oct 14, 2025

What changes were proposed in this pull request?

This PR aims to fix RBAC to allow Spark driver to create StatefulSet.

Why are the changes needed?

We need to fix this to allow Apache Spark's StatefulSetPodsAllocator which was introduced at Apache Spark 3.3.0.

Does this PR introduce any user-facing change?

No, this is an additional permission.

How was this patch tested?

Manual review.

Was this patch authored or co-authored using generative AI tooling?

No.

@github-actions github-actions bot added the BUILD label Oct 14, 2025
@dongjoon-hyun
Copy link
Member Author

Let me merge this since this is only a HelmChart change.

@dongjoon-hyun dongjoon-hyun deleted the SPARK-53909 branch October 15, 2025 00:23
@dongjoon-hyun
Copy link
Member Author

cc @jiangzho

Copy link
Member

@viirya viirya left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

@dongjoon-hyun
Copy link
Member Author

Thank you, @viirya !

dongjoon-hyun added a commit that referenced this pull request Oct 15, 2025
### What changes were proposed in this pull request?

This PR aims to add `StatefulSet`-based SparkApp example.

### Why are the changes needed?

To provide a `StatefulSet` example with `spark.kubernetes.allocation.pods.allocator=statefulset`.

Note that this requires the following is a part of `v0.6.0`.
- #389

### Does this PR introduce _any_ user-facing change?

No, this is a new example.

### How was this patch tested?

Manual review.

```
$ kubectl apply -f examples/pi-statefulset.yaml
sparkapplication.spark.apache.org/pi-statefulset created
```

```
$ kubectl get sparkapp
NAME             CURRENT STATE    AGE
pi-statefulset   RunningHealthy   5s
```

```
$ kubectl get pod
NAME                                        READY   STATUS    RESTARTS   AGE
pi-statefulset-0-driver                     1/1     Running   0          7s
spark-kubernetes-operator-cdb4b547d-5bhfm   1/1     Running   0          80s
spark-s-pi-statefulset-0-0-0                1/1     Running   0          5s
spark-s-pi-statefulset-0-0-1                1/1     Running   0          5s
spark-s-pi-statefulset-0-0-2                1/1     Running   0          5s
```

```
$ kubectl get sparkapp
NAME             CURRENT STATE      AGE
pi-statefulset   ResourceReleased   48s
```

### Was this patch authored or co-authored using generative AI tooling?

No.

Closes #390 from dongjoon-hyun/SPARK-53910.

Authored-by: Dongjoon Hyun <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants