SpanAttributeConstants is internal

[eddynaka]

Fixes #.

Changes

Please provide a brief description of the changes here.

For significant contributions please make sure you have completed the following items:

• CHANGELOG.md updated for non-trivial changes
• Design discussion issue #
• Changes in public API reviewed

[codecov]

Codecov Report

Merging #1457 into master will increase coverage by 0.27%.
The diff coverage is 85.71%.

@@            Coverage Diff             @@
##           master    #1457      +/-   ##
==========================================
+ Coverage   81.67%   81.94%   +0.27%     
==========================================
  Files         229      229              
  Lines        6095     6143      +48     
==========================================
+ Hits         4978     5034      +56     
+ Misses       1117     1109       -8     

Impacted Files	Coverage Δ
...lemetry/Internal/SelfDiagnosticsConfigRefresher.cs	0.00% <0.00%> (ø)
...Telemetry/Internal/SelfDiagnosticsEventListener.cs	90.90% <91.30%> (+90.90%)	⬆️
...ter.ZPages/Implementation/ZPagesActivityTracker.cs	100.00% <0.00%> (+2.85%)	⬆️
...ZPages/Implementation/ZPagesExporterEventSource.cs	62.50% <0.00%> (+6.25%)	⬆️
...emetry.Api/Internal/OpenTelemetryApiEventSource.cs	82.35% <0.00%> (+8.82%)	⬆️
...us/Implementation/PrometheusExporterEventSource.cs	72.72% <0.00%> (+9.09%)	⬆️

[cijothomas]

i am wondering why is the instrumentation populating those tags which are not part of semantic convention? we can eliminate that, and then eliminate the need for this at all.

(Lets address it separately, Okay to merge this)