Skip to content
This repository was archived by the owner on Dec 4, 2024. It is now read-only.

remove panic on unexpected number of Stacks statuses updated#334

Open
mijoco-btc wants to merge 3 commits into
mainfrom
333-romeo-exits-with-unexpected-number-of-stacks-statuses-updated
Open

remove panic on unexpected number of Stacks statuses updated#334
mijoco-btc wants to merge 3 commits into
mainfrom
333-romeo-exits-with-unexpected-number-of-stacks-statuses-updated

Conversation

@mijoco-btc

@mijoco-btc mijoco-btc commented Oct 28, 2023

Copy link
Copy Markdown

just removes the panic when the server is restarted and it processes all the previous mints and burns

@mijoco-btc mijoco-btc linked an issue Oct 28, 2023 that may be closed by this pull request
Romeo exits with Unexpected number of Stacks statuses updated #333
Open
@mijoco-btc

mijoco-btc commented Oct 28, 2023

Copy link
Copy Markdown
Author

After spinning up a node with this change i get lots of logging like this...

2023-10-28T15:37:38.671959Z DEBUG update: romeo::state: Ignoring Broadcasted status update for a Stacks transaction that is not acknowledged: 0000000000000000000000000000000000000000000000000000000000000000 event=StacksTransactionUpdate(0000000000000000000000000000000000000000000000000000000000000000, Broadcasted)
2023-10-28T15:37:38.672003Z DEBUG update: romeo::state: Igno anring Broadcasted status update for a Stacks transaction that doesn't have a pending task: 0000000000000000000000000000000000000000000000000000000000000000 event=StacksTransactionUpdate(0000000000000000000000000000000000000000000000000000000000000000, Broadcasted)
2023-10-28T15:37:38.672047Z DEBUG update: romeo::state: Igno anring Broadcasted status update for a Stacks transaction that doesn't have a pending task: 0000000000000000000000000000000000000000000000000000000000000000 event=StacksTransactionUpdate(0000000000000000000000000000000000000000000000000000000000000000, Broadcasted)
2023-10-28T15:37:38.672091Z DEBUG update: romeo::state: Igno anring Broadcasted status update for a Stacks transaction that doesn't have a pending task: 0000000000000000000000000000000000000000000000000000000000000000 event=StacksTransactionUpdate(0000000000000000000000000000000000000000000000000000000000000000, Broadcasted)
2023-10-28T15:37:38.672136Z DEBUG update: romeo::state: Igno anring Broadcasted status update for a Stacks transaction that doesn't have a pending task: 0000000000000000000000000000000000000000000000000000000000000000 event=StacksTransactionUpdate(0000000000000000000000000000000000000000000000000000000000000000, Broadcasted)
2023-10-28T15:37:38.672180Z DEBUG update: romeo::state: Igno anring Broadcasted status update for a Stacks transaction that doesn't have a pending task: 0000000000000000000000000000000000000000000000000000000000000000 event=StacksTransactionUpdate(0000000000000000000000000000000000000000000000000000000000000000, Broadcasted)

@friedger

friedger commented Oct 28, 2023

Copy link
Copy Markdown
Contributor

While this cures the symptom, I think the problem is elsewhere.

@CLAassistant

CLAassistant commented Nov 16, 2023 •
edited
Loading

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@mijoco-btc
Merge branch 'main' into 333-romeo-exits-with-unexpected-number-of-st…
eabf261 
…acks-statuses-updated

* main:
  test: add property-based tests for romeo asset contract
  test: add property-based tests for romeo asset contract
  test: add property-based tests for romeo asset contract
  test: add property-based tests for romeo asset contract
@CAGS295

CAGS295 commented Nov 17, 2023

Copy link
Copy Markdown
Contributor

I have a fix, but it unravels other issues. I'm waiting a bit longer until I can put the case together.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Romeo exits with Unexpected number of Stacks statuses updated

4 participants

@mijoco-btc @friedger @CLAassistant @CAGS295