Remove experimental from profile session tag

[pharring]

The OpenTelemetry profiling data model spec has been merged: open-telemetry/oteps#239
This confirms the use of the "profile_id" tag to link from spans to profiles.
We can now drop the "_experimental" suffix from the tag we write in ProfilingSessionTraceProcessor.

[github-actions]

Thank you for your contribution @pharring! We will review the pull request and get back to you soon.

[pallavit]

@pharring:

You are missing the needed permissions and public group memberships for an internal contributor. Please review the Azure SDK onboarding documentation and use the onboarding teams channel for any questions.

Azure SDK onboarding (Microsoft internal)
Azure SDK onboarding assistance (Microsoft internal)
Please also be sure to add yourself to CODEOWNERS for this library, if you will be maintaining it going forward.

[TimothyMothra]

LGTM.

Let me sync with Raj since he has more context on this.
But this is a very small change, I think we can get this merged today.

[rajkumar-rangaraj]

The OpenTelemetry profiling data model spec has been merged: open-telemetry/oteps#239

oteps is not a spec repository; it’s one step behind the spec. Initial ideas are discussed here, then they move to the spec. Should we remove the experimental flag based on this?

[TimothyMothra]

@rajkumar-rangaraj looks like the original PR didn't update the Changelog. #41594

@pharring how would you like this feature described in our changelog?

[pharring]

@TimothyMothra, I added a line in the CHANGELOG about removing the experimental suffix. As for documenting the original feature, is it OK to retro-actively add it to the CHANGELOG? Given the timeline, I think it would go in 1.1.0.

[pharring]

@TimothyMothra looks like this missed the last beta. Can we get it in now?

[azure-sdk]

API change check

API changes are not detected in this pull request.

[rajkumar-rangaraj]

The OpenTelemetry profiling data model spec has been merged: open-telemetry/oteps#239

oteps is not a spec repository; it’s one step behind the spec. Initial ideas are discussed here, then they move to the spec. Should we remove the experimental flag based on this?

@pharring Review was pending on this question.

[pharring]

Oh, I see. Sorry.