Skip to content

[OpAMP] Expose public CustomMessageMessage and CustomCapabilitiesMessage#3681

Merged
martincostello merged 3 commits intoopen-telemetry:mainfrom
RassK:opamp-custommessages
Jan 13, 2026
Merged

[OpAMP] Expose public CustomMessageMessage and CustomCapabilitiesMessage#3681
martincostello merged 3 commits intoopen-telemetry:mainfrom
RassK:opamp-custommessages

Conversation

@RassK
Copy link
Copy Markdown
Contributor

@RassK RassK commented Jan 13, 2026

Towards #3613

Changes

Exposes two related messages to public:

  • CustomMessageMessage
  • CustomCapabilitiesMessage

Merge requirement checklist

  • CONTRIBUTING guidelines followed (license requirements, nullable enabled, static analysis, etc.)
  • Unit tests added/updated
  • Appropriate CHANGELOG.md files updated for non-trivial changes
  • Changes in public API reviewed (if applicable)

@RassK RassK requested a review from a team as a code owner January 13, 2026 10:06
@github-actions github-actions Bot added the comp:opamp.client Things related to OpenTelemetry.OpAmp.Client label Jan 13, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 13, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 71.72%. Comparing base (b6723bb) to head (43417b0).
⚠️ Report is 5 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #3681      +/-   ##
==========================================
+ Coverage   71.58%   71.72%   +0.14%     
==========================================
  Files         455      445      -10     
  Lines       17700    17653      -47     
==========================================
- Hits        12670    12662       -8     
+ Misses       5030     4991      -39     
Flag Coverage Δ
unittests-Instrumentation.Cassandra ?
unittests-OpAmp.Client 80.84% <100.00%> (+0.10%) ⬆️

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

Files with missing lines Coverage Δ
...lient/Messages/Custom/CustomCapabilitiesMessage.cs 100.00% <100.00%> (ø)
...Amp.Client/Messages/Custom/CustomMessageMessage.cs 100.00% <100.00%> (ø)

... and 11 files with indirect coverage changes

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

Comment thread test/OpenTelemetry.OpAmp.Client.Tests/FrameProcessorTests.cs Outdated
Comment thread test/OpenTelemetry.OpAmp.Client.Tests/Messages/CustomMessageMessageTests.cs Outdated
Comment thread test/OpenTelemetry.OpAmp.Client.Tests/PlainHttpTransportTests.cs Outdated
Comment thread test/OpenTelemetry.OpAmp.Client.Tests/WsTransportTest.cs Outdated
@martincostello martincostello added this pull request to the merge queue Jan 13, 2026
Merged via the queue into open-telemetry:main with commit 5865003 Jan 13, 2026
67 checks passed
@RassK RassK mentioned this pull request Jan 27, 2026
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp:opamp.client Things related to OpenTelemetry.OpAmp.Client

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants