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

go/worker/common: Reset runtime provisioned status on failure #5361

Merged
merged 1 commit into from
Sep 11, 2023

Conversation

kostko
Copy link
Member

@kostko kostko commented Sep 8, 2023

No description provided.

@kostko kostko force-pushed the kostko/fix/rt-status-failed branch from 3993a8f to a012888 Compare September 8, 2023 08:55
@kostko kostko marked this pull request as ready for review September 8, 2023 08:55
@codecov
Copy link

codecov bot commented Sep 8, 2023

Codecov Report

Merging #5361 (a012888) into master (5fca705) will increase coverage by 0.29%.
Report is 19 commits behind head on master.
The diff coverage is 64.54%.

@@            Coverage Diff             @@
##           master    #5361      +/-   ##
==========================================
+ Coverage   66.93%   67.22%   +0.29%     
==========================================
  Files         525      525              
  Lines       55763    55729      -34     
==========================================
+ Hits        37323    37464     +141     
+ Misses      13909    13749     -160     
+ Partials     4531     4516      -15     
Files Changed Coverage Δ
go/beacon/api/api.go 75.00% <ø> (ø)
go/beacon/api/grpc.go 34.28% <0.00%> (+6.28%) ⬆️
go/common/crypto/mathrand/rand.go 83.33% <0.00%> (ø)
go/common/crypto/signature/signers/plugin/rpc.go 76.92% <0.00%> (ø)
go/common/grpc/auth/auth_tls.go 0.00% <0.00%> (ø)
go/common/grpc/auth/auth_tls_pubkeys.go 0.00% <0.00%> (ø)
go/common/grpc/errors.go 86.27% <ø> (ø)
go/common/grpc/service.go 35.84% <0.00%> (ø)
go/common/version/version.go 80.26% <0.00%> (ø)
go/common/workerpool/workerpool.go 77.27% <0.00%> (ø)
... and 146 more

... and 45 files with indirect coverage changes

📢 Have feedback on the report? Share it here.

@kostko kostko enabled auto-merge September 8, 2023 09:32
@kostko kostko merged commit 1c00e97 into master Sep 11, 2023
2 of 3 checks passed
@kostko kostko deleted the kostko/fix/rt-status-failed branch September 11, 2023 11:54
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