docs: use resourceFromAttributes to replace new Resource() to obtain Resource

[nekomeowww]

What?

By following the default method to integrate OpenTelemetry in Next.js applications... this error occurs:

'Resource' only refers to a type, but is being used as a value here.

Why?

Resource presented here as a interface or type only definition, as specified in OpenTelemetry's repo, https://github.com/open-telemetry/opentelemetry-js/blob/2d544088ac9a4bcb07986be7d9cdc2d0688e7b44/packages/opentelemetry-resources/src/Resource.ts#L20-L30:

This interface is NOT user-implementable. Valid ways to obtain a {@link Resource} are by using either of these functions

• resourceFromAttributes
• emptyResource
• defaultResource
• detectResources

How?

By default, we use resourceFromAttributes for OTEL resources with attributes.

Fixes: #77451

[ijjk]

Allow CI Workflow Run

• approve CI run for commit: 250c1f1

Note: this should only be enabled once the PR is ready to go and can only be enabled by a maintainer

[icyJoseph]

Hi ~ ty - there's two more things we could do - if you don't mind making a follow up PR:

• We need to add serverExternalPackages to next.config.s, as per: [node-sdk] Missing @opentelemetry/exporter-jaeger raises an error on Next.js app open-telemetry/opentelemetry-js#4297

// next.config.js
  serverExternalPackages: [
    "@opentelemetry/sdk-node",
  ],

Otherwise - a warning like this prints:

 ⚠ ./node_modules/@opentelemetry/sdk-node/build/src/utils.js
Module not found: Can't resolve '@opentelemetry/exporter-jaeger'

This should be part of the docs manual instructions, right after the changes you are making.

Also the two TS/JS snippets above your change, do await import("./instrumentation.node.ts"), but I think the extension is not needed/valid without some assumptions about the project.

[nekomeowww]

Sure, will check this too. BTW, thanks for merging so quickly.

[icyJoseph]

Sure, will check this too. BTW, thanks for merging so quickly.

No worries, it had been overdue for a while - and it was relatively easy to verify.