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

Gc enhancement 2.5 - fault tolerance & parallel deletion in gc #16090

Closed
wants to merge 3 commits into from

Conversation

zyyw
Copy link
Contributor

@zyyw zyyw commented Dec 8, 2021

Closes: #15469
Closes: #15822
Closes: #15807

Signed-off-by: Shengwen Yu yshengwen@vmware.com

Shengwen Yu added 3 commits December 7, 2021 16:07
Signed-off-by: Shengwen Yu <yshengwen@vmware.com>
Signed-off-by: Shengwen Yu <yshengwen@vmware.com>
Signed-off-by: Shengwen Yu <yshengwen@vmware.com>
@zyyw zyyw requested a review from wy65701436 December 8, 2021 03:46
@codecov
Copy link

codecov bot commented Dec 8, 2021

Codecov Report

Merging #16090 (c6f68b3) into main (7e67c1f) will decrease coverage by 26.36%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##             main   #16090       +/-   ##
===========================================
- Coverage   66.81%   40.44%   -26.37%     
===========================================
  Files         940      227      -713     
  Lines       77873    11998    -65875     
  Branches     2293     2293               
===========================================
- Hits        52032     4853    -47179     
+ Misses      21824     6326    -15498     
+ Partials     4017      819     -3198     
Flag Coverage Δ
unittests 40.44% <ø> (-26.37%) ⬇️

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

Impacted Files Coverage Δ
...es/vulnerability/vulnerability-config.component.ts 46.82% <0.00%> (-3.97%) ⬇️
...tegration/tag-retention/tag-retention.component.ts 28.26% <0.00%> (-3.92%) ⬇️
src/jobservice/job/impl/gc/garbage_collection.go
src/common/http/transport.go
...pkg/notifier/handler/notification/slack_handler.go
src/lib/response_buffer.go
src/pkg/config/rest/manager.go
src/lib/log/record.go
src/pkg/registry/auth/authorizer.go
src/common/security/secret/context.go
... and 705 more

@zyyw
Copy link
Contributor Author

zyyw commented Dec 21, 2021

Closing this PR as we will submit another two PRs to deal with failure tolerance in GC and parallel processing in GC, respectively.

@zyyw zyyw closed this Dec 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant