feat(detector-container)!: change implementation to DetectorSync interface

[david-luna]

Context

Another PR moving away from the deprecated Detector interface in order to prepare the codebase for a fix for #2320. This time is for Container detector.

Not doing all detectors at once:

• detectors are independent from each other
• smaller PRs for faster reviews
• other detectors have a similar approach on exceptions like the described in feat(detector-alibaba)!: change implementation to DetectorSync interface #2328

My reasons to doing this change:

• SDK handles both interfaces using detectResources from @opentelemetry/resources package. Ref
• it is unlikely to be used oustide the SDK or auto-instrumentations

Short description of the changes

• change detect method to be sync
• update tests

[codecov]

Codecov Report

Attention: Patch coverage is 85.71429% with 1 line in your changes missing coverage. Please review.

Project coverage is 90.44%. Comparing base (dfb2dff) to head (19a81e7).
Report is 205 commits behind head on main.

Files	Patch %	Lines
...ector-container/src/detectors/ContainerDetector.ts	85.71%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2334      +/-   ##
==========================================
- Coverage   90.97%   90.44%   -0.53%     
==========================================
  Files         146      150       +4     
  Lines        7492     7403      -89     
  Branches     1502     1535      +33     
==========================================
- Hits         6816     6696     -120     
- Misses        676      707      +31     

Files	Coverage Δ
...ector-container/src/detectors/ContainerDetector.ts	95.08% <85.71%> (+2.22%)	⬆️

... and 67 files with indirect coverage changes

[pichlermarc]

FYI @abhee11(component owner)