Skip to content

fix: add in-memory exporter to self when the exporter is defined #17

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
cuericlee merged 2 commits into from
Aug 5, 2025
Merged

fix: add in-memory exporter to self when the exporter is defined #17

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
fe71e66
chore: update version to 0.1.0
yaozheng-fang Aug 4, 2025
d65a36c
fix: add inmemory exporter when already defined
yaozheng-fang Aug 4, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion pyproject.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
[project]
name = "veadk-python"
version = "1.0.0"
version = "0.1.0"
description = "Volcengine agent development kit, integrations with Volcengine cloud services."
readme = "README.md"
requires-python = ">=3.10"
Expand Down
3 changes: 3 additions & 0 deletions veadk/tracing/telemetry/opentelemetry_tracer.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -54,15 +54,18 @@ class OpentelemetryTracer(BaseModel, BaseTracer):
def model_post_init(self, context: Any, /) -> None:
self._processors = []
self._inmemory_exporter: InMemoryExporter = None
self._apiserver_exporter: ApiServerExporter = None

# Inmemory & APIServer are the default exporters
have_inmemory_exporter = False
have_apiserver_exporter = False
for exporter in self.exporters:
if isinstance(exporter, InMemoryExporter):
have_inmemory_exporter = True
self._inmemory_exporter = exporter
elif isinstance(exporter, ApiServerExporter):
have_apiserver_exporter = True
self._apiserver_exporter = exporter

if not have_inmemory_exporter:
inmemory_exporter = InMemoryExporter()
Expand Down