feat(sdk-node): override IdGenerator when using NodeSDK

[haddasbronfman]

Which problem is this PR solving?

Add ability to override idGenerator when using NodeSDK
Fixes #3435

Short description of the changes

Add 'idGenerator' to the configuration of NodeSDK

Type of change

• New feature (non-breaking change which adds functionality)

How Has This Been Tested?

I added a test that creates NodeSDK with a custom idGenerator object, then I created span and verified it has the id that my custom generator generates. (a constant id just for testing)

[codecov]

Codecov Report

Merging #3645 (ea1659e) into main (a457d94) will increase coverage by 0.61%.
The diff coverage is 100.00%.

❗ Current head ea1659e differs from pull request most recent head f33fcf5. Consider uploading reports for the commit f33fcf5 to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3645      +/-   ##
==========================================
+ Coverage   93.10%   93.72%   +0.61%     
==========================================
  Files         183      274      +91     
  Lines        4538     8060    +3522     
  Branches      905     1671     +766     
==========================================
+ Hits         4225     7554    +3329     
- Misses        313      506     +193     

Impacted Files	Coverage Δ
api/src/diag/types.ts	100.00% <ø> (ø)
...imental/packages/opentelemetry-sdk-node/src/sdk.ts	93.87% <100.00%> (ø)
packages/sdk-metrics/src/aggregator/types.ts	100.00% <0.00%> (ø)
packages/sdk-metrics/src/state/MetricStorage.ts	100.00% <0.00%> (ø)
packages/sdk-metrics/src/state/HashMap.ts	100.00% <0.00%> (ø)
...dk-metrics/src/state/MultiWritableMetricStorage.ts	100.00% <0.00%> (ø)
...-metrics/src/exemplar/NeverSampleExemplarFilter.ts	100.00% <0.00%> (ø)
...ntal/packages/otlp-grpc-exporter-base/src/types.ts	100.00% <0.00%> (ø)
packages/sdk-metrics/src/state/MetricCollector.ts	100.00% <0.00%> (ø)
packages/sdk-metrics/src/InstrumentDescriptor.ts	100.00% <0.00%> (ø)
... and 87 more

[pichlermarc]

Looks great; thanks for adding this 🙂
Could you please also add the changelog entry? 🙂

Nit: I'm not sure about the scope of the conventional commit message in the PR title, as it looks like it changes the IdGenerator, maybe
feat(sdk-node): override IdGenerator when using NodeSDK #3645? 🙂

[blumamir]

Thanks! I think you do need to add an entry in CHANGELOG.md

[dyladan]

Please add a changelog entry