-
Notifications
You must be signed in to change notification settings - Fork 20
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
Only enable the failover fix if configured #109
Conversation
0887c26
to
780f52d
Compare
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #109 +/- ##
==========================================
+ Coverage 68.88% 69.14% +0.25%
==========================================
Files 10 10
Lines 990 995 +5
Branches 173 174 +1
==========================================
+ Hits 682 688 +6
+ Misses 253 252 -1
Partials 55 55
☔ View full report in Codecov by Sentry. |
780f52d
to
6c8cc2a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We have a test to test the new feature flag mechanism, but we don't have a test for the new feature itself? (if master_node
)
To make sure that it behaves differently than previous version.
6c8cc2a
to
1e6b598
Compare
To be able to roll it out in a more controlled fashion
1e6b598
to
0ac108d
Compare
The test I've adjusted should cover the feature and the disabling of it. |
To be able to roll it out in a more controlled fashion
BF-1847