Remove MySql.Data.OpenTelemetry

Remove the `MySql.Data.OpenTelemetry` package.

Author: martincostello

CHANGELOG.md

@@ -6,8 +6,11 @@
 
 * Remove reference on System.Text.Json for `net8.0` target framework
   ([#136](https://github.com/grafana/grafana-opentelemetry-dotnet/pull/136))
-* Replace dependency on `OpenTelemetry.Instrumentation.MySqlData`
-  with `MySql.Data.OpenTelemetry`
+* Remove dependency on `OpenTelemetry.Instrumentation.MySqlData`.
+  Add the [MySql.Data.OpenTelemetry](https://www.nuget.org/packages/MySql.Data.OpenTelemetry)
+  package to your project if you require MySQL instrumentation. **NOTE**: This
+  NuGet package is licensed under the GPL license which may not be suitable for
+  all projects.
   ([#146](https://github.com/grafana/grafana-opentelemetry-dotnet/pull/146))
 * Use 1.0.0-rc.18 of OpenTelemetry.Instrumentation.Wcf
   ([#146](https://github.com/grafana/grafana-opentelemetry-dotnet/pull/146))

docs/supported-instrumentations.md

@@ -20,8 +20,6 @@ and [minimal](./installation.md#install-the-base-package) dependencies:
 | `Hangfire`            | :heavy_check_mark: |                    | [OpenTelemetry.Instrumentation.Hangfire](https://www.nuget.org/packages/OpenTelemetry.Instrumentation.Hangfire) |
 | `HttpClient`          | :heavy_check_mark: | :heavy_check_mark: | [OpenTelemetry.Instrumentation.Http](https://www.nuget.org/packages/OpenTelemetry.Instrumentation.Http) |
 | `HostResource`        | :heavy_check_mark: | :heavy_check_mark: | [OpenTelemetry.Resources.Host](https://github.com/open-telemetry/opentelemetry-dotnet-contrib/tree/main/src/OpenTelemetry.Resources.Host) |
-| `MySqlData`           | :heavy_check_mark: | :heavy_check_mark: | [OpenTelemetry.Instrumentation.MySqlData](https://www.nuget.org/packages/OpenTelemetry.Instrumentation.MySqlData) |
-|                       |                    |                    | [MySql.Data.OpenTelemetry](https://www.nuget.org/packages/MySql.Data.OpenTelemetry) |
 | `NetRuntime`          | :heavy_check_mark: | :heavy_check_mark: | [OpenTelemetry.Instrumentation.Runtime](https://www.nuget.org/packages/OpenTelemetry.Instrumentation.Runtime) |
 | `Owin`                | :heavy_check_mark: |                    | [OpenTelemetry.Instrumentation.Owin](https://www.nuget.org/packages/OpenTelemetry.Instrumentation.Owin) |
 | `Process`             | :heavy_check_mark: | :heavy_check_mark: | [OpenTelemetry.Instrumentation.Process](https://www.nuget.org/packages/OpenTelemetry.Instrumentation.Process) |
@@ -37,5 +35,3 @@ and [minimal](./installation.md#install-the-base-package) dependencies:
 * The `ContainerResource` instrumentation is included but needs to be explicitly
   activated, as it currently adds container resource attributes for processes
   running not in containers.
-* The `MySqlData` instrumentation does not include the MySql.Data.OpenTelemetry
-  package. Install separately if needed.

src/Grafana.OpenTelemetry/Grafana.OpenTelemetry.csproj

@@ -29,11 +29,6 @@
     <PackageReference Include="CassandraCSharpDriver" Version="[3.17.0,)" />
   </ItemGroup>
 
-  <!-- OpenTelemetry.Instrumentation.MySqlData is deprecated -->
-  <ItemGroup>
-    <PackageReference Include="MySql.Data.OpenTelemetry" Version="9.3.0" />
-  </ItemGroup>
-
   <!-- Stable instrumentation packages with dependencies, only .NET -->
   <ItemGroup Condition=" '$(TargetFramework)' != 'net462' ">
     <PackageReference Include="OpenTelemetry.Extensions.Hosting" Version="1.9.0" /> <!-- needed for AspNetCore -->