Skip to content

feat(target allocator): add OTLP self-telemetry API to CRDs - #5293

Open
odubajDT wants to merge 6 commits into
open-telemetry:mainfrom
odubajDT:ta-selftelemetry-crd
Open

feat(target allocator): add OTLP self-telemetry API to CRDs#5293
odubajDT wants to merge 6 commits into
open-telemetry:mainfrom
odubajDT:ta-selftelemetry-crd

Conversation

@odubajDT

@odubajDT odubajDT commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

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 for endpoint, protocol (grpc/http), temporality, headers, TLS/insecure, export interval/timeout
  • operator renders the telemetry block into the TargetAllocator ConfigMap
  • generated CRDs, bundles, API docs, deepcopy

Refs #5047

Based on #5068

@odubajDT
odubajDT marked this pull request as ready for review July 6, 2026 05:23
@odubajDT
odubajDT requested a review from a team as a code owner July 6, 2026 05:23
@odubajDT
odubajDT requested review from jaronoff97 and yuriolisa July 6, 2026 05:23
@codecov

codecov Bot commented Jul 6, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 63.46%. Comparing base (adc7406) to head (c4cffde).
✅ All tests successful. No failed tests found.

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     
Flag Coverage Δ
integration 69.73% <ø> (ø)
unittests 63.46% <100.00%> (+0.07%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@odubajDT
odubajDT force-pushed the ta-selftelemetry-crd branch from a4beca9 to 18164b6 Compare July 6, 2026 05:41
@odubajDT
odubajDT force-pushed the ta-selftelemetry-crd branch from 18164b6 to b963c6b Compare July 28, 2026 10:07
@swiatekm

Copy link
Copy Markdown
Contributor

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?

@odubajDT
odubajDT marked this pull request as draft July 29, 2026 06:09
@odubajDT

Copy link
Copy Markdown
Contributor Author

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).

@odubajDT
odubajDT force-pushed the ta-selftelemetry-crd branch from 12a15bd to 219cf86 Compare July 29, 2026 06:25
odubajDT added 6 commits July 29, 2026 10:44
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>
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
@odubajDT
odubajDT force-pushed the ta-selftelemetry-crd branch from 3fef379 to c4cffde Compare July 29, 2026 08:54
@odubajDT
odubajDT marked this pull request as ready for review July 29, 2026 09:18
@odubajDT

Copy link
Copy Markdown
Contributor Author

@swiatekm should be ready :)

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.

2 participants