Skip to content

Commit

Permalink
Remove heapster manifests from addons
Browse files Browse the repository at this point in the history
* Heapster addon powers `kubectl top`
* In early Kubernetes, people legitimately used and expected
`kubectl top` to work, so the optional addon was provided
* Today the standards are different. Many better monitoring
tools exist, that are also less coupled to Kubernetes
`kubectl top` reliance on a non-core extensions means its not
in-scope for minimal Kubernetes clusters
* Finally, its just not that useful anymore and Heapster
manifests have no need for Typhoon-specific modification
so there's little value in maintaining them here
* Look to prior releases if you still wish to apply heapster
  • Loading branch information
dghubble committed Mar 14, 2019
1 parent e0bee2e commit f87d807
Show file tree
Hide file tree
Showing 11 changed files with 4 additions and 174 deletions.
4 changes: 4 additions & 0 deletions CHANGES.md
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,10 @@ Notable changes between versions.

* Update Prometheus from v2.7.1 to [v2.8.0](https://github.com/prometheus/prometheus/releases/tag/v2.8.0)
* Update Grafana from v6.0.0 to v6.0.1
* Remove heapster manifests from addons ([#427](https://github.com/poseidon/typhoon/pull/427))
* Heapster addon powers `kubectl top` (in early Kubernetes, running the addon was expected). Today,there are many better options in the monitoring space.
* `kubectl top`'s reliance on a non-core extension/addon means its not in-scope for minimal Kubernetes clusters
* Look to prior releases if you still wish to apply heapster

## v1.13.4

Expand Down
12 changes: 0 additions & 12 deletions addons/heapster/cluster-role-binding.yaml

This file was deleted.

30 changes: 0 additions & 30 deletions addons/heapster/cluster-role.yaml

This file was deleted.

62 changes: 0 additions & 62 deletions addons/heapster/deployment.yaml

This file was deleted.

13 changes: 0 additions & 13 deletions addons/heapster/role-binding.yaml

This file was deleted.

19 changes: 0 additions & 19 deletions addons/heapster/role.yaml

This file was deleted.

5 changes: 0 additions & 5 deletions addons/heapster/service-account.yaml

This file was deleted.

12 changes: 0 additions & 12 deletions addons/heapster/service.yaml

This file was deleted.

19 changes: 0 additions & 19 deletions docs/addons/heapster.md

This file was deleted.

1 change: 0 additions & 1 deletion docs/addons/overview.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@ Every Typhoon cluster is verified to work well with several post-install addons.

* [CLUO](cluo.md) (Container Linux only)
* Nginx [Ingress Controller](ingress.md)
* [Heapster](heapster.md)
* [Prometheus](prometheus.md)
* [Grafana](grafana.md)

1 change: 0 additions & 1 deletion mkdocs.yml
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,6 @@ nav:
- 'Addons':
- 'Overview': 'addons/overview.md'
- 'CLUO': 'addons/cluo.md'
- 'Heapster': 'addons/heapster.md'
- 'Nginx Ingress': 'addons/ingress.md'
- 'Prometheus': 'addons/prometheus.md'
- 'Grafana': 'addons/grafana.md'

0 comments on commit f87d807

Please sign in to comment.