Skip to content

Send out refresh notificiation when a target becomes "online"#55

Merged
rustybee42 merged 2 commits intomainfrom
rb/fix-target-states-notification
Feb 26, 2026
Merged

Send out refresh notificiation when a target becomes "online"#55
rustybee42 merged 2 commits intomainfrom
rb/fix-target-states-notification

Conversation

@rustybee42
Copy link
Copy Markdown
Collaborator

Not only when its consistency state changes. This fixes meta servers not recognizing a storage target as online after startup under certain conditions. Also improve the log message slightly.

Not only when its consistency state changes. This fixes meta servers not
recognizing a storage target as online after startup under certain
conditions. Also improve the log message slightly.
@rustybee42 rustybee42 self-assigned this Feb 26, 2026
@rustybee42 rustybee42 requested a review from a team as a code owner February 26, 2026 14:41
Comment thread mgmtd/src/bee_msg/common.rs
Copy link
Copy Markdown
Member

@philippfalk philippfalk left a comment

Choose a reason for hiding this comment

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

Looks good, thanks for the quick fix! Some initial testing shows that it's working correctly now and updating the states after ~3 seconds.

@rustybee42 rustybee42 force-pushed the rb/fix-target-states-notification branch from dec0c97 to 12fd2fc Compare February 26, 2026 16:45
@rustybee42 rustybee42 enabled auto-merge February 26, 2026 16:45
@rustybee42 rustybee42 merged commit 3c4a1c5 into main Feb 26, 2026
4 checks passed
@rustybee42 rustybee42 deleted the rb/fix-target-states-notification branch February 26, 2026 16:46
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.

Add a grace period on startup so nodes have time to reconnect before they are marked offline

2 participants