Skip to content

Comments

[tda] Investigation: code to test clock drift#952

Draft
maxkosty wants to merge 1 commit intomasterfrom
tda-test-clock-drift
Draft

[tda] Investigation: code to test clock drift#952
maxkosty wants to merge 1 commit intomasterfrom
tda-test-clock-drift

Conversation

@maxkosty
Copy link
Collaborator

This is for proving SauceLabs support that their VMs can have major clock drift which causes mis-alignment in the waterfall view in distributed traces.

@vercel
Copy link
Contributor

vercel bot commented Jul 17, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Review Updated (UTC)
empower Ignored Ignored Preview Jan 28, 2026 7:44pm

Request Review

@codecov
Copy link

codecov bot commented Jul 17, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 22.01%. Comparing base (40cad6b) to head (f69442b).
✅ All tests successful. No failed tests found.

❌ Your project check has failed because the head coverage (22.01%) is below the target coverage (35.00%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #952   +/-   ##
=======================================
  Coverage   22.01%   22.01%           
=======================================
  Files          42       42           
  Lines        1349     1349           
  Branches      174      174           
=======================================
  Hits          297      297           
- Misses       1026     1027    +1     
+ Partials       26       25    -1     
Flag Coverage Δ
api 5.83% <ø> (ø)
frontend 32.51% <ø> (ø)
Components Coverage Δ
checkout_module 5.83% <ø> (ø)
product_component 26.13% <ø> (ø)
see 1 file with indirect coverage changes

@maxkosty maxkosty force-pushed the tda-test-clock-drift branch from 861070e to f69442b Compare January 28, 2026 19:44
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.

1 participant