* Fix AlertManager discovery if chart name is not the default
Signed-off-by: Trond Nordheim <trond@nordheim.io>
* Bump chart patch version
Signed-off-by: Trond Nordheim <trond@nordheim.io>
* Switch from localhost to 127.0.0.1
The container is a binary only container without `/etc/hosts` or `/etc/resolv.conf` which will hit the dns server to resolv `localhost -> 127.0.0.1`
We had the case that there was a VM named `localhost` in the cloud and when hitting the DNS to resolv `localhost` we got the IP from this vm instead of `127.0.0.1`.
Took hours to find this issue :(
* Update prometheus chart version
* Bump version to 7.0.0
* [update] default pushgateway image tag to 0.5.1
* [fix] pushgateway-deployment readinessProbe path
the readinessProbe would fail if "web.route-prefix" is set in pushgateway extraArgs
* [update] Chart version
* [fix] Bump Chart version
This commit reverts a breaking change that was introduced in the
following PR: https://github.com/kubernetes/charts/pull/4915
The problems caused by this change are described in the following GitHub
issue: https://github.com/kubernetes/charts/issues/5128
The root of the problem is that the `clusterIP` field on services is not
mutable, so a simple helm upgrade is not sufficient to handle this type
of change. Furthermore, since these are overridable values, users who
have a specific use case for making the services non-headless can do so
in their overrides for the chart installation.
Closes https://github.com/kubernetes/charts/issues/5128
* [prometheus/stable] allow extra Ingress labels
A common pattern with the Traefik ingress controller is to run multiple
Traefik deployments with different Kubernetes label selectors, which
then discover only matching Ingress resources.
Add extraLabels configuration options to allow extra labels to be
configured on Prometheus Server and AlertManager resources.
* [prometheus/stable] Chart version => 6.2.2
* [prometheus/stable] error cases with empty extraLabels
I'd thought this was OK but upon further testing, discovered that it
generated errors in some cases. Use a range loop instead. Tested OK
with
* ingress enabled but no extra labels
* ingress enabled with extra labels
* both server and alertmanager components
* [stable/prometheus] behavioural changes => v6.3.1
* prometheus: Make the security context customizable for all deployments
* prometheus: Bump chart version to 6.3.0
* Describe custom security context options in the README
* [stable/prometheus]: remove unnecessary clusterrolebindings for alertmanager and node-exporter
* [stable/prometheus] RBAC resource creation complies with best practices
* [stable/prometheus]: setting serviceaccount creation to true by default
* [stable/prometheus]: bumping chart version
* [stable/prometheus]: proper compliance with RBAC best practices
* [stable/prometheus]: updated README.md
* Fix for version comparison from strings to semver
See #3002 for more detail
* Include pre-releases in the semver ranges
This is important when testing against alpha and beta builds of
Kubernetes along with environments that use pre-releases to denote
things other than pre-releases (e.g., gke denotes the environment
with a pre-releases)
Similar to other charts, this allows for configurable name
overrides so 'cleaner' resource names can be achieved, independent
of the wanted release names.
This PR also adds appVersion to Chart.yaml, as this seems to be
a requirement now for tests to pass.
* [stable/prometheus] extraSecretMounts support
Sometimes you need to monitor external services and sometimes
those services have authentication mechanisms, like TLS. For
example, you need to monitor external Etcd cluster. In such case
it would be handy to put all the TLS files into Kubernetes Secret
and then mount them into Prometheus Server pod, instead of
distributing them on all the worker nodes and using hostPath mounts.
* Incrementing the chart version
- Use yaml not literal for alertmanagerFiles (closes#3576).
- Raise minor release number to mark the updated values structure.
- Fix typo in README associated with yaml not literal for
serverFiles.