Skip to content

chore(executors): simplify adding source and writer parameters to otel span#519

Open
AndreyMarkinPPC wants to merge 1 commit into
mainfrom
otel-executor-parameters
Open

chore(executors): simplify adding source and writer parameters to otel span#519
AndreyMarkinPPC wants to merge 1 commit into
mainfrom
otel-executor-parameters

Conversation

@AndreyMarkinPPC

Copy link
Copy Markdown
Member

No description provided.

@AndreyMarkinPPC AndreyMarkinPPC self-assigned this Jul 16, 2026
@codecov-commenter

codecov-commenter commented Jul 16, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 88.88889% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 90.36%. Comparing base (5ebb9d2) to head (d26e222).
⚠️ Report is 318 commits behind head on main.

Files with missing lines Patch % Lines
libs/executors/garf/executors/executor.py 83.33% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #519      +/-   ##
==========================================
+ Coverage   88.23%   90.36%   +2.12%     
==========================================
  Files          61       88      +27     
  Lines        2600     4658    +2058     
==========================================
+ Hits         2294     4209    +1915     
- Misses        306      449     +143     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

* simplify adding source and writer parameters to otel span
* add support for GARF_MODE env variable in resource attributes
* add batch_size to execute_batch method of executor
@AndreyMarkinPPC AndreyMarkinPPC force-pushed the otel-executor-parameters branch from c482657 to d26e222 Compare July 16, 2026 12:27
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