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

feat: stamper #440

Open
wants to merge 27 commits into
base: master
Choose a base branch
from
Open

feat: stamper #440

wants to merge 27 commits into from

Conversation

gacevicljubisa
Copy link
Contributor

@gacevicljubisa gacevicljubisa commented Dec 11, 2024

This PR introduces a new command group, stamper, to manage postage batches for Bee nodes, and enhances the existing node-funder functionality. It also includes fixes and improvements related to cluster setup and configuration usage.

Features Introduced

Stamper Command

The stamper command group is designed to handle postage batches with the following subcommands:

  1. create:
    Creates a new postage batch for selected nodes with customizable parameters like amount, depth, and timeout.

  2. topup:
    Tops up postage batches for selected nodes based on specified thresholds for TTL and periodic checks.

  3. dilute:
    Dilutes existing postage batches by increasing the depth based on utilization thresholds.

  4. set:
    Combines the functionalities of topup, and dilute to manage postage batches comprehensively.

Periodic checks for topup, dilute, and set subcommands are handled by the scheduler package.

Node-Funder Enhancements
  • Added --periodic-check flag to support periodic execution of funding operations.

Fixes and Improvements

  • Enhanced cluster setup process.
  • Improved configuration handling for better flexibility and usability.

@gacevicljubisa gacevicljubisa linked an issue Dec 11, 2024 that may be closed by this pull request
@gacevicljubisa gacevicljubisa changed the title feat: stamp-funder feat: stamper Dec 13, 2024
@gacevicljubisa gacevicljubisa marked this pull request as ready for review January 18, 2025 14:49
@gacevicljubisa gacevicljubisa requested review from istae and janos January 20, 2025 12:46
pkg/stamper/node.go Outdated Show resolved Hide resolved
@gacevicljubisa gacevicljubisa requested a review from istae January 22, 2025 10:41
Copy link
Member

@janos janos left a comment

Choose a reason for hiding this comment

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

Very nice work, Ljubisa.

pkg/stamper/node.go Outdated Show resolved Hide resolved
pkg/stamper/stamper.go Outdated Show resolved Hide resolved
pkg/swap/http.go Show resolved Hide resolved
Copy link
Member

@janos janos left a comment

Choose a reason for hiding this comment

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

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants