Skip to content

Conversation

coot
Copy link
Contributor

@coot coot commented May 26, 2025

Description

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 May 26, 2025 15:44
@github-project-automation github-project-automation bot moved this to In Progress in Ouroboros Network May 26, 2025
@coot coot force-pushed the coot/ouroboros-network-0.21.1 branch from 35e3f15 to 0f27b8d Compare May 26, 2025 15:46
@coot coot added the release Issues / PRs related releases. label May 26, 2025
@coot coot force-pushed the coot/ouroboros-network-0.21.1 branch from 0f27b8d to 24100f8 Compare May 26, 2025 15:56
@coot coot enabled auto-merge May 26, 2025 15:56
@coot coot requested a review from karknu May 27, 2025 06:00
@coot coot mentioned this pull request May 27, 2025
9 tasks
A race can occur between peerMonitoringLoop closing an outbound
connection due to a warm protocol finish and the OG demoting
the same peer for different reason. Both will call
closePeerConnection, but only the winner should invoke CM's
releaseOutboundConnection. If the winner is the OG's demotion job,
close peer connection should not return wasWarm, because the
losing peerMonitoringLoop would see it as false, and would end
without setting peer status variable to PeerCold. Instead,
the return value of closePeerConnection is changed such that
the it will work regardless of who wins the race to close the outbound
connection.
@coot coot force-pushed the coot/ouroboros-network-0.21.1 branch from 24100f8 to 9d18722 Compare May 27, 2025 11:10
@coot coot self-assigned this May 27, 2025
@coot coot force-pushed the coot/ouroboros-network-0.21.1 branch from 9d18722 to f0c3f7e Compare May 27, 2025 11:23
@coot coot requested a review from a team as a code owner May 27, 2025 11:23
@coot coot force-pushed the coot/ouroboros-network-0.21.1 branch from f0c3f7e to 3477ed9 Compare May 27, 2025 11:35
@coot coot force-pushed the coot/ouroboros-network-0.21.1 branch from 3477ed9 to 92a9487 Compare May 27, 2025 12:34
@coot coot merged commit 2f7baf9 into release/ouroboros-network-0.21 May 27, 2025
13 of 14 checks passed
@coot coot deleted the coot/ouroboros-network-0.21.1 branch May 27, 2025 14:18
@github-project-automation github-project-automation bot moved this from In Progress to Done in Ouroboros Network May 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release Issues / PRs related releases.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants