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

Introduce a Block Persistence Failure response code #743

Open
ata-nas opened this issue Feb 28, 2025 · 6 comments
Open

Introduce a Block Persistence Failure response code #743

ata-nas opened this issue Feb 28, 2025 · 6 comments
Assignees
Labels
Block Node Issues/PR related to the Block Node. New Feature A new feature, service, or documentation. Major changes that are not backwards compatible.
Milestone

Comments

@ata-nas
Copy link
Contributor

ata-nas commented Feb 28, 2025

Persona

As a Block Node Developer/Operator/User/Client

Request

I want to send a failed persistence response code to publishers

Goal

So that I report that persistence has failed

Technical Notes

  • as of today there is no persistence failure PublishStreamResponseCode
  • as a fallback, today we use the bad state proof if persistence fails
  • we need to introduce a block persistence failure response code
@ata-nas ata-nas added Block Node Issues/PR related to the Block Node. New Feature A new feature, service, or documentation. Major changes that are not backwards compatible. labels Feb 28, 2025
@jsync-swirlds jsync-swirlds self-assigned this Feb 28, 2025
@jsync-swirlds
Copy link
Contributor

PR here to update in services.
Change to published protobufs will happen after release 61 is synchronized to hedera-protobufs (or possibly via some other mechanism if the publication method changes).

@jsync-swirlds jsync-swirlds added this to the 0.6.0 milestone Mar 1, 2025
@ata-nas
Copy link
Contributor Author

ata-nas commented Mar 4, 2025

@jsync-swirlds FYI, places where changes need to be made (be chosen the correct response code to publish) in the codebase currently are tagged with @todo(743) within PR #715. These are in addition to the introduction of the response code itself.

@jsync-swirlds
Copy link
Contributor

@jsync-swirlds FYI, places where changes need to be made (be chosen the correct response code to publish) in the codebase currently are tagged with @todo(743) within PR #715. These are in addition to the introduction of the response code itself.

Good point. I think I'll split out a separate task for that, so the work in other repos is separate from the work in block node repo.

@Nana-EC
Copy link
Contributor

Nana-EC commented Mar 8, 2025

@jsync-swirlds moving this to ongoing sprint since it's active but left in old sprint

@Nana-EC
Copy link
Contributor

Nana-EC commented Mar 9, 2025

@jsync-swirlds PR on services side was merged, what's the next step here or can this be closed assuming BN logic already has logic to send error code?

@jsync-swirlds
Copy link
Contributor

I was just leaving open to track that we see the updates in protobufs. It's closed from a work-to-do perspective (Block Node code changes are in a separate issue).
That's also why I left the old sprint, as all the work was done in that sprint. We should probably have a team guideline on when to update sprint/when not to.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Block Node Issues/PR related to the Block Node. New Feature A new feature, service, or documentation. Major changes that are not backwards compatible.
Projects
None yet
Development

No branches or pull requests

3 participants