[incubator/cassandra-k8s-operator] Add charts for the Orange's Cassandra Kubernetes operator

[allamand]

Signed-off-by: sebastien allamand sebastien.allamand@orange.com

What this PR does / why we need it:

This PR adds a new chart for cassandra-k8s-operator a Cassandra Kubernetes operator (CassKop).

This will ease the management of Cassandra cluster in Kubernetes.

Checklist

[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

• DCO signed
• Chart Version bumped
• Variables are documented in the README.md
• Title of the PR starts with chart name (e.g. [stable/chart])

[k8s-ci-robot]

Hi @allamand. Thanks for your PR.

I'm waiting for a helm member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[maorfr]

hey @allamand,

Have you considered hosting this chart and adding it to helm hub?
https://github.com/helm/hub/blob/master/Repositories.md

[allamand]

Hello @maorfr, actually I have nowhere to store the chart, so I thought it was better to add it in the helm/charts repository.

[allamand]

@maorfr, can we run e2e tests on this, is it ok for you ?

[maorfr]

/ok-to-test

sure, i'll let the tests run
but regarding self hosting - you can use github pages:
https://github.com/helm/hub/blob/master/docs/github.md

wdyt?

[allamand]

I can take a look at github page, but I already use github pages for documentation sharing in the project. It seems to be at this time that it would be more useful in the charts repository but I'm open mind :)

Regarding the tests, they are failing because the CRD is missing when testing the chart.

In fact, I don't create the CRD in the helm willingly to avoid garbage collector of the CRD when deleting the chart. Any suggestion on how I should be doing that?

thanks for help

[obeyler]

/approve

[allamand]

/retest

[allamand]

@maorfr I think It's Ready to merge. I would like to merge it in incubator/charts, and I will work in future to have the charts on our own repo.

Is that ok for you ?

[k8s-ci-robot]

@obeyler: adding LGTM is restricted to approvers and reviewers in OWNERS files.

In response to this:

/lgtm

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[allamand]

/assign @jlegrone

[allamand]

/assign @prydonius

[prydonius]

I would like to merge it in incubator/charts, and I will work in future to have the charts on our own repo.

I think that's fine for now. Just bear in mind that in the future we want to move towards a more distributed model, so folks will run their own chart repositories and add them to the Helm Hub (https://hub.helm.sh).

/lgtm

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: allamand, obeyler, prydonius

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [prydonius]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment