0chain / blobber

A storage provider (blobber) interface to the blockchain and consumers of storage.
Other
19 stars 22 forks source link

fix lock ordering #1436

Closed Hitenjain14 closed 1 month ago

Hitenjain14 commented 1 month ago

Changes

Fixes

Tests

Tasks to complete before merging PR:

Associated PRs (Link as appropriate):

codecov-commenter commented 1 month ago

Codecov Report

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

Project coverage is 14.88%. Comparing base (7ce03e2) to head (f193ae2). Report is 146 commits behind head on staging.

Files Patch % Lines
...go/0chain.net/blobbercore/allocation/connection.go 0.00% 1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## staging #1436 +/- ## =========================================== - Coverage 15.01% 14.88% -0.13% =========================================== Files 178 179 +1 Lines 19908 20182 +274 =========================================== + Hits 2989 3005 +16 - Misses 16548 16804 +256 - Partials 371 373 +2 ``` | [Flag](https://app.codecov.io/gh/0chain/blobber/pull/1436/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=0chain) | Coverage Δ | | |---|---|---| | [Unit-Tests](https://app.codecov.io/gh/0chain/blobber/pull/1436/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=0chain) | `14.88% <0.00%> (-0.13%)` | :arrow_down: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=0chain#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.