Skip to content

Retry NM-owned retryable suspend failures before releasing network teardown - #122

Merged
Staphylococcus merged 1 commit into
devfrom
issue-121-nm-owned-retry
Sep 2, 2026
Merged

Retry NM-owned retryable suspend failures before releasing network teardown#122
Staphylococcus merged 1 commit into
devfrom
issue-121-nm-owned-retry

Conversation

@Staphylococcus

Copy link
Copy Markdown
Owner

Summary

  • give NetworkManager-owned retryable suspend failures the same single follow-up opportunity as joined failures
  • record a completed cycle and ownership marker when that follow-up succeeds
  • release teardown after one failed follow-up without introducing a retry loop

Test-driven evidence

The owner-first regression initially failed with:

left:  Some(RetryableTransportFailure)
right: Some(Completed)

After the implementation change, both the recovery case and the bounded second-failure case pass.

Validation

  • cargo fmt --all --check
  • cargo clippy -p lg-buddy --all-targets -- -D warnings
  • cargo test -p lg-buddy --lib (737 passed, 1 ignored)
  • cargo test -p lg-buddy (124 Cucumber scenarios / 1,431 steps plus all integration tests)
  • git diff --check

Closes #121

@Staphylococcus
Staphylococcus merged commit b38f5a7 into dev Sep 2, 2026
4 checks passed
@Staphylococcus
Staphylococcus deleted the issue-121-nm-owned-retry branch September 2, 2026 14:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant