Skip to content

Commit d3c4afb

Browse files
ChrsMarktiffany76
andcommitted
Apply suggestions from code review
Co-authored-by: Tiffany Hrabusa <[email protected]>
1 parent dcced5f commit d3c4afb

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

content/en/blog/2025/otel-collector-k8s-discovery.md

+6-6
Original file line numberDiff line numberDiff line change
@@ -52,8 +52,8 @@ receivers:
5252
collection_interval: '15s'
5353
```
5454
55-
The above configuration will be enabled when a pod is discovered via the
56-
Kubernetes API that exposes port `80` (the known port for NGINX) and it's name
55+
The previous configuration is enabled when a pod is discovered via the
56+
Kubernetes API that exposes port `80` (the known port for NGINX) and its name
5757
matches the `nginx` keyword.
5858

5959
This is great, and as an SRE or Platform Engineer managing an observability
@@ -99,7 +99,7 @@ Each scraping receiver has a default configuration with only one required field:
9999
`endpoint`. Given that the endpoint information is provided by the Kubernetes
100100
observer, the only information that the user needs to provide explicitly is
101101
which receiver/scraper should be used to scrape data from a discovered endpoint.
102-
That information can be configured on the collector, but as mentioned before,
102+
That information can be configured on the Collector, but as mentioned before,
103103
this is inconvenient. A much more convenient place to define which receiver can
104104
be used to scrape telemetry from a particular pod is the pod itself. Pod’s
105105
annotations is the natural place to put that kind of detail. Given that the
@@ -115,7 +115,7 @@ can be used to scrape metrics from it:
115115
io.opentelemetry.discovery.metrics/scraper: nginx
116116
```
117117

118-
Apart from that, the discovery on the pod need to be explicitly enabled with the
118+
Apart from that, the discovery on the pod needs to be explicitly enabled with the
119119
following annotation:
120120

121121
```yaml
@@ -135,7 +135,7 @@ io.opentelemetry.discovery.metrics/config: |
135135
```
136136
137137
It’s important to mention that the configuration defined in the annotations
138-
cannot point the receiver creator to another pod. The collector will reject such
138+
cannot point the receiver creator to another pod. The Collector will reject such
139139
configurations.
140140
141141
In addition to the metrics scraping, the annotation-based discovery also
@@ -164,7 +164,7 @@ including the default container parser, has to be redefined because list config
164164
fields are entirely replaced when merged into the default configuration struct.
165165
166166
The discovery functionality has to be explicitly enabled in the receiver creator
167-
just by adding the following configuration field:
167+
by adding the following configuration field:
168168
169169
```yaml
170170
receivers:

0 commit comments

Comments
 (0)