Skip to content
This repository has been archived by the owner on May 18, 2020. It is now read-only.

Convert production cassandra config to a template with volumeClaimTemplate support #89

Merged
merged 3 commits into from
Sep 14, 2018
Merged

Conversation

gregswift
Copy link
Contributor

@gregswift gregswift commented Aug 19, 2018

Which problem is this PR solving?

  • Resolves Add VolumeClaimTemplates for Cassandra #54 (Adding support for volumeClaimTemplates to the cassandra statefulset)
  • Setups the StatefulSet to support scaling up the cluster with persistent volume claims
  • Conversion to template is required to parameterize things like size and pool names for the PVC

Short description of the changes

The built in cassadnra template doesnt really work with persistent data. Changing it to a template, and introducing volumeClaimTemplate in the StatefulSet makes it so its usable for being able to scaling at all.

That being said apparently according to #30 it doesnt seem like there is a goal of making the cassandra* bits in this repo prod ready?

Signed-off-by: Greg Swift <gregswift@gmail.com>
…s its own pvc

Signed-off-by: Greg Swift <gregswift@gmail.com>
@gregswift gregswift changed the title WIP: Convert production cassandra config to a template with volumeClaimTemplate support Convert production cassandra config to a template with volumeClaimTemplate support Aug 20, 2018
@jpkrohling
Copy link
Collaborator

That being said apparently according to #30 it doesnt seem like there is a goal of making the cassandra* bits in this repo prod ready?

That's correct: we provide the template as a starting point, but we have no intention in getting into the business of providing a production-grade Cassandra installation :-)

spec:
terminationGracePeriodSeconds: 1800
containers:
- name: ${CASSANDRA_SERVICE_NAME
Copy link
Collaborator

Choose a reason for hiding this comment

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

Isn't it missing a }?

app: jaeger
jaeger-infra: cassandra-statefulset
spec:
serviceName: ${CASSANDRA_SERVICE_NAME
Copy link
Collaborator

Choose a reason for hiding this comment

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

Isn't it missing a }?

@jpkrohling
Copy link
Collaborator

Thanks for the PR! Looks good to me, except for a couple of places where it seems a } is missing.

Signed-off-by: Greg Swift <gregswift@gmail.com>
@rackergs
Copy link

I thought i had caught all of those :/

updated.

my whole point in making this happen was to actually use this as the prod data store because it was already openshift prepared. So i'm a bit torn around what to move forward with :/

@jpkrohling
Copy link
Collaborator

my whole point in making this happen was to actually use this as the prod data store because it was already openshift prepared. So i'm a bit torn around what to move forward with :/

You probably know this better than me, but having Cassandra on Kubernetes/OpenShift is tricky, as pods come and go without notice. As far as I understand, this is not a pattern that Cassandra is particularly fond of. @pavolloffay and/or @burmanm can probably give you more details, as they spent more time on it than I have. Besides, at the time we wrote this OpenShift template, there hasn't been any official/supported way of running Cassandra on them, other than a few quick starts/tutorials here and there. This might have changed, though.

In any case, the main point is that we expect users to understand what they are doing with their Cassandra installation and be able to self-support: we can help with the Jaeger parts, but we can't help with the Cassandra parts.

Copy link
Collaborator

@jpkrohling jpkrohling left a comment

Choose a reason for hiding this comment

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

LGTM, but I'll give other a chance to review as well and will merge by tomorrow at most, if there are no concerns.

@jpkrohling jpkrohling merged commit b9909ba into jaegertracing:master Sep 14, 2018
@jpkrohling
Copy link
Collaborator

Sorry for taking so long to merge this!

@gregswift gregswift deleted the cassandra-pvc branch September 14, 2018 14:01
@gregswift
Copy link
Contributor Author

thanks

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants