Skip to content

tx-undecision: awaitSharedChange as an STM action#5399

Merged
coot merged 1 commit into
mainfrom
coot/tx-undecision-stm
Jul 13, 2026
Merged

tx-undecision: awaitSharedChange as an STM action#5399
coot merged 1 commit into
mainfrom
coot/tx-undecision-stm

Conversation

@coot

@coot coot commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Description

When a registered delay is created out of the scope of awaitSharedChange it
can become an STM action. This way we won't need to race IO actions in
dmq-node but instead resolve the race in an STM transaction.

Checklist

Quality

  • Commit sequence makes sense and have useful messages, see ref.
  • New tests are added and existing tests are updated.
  • Self-reviewed the PR.

Maintenance

  • Linked an issue or added the PR to the current sprint of ouroboros-network project.
  • Added labels.
  • Updated changelog files.
  • The documentation has been properly updated, see ref.

@coot coot requested a review from a team as a code owner July 9, 2026 14:04
@github-project-automation github-project-automation Bot moved this to In Progress in Ouroboros Network Jul 9, 2026
This way we won't need to `race` `IO` actions in `dmq-node` but instead
resolve the race in an STM transaction.
@coot coot force-pushed the coot/tx-undecision-stm branch from e275a4f to 6d93899 Compare July 9, 2026 14:24
@coot coot self-assigned this Jul 9, 2026
@coot coot added the tx-submission Issues related to tx-submission protocol label Jul 9, 2026

@crocodile-dentist crocodile-dentist left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@coot coot added this pull request to the merge queue Jul 10, 2026
@coot coot moved this from In Progress to In Review in Ouroboros Network Jul 10, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 10, 2026
@coot coot added this pull request to the merge queue Jul 10, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 10, 2026
@coot coot added this pull request to the merge queue Jul 13, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 13, 2026
@coot coot added this pull request to the merge queue Jul 13, 2026
Merged via the queue into main with commit 5860d50 Jul 13, 2026
25 checks passed
@coot coot deleted the coot/tx-undecision-stm branch July 13, 2026 14:35
@github-project-automation github-project-automation Bot moved this from In Review to Done in Ouroboros Network Jul 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tx-submission Issues related to tx-submission protocol

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants