Skip to content

Commit

Permalink
Grammar & spelling (#55)
Browse files Browse the repository at this point in the history
  • Loading branch information
pyohannes authored Nov 13, 2023
1 parent c12b01c commit cc09464
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 9 deletions.
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ The `UseGrafana` extension method on the `TracerProviderBuilder` or the
default, telemetry data will be sent to a Grafana agent or an OTel collector
that runs locally and listens to default OTLP ports.

Given the zone, instance id, and API token, telemetry data can be sent directly
Given the zone, instance ID, and API token, telemetry data can be sent directly
to the Grafana Cloud without involving an agent or collector:

```csharp
Expand Down
13 changes: 6 additions & 7 deletions docs/configuration.md
Original file line number Diff line number Diff line change
Expand Up @@ -114,7 +114,7 @@ documentation](https://opentelemetry.io/docs/specs/otel/protocol/exporter/#endpo

### Sending data directly to Grafana Cloud via OTLP

Given the zone, instance id, and API token, telemetry data can be sent directly
Given the zone, instance ID, and API token, telemetry data can be sent directly
to the Grafana Cloud without involving an agent or collector:

```csharp
Expand Down Expand Up @@ -209,7 +209,7 @@ can be added according to the documentation provided with it.

To active ASP.NET instrumentation, it is necessary to add an additional HTTP
module `OpenTelemetry.Instrumentation.AspNet.TelemetryHttpModule` to the web
server. This module is shipped as dependency of the
server. This module is shipped as a dependency of the
`OpenTelemetry.Instrumentation.AspNet` package. When using the IIS web server,
the following changes to `Web.config` are required.

Expand Down Expand Up @@ -253,9 +253,8 @@ The distribution is designed to be easily extensible with components that it
doesn't contain. This can be done by invoking additional extension methods on
any of the provider builders in addition to the `UseGrafana` extension method.

The example below initializes the distribution with default settings, but
additionally also initializes a console exporter which prints traces to the
console.
The example below initializes the distribution with default settings but
also initializes a console exporter which prints traces to the console.

```csharp
using var tracerProvider = Sdk.CreateTracerProviderBuilder()
Expand All @@ -264,7 +263,7 @@ using var tracerProvider = Sdk.CreateTracerProviderBuilder()
.Build();
```

In the same way it is possible to add additional instrumentation libraries that
In the same way, it is possible to add additional instrumentation libraries that
are not contained in the distribution.

## Supported environment variables
Expand All @@ -273,5 +272,5 @@ are not contained in the distribution.
| ----------------------------------------- | ------------------ | ----------- |
| `GRAFANA_DOTNET_DISABLE_INSTRUMENTATIONS` | "Process,NetRuntime" | A comma-separated list of instrumentations to disable. |
| `GRAFANA_CLOUD_ZONE` | "prod-us-east-0" | Zone of the Grafana Cloud stack to send data to. |
| `GRAFANA_CLOUD_INSTANCE_ID` | "123456" | Instance id of the Grafana Cloud stack to send data to. |
| `GRAFANA_CLOUD_INSTANCE_ID` | "123456" | Instance ID of the Grafana Cloud stack to send data to. |
| `GRAFANA_CLOUD_API_KEY` | | API key of the Grafana Cloud Stack to send data to. |
2 changes: 1 addition & 1 deletion docs/supported-instrumentations.md
Original file line number Diff line number Diff line change
Expand Up @@ -26,5 +26,5 @@ and [minimal](./installation.md#install-the-base-package) dependencies:
| `StackExchangeRedis` | :heavy_check_mark: | | [OpenTelemetry.Instrumentation.StackExchangeRedis](https://www.nuget.org/packages/OpenTelemetry.Instrumentation.StackExchangeRedis) |
| `Wcf` | :heavy_check_mark: | | [OpenTelemetry.Instrumentation.Wcf](https://www.nuget.org/packages/OpenTelemetry.Instrumentation.Wcf) |

The `AWSLambda` instrumentation is included, but needs to be explicitly
The `AWSLambda` instrumentation is included but needs to be explicitly
activated, as activating it in non-AWS scenarios causes errors.

0 comments on commit cc09464

Please sign in to comment.