Skip to content
New issue

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

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

Already on GitHub? # to your account

Check free space when Registering Task #585

Merged
merged 16 commits into from
Sep 1, 2021
Merged

Check free space when Registering Task #585

merged 16 commits into from
Sep 1, 2021

Conversation

zzy987
Copy link
Contributor

@zzy987 zzy987 commented Aug 26, 2021

In the hybridStorageMgr, the remaining space of the memory is checked. I want to introduced this feature in other situations.

@zzy987 zzy987 requested a review from 244372610 August 26, 2021 09:00
@zzy987 zzy987 requested a review from a team as a code owner August 26, 2021 09:00
zzy987 added 3 commits August 26, 2021 17:21
Signed-off-by: zzy987 <67889264+zzy987@users.noreply.github.com>
Signed-off-by: zzy987 <67889264+zzy987@users.noreply.github.com>
Signed-off-by: zzy987 <67889264+zzy987@users.noreply.github.com>
@codecov-commenter
Copy link

codecov-commenter commented Aug 27, 2021

Codecov Report

Merging #585 (89716cc) into main (4464dae) will decrease coverage by 0.15%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #585      +/-   ##
==========================================
- Coverage   39.26%   39.10%   -0.16%     
==========================================
  Files          86       86              
  Lines        6701     6712      +11     
==========================================
- Hits         2631     2625       -6     
- Misses       3697     3714      +17     
  Partials      373      373              
Flag Coverage Δ
e2etests ?
unittests 39.10% <0.00%> (-0.16%) ⬇️

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

Impacted Files Coverage Δ
cdnsystem/errors/errors.go 76.66% <0.00%> (-2.65%) ⬇️
cdnsystem/rpcserver/rpcserver.go 0.64% <0.00%> (-0.02%) ⬇️
cdnsystem/supervisor/cdn/manager.go 60.34% <0.00%> (-1.61%) ⬇️
cdnsystem/supervisor/task/manager.go 10.37% <0.00%> (+0.28%) ⬆️
cdnsystem/supervisor/task/manager_util.go 0.00% <0.00%> (ø)
client/daemon/peer/peertask_base.go 32.86% <0.00%> (-1.22%) ⬇️

@zzy987 zzy987 changed the title Check free space when ResetRepo Check free space when Registering Task Aug 27, 2021
zzy987 added 9 commits August 30, 2021 13:58
Signed-off-by: zzy987 <67889264+zzy987@users.noreply.github.com>
Signed-off-by: zzy987 <67889264+zzy987@users.noreply.github.com>
Signed-off-by: zzy987 <67889264+zzy987@users.noreply.github.com>
Signed-off-by: zzy987 <67889264+zzy987@users.noreply.github.com>
Signed-off-by: zzy987 <67889264+zzy987@users.noreply.github.com>
Signed-off-by: zzy987 <67889264+zzy987@users.noreply.github.com>
Signed-off-by: zzy987 <67889264+zzy987@users.noreply.github.com>
@jim3ma jim3ma merged commit fa22d7c into dragonflyoss:main Sep 1, 2021
@zzy987 zzy987 deleted the fix/cdn-gc branch September 1, 2021 06:49
gaius-qi pushed a commit that referenced this pull request Jun 28, 2023
Signed-off-by: zzy987 <67889264+zzy987@users.noreply.github.com>
e-ngo pushed a commit to e-ngo/dragonfly that referenced this pull request Jan 23, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants