Skip to content

[exporter/exporterhelper] Migrate TimeoutConfig to schema-based configuration generated by mdatagen. - #15505

Merged
dmitryax merged 20 commits into
open-telemetry:mainfrom
jkoronaAtCisco:migrate_timeout_config
Aug 31, 2026
Merged

[exporter/exporterhelper] Migrate TimeoutConfig to schema-based configuration generated by mdatagen.#15505
dmitryax merged 20 commits into
open-telemetry:mainfrom
jkoronaAtCisco:migrate_timeout_config

Conversation

@jkoronaAtCisco

@jkoronaAtCisco jkoronaAtCisco commented Jun 26, 2026

Copy link
Copy Markdown
Member

Description

Migrate TimeoutConfig (part of exporterhelper) to mdatagen schema-based configuration. Removes the hand-written TimeoutConfig struct and NewTimeoutConfig constructor in favor of generated code driven by metadata.yaml and config.schema.json.

Authorship

  • I, a human, wrote this pull request description myself.

@codecov

codecov Bot commented Jun 26, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.08%. Comparing base (a183c9c) to head (2e5c09a).

Additional details and impacted files
@@           Coverage Diff           @@
##             main   #15505   +/-   ##
=======================================
  Coverage   91.08%   91.08%           
=======================================
  Files         733      734    +1     
  Lines       48679    48680    +1     
=======================================
+ Hits        44337    44340    +3     
+ Misses       3018     3016    -2     
  Partials     1324     1324           

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

@github-actions

Copy link
Copy Markdown
Contributor

This PR was marked stale due to lack of activity. It will be closed in 14 days.

@github-actions github-actions Bot added the Stale label Jul 15, 2026
@jkoronaAtCisco
jkoronaAtCisco marked this pull request as ready for review July 23, 2026 12:09
@jkoronaAtCisco
jkoronaAtCisco requested review from a team, bogdandrutu and dmitryax as code owners July 23, 2026 12:09
@jkoronaAtCisco
jkoronaAtCisco marked this pull request as draft July 24, 2026 20:02
@github-actions github-actions Bot removed the Stale label Jul 29, 2026
@jkoronaAtCisco
jkoronaAtCisco marked this pull request as ready for review July 30, 2026 14:56

@dmitryax dmitryax left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

Comment thread .chloggen/config.yaml Outdated
@github-actions

Copy link
Copy Markdown
Contributor

This PR was marked stale due to lack of activity. It will be closed in 14 days.

@github-actions github-actions Bot added the Stale label Aug 15, 2026
@codspeed-hq

codspeed-hq Bot commented Aug 23, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

⚠️ Unknown Walltime execution environment detected

Using the Walltime instrument on standard Hosted Runners will lead to inconsistent data.

For the most accurate results, we recommend using CodSpeed Macro Runners: bare-metal machines fine-tuned for performance measurement consistency.

⏩ 4 skipped benchmarks1
🗄️ 2 archived benchmarks run2


Comparing jkoronaAtCisco:migrate_timeout_config (9afefe4) with main (a693ede)

Open in CodSpeed

Footnotes

  1. 4 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

  2. 2 benchmarks were run, but are now archived. If they were deleted in another branch, consider rebasing to remove them from the report. Instead if they were added back, click here to restore them.

@dmitryax
dmitryax added this pull request to the merge queue Aug 31, 2026
Merged via the queue into open-telemetry:main with commit c62353b Aug 31, 2026
73 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants