Skip to content
This repository has been archived by the owner on Apr 30, 2021. It is now read-only.

Commit

Permalink
Remove unwanted prometheus rules
Browse files Browse the repository at this point in the history
The kube-apiserver is part of the EKS control plane, so, from a metrics
and alerting point of view we aren't interested. So remove the associated
rules.
  • Loading branch information
blairboy362 committed Aug 7, 2020
1 parent e6e6cf2 commit dfa6a53
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 0 deletions.
2 changes: 2 additions & 0 deletions charts/gsp-cluster/charts/gsp-monitoring/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,8 @@ prometheus-operator:
general: false # see templates/rules-general.yaml for replacement
kubernetesSystem: false # see templates/rules-kubernetes-system.yaml for replacement
alertmanager: false
kubeApiServer:
enabled: false
kubeControllerManager:
enabled: false
kubeScheduler:
Expand Down
6 changes: 6 additions & 0 deletions pipelines/deployer/deployer.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -209,6 +209,12 @@ apply_cluster_chart: &apply_cluster_chart
echo "RELEASE_TAG=${RELEASE_TAG}"
echo "Removing EKS-provided pod security policy clusterrolebinding (if it exists)"
kubectl delete --ignore-not-found=true clusterrolebinding eks:podsecuritypolicy:authenticated
### Start of temporary tidyup code ###
kubectl delete --ignore-not-found=true --namespace gsp-system servicemonitor gsp-prometheus-operator-apiserver
kubectl delete --ignore-not-found=true --namespace gsp-system configmap gsp-prometheus-operator-apiserver
kubectl delete --ignore-not-found=true --namespace gsp-system prometheusrule gsp-prometheus-operator-kube-apiserver-slos
kubectl delete --ignore-not-found=true --namespace gsp-system prometheusrule gsp-prometheus-operator-kube-apiserver.rules
### End of temporary tidyup code ###
echo "rendering ${CHART_NAME} chart..."
mkdir -p manifests
mkdir -p kube-system-manifests
Expand Down

0 comments on commit dfa6a53

Please sign in to comment.