Branch: refs/heads/master
Home:
https://github.com/Checkmk/checkmk
Commit: 46400222a9c90aefa78e2bf38adae680baa681d7
https://github.com/Checkmk/checkmk/commit/46400222a9c90aefa78e2bf38adae680b…
Author: Lars Michelsen <lm(a)checkmk.com>
Date: 2024-08-02 (Fri, 02 Aug 2024)
Changed paths:
M cmk/gui/background_job/_process.py
M cmk/gui/log.py
M cmk/gui/wsgi/applications/index.wsgi
M packages/cmk-trace/cmk/trace/__init__.py
A packages/cmk-trace/cmk/trace/_config.py
A packages/cmk-trace/cmk/trace/export.py
A packages/cmk-trace/cmk/trace/logs.py
M packages/cmk-trace/tests/test_trace.py
Log Message:
-----------
Separate cmk.trace into modules
It turned out that the dependencies that are pulled in by cmk.trace
are too much for basic instrumenting use cases. The extreme is the OTLP
exporter, which pulls in protobuf. This is not an acceptable dependency
for code which just wants to create a span.
Luckily the issue was caused by own code. So the solution is fairly
straight forward: Separate `cmk.trace` into smaller parts which then can
be pulled in individually by the call sites as needed.
This is the first step which extracts the exporter setup and it's
dependencies.
Change-Id: Ib3791e4f9e5688bb923b0cf44d5d8f3be9eb6c51
Commit: 7efe9b6d5e0d305dd991b5cea2da343a58fbc20e
https://github.com/Checkmk/checkmk/commit/7efe9b6d5e0d305dd991b5cea2da343a5…
Author: Lars Michelsen <lm(a)checkmk.com>
Date: 2024-08-02 (Fri, 02 Aug 2024)
Changed paths:
M cmk/gui/background_job/_base.py
M cmk/gui/background_job/_interface.py
M cmk/gui/wsgi/applications/index.wsgi
M packages/cmk-trace/cmk/trace/__init__.py
M packages/cmk-trace/cmk/trace/export.py
M tests/unit/cmk/gui/test_background_job.py
Log Message:
-----------
Move span processor to cmk.trace.export
It is closely related to the actual exporter, so it's better to keep it
in the same module.
Change-Id: Icd17fbf1877a685c50addafc7f18227dd5950b19
Compare:
https://github.com/Checkmk/checkmk/compare/135b23b062e5...7efe9b6d5e0d
To unsubscribe from these emails, change your notification settings at
https://github.com/Checkmk/checkmk/settings/notifications