-
Notifications
You must be signed in to change notification settings - Fork 882
Donate https://github.com/aspnet/Microsoft.AspNet.TelemetryCorrelation to OpenTelemetry #2222
Copy link
Copy link
Closed
Description
This has been discussed during the OpenTelemetry .NET SIG Meeting at 08/03/2021 4PM PT.
Here are the steps that we will be following:
- Fork a “aspnet-telemetrycorrelation” branch from main
- Just copy the code, with no change (Copy the code from ASP.NET repo #2223)
- Make minimum change to clean up Microsoft specific stuff (comments, types, etc.) (initial cleanup #2224, Update copyright info #2225)
- Fix the styling/sanity check/project/solution/build (Fix build #2226, Fix build #2229, Fix format #2231, Format code #2235, Disable TelemetryHttpModule API Compat for now #2244)
- Integrate it with the instrumentation library, major change could happen at this stage. We should also pick a better namespace and package name (Rename Microsoft.AspNet.TelemetryCorrelation -> OpenTelemetry.Instrumentation.AspNet.TelemetryHttpModule #2238, AspNet.TelemetryHttpModule public api + test fixes + renames #2240)
- Update test cases (AspNet.TelemetryHttpModule public api + test fixes + renames #2240)
- Update changelog (Add changelog for OpenTelemetry.Instrumentation.AspNet.TelemetryHttpM… #2242)
- Merge the “aspnet-telemetrycorrelation” branch back to main, keep individual commit history rather than squash the commits (Merge aspnet-telemetrycorrelation branch back to main #2233)
- Delete the “aspnet-telemetrycorrelation” branch
- Do a test release (most likely we'll just catch the 1.2.0 alpha release train)
FYI @open-telemetry/dotnet-maintainers
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
enhancementNew feature or requestNew feature or request