Skip to content

Commit

Permalink
Combine payload update and setting healthy status. (#107)
Browse files Browse the repository at this point in the history
This PR revises how updates are dispatched. Instead of having separate
status and payload updates, these are now merged into one with a payload
update implying a status change to healthy. This should avoid the potential
for strange races between setting the status back to healthy and updating
the data.

A resulting limitation is that if a unit wants to change from stalled to
healthy it has to submit a payload update, but in practice this health
status change is pretty much always triggered by a new payload update
becoming available, so this should be fine.

This is a breaking change.
  • Loading branch information
partim authored Apr 10, 2024
1 parent 0f61284 commit a6df70e
Show file tree
Hide file tree
Showing 10 changed files with 457 additions and 312 deletions.
95 changes: 74 additions & 21 deletions Cargo.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

1 change: 1 addition & 0 deletions Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,7 @@ url = { version = "2.2", features = ["serde"] }
webpki-roots = "0.25.2"

[dev-dependencies]
stderrlog = "0.6"
rand_pcg = "0.3"

[profile.release]
Expand Down
Loading

0 comments on commit a6df70e

Please sign in to comment.