Skip to content

Conversation

@ktoso
Copy link
Member

@ktoso ktoso commented Jul 19, 2022

Final step for Resolves #824 cluster singleton

@ktoso ktoso changed the title Wip last singleton step Distributed Actor support in Cluster Singleton Jul 19, 2022
@ktoso ktoso force-pushed the wip-last-singleton-step branch 3 times, most recently from e73f180 to b587d84 Compare July 20, 2022 10:30
@ktoso
Copy link
Member Author

ktoso commented Jul 20, 2022

Let's hope I got the rebase right 🤞
Locally we're green 🟢

@ktoso ktoso force-pushed the wip-last-singleton-step branch from b587d84 to f369625 Compare July 20, 2022 11:31
@ktoso
Copy link
Member Author

ktoso commented Jul 20, 2022

Had to change out metrics internal API usage because of apple/swift-metrics@53be786 but good now

@ktoso
Copy link
Member Author

ktoso commented Jul 20, 2022

Flaky cluster join - #1016 need to check it out, maybe just timing.

@ktoso
Copy link
Member Author

ktoso commented Jul 20, 2022

@swift-server-bot test this please

@ktoso
Copy link
Member Author

ktoso commented Jul 20, 2022

Hm same but differnet test #1018

It seems we have some timing hiccups with node joins; the singleton things working well though 🥳

I'll follow up on these tomorrow in depth.

@ktoso ktoso merged commit ad8e6b6 into apple:main Jul 20, 2022
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.

Implement ActorSingleton for DistributedActor

2 participants