Skip to content

[multicast] Remove recovery path and update to latest dendrite#9951

Merged
zeeshanlakhani merged 2 commits intomainfrom
zl/dendrite-232
Mar 16, 2026
Merged

[multicast] Remove recovery path and update to latest dendrite#9951
zeeshanlakhani merged 2 commits intomainfrom
zl/dendrite-232

Conversation

@zeeshanlakhani
Copy link
Collaborator

@zeeshanlakhani zeeshanlakhani commented Mar 2, 2026

This incorporates oxidecomputer/dendrite#232 and oxidecomputer/dendrite#195.

Note: No API changes here.

@zeeshanlakhani zeeshanlakhani changed the title [multicast] Bump dendrite to zl/fix-mcast-empty-transition and remove unnecessary recovery path [multicast] Bump dendrite and remove unnecessary recovery path Mar 2, 2026
Base automatically changed from ry/dendrite-220 to main March 2, 2026 19:34
@zeeshanlakhani zeeshanlakhani changed the title [multicast] Bump dendrite and remove unnecessary recovery path [multicast] Remove recovery path and update to latest dendrite Mar 10, 2026
@zeeshanlakhani zeeshanlakhani marked this pull request as ready for review March 10, 2026 08:46
Copy link
Contributor

@FelixMcFelix FelixMcFelix left a comment

Choose a reason for hiding this comment

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

My read is that all the removed code was working around the bug you fixed in dendrite#232? If so then it's good to be rid of the complexity.

@zeeshanlakhani
Copy link
Collaborator Author

My read is that all the removed code was working around the bug you fixed in dendrite#232? If so then it's good to be rid of the complexity.

exactly.

@zeeshanlakhani zeeshanlakhani merged commit 2353600 into main Mar 16, 2026
18 checks passed
@zeeshanlakhani zeeshanlakhani deleted the zl/dendrite-232 branch March 16, 2026 02:49
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.

2 participants