Skip to content

Commit 2fa5594

Browse files
committed
Address review comments for the grpc-observability example.
1 parent 14793d2 commit 2fa5594

File tree

10 files changed

+13
-11
lines changed

10 files changed

+13
-11
lines changed

examples/grpc-observability/README.md

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -97,7 +97,7 @@ kubectl apply -f ./examples/grpc-observability/frontend/frontend.yaml
9797
## Set up the Prometheus to collect the gRPC metrics
9898

9999
Once the frontend/backend microservices are deployed (either locally or on
100-
cloud), you may set up the Prometheus to start scrape the metrics from them.
100+
cloud), you may set up the Prometheus to start scraping the metrics from them.
101101
Depends on where you run the frontend/backend microservices, you may need to
102102
deploy the Prometheus to a proper location to be able to access them, such as,
103103
the same cloud, etc.
@@ -106,6 +106,7 @@ If you
106106
use [Google Managed Prometheus](https://cloud.google.com/stackdriver/docs/managed-prometheus),
107107
you may need to configure the PodMonitoring resource to tell where are endpoints
108108
to scrape the Prometheus metrics.
109+
109110
```
110111
kubectl apply -f ./examples/grpc-observability/pod_monitoring.yaml
111112
```
@@ -118,7 +119,8 @@ server to visualize them.
118119
- Set up a Grafana server, deploy it to a place where can connect the Prometheus
119120
server as its data source.
120121
- Create a Grafana dashboard by importing the
121-
grpc-observability/grafana/prometheus/microservices-grpc-dashboard.json file.
122+
[examples/grpc-observability/grafana/prometheus/microservices-grpc-dashboard.json](http://github.com/grpc-ecosystem/grpc-spring/blob/master/examples/grpc-observability/grafana/prometheus/microservices-grpc-dashboard.json)
123+
file.
122124

123125
If you
124126
use [Google Managed Prometheus](https://cloud.google.com/stackdriver/docs/managed-prometheus),
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,3 @@
11
FROM eclipse-temurin:17-jdk-alpine
22
COPY build/libs/backend.jar backend.jar
3-
ENTRYPOINT ["java","-jar","/backend.jar"]
3+
ENTRYPOINT ["java","-jar","/backend.jar"]

examples/grpc-observability/backend/backend.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,4 +49,4 @@ spec:
4949
- name: monitoring
5050
port: 8081
5151
- name: grpc
52-
port: 9091
52+
port: 9091

examples/grpc-observability/backend/src/main/resources/application.properties

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,4 +4,4 @@ server.port=8081
44
# By default, expose on `/actuator/prometheus`.
55
management.endpoints.web.exposure.include=prometheus
66
# Port serves the gRPC traffic.
7-
grpc.server.port=9091
7+
grpc.server.port=9091
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,3 @@
11
FROM eclipse-temurin:17-jdk-alpine
22
COPY build/libs/frontend.jar frontend.jar
3-
ENTRYPOINT ["java","-jar","/frontend.jar"]
3+
ENTRYPOINT ["java","-jar","/frontend.jar"]

examples/grpc-observability/frontend/frontend.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,4 +46,4 @@ spec:
4646
app: frontend
4747
ports:
4848
- name: monitoring
49-
port: 8080
49+
port: 8080

examples/grpc-observability/frontend/src/main/resources/application.properties

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,4 +9,4 @@ management.endpoint.configprops.show-values=ALWAYS
99
grpc.client.backend.address=static://localhost:9091
1010
# Teh backend service address, for kubernetes.
1111
# grpc.client.backend.address=dns:///backend.default.svc.cluster.local:9091
12-
grpc.client.backend.negotiationType=PLAINTEXT
12+
grpc.client.backend.negotiationType=PLAINTEXT

examples/grpc-observability/grafana/prometheus/microservices-grpc-dashboard.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1402,4 +1402,4 @@
14021402
"uid": "-ABC1NOTy",
14031403
"version": 17,
14041404
"weekStart": ""
1405-
}
1405+
}

examples/grpc-observability/pod_monitoring.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,4 +9,4 @@ spec:
99
endpoints:
1010
- port: monitoring
1111
path: /actuator/prometheus
12-
interval: 30s
12+
interval: 30s

examples/grpc-observability/proto/src/main/proto/backend.proto

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,4 +58,4 @@ message BidiStreamingRequest {
5858

5959
message BidiStreamingResponse {
6060
string message = 1;
61-
}
61+
}

0 commit comments

Comments
 (0)