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

update action statuses and indexing rules #665

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

hopeyen
Copy link
Contributor

@hopeyen hopeyen commented May 17, 2023

As reported in issue #575, we observed cases in which the agent allocate to a subgraph deployment right after unallocate actions. The two possible causes are

  • allocation was closed successfully but failed to confirm due to IE053 when identifying the receipts, thus exited with error early without updating the indexing rules
    • moved update indexing rules to right after closing allocation, and before identifying receipts to collect. Regardless of the result for query fee collection, the indexing rules should be updated promptly
  • transaction to update action statuses did not include all the approved actions, might be due to agent creating allocate actions too early from indexing rules
    • added an additional step to update actions not included, and added debug logging

@fordN fordN assigned fordN and hopeyen and unassigned fordN Dec 5, 2023
@fordN fordN added the enhancement New feature or request label Dec 5, 2023
@hopeyen hopeyen force-pushed the hope/confirm-unallocate-db-update branch from 592960a to ee1b76d Compare December 7, 2023 20:24
@hopeyen hopeyen requested a review from fordN December 7, 2023 20:24
@hopeyen hopeyen force-pushed the hope/confirm-unallocate-db-update branch from ee1b76d to 3d6c8dc Compare December 7, 2023 20:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: 🛑 Blocked
Development

Successfully merging this pull request may close these issues.

2 participants