Skip to content

Disable flaky JSON-RPC test#272

Merged
grcevski merged 1 commit into
open-telemetry:mainfrom
rafaelroquetto:disable_jsonrpc_flaky
Jul 8, 2025
Merged

Disable flaky JSON-RPC test#272
grcevski merged 1 commit into
open-telemetry:mainfrom
rafaelroquetto:disable_jsonrpc_flaky

Conversation

@rafaelroquetto
Copy link
Copy Markdown
Contributor

The test is flaky, working only sometimes, and is preventing other PRs from progressing. It was first introduced by #161.

@rafaelroquetto rafaelroquetto requested a review from a team as a code owner July 7, 2025 21:24
@rafaelroquetto
Copy link
Copy Markdown
Contributor Author

Tracked at #273

@codecov
Copy link
Copy Markdown

codecov Bot commented Jul 7, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.81%. Comparing base (568eedf) to head (9292c41).
⚠️ Report is 838 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #272      +/-   ##
==========================================
+ Coverage   72.44%   76.81%   +4.36%     
==========================================
  Files         178      179       +1     
  Lines       19078    19507     +429     
==========================================
+ Hits        13822    14985    +1163     
+ Misses       4489     3735     -754     
- Partials      767      787      +20     
Flag Coverage Δ
integration-test 56.63% <ø> (-0.02%) ⬇️
integration-test-arm 34.99% <ø> (-0.49%) ⬇️
k8s-integration-test 51.80% <ø> (?)
oats-test 35.04% <ø> (-0.46%) ⬇️
unittests 47.39% <ø> (+0.51%) ⬆️

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

☔ View full report in Codecov by Sentry.
📢 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@grcevski grcevski merged commit 73c26bc into open-telemetry:main Jul 8, 2025
12 of 13 checks passed
@rafaelroquetto rafaelroquetto deleted the disable_jsonrpc_flaky branch July 10, 2025 22:56
@MrAlias MrAlias added this to the v0.1.0 milestone Jul 17, 2025
marctc pushed a commit to grafana/opentelemetry-ebpf-instrumentation that referenced this pull request Dec 9, 2025
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.

4 participants