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

Node Split beta #338

Open
5 of 8 tasks
pigmej opened this issue Jan 14, 2025 · 2 comments
Open
5 of 8 tasks

Node Split beta #338

pigmej opened this issue Jan 14, 2025 · 2 comments

Comments

@pigmej
Copy link
Member

pigmej commented Jan 14, 2025

This is the next phase after #337

  1. API Changes and Architecture:

    • Remove local function calls in favor of HTTP-only calls
    • Support monolithic application using two goroutines (node and smesher service)
      • Add a warning that this combined mode might be deprecated.
    • Implement communication between services via localhost
  2. Service Refactoring:

  3. API Improvements:

  4. State Management:

    • UI to deduce per-epoch state and execute the API calls
    • Carefully manage epoch state retrieval, particularly for coinbase from previous ATX
      • we need to ask proper epoch ATX to get information about rewards (n-1) otherwise we may endup with wrong coinbase
    • Create documentation explaining state retrieval and understanding
    • Allow event duplication for UI purposes
  5. Documentation:

    • Create visual documentation (Excalidraw) for state management and understanding
    • Documment config changes and adjust official docs
  6. UX / UI

@pigmej pigmej mentioned this issue Jan 14, 2025
5 tasks
@pigmej pigmej moved this to 🏗 Doing in Dev team kanban Jan 14, 2025
@brusherru
Copy link
Member

Remove current 100-item limit from smesher states
Add pagination and filtering capabilities to states API

I think it's also valid for eligibilities and proposals endpoints (in SmeshingIdentitiesService):
https://github.com/spacemeshos/api/blob/node-split-poc/spacemesh/v2alpha1/smeshing_identities.proto

@kacpersaw
Copy link

@pigmej This can be removed:
Remove current 100-item limit from smesher states
The limit was removed along with the introduction of persistent data.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: 🏗 Doing
Development

No branches or pull requests

3 participants