Skip to content

Commit

Permalink
Use ClusterIP for our monitoring services so we can still access them…
Browse files Browse the repository at this point in the history
… using kubectl proxy (knative#7867)

* Revert "Remove NodePort exposure from Grafana, Prometheus and Kibana (knative#7640)"

This reverts commit 6ec7f43.

* use clusterIP to expose monitoring services
  • Loading branch information
dprotaso authored May 6, 2020
1 parent 877aae3 commit 7227e67
Show file tree
Hide file tree
Showing 3 changed files with 47 additions and 0 deletions.
16 changes: 16 additions & 0 deletions config/monitoring/metrics/prometheus/100-grafana.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -118,6 +118,22 @@ data:
options:
path: /grafana-dashboard-definition/scaling
---
apiVersion: v1
kind: Service
metadata:
name: grafana
namespace: knative-monitoring
labels:
app: grafana
serving.knative.dev/release: devel
spec:
ports:
- port: 30802
protocol: TCP
targetPort: 3000
selector:
app: grafana
---
apiVersion: apps/v1
kind: Deployment
metadata:
Expand Down
14 changes: 14 additions & 0 deletions config/monitoring/metrics/prometheus/300-prometheus.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -219,6 +219,20 @@ subjects:
name: prometheus-system
namespace: knative-monitoring
---
apiVersion: v1
kind: Service
metadata:
name: prometheus-system-np
namespace: knative-monitoring
labels:
serving.knative.dev/release: devel
spec:
selector:
app: prometheus
ports:
- port: 8080
targetPort: 9090
---
apiVersion: apps/v1
kind: StatefulSet
metadata:
Expand Down
17 changes: 17 additions & 0 deletions third_party/config/monitoring/logging/elasticsearch/kibana.yaml
Original file line number Diff line number Diff line change
@@ -1,3 +1,20 @@
apiVersion: v1
kind: Service
metadata:
name: kibana-logging
namespace: knative-monitoring
labels:
app: kibana-logging
kubernetes.io/cluster-service: "true"
kubernetes.io/name: "Kibana"
spec:
selector:
app: kibana-logging
ports:
- port: 5601
protocol: TCP
targetPort: ui
---
apiVersion: apps/v1
kind: Deployment
metadata:
Expand Down

0 comments on commit 7227e67

Please sign in to comment.