Enable SDK to create package for auto-instrumentation

[rajkumar-rangaraj]

Thanks @TimothyMothra for the PoC

To upgrade Application Insights SDK for .NET Core instrumentation following are the requirements and this PR propose the changes for it.

1. Downgrade System.Diagnostics.DiagnosticSource package reference to 4.7.0 from 5.0.0.
2. Rename event source names to support side-by-side load from both Nuget and auto-instrumentation AI SDK.
3. Build the package from AI SDK’s official release pipeline

Changes

(Please provide a brief description of the changes here.)

• Downgraded DiagnosticSource
• Added conditional compilation field REDFIELD which is used as preprocessor to decide the event source name
• Enabled redfield compilation switch to build and generate package for auto-instrumentation

[reyang]

LGTM with some suggestions.

[cijothomas]

Note: Won't be able to support https://github.com/microsoft/ApplicationInsights-dotnet/pull/2185/files with this change

[rajkumar-rangaraj]

Note: Won't be able to support https://github.com/microsoft/ApplicationInsights-dotnet/pull/2185/files with this change

We added a redfied sanity check in GitHub actions. This ensures builds are not broken with new changes.

This is a temporary workaround until we resolve the DiagnosticSource versioning issue. We will remove these changes once the DiagnosticSource issue is resolved.

[cijothomas]

only building Base sdk?

[TimothyMothra]

Running our full test suite here will fail.
We have too many flaky tests and Github Actions does not have a mechanism for retries.
(I'm investigating this in #2345)
If we want to run more test projects here, we must filter those tests.

[cijothomas]

Can you open an issue to completely remove this class. it should not be required.

[rajkumar-rangaraj]

Investigated a little and found methods from this class is called in several places.
For example Tryrun is called in OperationCorrelationTelemetryInitializer.cs.

Did I understand your comment incorrectly?

[cijothomas]

yea all the places where this ActivityExtensions is used can be modified to directly use Activity API, instead of doing it via this class.

[cijothomas]

for a separate PR , this file seems doing nothing and could be removed!

[cijothomas]

Note: Won't be able to support https://github.com/microsoft/ApplicationInsights-dotnet/pull/2185/files with this change

We added a redfied sanity check in GitHub actions. This ensures builds are not broken with new changes.

This is a temporary workaround until we resolve the DiagnosticSource versioning issue. We will remove these changes once the DiagnosticSource issue is resolved.

I think we can support PRs like the above, by doing it inside IF !REDFIELD check, as we'll have 5.0.0 of DS when not-redfield.