Change ConfigProvider.getInstrumentationConfig() to not return null

[trask]

null-checking this all the time is awkward

This was being used to detect whether in declarative configuration world or not, but I think instrumentations can still be split brained if they really want to by first looking for the specific declarative configuration they care about and falling back to system properties.

cc @zeitlinger

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.11%. Comparing base (40dbfa4) to head (95652e8).
⚠️ Report is 29 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff            @@
##               main    #7920   +/-   ##
=========================================
  Coverage     90.11%   90.11%           
- Complexity     7318     7319    +1     
=========================================
  Files           824      824           
  Lines         22027    22030    +3     
  Branches       2177     2177           
=========================================
+ Hits          19850    19853    +3     
  Misses         1500     1500           
  Partials        677      677           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[zeitlinger]

great idea!

[trask]

Looks like this is against spec language https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/configuration/api.md#get-instrumentation-config

If the .instrumentation node is not set, get instrumentation config MUST
return nil, null, undefined or another language-specific idiomatic pattern
denoting empty.

[jack-berg]

Closing because #7954 was a duplicate of this, but somehow got merged first

[zeitlinger]

I didn't see that we already had a PR for this