Skip to content

Conversation

@cijothomas
Copy link
Member

AzureMonitor/LogAnalytics fields names must be unique. Given this is coming from config, we can easily validate it upfront.

Few other nits also.

@cijothomas cijothomas requested a review from a team as a code owner January 7, 2026 23:21
@github-actions github-actions bot added the rust Pull requests that update Rust code label Jan 7, 2026
@codecov
Copy link

codecov bot commented Jan 7, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.83%. Comparing base (a2b3698) to head (e0b71ca).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1739      +/-   ##
==========================================
- Coverage   84.09%   81.83%   -2.26%     
==========================================
  Files         470      181     -289     
  Lines      136558    51421   -85137     
==========================================
- Hits       114836    42080   -72756     
+ Misses      21188     8807   -12381     
  Partials      534      534              
Components Coverage Δ
otap-dataflow ∅ <ø> (∅)
query_abstraction 80.61% <ø> (ø)
query_engine 90.22% <ø> (-0.17%) ⬇️
syslog_cef_receivers ∅ <ø> (∅)
otel-arrow-go 53.50% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@cijothomas
Copy link
Member Author

Closing as discussed with @gouslu who has this incoming in a separate PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

rust Pull requests that update Rust code

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants