Cerebro service annotations, image update and configuration overrides

[NicolasT]

This PR updates the stable/cerebro chart to

• Allow to set annotations on the Service
• Update the default Docker image to use the official upstream one
• Allow to set various application configuration values through values.yaml
• Use out-of-container storage (emptyDir) for the application database

Note to reviewers:

• Unlike the original Docker image, the official one doesn't run as root, hence the initContainer
• Version bumped according to SEMVER with every commit for consistency, so 0.1.1 -> 0.2.0 -> 0.2.1 -> 0.3.0

[NicolasT]

/assign @prydonius
/assign @unguiculus
/assign @timstoop

[timstoop]

Where did 0.2 go to?

[NicolasT]

I tried to explain in the PR comment. See e.g. 0dd516d#diff-0fcab9ba30f2d528e9326bf1299ef520R9

[NicolasT]

Basically wanted to make sure every commit (~ version) introduced by this PR has a correct Chart version attached to it.

[desaintmartin]

LGTM

Running it right now without issue, I was about to write it myself. Good timing, thanks.

[davidkarlsen]

this should probably be removed if using another image

[NicolasT]

Good point, will do (also in README).

[NicolasT]

Updated to use the 'new' Docker image source repository in Chart.yaml, and document the new 'values' in README.md.

[unguiculus]

/ok-to-test

[unguiculus]

/lgtm

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: NicolasT, unguiculus

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

The pull request process is described here

Details

Needs approval from an approver in each of these files:

• OWNERS [unguiculus]

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

[johnmarcou]

Hello!

I might have found a little typo in the helm chart.
When I am specifying a config.hosts, I can found a , in the json config.

Here the command:
helm upgrade --install my-cerebro stable/cerebro --set ingress.enabled=true --set ingress.hosts.0=cerebro.asci.synchrotron.org.au --set config.hosts.0.host=http://my-release-elasticsearch-client:9200 --set config.hosts.0.name=kube-logging

The resulting K8s config manifest:

secret = "[redacted]"
basePath = "/"
pidfile.path = /dev/null
rest.history.size = 50
data.path = "/var/db/cerebro/cerebro.db"
es = {
  gzip = true
}
auth = {
}
hosts = [
  ,
  {
    host = "http://my-release-elasticsearch-client:9200"
    name = "kube-logging"
  }
]

https://github.com/helm/charts/blob/master/stable/cerebro/templates/configmap.yaml#L31

This one should be after the item, only if there is multiple hosts, I guess?

Thx