Skip to content

[I6] Add distributed tracing via OpenTelemetry TraceInterceptor#814

Open
portableDD wants to merge 3 commits into
MettaChain:mainfrom
portableDD:feat-808-I6-tracing
Open

[I6] Add distributed tracing via OpenTelemetry TraceInterceptor#814
portableDD wants to merge 3 commits into
MettaChain:mainfrom
portableDD:feat-808-I6-tracing

Conversation

@portableDD

Copy link
Copy Markdown
Contributor

Closes #808

Implemented from fork portableDD/PropChain-BackEnd

Copy link
Copy Markdown
Contributor

❌ CI is failing on this one. Could you take a look at the failing checks and push a fix? Once it's green I'll merge. 🙏

Copy link
Copy Markdown
Contributor

❌ CI is still red on this one (same failing stage as before). Could you take a look and push a fix? Once it's green I'll merge. 🙏

- fix: use NodeTracerProvider instead of NodeSDK (not in deps)
- fix: use resourceFromAttributes instead of new Resource()
- fix: use HttpInstrumentation instead of getNodeAutoInstrumentations
- fix: use latest ATTR_SERVICE_NAME/ATTR_SERVICE_VERSION constants
- fix: pass BatchSpanProcessor via spanProcessors constructor option
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[I6] Add distributed tracing via OpenTelemetry TraceInterceptor

2 participants