Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Distributed Actor support in Cluster Singleton #1010

Merged
merged 21 commits into from
Jul 20, 2022

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