feat(target allocator): add OTLP self-telemetry API to CRDs - #5293
feat(target allocator): add OTLP self-telemetry API to CRDs#5293odubajDT wants to merge 6 commits into
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #5293 +/- ##
==========================================
+ Coverage 63.39% 63.46% +0.07%
==========================================
Files 217 217
Lines 15505 15541 +36
==========================================
+ Hits 9829 9863 +34
- Misses 4986 4987 +1
- Partials 690 691 +1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
a4beca9 to
18164b6
Compare
18164b6 to
b963c6b
Compare
|
I think this configuration should at the very least be a subset of the declarative config, same as what we implemented for the target allocator itself. @pavolloffay as the person who investigated generating CRDs based on the declarative config spec, what do you think the easiest way to accomplish this is? |
Hi, totally agree and already working on it. Honestly, I forgot to change the PR to draft, as I didn't manage to finish and push the changes yesterday. Sorry for that, will try to put it together today. The version here still contained the old state of CRDs (before the TA binary OTLP selfmon changes were merged). |
12a15bd to
219cf86
Compare
CRD + operator wiring for the TargetAllocator OTLP self-telemetry feature: - spec.targetAllocator.telemetry.metrics.otlp (OpenTelemetryCollector CR) and spec.telemetry.metrics.otlp (TargetAllocator CR), typed fields following the Instrumentation CRD pattern (endpoint, protocol, temporality, headers, TLS, export interval/timeout) - operator renders the telemetry block into the TargetAllocator ConfigMap - generated CRDs, bundles, API docs, deepcopy Refs open-telemetry#5047
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
3fef379 to
c4cffde
Compare
|
@swiatekm should be ready :) |
CRD + operator wiring for the TargetAllocator OTLP self-telemetry feature:
Refs #5047
Based on #5068