Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[wip] Stateful sets #262

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

[wip] Stateful sets #262

wants to merge 2 commits into from

Conversation

jkremser
Copy link
Member

Changing what operator actually deploys. Replication controllers are considered as a low-lvl detail k8s resources and don't allow for durable data (one can't assign the persistent volume to a certain replica managed by RC).

This changes replication controllers to stateful sets, it's wip, because the volumeClaimTemplates are still not there.

Description

after applying the example/cluster.yaml:

λ oc get rc
No resources found.

λ oc get statefulsets
NAME                 READY   AGE
my-spark-cluster-m   1/1     3m6s
my-spark-cluster-w   2/2     3m6s

✨ New feature (non-breaking change which adds functionality)

@jkremser jkremser added the enhancement New feature or request label Oct 22, 2019
@todo
Copy link

todo bot commented Oct 22, 2019

transform the replication controllers into stateful sets if the PV is asked

// // TODO: transform the replication controllers into stateful sets if the PV is asked
// List<Volume> volumes = getVolumes(cluster, isMaster);
// if (volumes != null) {
// rcBuilder.withVolumes(volumes);
// }


This comment was generated by todo based on a TODO comment in fc2d038 in #262. cc @radanalyticsio.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant