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

fix(share/pruner) enable Light node pruner service #3455

Merged
merged 7 commits into from
Jun 4, 2024

Conversation

walldiss
Copy link
Member

@walldiss walldiss commented Jun 3, 2024

Fixes 2 issues that prevented Light pruner service from launching:

  • pruning flag was not parsed by Light node
  • Pruner service was not provided to Light node

@walldiss walldiss added the kind:fix Attached to bug-fixing PRs label Jun 3, 2024
@walldiss walldiss self-assigned this Jun 3, 2024
nodebuilder/pruner/module.go Outdated Show resolved Hide resolved
@codecov-commenter
Copy link

codecov-commenter commented Jun 3, 2024

Codecov Report

Attention: Patch coverage is 11.11111% with 16 lines in your changes missing coverage. Please review.

Project coverage is 44.97%. Comparing base (2469e7a) to head (d75914b).
Report is 94 commits behind head on main.

Files Patch % Lines
nodebuilder/pruner/module.go 12.50% 13 Missing and 1 partial ⚠️
cmd/util.go 0.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3455      +/-   ##
==========================================
+ Coverage   44.83%   44.97%   +0.14%     
==========================================
  Files         265      273       +8     
  Lines       14620    15345     +725     
==========================================
+ Hits         6555     6902     +347     
- Misses       7313     7653     +340     
- Partials      752      790      +38     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

renaynay
renaynay previously approved these changes Jun 3, 2024
Copy link
Member

@renaynay renaynay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lord.

nodebuilder/pruner/module.go Outdated Show resolved Hide resolved
nodebuilder/pruner/module.go Outdated Show resolved Hide resolved
nodebuilder/pruner/module.go Outdated Show resolved Hide resolved
ramin
ramin previously approved these changes Jun 3, 2024
@walldiss walldiss dismissed stale reviews from ramin and renaynay via 2ecb679 June 3, 2024 13:27
@walldiss walldiss requested a review from renaynay June 3, 2024 13:29
ramin
ramin previously approved these changes Jun 3, 2024
cristaloleg
cristaloleg previously approved these changes Jun 3, 2024
Copy link
Contributor

@cristaloleg cristaloleg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

SGTM

nodebuilder/pruner/module.go Outdated Show resolved Hide resolved
cmd/util.go Show resolved Hide resolved
@walldiss walldiss dismissed stale reviews from cristaloleg and ramin via d2d325b June 4, 2024 08:12
@walldiss walldiss enabled auto-merge (squash) June 4, 2024 12:03
@walldiss walldiss merged commit 0a6beb1 into celestiaorg:main Jun 4, 2024
28 checks passed
ramin pushed a commit that referenced this pull request Jun 6, 2024
Fixes 2 issues that prevented Light pruner service from launching:
- pruning flag was not parsed by Light node
- Pruner service was not provided to Light node
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind:fix Attached to bug-fixing PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants