rpc: loadtxoutset should return errors that currently go to logs #28621

issue Sjors openend this issue on October 9, 2023
  1. Sjors commented at 1:23 pm on October 9, 2023: member

    Please describe the feature you’d like to see added.

    loadtxoutset should return (more of) the errors that currently only appear in the log.

    Currently most errors encountered during loadtxoutset can only be found via the log.

    Describe the solution you’d like

    No response

    Describe any alternatives you’ve considered

    No response

    Please leave any additional context

    Errors could be shown by loadtxoutset status if the method is made async (see #28620), otherwise just returned.

  2. Sjors added the label Feature on Oct 9, 2023
  3. pablomartin4btc commented at 3:13 am on October 10, 2023: member

    Concept ACK.

    eg. currently when re-try already processed and completed loadtxoutset on mainnet the returned error message is:

    0./src/bitcoin-cli -datadir=${AU_DATADIR} loadtxoutset ${AU_DATADIR}/utxo-800000.dat
    1error code: -32603
    2error message:
    3Unable to load UTXO snapshot .../.test_utxo_2/utxo-800000.dat
    

    While in the logs a user would see:

    02023-10-09T20:26:13Z [httpworker.2] [snapshot] waiting to see blockheader 00000000000000000002a7c4c1e48d76c5a37902165a270156b7a8d72728a054 in headers chain before snapshot activation
    12023-10-09T20:26:13Z [httpworker.2] [snapshot] can't activate a snapshot-based chainstate more than once
    

    I like also the idea to extend the rpc command with the status param: loadtxoutset status, in order to see percentage of processing, errors or warnings, especially if we manage to make it async.

  4. brunoerg commented at 8:52 pm on October 12, 2023: contributor
    Perhaps a good first issue?
  5. fanquake closed this on Oct 17, 2023

  6. Sjors commented at 7:18 am on October 18, 2023: member
    This shouldn’t be closed yet. #28652 only fixes one failure mode.
  7. maflcko reopened this on Oct 18, 2023

  8. Julio-Rats referenced this in commit 738ef44abb on Jan 26, 2024
  9. ryanofsky closed this on Aug 5, 2024

  10. ryanofsky referenced this in commit 69df012e74 on Aug 5, 2024
  11. bitcoin deleted a comment on Sep 11, 2024

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-11-17 12:12 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me