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

Vas prometheus operator #5

Merged
merged 11 commits into from
Oct 22, 2018
Merged

Vas prometheus operator #5

merged 11 commits into from
Oct 22, 2018

Conversation

vsliouniaev
Copy link

No description provided.

@gianrubio gianrubio merged this pull request into gianrubio:prometheus-operator Oct 22, 2018
gianrubio pushed a commit that referenced this pull request Oct 22, 2018
* Fix lint

Signed-off-by: Vasily Sliouniaev <[email protected]>

* Fix missing selector on kubelet

helm#6765 (comment)

Signed-off-by: Vasily Sliouniaev <[email protected]>

* Fix braces in files

Signed-off-by: Vasily Sliouniaev <[email protected]>

* Fix naming

Signed-off-by: Vasily Sliouniaev <[email protected]>

* Move non-exporters out of exporters folder

Signed-off-by: Vasily Sliouniaev <[email protected]>

* Use common value names, add to readme

Signed-off-by: Vasily Sliouniaev <[email protected]>

* Change value names deploy -> enabled

helm#6765 (comment)
helm#6765 (comment)

Signed-off-by: Vasily Sliouniaev <[email protected]>

* Update etcd rules

https://github.com/etcd-io/etcd/blob/master/Documentation/op-guide/etcd3_alert.rules.yml

Signed-off-by: Vasily Sliouniaev <[email protected]>

* Fix etcd rules to match job name

Signed-off-by: Vasily Sliouniaev <[email protected]>

* Review readme

Signed-off-by: Vasily Sliouniaev <[email protected]>

* Change initial version to 0.1.0

Signed-off-by: Vasily Sliouniaev <[email protected]>
@vsliouniaev vsliouniaev deleted the vas-prometheus-operator branch October 22, 2018 16:05
gianrubio pushed a commit that referenced this pull request Oct 23, 2018
Changed:
- Update Prometheus to 2.4.3, alertmanager to 0.15.2
- Change some job labels - should refer to a label appearing on the service. Tried to keep this consistent with what was used in the coreos version
- Add confguration for using CoreDNS
- Add insecureSkipVerify to kubeApiServer
- Add `prometheus.additionalScrapeConfigs`, `prometheus.additionalAlertManagerConfigs`

- Add ability to deploy an operated prometheus without installing anything else
  - Switch `prometheus.serviceAccountName` to be based on `prometheus.fullname` to prevent clashes
- Fix `serviceMonitorSelector` typo
- Add `serviceMonitorNamespaceSelector` to values.yaml
- Fix grafana typo (helm#6765 (comment))

Add CoreDNS dashboard

- Fixed dashboard use of updated name metric

Signed-off-by: Giancarlo Rubio <[email protected]>
Signed-off-by: Vasily Sliouniaev <[email protected]>

Vas prometheus operator (#5)

* Fix lint

* Fix missing selector on kubelet

helm#6765 (comment)

* Fix braces in files

* Fix naming

* Move non-exporters out of exporters folder

* Use common value names, add to readme

* Change value names deploy -> enabled

helm#6765 (comment)
helm#6765 (comment)

* Update etcd rules

https://github.com/etcd-io/etcd/blob/master/Documentation/op-guide/etcd3_alert.rules.yml

* Fix etcd rules to match job name

* Review readme

* Change initial version to 0.1.0

Additional Fixes While Github is down (#6)

* Fix additional service monitor template

* Fix label indentation

helm#6765 (comment)

Fix cleanup, requirements.lock file (#7)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants