-
Notifications
You must be signed in to change notification settings - Fork 13.1k
chore: fix race condition on LocalBroker service's dependencies
#35997
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
Conversation
|
Looks like this PR is ready to merge! 🎉 |
|
Code Review Completed! 🔥The code review was successfully completed based on your current configurations. Kody Guide: Usage and ConfigurationInteracting with Kody
Current Kody ConfigurationReview OptionsThe following review options are enabled or disabled:
|
|
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## develop #35997 +/- ##
===========================================
+ Coverage 65.83% 65.84% +0.01%
===========================================
Files 3186 3186
Lines 106646 106645 -1
Branches 20315 20316 +1
===========================================
+ Hits 70207 70223 +16
+ Misses 33768 33755 -13
+ Partials 2671 2667 -4
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
5e0058c to
dd25cff
Compare
dd25cff to
0987b80
Compare
Proposed changes (including videos or screenshots)
Sometimes you could get stuck on startup due to a dependency on the settings service not being met:
I was able to reproduce this issue while using a remote MongoDB server, so things were much slower and apparently ran in a different order than usual causing it to get stuck.
Issue(s)
ARCH-1725
Steps to test or reproduce
Further comments
Pull Request Description
This pull request addresses a race condition issue in the
LocalBrokerservice within the Rocket.Chat repository. The changes focus on enhancing the service registration logic by implementing a filter. This filter ensures that only services which are not already started or not present in the services map are registered. This improvement aims to prevent potential conflicts and ensure smoother operation of theLocalBrokerservice's dependencies. The changes are made in thepackages/core-services/src/LocalBroker.tsfile.