Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

interop: add reset handling based on db write error #13497

Closed
wants to merge 1 commit into from

Conversation

axelKingsley
Copy link
Contributor

No description provided.

@axelKingsley axelKingsley requested review from a team as code owners December 19, 2024 22:45
@axelKingsley axelKingsley requested review from protolambda and removed request for a team December 19, 2024 22:45
Copy link

codecov bot commented Dec 19, 2024

Codecov Report

Attention: Patch coverage is 0% with 24 lines in your changes missing coverage. Please review.

Project coverage is 46.20%. Comparing base (78042a7) to head (57d7889).

Files with missing lines Patch % Lines
op-supervisor/supervisor/backend/db/query.go 0.00% 24 Missing ⚠️
Additional details and impacted files
@@                 Coverage Diff                 @@
##           interop-managed   #13497      +/-   ##
===================================================
+ Coverage            45.03%   46.20%   +1.16%     
===================================================
  Files                  880      934      +54     
  Lines                73688    78083    +4395     
  Branches               803      803              
===================================================
+ Hits                 33188    36081    +2893     
- Misses               37977    39362    +1385     
- Partials              2523     2640     +117     
Flag Coverage Δ
cannon-go-tests-32 62.27% <ø> (?)
cannon-go-tests-64 57.35% <ø> (?)
contracts-bedrock-tests 89.28% <ø> (-2.70%) ⬇️

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

Files with missing lines Coverage Δ
op-supervisor/supervisor/backend/db/query.go 0.00% <0.00%> (ø)

... and 90 files with indirect coverage changes

@axelKingsley axelKingsley force-pushed the interop-managed-resetNode branch from 57d7889 to b7adca8 Compare December 20, 2024 16:05
@protolambda
Copy link
Contributor

Merged with rebase into #13406

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