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

Jorge/10574 read fully #10575

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from
Draft

Jorge/10574 read fully #10575

wants to merge 3 commits into from

Conversation

Jorge-Lopes
Copy link
Collaborator

@Jorge-Lopes Jorge-Lopes commented Nov 26, 2024

refs: #10574

Description

This pull request aims to facilitate the observation of the behavior described in issue #10574 where the readFully method from the Agoric client-utils package fails when attempting to read a vstorage node.

To achieve this goal, the following changes have been made:

  • A simple test has been added to the acceptance proposal of a3p-integration to demonstrate the issue in a controlled environment.
  • Existing console.debug calls in packages/client-utils/src/vstorage.js have been uncommented to provide additional debugging insights during execution.

These changes are intended solely to help identify and understand the root cause of the issue, not to provide a final solution.

Error analysis

The error arises during the readFully method's execution, as observed in the log output. When invoked, the method follows this sequence:

  • Initial Behavior:
    The readAt method successfully retrieves data for the latestQuestion node at the highest block height (439) and appends it to the parts list. Subsequently, the method performs the same operation for block height 367, adding the respective node data to the list.

  • Error Occurrence:
    During the next iteration of the while loop, the readStorage function throws an error. This failure is signaled by the response code of 38.

Copy link

cloudflare-workers-and-pages bot commented Nov 26, 2024

Deploying agoric-sdk with  Cloudflare Pages  Cloudflare Pages

Latest commit: 603977a
Status: ✅  Deploy successful!
Preview URL: https://be3829f3.agoric-sdk.pages.dev
Branch Preview URL: https://jorge-10574-readfully.agoric-sdk.pages.dev

View logs

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
force:integration Force integration tests to run on PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant