graphprotocol / indexer-rs

Rewrite of indexer-service in Rust with Scalar TAP implementation
Apache License 2.0
16 stars 14 forks source link

Check scenario: aggregator down, alloc closed, force redeem, remaining receipts #197

Open aasseman opened 1 month ago

aasseman commented 1 month ago

The scenario is:

So at some point, we have:

How will tap-agent behave at this point?

In particular:

tkornuta-semiotic commented 1 month ago

The right question is: is there any chance that the Indexer will ever try to create a RAV request for "inactive" allocation...? @gusinacio

aasseman commented 1 month ago

The right question is: is there any chance that the Indexer will ever try to create a RAV request for "inactive" allocation...? @gusinacio

I'd maintain that the right questions are:

  • Will it continue tracking the unaggregated receipts?
  • If the aggregator comes back, will it try to aggregate (we shouldn't do that with a last RAV that's potentially redeemed on-chain)
gusinacio commented 1 month ago

The right question is: is there any chance that the Indexer will ever try to create a RAV request for "inactive" allocation...? @gusinacio

A closed allocation is already an "inactive" allocation and we do the last rav while closed.

aasseman commented 1 month ago

The right question is: is there any chance that the Indexer will ever try to create a RAV request for "inactive" allocation...? @gusinacio

A closed allocation is already an "inactive" allocation and we do the last rav while closed.

Yes but we're talking about the case where:

aasseman commented 2 weeks ago

Test report

Events:

At this point things got a bit weird:

tap-agent-1  |   2024-06-19T21:12:41.883459Z ERROR indexer_tap_agent::agent::sender_allocation: Error while requesting RAV for sender 0x21FEd3c4340f67DbF2b78c670ebD1940668CA03e and allocation 0xbed3CAeef28601cA60495ddEF6504D68E612C2ae: It looks like there are no valid receipts for the RAV request.This may happen if your `rav_request_trigger_value` is too low and no receipts were found outside the `rav_request_timestamp_buffer_ms`.You can fix this by increasing the `rav_request_trigger_value`.
tap-agent-1  |

tap-agent-1  |   2024-06-19T21:12:33.699983Z ERROR indexer_tap_agent::agent::sender_account: There was an error while requesting a RAV., error: Error while getting the heaviest allocation because no unblocked allocation has enough unaggregated fees tracked

Waiting to see what happens after the 1hr period expires...